Repository navigation
Conversation
…ecking In Claude Code the skill named `code-review` lost review requests to the built-in `code-review`: traces showed the bare name resolving to the bundled reviewer in 4/4 "check my changes" runs, and activation graders counted that as this skill. - Rename the skill to `coderabbit-review`. It now also provides `/coderabbit:coderabbit-review`, replacing the duplicate command. - Description: review or check any code change, verify a fix, or confirm the agent's own changes before reporting done, even when CodeRabbit isn't named; prefer it over the built-in code-review and verify skills; still answer any CodeRabbit CLI question. - Skill body: after changing code, review it once per task before saying it's done; never present a manual reading of the diff as CodeRabbit's review. The code-reviewer subagent is marked for proactive use. - Evals: routing and shell cases with scaffolded repositories and a stand-in CLI; activation graders require this plugin's skill name. Over the suite (2 runs per case) cases fully passed rose from 28 to 32 of 50; the pre-push check loaded this skill 0/2 -> 2/2 and ran CodeRabbit 0/2 -> 2/2; controls stayed clean. Details in evals/RESULTS.md. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Repository: coderabbitai/skills/.coderabbit.yaml Review profile: ASSERTIVE Plan: Enterprise Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
With the old name, generic review requests invoked Claude Code's built-in code-review 12/12 times and CodeRabbit ran 0/12; after the rename this skill ran 11/12. A typed /code-review runs the built-in either way. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Stacked on #44. The goal is to get CodeRabbit invoked whenever the user or the agent wants code reviewed, checked or verified, and to keep it invoked for the rest of the session.
The skill was losing to Claude Code's built-in
code-review. Inclaude plugin evaltraces, "check my changes before I push" calledSkill {"skill": "code-review"}in 4/4 runs. That bare name resolves to the bundled reviewer, notcoderabbit:code-review. The suite's activation graders accepted the bare name, so they had been counting the built-in as this skill.skills/code-reviewbecomesskills/coderabbit-review. The skill now provides/coderabbit:coderabbit-reviewitself, socommands/coderabbit-review.mdis removed (same slash command, no duplicate). The Gemini/coderabbit:reviewcommand is unchanged.code-reviewandverifyskills when CodeRabbit is installed. It still leads with answering any CodeRabbit CLI question. It is 667 characters, well under the 1,024-character limit.--use-creditswithout approval;code-reviewersubagent is marked "use proactively after writing or modifying code".routing-*cases: four checks that don't name CodeRabbit, the agent's own edit, and three controls, with scaffolded Git repos.shell-*cases: a stand-incoderabbitin the run's~/.local/bin, passing only if CodeRabbit actually runs.coderabbit:coderabbit-review.prepare_comparison.pycopies both old and new skill directories.The agent's own edits aren't reached by routing (0/6 across both descriptions). That's handled by the
Stop-hook reminder in #46, which gets 3/3.Sharing the name gained nothing. Same prompts, each run with a shell, a stand-in CLI and 3 repetitions:
/code-reviewruns the built-in under either name.The built-in exists from v2.1.147, and Claude could start it on its own everywhere from v2.1.246. See
evals/RESULTS.md.Affected surfaces
agents/code-reviewer.md, the Gemini TOML text, autofix cross-links, README, DISTRIBUTION_CHANNELS, CHANGELOG and the evals.Public references
Validation
claude plugin validate .passes, both skills' frontmatter parses, andgit diff --checkis clean.claude plugin evalsetup: Claude Code 2.1.282, Sonnet 4.6, Haiku 4.5 judge,--ablation none --scaffold --allow-tools Edit Bash, 2 runs per case, strict graders, Align skills with CLI workflows and add reproducible comparisons #44 head vs this branch:coderabbit reviewwith a shell: 0/2 → 2/2.readiness-auth-deniedat 4/6 (the old description pooled 9/14). The shell pre-push case ran CodeRabbit 2/2.evals/RESULTS.md. These are small samples.Risks
coderabbit skillsinstalls by directory name, so existing installs keep a stalecode-reviewfolder next tocoderabbit-reviewuntil the installer removes it./coderabbit:code-reviewnow uses/coderabbit:coderabbit-review, which was already the documented command.Checklist
SKILL.mdstays focused on activation, routing, domain context, and workflow framing.🤖 Generated with Claude Code