Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughThe Claude review workflow now stages pull request context, restricts review tools, pins its action version, prevents bot-triggered reviews, limits oversized inputs and outputs, and posts Claude’s validated final message as a pull request comment. ChangesClaude review workflow
Estimated code review effort: 4 (Complex) | ~45 minutes Possibly related PRs
Suggested reviewers: Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
Tick the box to add this pull request to the merge queue (same as
|
There was a problem hiding this comment.
🟡 Not ready to approve
The new “Post review summary” step can incorrectly skip posting valid short summaries due to a brittle length-based guard.
Once you've addressed the issues Copilot identified, you can request another Copilot review.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
This PR updates the repository’s Claude-based automated code review workflow to align with claude-code-action v7-era behavior by removing reliance on authenticated gh subprocesses and instead staging PR context on disk, pinning the action by SHA, and publishing the final review summary from the action’s execution log.
Changes:
- Adds explicit documentation and workflow structure to ensure no GitHub token is available to Claude subprocesses, and stages PR diff/metadata to
pr-context/for Read/Grep/Glob. - Updates the checkout action version and pins
anthropics/claude-code-actionto a specific commit SHA; removes Bash/gh tools from--allowedTools. - Adds a workflow step to post the final summary by extracting it from
execution_fileviajq.
File summaries
| File | Description |
|---|---|
| .github/workflows/claude-code-review.yml | Reworks the Claude review workflow to stage PR context locally, pin the action SHA, remove gh Bash tools, and post the final summary from execution_file. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
| # A run that errored, hit a permission wall, or produced only | ||
| # boilerplate should stay silent rather than post an empty comment. | ||
| if [ "$(wc -c < "$summary_file")" -lt 40 ]; then | ||
| echo "::warning::Claude produced no usable review summary; skipping the comment" | ||
| echo "--- begin captured summary ---" | ||
| cat "$summary_file" | ||
| echo "--- end captured summary ---" | ||
| exit 0 | ||
| fi |
|
🚀 Deployed on https://6a6e91bcb77416ce6def27eb--opengeos.netlify.app |
Summary by CodeRabbit