CtrlK
BlogDocsLog inGet started
Tessl Logo

code-review-and-quality

Conducts multi-axis code review. Use before merging any change. Use when reviewing code written by yourself, another agent, or a human. Use when you need to assess code quality across multiple dimensions before it enters the main branch. Use when asked to review a diff or a pull request, even when the diff is pasted inline.

63

Quality

74%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide

SecuritybySnyk

Low

Low-risk findings worth noting

Fix and improve this skill with Tessl

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

Quality

Content

70%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, highly actionable review policy with an exemplary five-step workflow and concrete checklists. Its weaknesses are verbosity from repeating the same guidance across multiple sections and a broken progressive-disclosure story: two referenced files do not exist and no detail is actually offloaded from the main body.

Suggestions

Create the referenced files or fix the dangling "../../references/security-checklist.md" and "../../references/performance-checklist.md" paths — no references/ directory exists in the bundle.

Move detail-heavy sections (Dependency Discipline upgrade workflow, Change Sizing) into reference files and keep SKILL.md as a lean overview, as the See Also section already implies.

Deduplicate the file-size guidance that appears in Change Sizing, the Review Checklist, Red Flags, Common Rationalizations, and Verification — state it once and reference it.

DimensionReasoningScore

Conciseness

The ~390-line body is mostly non-trivial policy guidance Claude would not inherently know, but it is noticeably repetitive: file-size guidance recurs in Change Sizing, the checklist, Red Flags, Common Rationalizations, and Verification, and the 'bump deps' material likewise appears three times. It is above the padded level 2 but clearly below the trimmed level 4.

3 / 5

Actionability

Concrete, executable artifacts throughout — a severity-prefix table with author actions, numeric sizing thresholds (~100/~300/~1000 lines), a splitting-strategy table, a copy-paste review-agent prompt, and a dead-code report template — with minor abstract gaps like "follow existing patterns". Not level 5 because several checklist and axis bullets remain directional rather than fully executable.

4 / 5

Workflow Clarity

The five-step Review Process is clearly sequenced with explicit checkpoints: severity labeling in Step 4, "Verify the Verification" in Step 5, and a final checkbox checklist covering Critical/Required resolution — matching the level-5 anchor of explicit validation steps, feedback loops, and checklists.

5 / 5

Progressive Disclosure

Sections are well organized and the "See Also" references are clearly signaled, but no bundle exists — the cited "../../references/security-checklist.md" and "../../references/performance-checklist.md" are dangling paths — and content that belongs in separate files (dependency upgrade workflow, change sizing) is fully inlined in the monolithic body. Structure exists but the split and the references are broken, matching level 3 rather than level 4.

3 / 5

Total

15

/

20

Passed

Description

78%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 that explicitly answers both what and when with multiple natural trigger phrases and a distinctive pre-merge review niche. Its main weakness is that the capability statement stays abstract — the multiple review axes are alluded to but never named.

Suggestions

Enumerate the concrete review capabilities (e.g., 'Reviews correctness, security, architecture, readability, and performance') instead of the abstract 'multi-axis' framing.

Add common trigger synonyms such as 'PR' or 'code review' to widen natural-keyword coverage.

DimensionReasoningScore

Specificity

"Conducts multi-axis code review" and "assess code quality across multiple dimensions" name the domain and one to two actions, but they restate the same action rather than enumerating several concrete capabilities (the five axes are never named). It sits above level 2 (which names only a domain with generic actions) but below level 4 (which lists several specific actions).

3 / 5

Completeness

What it does ("Conducts multi-axis code review... assess code quality across multiple dimensions") and when to use it (three explicit "Use when..." clauses, including the edge case "even when the diff is pasted inline") are both clearly and explicitly answered with concrete trigger phrases, matching the level-5 anchor; the level-4 weakness ('when' could be more explicit) does not apply.

5 / 5

Trigger Term Quality

Natural phrases users would say are present — "reviewing code", "review a diff or a pull request", "before merging any change", "code quality" — but common synonyms like "PR", "LGTM", or "critique my code" are missing, so it falls short of comprehensive synonym coverage at level 5.

4 / 5

Distinctiveness Conflict Risk

A clear pre-merge code-review niche with distinct triggers (diff, pull request, merging, agent-written code), but it overlaps slightly with dedicated security- or performance-review skills it itself defers to, so it is mostly rather than fully distinct.

4 / 5

Total

16

/

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
addyosmani/agent-skills
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.