CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review-specialist

Comprehensive code review with security, performance, and quality analysis. Use when users ask to review code, analyze code quality, evaluate pull requests, or mention code review, security analysis, or performance optimization.

Quality

67%

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 ./03-skills/code-review-specialist/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

46%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.

The body is a reasonably organized catalog with a useful output template, but it is undermined by references to two bundle files that do not exist, no invocation examples for the scripts that do, and a padded version-history/footer full of time-sensitive metadata. The skill tells Claude what to look for more than how to proceed.

Suggestions

Create the missing 'templates/review-checklist.md' and 'templates/finding-template.md' (or remove the references and inline the essential checklist items) — instructions to read nonexistent files are the body's most damaging defect.

Add concrete invocation examples for the two scripts, e.g. 'python scripts/analyze-metrics.py <file-under-review>' and 'python scripts/compare-complexity.py <before> <after>', so the metrics-gathering step is executable.

Delete the Version History, Last Updated, Claude Code Version, Sources, and Compatible Models footer — this time-sensitive metadata penalizes conciseness and adds nothing Claude needs to perform a review.

DimensionReasoningScore

Conciseness

The core sections (category outline, reference list, review template) are reasonably lean and scannable, but the four category bullet-lists enumerate concepts Claude already knows (SOLID principles, Big O, naming conventions, injection vulnerabilities), and the footer pads the file with time-sensitive noise — 'Last Updated: August 4, 2026', 'v1.0.0 (2024-12-10)', 'Claude Code Version: 2.1.220', and a 'Compatible Models' list — which the guidelines explicitly penalize outside a deprecated/old-patterns section. Not anchor 2 because there is no prose-style over-explanation and the structure stays tight.

3 / 5

Actionability

The Review Template gives a concrete output contract (Summary, Critical Issues with Issue/Location/Impact/Severity/Fix, findings by category), but two of the four 'Reference Files' entries point to files that do not exist — 'templates/review-checklist.md' ('Read this file and use it as a guide') and 'templates/finding-template.md' ('use its format when reporting issues') — and the two scripts that do exist are described but never invoked with a concrete command line. Anchor 3 ('some concrete guidance but incomplete; missing key details') fits: guidance is partially executable but broken where it depends on missing files.

3 / 5

Workflow Clarity

There is no explicit sequence for conducting a review — the body is a category catalog plus an output format, with no ordering of when to read references, run 'analyze-metrics.py' vs 'compare-complexity.py', or how to combine their output into the report. The implied flow (read references → review → report in template form) matches anchor 3 ('sequence present but checkpoints missing or implicit'), not anchor 4 which requires most checkpoints to be explicit. Code review is read-only, so the destructive/batch validation cap does not apply.

3 / 5

Progressive Disclosure

The bundle structure is partially sound — 'scripts/analyze-metrics.py' and 'scripts/compare-complexity.py' exist and are one level deep with clear descriptions — but the body's navigation is broken: 'templates/review-checklist.md' and 'templates/finding-template.md' are referenced with read/use instructions yet no templates/ directory exists. References that lead nowhere are worse than the anchor-3 'references present but not clearly signaled' case, landing at anchor 2 ('minimal structure; references... buried/broken'), while anchor 1's nested-reference failure mode is not present.

2 / 5

Total

11

/

20

Passed

Description

71%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 solid, well-formed description with an explicit 'Use when' clause, third-person voice, and natural trigger phrases. Its main weaknesses are a slightly generic 'what' statement, missing common trigger synonyms, and two over-broad triggers ('security analysis', 'performance optimization') that raise conflict risk with adjacent skills.

DimensionReasoningScore

Specificity

The description lists several concrete analysis areas — 'security, performance, and quality analysis' — rather than vague filler, though the actions are analysis-domain names rather than fully concrete verbs, and 'Comprehensive' is mild buzzword padding. Anchor 4 ('lists several specific actions; minor gaps in coverage') fits; it is not anchor 5 because it lacks the exhaustive, verb-driven action list, and not anchor 3 because it goes beyond naming a domain with 1-2 generic actions.

4 / 5

Completeness

Both halves are present: what ('Comprehensive code review with security, performance, and quality analysis') and an explicit when ('Use when users ask to review code... or mention code review, security analysis, or performance optimization'). It falls just below anchor 5 because the 'what' is a single generic noun phrase whose specificity rests entirely on the three analysis areas — less explicit than the anchor-5 exemplar's action enumeration.

4 / 5

Trigger Term Quality

Trigger phrases are natural user language: 'review code', 'analyze code quality', 'evaluate pull requests', 'code review', 'security analysis', 'performance optimization'. Good coverage, but common variations like 'PR review', 'refactoring check', or file/extension cues are missing, so it stops short of the anchor-5 'synonyms and variations' level.

4 / 5

Distinctiveness Conflict Risk

Code review is a recognizable niche, but the triggers 'security analysis' and 'performance optimization' are broad enough to fire on requests that are not code reviews at all (e.g., app-config security review, general optimization help), and it overlaps with lint/refactor/test-review skills. Anchor 3 ('somewhat specific but could still overlap with similar skills') is the best fit — anchor 4's 'minor overlap risk' understates the catch-all breadth of those two trigger terms.

3 / 5

Total

15

/

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

luongnv89/claude-howto

Path

03-skills/code-review-specialist/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.