Skip to content

Fix Claude Code Review workflow: use built-in review#17

Merged
justin808 merged 3 commits intomainfrom
jg/rm-broken-claude-review
Feb 17, 2026
Merged

Fix Claude Code Review workflow: use built-in review#17
justin808 merged 3 commits intomainfrom
jg/rm-broken-claude-review

Conversation

@justin808
Copy link
Member

@justin808 justin808 commented Feb 17, 2026

Summary

  • Fixes the claude-code-review.yml workflow to use the built-in review instead of the broken plugin approach
  • The plugin-based setup fails because it tries to call gh api via the Bash tool, which the action's sandbox blocks
  • Replaces plugin_marketplaces/plugins/prompt with review_triggered_on: pr and use_sticky_comment: true

Test plan

  • Open a test PR and verify Claude auto-reviews it

🤖 Generated with Claude Code

The plugin-based review setup is broken — the plugin tries to call
gh api via the Bash tool, which the action sandbox blocks by default.
Removing in favor of the working default claude.yml (@claude mentions).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@coderabbitai
Copy link

coderabbitai bot commented Feb 17, 2026

Warning

Rate limit exceeded

@justin808 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 14 minutes and 52 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch jg/rm-broken-claude-review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@greptile-apps
Copy link

greptile-apps bot commented Feb 17, 2026

Greptile Summary

This PR removes the claude-code-review.yml workflow that was attempting to use a plugin-based approach with gh api commands, which were being blocked by the action's sandbox. The removal is appropriate and matches a similar cleanup already merged in the shakacode/.github repository.

  • The deletion is clean with no remaining references or dependencies
  • The functional claude.yml workflow (triggered by @claude mentions) remains intact and unaffected
  • The unit-tests.yml workflow is also unaffected

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The change is a straightforward deletion of a non-functional workflow file with no code logic changes, no dependencies, and no impact on existing functionality
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/claude-code-review.yml Removed broken plugin-based workflow that was blocked by sandbox restrictions; claude.yml remains functional

Last reviewed commit: 6ec6893

…n plugin

Replace the plugin-based review (which fails because the sandbox blocks
Bash tool access) with the built-in review_triggered_on and
use_sticky_comment options.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@justin808 justin808 changed the title Remove broken Claude Code Review workflow Fix Claude Code Review workflow: use built-in review Feb 17, 2026
The action auto-detects review mode from the pull_request trigger.
Only use_sticky_comment and prompt are needed.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@justin808 justin808 merged commit 3b0775e into main Feb 17, 2026
2 of 3 checks passed
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