Skip to content

Comments

Fix Claude Code Review workflow comment tooling#22

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

Fix Claude Code Review workflow comment tooling#22
justin808 merged 1 commit intomasterfrom
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 26 minutes and 55 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 the Claude Code Review workflow to use inline job definition instead of a reusable workflow, enabling direct control over Claude's comment tooling. Key improvements include:

  • Added ready_for_review and reopened trigger types for better PR coverage
  • Upgraded to actions/checkout@v6 from the reusable workflow pattern
  • Introduced explicit prompt instructions directing Claude to use GitHub comment tools
  • Configured --allowedTools to grant Claude access to mcp__github_inline_comment__create_inline_comment and specific gh pr commands
  • Removed dependency on centralized workflow, making this configuration self-contained

The workflow syntax is correct and the permissions are properly configured. This change aligns with the referenced fix from shakacode/hichee-data#367.

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The workflow changes are well-structured, use proper GitHub Actions syntax, maintain all necessary permissions, and implement a tested pattern from another repository. The changes are configuration-only with no impact on the main codebase functionality.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/claude-code-review.yml Migrates from reusable workflow to inline definition with explicit tool permissions and improved trigger types

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[PR Event: opened/synchronize/ready_for_review/reopened] --> B[Checkout repository]
    B --> C[Run Claude Code Review Action]
    C --> D{Analyze PR with Claude}
    D --> E[Use gh pr comment for top-level feedback]
    D --> F[Use inline comment tool for code-specific issues]
    E --> G[Post GitHub Comments]
    F --> G
    G --> H[Review Complete]
Loading

Last reviewed commit: fb33df5

@justin808 justin808 merged commit 9c4cad6 into master Feb 19, 2026
2 of 6 checks passed
@justin808 justin808 deleted the codex/fix-claude-code-review-tools-20260219 branch February 19, 2026 00:48
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