Skip to content

chore: bump claude-blocking-review @v2.0.2 → @v3 - #67

Merged
twistedmelonman merged 2 commits into
mainfrom
claude/bump-claude-blocking-review-v3-20260428
Apr 29, 2026
Merged

chore: bump claude-blocking-review @v2.0.2 → @v3#67
twistedmelonman merged 2 commits into
mainfrom
claude/bump-claude-blocking-review-v3-20260428

Conversation

@twistedmelonman

Copy link
Copy Markdown
Owner

Summary

Bumps the pinned version of smartwatermelon/github-workflows/.github/workflows/claude-blocking-review.yml from @v2.0.2 to @v3 (released today via github-workflows#62).

What v3 changed

  • Drops the --max-turns cap on the Claude review agent. Reviews are now bounded only by the wall-clock timeout-minutes (the hard safety net), the prompt's scope discipline, and the OAuth subscription quota. The "exceeded turn limit" failure mode that recently force-killed reviews on small/medium diffs is gone.
  • Removes the max_turns input parameter. This caller workflow does not pass it, so no other change is required.
  • Re-bases timeout estimation on diff size directly: 10 + lines/100 minutes (floor 10, cap 30).
  • Tightens the prompt with explicit self-bounding (3-file ceiling per hypothesis) since it's now the soft constraint where the turn cap used to be the hard one.

Test plan

  • CI on this PR exercises the bumped workflow on a workflow-only diff — the workflow's PRIORITY SKIP path should fire (claude-code-action refuses to review PRs that modify workflows). Status check should pass quickly.
  • Next non-workflow PR after merge gets the real v3 review.

🤖 Generated with Claude Code

@netlify

netlify Bot commented Apr 28, 2026

Copy link
Copy Markdown

Deploy Preview for projectinsomnia ready!

Name Link
🔨 Latest commit 9eb2e00
🔍 Latest deploy log https://app.netlify.com/projects/projectinsomnia/deploys/69f20dd4cff0d2000813df44
😎 Deploy Preview https://deploy-preview-67--projectinsomnia.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@twistedmelonman
twistedmelonman marked this pull request as ready for review April 29, 2026 13:55
@twistedmelonman
twistedmelonman merged commit 029a4c0 into main Apr 29, 2026
6 checks passed
@twistedmelonman
twistedmelonman deleted the claude/bump-claude-blocking-review-v3-20260428 branch April 29, 2026 13:57
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.

1 participant