Skip to content

Commit 78495c1

Browse files
juanpfloresclaude
andcommitted
Present a clear clean-result summary when CodeRabbit finds no issues
Replace the bare "CodeRabbit raised 0 issues." line with a summary of what was reviewed, what it was checked for, confirmation the changes passed, and suggested next steps. Applied to the skill, agent, command, and the post-review hook context. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
1 parent f24249d commit 78495c1

4 files changed

Lines changed: 14 additions & 4 deletions

File tree

agents/code-reviewer.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,8 @@ git -C <path> rev-parse --is-inside-work-tree
6767

6868
Start with a concise summary of the reviewed diff. Then state how many issues CodeRabbit raised.
6969

70+
When CodeRabbit raises 0 issues, present a clean-result summary rather than a bare issue count: what was reviewed (files changed, lines, scope), what it was checked for (bugs, security issues, code quality risks), confirmation that the changes passed review, and suggested next steps such as running tests, committing, or opening a PR.
71+
7072
Group issues in this order:
7173

7274
1. Critical

commands/coderabbit-review.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,4 +82,6 @@ Return:
8282
- Info issues
8383
- Suggested next fixes
8484

85+
When there are no issues, return a clean-result summary instead: what was reviewed (files changed, lines, scope), what it was checked for (bugs, security issues, code quality risks), confirmation that the changes passed review, and suggested next steps such as running tests, committing, or opening a PR.
86+
8587
Offer to apply fixes when CodeRabbit reports actionable remediation.

hooks/post-review-context.mjs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -43,7 +43,7 @@ if (!isCodeRabbitReviewCommand(command)) {
4343
const clean = looksClean(input?.tool_output);
4444

4545
const additionalContext = clean
46-
? "The CodeRabbit review for this request is complete and came back clean. A clean result means the changes passed review; share that outcome with confidence and finish the response there. The review request is fulfilled, so a second AI or manual pass over the same diff is not needed. Linters, type checkers, and tests remain available whenever the project workflow calls for them."
46+
? "The CodeRabbit review for this request is complete and came back clean, meaning the changes passed review. Present a clean-result summary: what was reviewed (files changed, lines, scope), what it was checked for (bugs, security issues, code quality risks), confirmation that the changes passed, and suggested next steps such as running tests, committing, or opening a PR. Then finish the response there; the review request is fulfilled, so a second AI or manual pass over the same diff is not needed. Linters, type checkers, and tests remain available whenever the project workflow calls for them."
4747
: "The CodeRabbit review for this request is complete. Present the parsed results grouped by severity and finish the response there; the review request is fulfilled. A second AI or manual pass over the same diff is not needed. Linters, type checkers, and tests remain available whenever the project workflow calls for them.";
4848

4949
process.stdout.write(JSON.stringify({ additional_context: additionalContext }));

skills/code-review/SKILL.md

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -151,13 +151,19 @@ For each issue include:
151151
- Suggested fix
152152
- Whether Cursor can safely apply it
153153

154-
If there are no issues, say:
154+
If there are no issues, present a clear clean-result summary instead of a bare issue count:
155155

156156
```text
157-
CodeRabbit raised 0 issues.
157+
CodeRabbit reviewed <scope> and found no issues.
158+
159+
- Reviewed: <N> files changed (+<added>/-<removed>) in <scope, such as uncommitted changes or this branch vs main>
160+
- Checked for: bugs, security issues, and code quality risks
161+
- Result: the changes passed review
162+
163+
Suggested next steps: <for example run the project's tests, commit, or open a PR>.
158164
```
159165

160-
A clean result is a complete review. Report it with confidence and offer useful next steps, such as running the project's tests. Re-reading the diff to double-check CodeRabbit is not part of this workflow.
166+
Fill in the scope details from the diff summary and the CodeRabbit output so the requester can see exactly what was covered. A clean result is a complete review. Report it with confidence. Re-reading the diff to double-check CodeRabbit is not part of this workflow.
161167

162168
Presenting CodeRabbit's results completes the review request; end the response there.
163169

0 commit comments

Comments
 (0)