CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-review

Use when code changes need review before merge - validates architecture, types, security, and test coverage.

Quality

83%

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

SKILL.md
Quality
Evals
Security

Quality

Content

86%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 lean, highly actionable review workflow with executable commands, a concrete architecture checklist, and a clear classification scheme. The main issues are presentational inconsistencies (2b numbering, undefined $SCOPE) and an unspecified report format in the final step.

Suggestions

Fix the step numbering (renumber 2b into the main sequence) and use one variable consistently — either $ARGUMENTS or $SCOPE — since $SCOPE is currently undefined.

Add a brief report template in step 5 (e.g., findings grouped by 🔴/🟡/🟢/✨ with file:line references, then the verdict line) so output shape is unambiguous.

State what to do when automated checks fail (e.g., treat each failure as a finding and include it in the classification step) to close the validation feedback loop.

DimensionReasoningScore

Conciseness

The body is roughly 44 lines with zero padding: every line is an instruction, command, or checklist item, and it assumes Claude's competence without explaining concepts it already knows. This matches anchor 5 ('every token earns its place').

5 / 5

Actionability

Steps 1-2 provide copy-paste ready commands (git diff, tsc, eslint, vitest) and step 3 is a concrete, checkable pattern list, but step 5 ('Produce report with verdict') gives no report structure. This is anchor 4 ('concrete commands with minor gaps'), not 5.

4 / 5

Workflow Clarity

The sequence (identify files, automated checks, architecture validation, manual pattern check, classify, report) is clear with the automated checks acting as validation checkpoints — anchor 4. It falls short of 5 due to inconsistent step numbering (2b), the undefined $SCOPE variable after 'Scope: $ARGUMENTS', and no guidance on handling check failures, though as a read-only review skill it avoids the destructive-operation cap.

4 / 5

Progressive Disclosure

The skill is under 50 lines with no need for external bundle files, and its sections are well organized; per the simple-skill guideline this earns anchor 5. The only external pointers (docs/ARCHITECTURE.md and the /review-check-architecture skill) are clearly signaled and one level deep.

5 / 5

Total

18

/

20

Passed

Description

75%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 description that explicitly pairs a clear when-clause with a concrete list of validation capabilities in third person. Its main gaps are missing natural trigger synonyms (PR, pull request, diff) and reliance on a single generic action verb.

Suggestions

Add natural trigger variations such as "pull request", "PR", or "diff" to the when-clause: e.g., 'Use when code changes, a PR, or a diff need review before merge'.

Describe the outcome as well as the checks, e.g., '...and produces a classified findings report with an approve/caveats/requires-changes verdict' to strengthen the what-clause.

DimensionReasoningScore

Specificity

"validates architecture, types, security, and test coverage" names several concrete validation targets, matching anchor 4. It is not a 5 because a single generic verb ("validates") carries all the action and there is no mention of the report/verdict output.

4 / 5

Completeness

Both parts are present: what ("validates architecture, types, security, and test coverage") and when ("Use when code changes need review before merge"). The when-clause could be more explicit with concrete trigger phrases (e.g., reviewing a PR or diff), keeping it at anchor 4 rather than 5.

4 / 5

Trigger Term Quality

"code changes", "review", and "merge" are natural phrases users would say, giving good keyword coverage. Common variations like "PR", "pull request", or "diff" are missing, so it does not reach anchor 5's comprehensive synonym coverage.

4 / 5

Distinctiveness Conflict Risk

The pre-merge change-review scope with named validation dimensions makes it mostly distinct with only minor overlap risk against generic code-review skills. It is not a 5 because "code review" is a crowded niche and the description does not include triggers that fully separate it from siblings.

4 / 5

Total

16

/

20

Passed

Validation

87%

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

Validation — 14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

allowed_tools_field

'allowed-tools' contains unusual tool name(s)

Warning

frontmatter_unknown_keys

Unknown frontmatter key(s) found; consider removing or moving to metadata

Warning

Total

14

/

16

Passed

Repository

fernandezbaptiste/specialist-agent

Path

packs/vue/skills/review-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.