CtrlK
BlogDocsLog inGet started
Tessl Logo

code-reviewer

Provides structured code review against plans and standards. Use when a feature is complete and needs validation, when reviewing code before merge, or when assessing quality and test coverage—e.g., "finished step X", "ready for review", "validate architecture", "check quality and tests".

89

1.71x
Quality

87%

Does it follow best practices?

Impact

89%

1.71x

Average score across 3 eval scenarios

SecuritybySnyk

Passed

No known issues

SKILL.md
Quality
Evals
Security

Quality

Discovery

89%

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

This is a well-crafted description that excels at completeness and trigger term quality with explicit 'Use when' guidance and natural user phrases. The main weakness is that the capabilities could be more specific—listing concrete review actions rather than general terms like 'validation' and 'assessing quality'. Overall, it should effectively differentiate itself in a skill selection context.

Suggestions

Add more specific concrete actions like 'checks adherence to coding standards, validates test coverage percentages, verifies architectural patterns' to improve specificity

DimensionReasoningScore

Specificity

Names the domain (code review) and mentions some actions like 'validation', 'reviewing code', 'assessing quality and test coverage', but doesn't list multiple concrete specific actions like 'check for security vulnerabilities, verify naming conventions, analyze cyclomatic complexity'.

2 / 3

Completeness

Clearly answers both what ('structured code review against plans and standards') and when ('when a feature is complete', 'before merge', 'assessing quality') with explicit 'Use when' clause and concrete trigger examples.

3 / 3

Trigger Term Quality

Includes natural keywords users would say: 'finished step X', 'ready for review', 'validate architecture', 'check quality and tests', 'before merge'. These are realistic phrases developers use when requesting code reviews.

3 / 3

Distinctiveness Conflict Risk

Clear niche focused on structured code review with specific triggers like 'finished step X', 'ready for review', 'validate architecture'. The combination of plan-based review and quality/test assessment creates a distinct identity unlikely to conflict with general coding skills.

3 / 3

Total

11

/

12

Passed

Implementation

85%

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 code review skill with strong actionability and clear workflow. The six-step procedure, output contract, and progressive disclosure to reference files demonstrate good skill design. Minor verbosity with emphatic phrases ('Every time.', 'No exceptions.') slightly reduces token efficiency without adding instructional value.

Suggestions

Remove repetitive emphatic phrases like 'Every time.' and 'No exceptions.' - Claude understands importance from structure and severity labels

Consolidate the motivational warnings into the procedure itself rather than stating consequences separately

DimensionReasoningScore

Conciseness

Generally efficient but includes some unnecessary emphatic phrases ('Every time.', 'No exceptions.') and repetitive warnings that could be tightened. The core content is valuable but slightly padded with motivational language.

2 / 3

Actionability

Provides concrete, specific guidance with clear steps, exact output structure, severity labels, and required elements. The six-step procedure and output contract are immediately actionable without ambiguity.

3 / 3

Workflow Clarity

Clear six-step sequence with explicit ordering, validation checkpoint ('Before finalizing: Verify you checked every file'), and communication protocol for handling deviations. The workflow is well-structured with feedback loops for plan updates.

3 / 3

Progressive Disclosure

Excellent structure with main content as overview and clear one-level-deep references to checklists.md and examples.md. References are well-signaled with context about when to load them ('Load when reviewing specific domains').

3 / 3

Total

11

/

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
jjjermiah/dotagents
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.