Skip to content

feat(openapi): document GET pulls/:number/ai-review-findings - #9404

Closed
philluiz2323 wants to merge 1 commit into
JSONbored:mainfrom
philluiz2323:fix/openapi-ai-review-findings-9305
Closed

feat(openapi): document GET pulls/:number/ai-review-findings#9404
philluiz2323 wants to merge 1 commit into
JSONbored:mainfrom
philluiz2323:fix/openapi-ai-review-findings-9305

Conversation

@philluiz2323

Copy link
Copy Markdown
Contributor

What

GET /v1/repos/:owner/:repo/pulls/:number/ai-review-findings is the only sibling of maintainer-packet and reviewability missing from the OpenAPI contract, despite sharing the same path family, auth boundary, and MCP-tool parity (loopover_get_pr_ai_review_findings / prAiReviewFindingsOutputSchema).

Change

  • Add PullRequestAiReviewFindingsSchema in src/openapi/schemas.ts matching prAiReviewFindingsOutputSchema
  • Register the schema and GET .../ai-review-findings path in src/openapi/spec.ts next to its siblings
  • Assert path + schema keys in test/unit/openapi.test.ts
  • Regenerate apps/loopover-ui/public/openapi.json

Closes #9305

…ed#9305)

ai-review-findings is the only sibling of maintainer-packet and reviewability
missing from the OpenAPI contract. Add PullRequestAiReviewFindingsSchema from
prAiReviewFindingsOutputSchema, register the path, assert it in openapi.test,
and regenerate apps/loopover-ui/public/openapi.json.

Closes JSONbored#9305
@philluiz2323
philluiz2323 requested a review from JSONbored as a code owner July 27, 2026 16:18
@superagent-security

Copy link
Copy Markdown
Contributor

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

@philluiz2323

Copy link
Copy Markdown
Contributor Author

Superseded: base advanced with openapi.json changes from #9389; opening a fresh PR rebased on current main.

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.

openapi: pulls/:number/ai-review-findings missing from spec, unlike its maintainer-packet + reviewability siblings

1 participant