Add public API documentation coverage workflow - #530
Open
yuecideng wants to merge 3 commits into
Open
Conversation
Greptile SummaryThe PR adds a read-only public API documentation coverage checker and integrates it into CI and contributor workflows.
Confidence Score: 5/5The PR appears safe to merge. No blocking failure remains.
|
| Filename | Overview |
|---|---|
| docs/scripts/check_api_docs.py | Adds static public-export discovery, Sphinx directive parsing, coverage comparison, and text/JSON reporting; the previously reported nested-export omission is no longer present. |
| tests/docs/test_check_api_docs.py | Adds focused regression coverage for branch-scoped exports, nested-scope exclusion, directive parsing, generated-page exclusion, reporting, and read-only behavior. |
| .github/workflows/main.yml | Runs the API documentation coverage gate after formatting checks in the lint job. |
| docs/source/api_reference/public_api.rst | Adds the agent-maintained fallback API supplement used to cover exports without curated reference entries. |
| embodichain/lab/gym/envs/tasks/init.py | Removes the deprecated task compatibility shim that originally exposed the prior branch-scoped __all__ issue. |
Reviews (3): Last reviewed commit: "refactor(gym): remove legacy task import..." | Re-trigger Greptile
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
This PR adds a read-only public API documentation coverage workflow based on static
__all__declarations.It introduces:
docs/scripts/check_api_docs.pyfor dependency-free structural coverage checks with text and JSON reports$update-api-docsfor AI-driven API-reference generation and docstring improvementembodichain.lab.gym.envs.taskscompatibility module and its stale development guidanceThe checker deliberately performs no writes. When it reports a missing export, the specialized skill owns generation and validation.
Migration: import official task environments from
embodichain_tasks; the legacyembodichain.lab.gym.envs.taskspath is no longer available.Dependencies: None.
Type of change
Screenshots
Not applicable.
Validation
black --check --diff --color ./— 681 files unchangedpytest tests/docs -q --confcutdir=tests/docs— 20 passedpytest tests/test_release_metadata.py -q— 5 passedpython docs/scripts/check_api_docs.py— 1027/1027 exports documentedquick_validate.py— affected agent skills validpython -m sphinx -Q -b dummy docs/source docs/build/api-docs-remove-tasks— passedgit diff --check— passedThe full simulation/GPU test suite was not run because this change affects documentation tooling, package compatibility guidance, and agent workflows rather than simulation behavior.
Checklist
black .command to format the code base.python docs/scripts/check_api_docs.py), if applicable