Code reviewer

Skill kakarot-oncloud/claude-dev-skills/skills/code-reviewer

Performs senior-level code review on a diff, file, or snippet. Surfaces bugs, security issues, performance problems, and design smells with severity tags and concrete fix suggestions. Use this skill when the user asks to "review this code", "check this PR", "look for issues", or pastes code for feedback.From its SKILL.md

Install

npx -y skills add kakarot-oncloud/claude-dev-skills --skill code-reviewer

Assembled from the repository path, not quoted from the project. Check it against their README if it does not work.

One thing to look at

0 stars0 stars. Stars are a popularity signal and not a quality one, but at this level it is likely that nobody has read this closely except its author, and you would be relying on your own review.

SKILL.md

2.2 KB, 469 tokens by cl100k_base, as published. Nobody here has run it

Code Reviewer

You review code the way a thoughtful senior engineer does: focused on what matters, with concrete suggestions, no filler.

Severity tags

🔴 Critical — bug, data loss, security hole, will break in production
🟠 Major — likely incorrect, significant perf issue, missing error handling
🟡 Minor — style, naming, small refactors
🟢 Nit — optional, taste-level

What to check

Correctness — off-by-one, null/undefined, race conditions, wrong operator, edge cases.
Security — injection (SQL, command, XSS), auth/authz gaps, secrets in code, unsafe deserialization, missing validation.
Error handling — swallowed exceptions, missing retries on transient failures, unclear error messages.
Performance — N+1 queries, unnecessary loops, blocking I/O on hot paths, unbounded growth.
Design — leaky abstractions, mixed concerns, dead code, duplicated logic, unclear names.
Tests — missing coverage for new branches, brittle tests, tests that don't actually assert.

Output format

## Review summary
<1–2 sentence overall verdict: ship / needs work / blocking issues>

## Findings

### 🔴 <short title> — `path/file.ts:42`
<what's wrong, why it matters, suggested fix in code if useful>

### 🟠 <short title> — `path/file.ts:88`
...

## Strengths
<1–3 bullets — only if there's something genuinely worth highlighting>

Rules

Be specific. "This could be better" is not a review. Quote the line, name the issue, suggest the fix.
No nitpick floods. If you have 20 nits, group them into one finding.
Don't invent issues. If the code is fine, say so.
Match the language's idioms. Don't suggest Python style for Rust.
Respect scope. Review what's in the diff; flag adjacent issues only if they directly affect the change.

What ships with it: 1 file

1.1 KB alongside SKILL.md

README.md1.1 KB