Skip to content

feat(spec-drift): PR-mode coverage check (2/3)#10

Open
andrewyu47 wants to merge 1 commit into
split/8a-spec-diff-enginefrom
split/8b-coverage-check
Open

feat(spec-drift): PR-mode coverage check (2/3)#10
andrewyu47 wants to merge 1 commit into
split/8a-spec-diff-enginefrom
split/8b-coverage-check

Conversation

@andrewyu47
Copy link
Copy Markdown

Part of splitting #8 into a reviewable stack. PR 2 of 3 — stacked on #9.

Base is split/8a-spec-diff-engine, so review #9 first. The diff shown here is only the coverage-check layer.

  • .github/scripts/check_spec_coverage.py — reads spec-diff.json (produced by the engine in feat(spec-drift): OpenAPI diff + classification engine (1/3) #9) and joins it with spec-manifest.json to build the PR-comment checklist of affected docs pages + impacted SDKs (breaking changes first). Unmapped surface is logged to spec-gaps-unmapped.json.
  • .github/scripts/tests/test_check_spec_coverage.py — pytest case (passing locally).

Stack: #9 (engine) -> this -> PR 3 (opener + workflow).

Umbrella: #8

Made with Cursor

check_spec_coverage.py reads spec-diff.json (from the engine) and joins it
against spec-manifest.json to produce the PR-comment checklist of affected
docs.pinecone.io pages and impacted SDKs, breaking changes first. Unmapped
surface is logged to spec-gaps-unmapped.json. Adds a pytest case.

Part of splitting #8 (2/3, stacked on 8a).

Co-authored-by: Cursor <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant