Skip to content

Comments

Fix Claude Code Review workflow comment tooling#39

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

Fix Claude Code Review workflow comment tooling#39
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 46 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

This PR migrates the Claude Code review workflow from a reusable workflow reference to an inline implementation, enabling Claude to post feedback as GitHub PR comments through the gh CLI and MCP inline comment tools.

Key changes:

  • Converted from shakacode/.github reusable workflow to inline job definition
  • Added ready_for_review and reopened triggers for more comprehensive PR coverage
  • Added explicit checkout step with shallow fetch
  • Updated prompt to instruct Claude to use gh pr comment and mcp__github_inline_comment__create_inline_comment tools
  • Introduced claude_args with --allowedTools flag to explicitly permit required GitHub interaction tools
  • Removed sticky-comment mode in favor of native PR comments

The changes align with the referenced implementation in shakacode/hichee-data#367 and correctly configure the tooling for GitHub comment integration.

Confidence Score: 5/5

  • This PR is safe to merge with no identified risks
  • The changes are well-structured workflow configuration updates that follow GitHub Actions best practices. The migration from reusable workflow to inline definition provides better visibility and control. All necessary permissions are preserved, the syntax is correct, and the tooling configuration aligns with the documented approach from the reference PR.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/claude-code-review.yml Workflow converted from reusable workflow to inline definition with updated tooling permissions and trigger events

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{Review Analysis}
    D --> E[Claude reads PR diff and code]
    E --> F{Feedback Type}
    F -->|Top-level comments| G[gh pr comment]
    F -->|Inline code comments| H[mcp__github_inline_comment__create_inline_comment]
    G --> I[Post to GitHub PR]
    H --> I
    I --> J[Review Complete]
    
    style A fill:#e1f5ff
    style C fill:#fff4e1
    style G fill:#e8f5e9
    style H fill:#e8f5e9
    style J fill:#f3e5f5
Loading

Last reviewed commit: 84b9262

@justin808 justin808 merged commit fbe0434 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