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 that earns its context budget with project-specific rules Claude could not infer. The main improvements are small: define the undefined $SCOPE variable, add guidance for handling failed checks (feedback loop), and tidy the step numbering.

DimensionReasoningScore

Conciseness

The body is ~45 lines of dense, project-specific rules (Svelte 5 runes vs. Svelte 4 patterns, service/adapter/load-function boundaries, < 200 line components) with zero padding and no explanation of concepts Claude already knows. Every token carries project-specific signal, matching the lean anchor 5.

5 / 5

Actionability

Steps 1-2 provide copy-paste ready commands (git diff, svelte-check, eslint, vitest) and steps 3-5 give concrete, checkable criteria with a defined classification and verdict scheme. However, step 2b references "$SCOPE" which is never defined (only "$ARGUMENTS" is), leaving a minor gap that keeps it below fully executable anchor 5.

4 / 5

Workflow Clarity

The sequence (identify files → automated checks → architecture checks → classify → report) is clear with an explicit checklist and defined output verdict, matching anchor 4. It falls short of anchor 5 because there is no feedback loop for error recovery (e.g., what to do when svelte-check or vitest fail) and the step numbering is slightly jumbled (an inserted "2b" and a missing blank line before step 3).

4 / 5

Progressive Disclosure

This is a single-file skill under 50 lines with no bundle files (references/, scripts/, assets/ are absent) and well-organized sections; the architecture checklist is compact and central to every run so inlining is appropriate. Per the rubric's simple-skill guideline, this earns the full score on structure alone.

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 covers both what (validates architecture, types, security, test coverage) and when (pre-merge code review) in third person without fluff. Its main limitation is narrow trigger phrasing — it lacks common variations like "PR", "pull request", or "diff" that users naturally say, and it names validation areas rather than the full set of concrete actions performed.

Suggestions

Add natural trigger variations to the when clause, e.g., "Use when code changes need review before merge (PRs, pull requests, diffs)".

Name the concrete actions performed beyond validation targets, such as running automated checks (svelte-check, eslint, vitest) and producing an approval verdict report.

Distinguish the skill from generic code-review skills by signaling its SvelteKit/Svelte 5 architecture-validation focus in the description.

DimensionReasoningScore

Specificity

"validates architecture, types, security, and test coverage" enumerates four concrete validation areas, going beyond 1-2 items, though it describes one verb applied to objects rather than multiple distinct actions (e.g., no mention of report/verdict generation). It matches anchor 4 (several specific items, minor coverage gaps) better than anchor 3, and not anchor 5 since the action coverage is not comprehensive.

4 / 5

Completeness

Both parts are present: an explicit "Use when code changes need review before merge" clause and a clear what ("validates architecture, types, security, and test coverage"). The when clause is explicit but single-phrased without concrete trigger variations, matching anchor 4 rather than anchor 5's multiple concrete trigger phrases.

4 / 5

Trigger Term Quality

"code changes", "review", and "merge" are natural terms users would say when needing this skill, giving good keyword coverage. Common variations such as "PR", "pull request", or "diff" are missing, which keeps it below anchor 5's comprehensive synonym coverage.

4 / 5

Distinctiveness Conflict Risk

The pre-merge validation framing ("code changes need review before merge") carves out a fairly distinct niche with specific validation targets. There is minor overlap risk with generic code-review skills that share the same trigger terms, so it fits anchor 4 rather than anchor 5's minimal-conflict profile.

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/svelte/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.