-
Notifications
You must be signed in to change notification settings - Fork 1
feat: Automated reference documentation generation from Weave source #7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
Warning Review the following alerts detected in dependencies. According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.
|
- Add GitHub Action workflow for weekly doc generation - Create scripts for Service API, Python SDK, and TypeScript SDK docs - Use Mintlify's native OpenAPI support for Service API - Minimal dependencies to avoid Socket Security issues - Generate docs from Weave source repository
…ng source - Python SDK: Install from PyPI or GitHub using version/tag/hash - TypeScript SDK: Download source archive from GitHub for specific version - Remove repository cloning from GitHub Action - Update documentation to reflect version-based approach
- Add pull_request trigger for testing (to be removed before merging) - Add PR test mode reporting to show what would happen in production - Prevent actual PR creation when running from a pull request
- Add module import testing before documentation generation - Better error messages and debugging output - Default to 'latest' PyPI version instead of 'main' branch - Add fallback to latest PyPI if main branch fails - Handle 'latest' version properly in TypeScript script
- Pin typedoc to 0.25.13 and typedoc-plugin-markdown to 3.17.1 - Remove unsupported config options (hideBreadcrumbs, hideInPageTOC) - Add better error handling and diagnostic output - Check existing package.json versions before installing
40752b0
to
c1007f4
Compare
Broken Link Validation IssueThe PR is currently failing the "Mintlify Validation" check due to pre-existing broken links in the repository that are unrelated to this PR's changes. Issue CreatedI've created Issue #11 to track the broken links that need to be fixed in the main branch. Current StatusUnfortunately, Mintlify doesn't provide a configuration option to temporarily disable link validation. The broken links are in:
Recommendations
The documentation generation system implemented in this PR is working correctly - the failures are solely due to pre-existing issues in the repository. |
Overview
This PR implements an automated system for generating reference documentation from the Weave source code. It addresses the need to move away from static markdown files to dynamically generated documentation while avoiding the Socket Security issues identified in the previous approach.
Key Features
lazydocs
from any Weave versiontypedoc
with markdown outputTesting the Action
The workflow includes a temporary PR trigger for testing. When you push changes to this PR that modify:
.github/workflows/generate-reference-docs.yml
scripts/generate_*.py
filesscripts/requirements.txt
The action will run in test mode and:
Check the Actions tab to see the test results.
Production Usage
Manual Trigger
v0.50.0
), commit SHA, or branch name (default:main
)true
)Automatic Updates
Version Support
latest
v0.50.0
or0.50.0
main
,feature/branch-name
, etc.Implementation Details
Improvements over previous approach:
Scripts:
generate_service_api_spec.py
: Downloads OpenAPI spec from productiongenerate_python_sdk_docs.py
: Installs Weave version and generates docsgenerate_typescript_sdk_docs.py
: Downloads source and generates docsImportant Notes
pull_request
trigger from the workflow (it's marked with a comment)Checklist