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.

62

Quality

72%

Does it follow best practices?

Impact

No eval scenarios have been run

SecuritybySnyk

Passed

No known issues

Optimize this skill with Tessl

npx tessl skill review --optimize ./skills/code-review-and-quality/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Discovery

67%

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

The description has strong completeness with clear 'what' and 'when' clauses covering multiple trigger scenarios. However, it lacks specificity about what 'multi-axis' actually entails—listing the concrete dimensions (e.g., security, performance, readability, test coverage) would significantly improve both specificity and distinctiveness. Trigger terms are adequate but miss common user phrasings like 'PR review' or 'pull request.'

Suggestions

List the specific review axes/dimensions (e.g., 'Evaluates security vulnerabilities, performance bottlenecks, readability, test coverage, and style consistency') to improve specificity.

Add common trigger term variations such as 'PR review,' 'pull request,' 'diff review,' or 'code feedback' to improve discoverability.

DimensionReasoningScore

Specificity

It names the domain ('code review') and mentions 'multi-axis' and 'multiple dimensions,' but does not list the specific actions or axes involved (e.g., security checks, performance analysis, style linting). The concrete actions remain vague.

2 / 3

Completeness

Clearly answers both 'what' (conducts multi-axis code review, assesses code quality across multiple dimensions) and 'when' (before merging any change, when reviewing code written by yourself/another agent/a human, when assessing code quality before it enters the main branch) with explicit trigger guidance.

3 / 3

Trigger Term Quality

Includes relevant terms like 'code review,' 'merging,' 'code quality,' and 'main branch,' which users might naturally say. However, it misses common variations like 'PR review,' 'pull request,' 'diff review,' 'code feedback,' or 'lint.'

2 / 3

Distinctiveness Conflict Risk

The 'code review' niche is reasonably specific, but 'multi-axis' is vague enough that it could overlap with other code quality, linting, or security-focused skills. Without naming the specific axes or distinguishing features, there's moderate conflict risk with adjacent skills.

2 / 3

Total

9

/

12

Passed

Implementation

77%

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

This is a well-structured, highly actionable code review skill that provides a comprehensive multi-axis framework with clear workflows and concrete guidance. Its main weakness is length — at 300+ lines it includes some redundant sections (Red Flags overlaps with Common Rationalizations and earlier content) and could be more concise by trimming explanations of concepts Claude already understands. The progressive disclosure structure is partially implemented with references to external files, but those files don't exist in the bundle.

Suggestions

Trim redundant sections: 'Common Rationalizations' and 'Red Flags' overlap with guidance already stated in the review process and honesty sections — consolidate into one compact table.

Move supplementary content (Change Sizing, Change Descriptions, Dependency Discipline) into referenced files to keep the core SKILL.md focused on the review workflow itself.

Provide the referenced bundle files (references/security-checklist.md, references/performance-checklist.md) or remove the references to avoid broken navigation.

DimensionReasoningScore

Conciseness

The skill is quite long (~300+ lines) and includes some content Claude already knows (e.g., what N+1 queries are, what XSS is, general code review principles). Several sections like 'Common Rationalizations' and 'Red Flags' overlap significantly. However, much of the content provides genuinely useful structure (the five-axis framework, severity labels, checklist) that earns its place.

2 / 3

Actionability

The skill provides highly concrete, actionable guidance: a specific five-axis review framework, severity label prefixes with clear definitions, a copy-paste-ready review checklist, change sizing guidelines with exact line counts, splitting strategies in a table, and a step-by-step review process. The dead code example and multi-model review pattern are specific and executable.

3 / 3

Workflow Clarity

The five-step review process is clearly sequenced (understand context → review tests → review implementation → categorize findings → verify verification). The severity labeling system provides clear decision criteria. The final verification checklist serves as an explicit quality gate before merge approval, and the categorization step creates a feedback loop for the author.

3 / 3

Progressive Disclosure

The skill references external files (`security-and-hardening`, `performance-optimization`, `references/security-checklist.md`, `references/performance-checklist.md`) which is good progressive disclosure design, but no bundle files are provided to support these references. The main file itself is quite long and could benefit from splitting — the change sizing, change descriptions, and dependency discipline sections could be separate reference files to keep the core review workflow leaner.

2 / 3

Total

10

/

12

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.

Validation11 / 11 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.