Sync address-review commands and add AGENTS.md/CLAUDE.md#54
Conversation
Pull latest address-review command improvements from react_on_rails and establish agent policy files for this Docusaurus site. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the 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. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (4)
✨ 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 |
|
Cloudflare preview deployed.
|
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: c448e166be
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| # Review body (often contains summary feedback) | ||
| gh api repos/${REPO}/pulls/{PR_NUMBER}/reviews/{REVIEW_ID} | jq '{id: .id, body: .body, state: .state, user: .user.login, html_url: .html_url}' | ||
|
|
||
| # Inline comments for this review | ||
| gh api --paginate repos/${REPO}/pulls/{PR_NUMBER}/reviews/{REVIEW_ID}/comments | jq -s '[.[].[] | {id: .id, node_id: .node_id, path: .path, body: .body, line: .line, start_line: .start_line, user: .user.login, in_reply_to_id: .in_reply_to_id}]' |
There was a problem hiding this comment.
Tag specific review bodies as review summaries
When /address-review is invoked with a #pullrequestreview-... URL and the actionable feedback lives in the review body rather than an inline comment, Step 3 stores that payload as only {id, body, state, user, html_url}. Unlike the full-PR path later in this file, there is no type: "review_summary" marker, so the later reply logic in Step 7 cannot distinguish a review summary from a normal review comment and can route the response through /comments/{COMMENT_ID}/replies using the review ID. That endpoint only accepts review-comment IDs, so this case cannot be replied to correctly.
Useful? React with 👍 / 👎.
|
|
||
| ## Behavioral Defaults | ||
|
|
||
| - When confident in your changes, **commit and push without asking for permission**. Always monitor CI after pushing. |
There was a problem hiding this comment.
Remove the auto-push default that bypasses ask-first rules
This new default tells Claude Code to commit and push immediately whenever it is confident, but the canonical policy added in AGENTS.md requires asking first for .github/workflows/, scripts/, and Docusaurus config changes and waiting for the first full review pass before follow-up pushes. On tasks that touch those areas, Claude will now be instructed to violate the repo's own safety boundaries instead of pausing for approval.
Useful? React with 👍 / 👎.
Summary
Pulls over the latest address-review command improvements from the react_on_rails repository and establishes agent policy files (AGENTS.md, CLAUDE.md) for this Docusaurus site.
What changed
.claude/commands/address-review.md— Updated to the latest version from react_on_rails main. Key improvements:$ARGUMENTSvariable for Claude Code template input--paginateon all REST API calls to handle multi-page responsesnode_id/thread_idtracking via REST + GraphQL for reliable thread resolution/replies)comments(first:100)inner pagination limitation documentedin_reply_to_idused as thread context instead of silently dropped.agents/workflows/address-review.md— New shared prompt for Codex CLI, ChatGPT, Copilot, Cursor, and other non-Claude tools. Mirrors the Claude/address-reviewworkflow with behavior rules for assistants that may lack terminal access.AGENTS.md— New canonical agent policy adapted for this Docusaurus site. Includes Reusable Workflows section, project-specific commands, content flow, project structure, review workflow, and boundaries.CLAUDE.md— New Claude-specific guidance that defers to AGENTS.md as the canonical source, with behavioral defaults and git safety rules.Source PRs from react_on_rails
These changes incorporate work from the following merged PRs in shakacode/react_on_rails:
$ARGUMENTS, review body fetch,node_id/thread_idtracking,--paginate+jq -s, GraphQL thread metadata query, known limitations section.agents/workflows/address-review.mdshared prompt for non-Claude tools, established the.agents/workflows/convention, added "Reusable Workflows" section to AGENTS.mdEarlier foundational PRs:
Design decisions
.claude/commands/address-review.mdfor Claude Code slash commands,.agents/workflows/address-review.mdfor all other tools — following the convention established in react_on_rails PR #2761Test plan
/address-reviewslash command works in Claude Code against a PR in this repo.agents/workflows/address-review.mdprompt works when pasted into Codex CLI🤖 Generated with Claude Code
Note
Low Risk
Documentation/workflow-only changes affecting AI agent prompts and repo guidance; no runtime code or build logic is modified.
Overview
Updates the Claude
/address-reviewcommand to a more robust workflow: supports PR/review/comment URLs, paginates GitHub API fetches, includes review summary bodies and general PR comments, mapsnode_idto GraphQLthread_idfor resolving threads, and refines triage/UX (reply handling, sequential numbering, optional rationale replies).Adds a non-Claude equivalent prompt in
.agents/workflows/address-review.md, and introducesAGENTS.md(canonical agent policy for this Docusaurus repo) plusCLAUDE.md(Claude-specific guidance that defers toAGENTS.md).Written by Cursor Bugbot for commit c448e16. This will update automatically on new commits. Configure here.