CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review

Perform thorough code reviews with security, performance, and maintainability analysis. Use when user asks to review code, check for bugs, or audit a codebase.

Quality

80%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./skills/code-review/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

67%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

A well-structured, actionable review guide with concrete commands and a clear output format. Its main weakness is token efficiency: a persona opener and a section re-teaching basic vulnerability patterns Claude already knows, plus duplicated grep examples.

Suggestions

Delete the 'You now have expertise... Follow this structured approach' persona opener and start directly with the checklist.

Cut or drastically trim 'Common Patterns to Flag' — Claude already knows SQL injection, eval, and mutable default arguments — or move it to a reference file for rare cases, keeping only project-specific patterns.

Deduplicate the password/secret/token grep commands, which appear in both the Security section and 'Review Commands'.

DimensionReasoningScore

Conciseness

The body is mostly checklist- and command-driven, but the 'You now have expertise...' persona opener and the 'Common Patterns to Flag' section re-teach concepts Claude already knows (SQL injection, eval, mutable default arguments, callback hell), and the password/secret grep appears in two places. It is not severely padded enough for level 2, but the re-taught basics and duplication keep it from level 4.

3 / 5

Actionability

Concrete, executable commands (npm audit, radon cc, git diff) and a copy-paste-ready output-format template cover the common cases, with minor gaps such as bare grep patterns lacking context on how to act on hits. It falls just short of level 5's fully-comprehensive coverage of common cases.

4 / 5

Workflow Clarity

The 'Review Workflow' section gives a clear 7-step sequence with checkpoints present (run the code, check tests pass, run automated scans), but they are implicit rather than explicit validate-then-proceed gates with error-recovery loops, matching level 4 rather than level 5.

4 / 5

Progressive Disclosure

No bundle files exist, and the body is well-organized into clear sections (checklist, output format, patterns, commands, workflow) with no buried references. The language-specific 'Common Patterns' section could arguably live in a separate reference file, which is the minor organization gap that keeps it below level 5.

4 / 5

Total

15

/

20

Passed

Description

83%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

A strong description in third-person voice with an explicit 'Use when...' clause covering natural trigger phrases. Its main gaps are minor: trigger coverage could include more synonyms (PR, diff, security review) and the 'what' could enumerate more concrete actions.

Suggestions

Add common trigger synonyms such as 'PR review', 'review my diff/changes', or 'security review' to the 'Use when' clause.

Enumerate one or two more concrete actions (e.g., 'run dependency audits, check test coverage') to push specificity toward comprehensive coverage.

DimensionReasoningScore

Specificity

The description lists several specific analysis dimensions — 'security, performance, and maintainability analysis' — but stops short of comprehensive concrete actions (no mention of dependency audits, testing, or output format), matching the 'several specific actions; minor gaps in coverage' anchor rather than the level-5 'comprehensive coverage'.

4 / 5

Completeness

It explicitly answers 'what' ('Perform thorough code reviews with security, performance, and maintainability analysis') and 'when' with concrete trigger phrases ('Use when user asks to review code, check for bugs, or audit a codebase'), matching the level-5 anchor; it exceeds level 4, whose 'when' is present but less explicit.

5 / 5

Trigger Term Quality

'review code, check for bugs, or audit a codebase' are natural phrases users would say, giving good keyword coverage, but common variations like 'PR review', 'diff', 'security review', or 'look over my changes' are missing, so it is not comprehensive per the level-5 anchor.

4 / 5

Distinctiveness Conflict Risk

Code review is a distinct niche with clear triggers, but 'check for bugs' and 'audit a codebase' could overlap with security-audit or debugging skills, so there is minor overlap risk with closely related skills rather than the minimal conflict of level 5.

4 / 5

Total

17

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

shareAI-lab/learn-claude-code

Path

skills/code-review/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.