fix(mcp): require visible APPROVED reviews before merge (closes #244) #251

Merged

sysadmin merged 2 commits from feat/issue-244-review-submit-visible into master

2026-07-06 11:22:14 -05:00

Author	SHA1	Message	Date
sysadminandClaude Opus 4.8	9d8ab0a7b5	test: fix PR queue inventory mock for visible APPROVED verification (#244 ) Update test_reviewer_profile_can_proceed_from_inventory_to_review_gates to return APPROVED state on POST and include GET /reviews verification responses required by #244 review submission gates. Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>	2026-07-06 12:21:26 -04:00
sysadmin	a1ba69eebb	fix(mcp): require visible APPROVED reviews before merge (closes #244 ) Gitea expects review event APPROVED, not APPROVE; the wrong event left submissions as PENDING drafts. Verify terminal verdicts after submit, submit pending drafts when needed, and block merge when approval_visible is false.	2026-07-06 11:51:29 -04:00

Author

SHA1

Message

Date

sysadminandClaude Opus 4.8

9d8ab0a7b5

test: fix PR queue inventory mock for visible APPROVED verification (#244 )

Update test_reviewer_profile_can_proceed_from_inventory_to_review_gates
to return APPROVED state on POST and include GET /reviews verification
responses required by #244 review submission gates.

Co-Authored-By: Claude Opus 4.8 (1M context) <[email protected]>

2026-07-06 12:21:26 -04:00

sysadmin

a1ba69eebb

fix(mcp): require visible APPROVED reviews before merge (closes #244 )

Gitea expects review event APPROVED, not APPROVE; the wrong event left
submissions as PENDING drafts. Verify terminal verdicts after submit,
submit pending drafts when needed, and block merge when approval_visible
is false.

2026-07-06 11:51:29 -04:00