docs(openapi): document the five repo maintainer-report routes in the spec - #9412
docs(openapi): document the five repo maintainer-report routes in the spec#9412bitfathers94 wants to merge 1 commit into
Conversation
|
Superagent didn't find any vulnerabilities or security issues in this PR. |
|
Important 🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨🟨 ⏳ LoopOver is waiting…LoopOver has seen this pull request and is waiting on CI checks to finish before reviewing it. This comment will update once the review runs. 🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed · 🟨 Waiting |
… spec The maintainer-noise, ams-miner-cohort, gate-precision, outcome-calibration, and activation-preview GET routes are all live and maintainer-authenticated with an MCP tool counterpart, but none appeared in the generated OpenAPI spec. MaintainerNoiseReport and AmsMinerCohortComparison were even registered as components yet never attached to a path. Add registerPath entries for all five, define GatePrecisionResponse, OutcomeCalibrationResponse, and ActivationPreviewResponse schemas mirroring their MCP tools' output shapes field-for-field, and regenerate the committed openapi.json. A regression test asserts each path is documented with a 200 response referencing a component whose keys match its MCP tool shape.
cfc180f to
14a8461
Compare
|
This repository reviews pull requests one-shot: the PR must be correct as originally opened. Pushing an additional commit closes it automatically instead of restarting review — open a fresh pull request with every fix included. |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #9412 +/- ##
==========================================
+ Coverage 75.51% 76.08% +0.57%
==========================================
Files 275 278 +3
Lines 58032 59590 +1558
Branches 6209 6456 +247
==========================================
+ Hits 43820 45342 +1522
- Misses 13942 13955 +13
- Partials 270 293 +23
Flags with carried forward coverage won't be shown. Click here to find out more.
|
1 participant
docs(openapi): document the five repo maintainer-report routes in the spec
The maintainer-noise, ams-miner-cohort, gate-precision, outcome-calibration,
and activation-preview GET routes are all live and maintainer-authenticated with
an MCP tool counterpart, but none appeared in the generated OpenAPI spec.
MaintainerNoiseReport and AmsMinerCohortComparison were even registered as
components yet never attached to a path.
Add registerPath entries for all five, define GatePrecisionResponse,
OutcomeCalibrationResponse, and ActivationPreviewResponse schemas mirroring
their MCP tools' output shapes field-for-field, and regenerate the committed
openapi.json. A regression test asserts each path is documented with a 200
response referencing a component whose keys match its MCP tool shape.
Closes #9302
Validation
Verified locally on this branch before opening:
npm run typechecknpx turbo run build:tsc build:verifynpm run test:coverage— patch coverage 100.0% of changed lines