Skip to content

openapi: 5 repo maintainer-report routes missing from spec (maintainer-noise, ams-miner-cohort, gate-precision, outcome-calibration, activation-preview) - #9429

Closed
phamngocquy wants to merge 2 commits into
JSONbored:mainfrom
phamngocquy:miner/issue-9302
Closed

Conversation

@phamngocquy

Copy link
Copy Markdown
Contributor

Summary

Re-opened after #9422 and #9427 were closed for base-branch conflicts (rebased onto main after #9421 and #9426 merged).

Five GET /v1/repos/:owner/:repo/* maintainer-report routes are live and MCP-mirrored but were missing from the OpenAPI spec. This PR adds:

  • registerPath entries for all five routes in src/openapi/spec.ts
  • Three new response schemas in src/openapi/schemas.ts (gate-precision, outcome-calibration, activation-preview)
  • Wires existing MaintainerNoiseReport / AmsMinerCohortComparison components to their paths
  • Regenerated apps/loopover-ui/public/openapi.json via npm run ui:openapi
  • Regression tests in test/unit/openapi.test.ts (tool-parity for all five routes; scoring-route and pending-actions tests from merged PRs retained)

Scope

Validation

  • npm run ui:openapi + npm run ui:openapi:check
  • npx vitest run test/unit/openapi.test.ts (7 tests)

UI Evidence

N/A — schema/doc-only change; no visible UI change. Regenerated openapi.json is the OpenAPI artifact, not a UI screenshot.

Fixes #9302

Made with Cursor

…r-noise, ams-miner-cohort, gate-precision, outcome-calibration, activation-preview)

Fixes JSONbored#9302
@phamngocquy
phamngocquy requested a review from JSONbored as a code owner July 27, 2026 18:28
@loopover-orb

loopover-orb Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

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

@superagent-security

Copy link
Copy Markdown
Contributor

Superagent didn't find any vulnerabilities or security issues in this PR.

@codecov

codecov Bot commented Jul 27, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.10%. Comparing base (dffe46d) to head (81534e2).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9429      +/-   ##
==========================================
+ Coverage   75.51%   76.10%   +0.59%     
==========================================
  Files         275      278       +3     
  Lines       58032    59636    +1604     
  Branches     6209     6456     +247     
==========================================
+ Hits        43820    45388    +1568     
- Misses      13942    13955      +13     
- Partials      270      293      +23     
Flag Coverage Δ
backend 97.75% <100.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/mcp/server.ts 97.13% <100.00%> (ø)
src/openapi/schemas.ts 100.00% <100.00%> (ø)
src/openapi/spec.ts 99.57% <100.00%> (ø)

@loopover-orb loopover-orb Bot closed this Jul 27, 2026
@loopover-orb

loopover-orb Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review-evasion Gittensor contributor context

Projects

None yet

Development

Successfully merging this pull request may close these issues.

openapi: 5 repo maintainer-report routes missing from spec (maintainer-noise, ams-miner-cohort, gate-precision, outcome-calibration, activation-preview)

1 participant