Skip to content
This repository was archived by the owner on Apr 3, 2026. It is now read-only.

feat: migrate to blocking review workflow - #25

Merged
twistedmelonman merged 2 commits into
mainfrom
claude/migrate-blocking-review-20260320
Mar 21, 2026
Merged

twistedmelonman merged 2 commits into
mainfrom
claude/migrate-blocking-review-20260320

Conversation

@twistedmelonman

Copy link
Copy Markdown
Owner

Summary

Dependency

Requires github-workflows PR #17 to be merged and v1 tag updated first, since this workflow references @v1.

Test plan

  • After github-workflows PR is merged + tagged: verify this PR's CI runs the new blocking review
  • Review completes in <4 minutes with <=6 turns
  • API cost < $0.15
  • Verdict (BLOCK/PASS) appears in PR comment and step summary

🤖 Generated with Claude Code

Claude Code Bot and others added 2 commits March 20, 2026 17:28
Replace plugin-based claude-code-action with the reusable
claude-blocking-review.yml from smartwatermelon/github-workflows.

This adds:
- max_turns limit (default 6) to prevent runaway API usage
- timeout (default 4min) to cap CI cost
- BLOCK/PASS verdict system with merge enforcement
- Scope constraints so review focuses on the diff only

Motivated by PR #23 where the plugin-based review consumed 20 turns,
$1.00, and 11 minutes for a 109-line diff.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Add explicit permissions required by the called reusable workflow:
contents:read, pull-requests:write, issues:write, id-token:write.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@twistedmelonman
twistedmelonman merged commit b2ab9ef into main Mar 21, 2026
10 checks passed
@twistedmelonman
twistedmelonman deleted the claude/migrate-blocking-review-20260320 branch March 21, 2026 01:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant