Skip to content

Fix Claude Code Review workflow comment tooling#267

Merged
justin808 merged 1 commit intomainfrom
codex/fix-claude-code-review-tools-20260219
Feb 19, 2026
Merged

Fix Claude Code Review workflow comment tooling#267
justin808 merged 1 commit intomainfrom
codex/fix-claude-code-review-tools-20260219

Conversation

@justin808
Copy link
Member

This ports the Claude Code review workflow fix from shakacode/hichee-data#367:

  • update prompt instructions so Claude posts feedback via GitHub comments
  • allow required tools via claude_args --allowedTools
  • remove sticky-comment mode

This makes Claude review output appear as top-level and inline PR comments.

@coderabbitai
Copy link

coderabbitai bot commented Feb 19, 2026

Warning

Rate limit exceeded

@justin808 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 29 minutes and 34 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 codex/fix-claude-code-review-tools-20260219

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 19, 2026

Greptile Summary

Updated Claude Code Review workflow to use GitHub PR comments instead of sticky comments for better integration.

  • Removed use_sticky_comment: true parameter
  • Enhanced prompt with detailed review focus areas and explicit tool usage instructions
  • Added claude_args with --allowedTools to enable specific GitHub comment and PR viewing tools
  • Made Claude output appear as standard PR comments rather than a single sticky comment

This aligns the workflow with the updated Claude Code action capabilities and improves review visibility.

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • The changes are configuration-only updates to a GitHub Actions workflow that improve the Claude Code review integration. The modifications remove deprecated functionality (sticky comments) and add proper tool permissions. YAML syntax is valid, permissions are appropriate, and the changes follow the established pattern from a proven implementation in another repository.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/claude-code-review.yml Updated workflow to use GitHub comments instead of sticky comments, added detailed prompt instructions and allowed tools configuration

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR Event Triggered] --> B[Checkout Repository]
    B --> C[Run Claude Code Action]
    C --> D{Review Mode}
    D -->|Old: Sticky Comment| E[Single Updatable Comment]
    D -->|New: GitHub Comments| F[Multiple PR Comments]
    F --> G[Top-level via gh pr comment]
    F --> H[Inline via MCP tool]
    C --> I[Parse Allowed Tools]
    I --> J[Enable gh pr commands]
    I --> K[Enable inline comment tool]
    I --> L[Enable gh pr diff/view]
Loading

Last reviewed commit: e216f50

@justin808 justin808 merged commit 66df734 into main Feb 19, 2026
7 of 8 checks passed
@justin808 justin808 deleted the codex/fix-claude-code-review-tools-20260219 branch February 19, 2026 00:49
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