Skip to content

Add compatible structured output to GitHub PR review (#70) - #526

Merged
amirbena merged 2 commits into
mainfrom
feat/github-pr-review-structured-output
Sep 24, 2026
Merged

amirbena merged 2 commits into
mainfrom
feat/github-pr-review-structured-output

Conversation

@amirbena

Copy link
Copy Markdown
Owner

Fixes #70

What

  • Behavior / contracts: github-pr-review can return the shared review-result document on explicit request, with reviewed head SHA, base/merge-base and the derived decision populated. Caller-only and never published; not the Research and design GitHub-native review enforcement #49 commit status. Canonical: skills/github-pr-review/policies/structured-output.md.
  • Governance / policy: None
  • Packaging / portability: Policy registered under reviewer-assist; package manifest regenerated. The document shape is stated inline so the packaged Skill is self-sufficient.
  • Changelog: generated at release from the two lines below — never edit CHANGELOG.md (see docs/RELEASE.md)
  • Release category: Added
  • Release entry: GitHub PR review can return a machine-readable review result on request.

Validation

  • Relevant validation was run, or the reason it could not be run is stated.
  • Full suite passed on the first run (4954); after the final commit tests/policy and tests/integration pass. A later full run had one failure in the live benchmark test dd-blocking-p1-inverted-error-rate, judged flaky and unrelated; not re-run.
  • Local emission (Add structured output to local review #69) is not implemented yet; "same schema" is defined by docs/review-result/.

Review

Local review of the diff: findings F1–F4 fixed, re-review clean at the fix commit.

🤖 Generated with Claude Code

amirbena and others added 2 commits September 24, 2026 16:05
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…ule, drift tests (#70)

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@amirbena amirbena self-assigned this Sep 24, 2026
@amirbena
amirbena merged commit 0336bb7 into main Sep 24, 2026
7 checks passed
@amirbena
amirbena deleted the feat/github-pr-review-structured-output branch September 24, 2026 13:17
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.

Add compatible structured output to GitHub PR review

1 participant