CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-change

Review a Go change when the user asks to inspect a diff for correctness.

Quality

68%

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

Fix and improve this skill with Tessl

tessl review fix ./examples/complete/skills/review-change/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

57%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.

The body is maximally concise and cleanly structured, but it is so sparse that it provides almost no actionable detail: no commands, no review criteria, no output format. Its companion reference file exists in the bundle yet is never linked, so prioritization guidance it contains is undiscoverable.

Suggestions

Add the concrete mechanics of Step 1, e.g. the command to obtain the diff ('git diff HEAD', 'gh pr diff <n>') and how much surrounding context to read.

Link the existing reference from the body, e.g. 'Prioritize findings per [review-guide.md](references/review-guide.md)', so the bundle's guidance is discoverable.

Specify the report format for Step 2 — severity levels, finding structure (file:line, issue, why it matters), and what to do when no findings exist.

DimensionReasoningScore

Conciseness

The body is lean with zero padding — 'Process steps in order. Do not skip ahead.' plus two two-line steps — and assumes Claude's competence, explaining nothing Claude already knows. It matches 'lean and efficient; every token earns its place'.

5 / 5

Actionability

Guidance is high-level: 'Inspect the complete diff and the surrounding code' and 'Report correctness, security, and test-coverage findings with file locations' give no concrete commands (how to obtain the diff), no criteria for severity or what counts as a finding, and no output format. This matches 'minimal concrete guidance; high-level hints but missing the specific steps to execute' — above level 1 only because the instructions are imperative and name specific finding categories and locations.

2 / 5

Workflow Clarity

The two-step sequence is present and explicitly ordered ('Proceed immediately to Step 2', 'Finish here'), but there are no checkpoints — nothing telling Claude how to confirm the diff was fully read or how findings should be structured before reporting. It matches 'steps listed but... checkpoints missing or implicit'; it is above level 2 because the sequence is coherent and unambiguous in ordering.

3 / 5

Progressive Disclosure

The body itself is short and well-sectioned, but the bundle's reference file (references/review-guide.md, which contains prioritization guidance like 'Prioritize behavior, security, and missing outcome coverage') is never mentioned or linked from the body. Scoring against the actual bundle structure, this is 'references present but not clearly signaled' — an orphaned reference and a clear organization gap; it stays at 3 rather than 2 because the inline content itself is appropriately placed and minimal.

3 / 5

Total

13

/

20

Passed

Description

62%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 concise, third-person description that answers both what and when without padding, but its capability coverage and trigger vocabulary are narrow. It names one action (correctness review of a Go diff) and omits natural synonyms users would say, limiting both completeness and trigger reach.

Suggestions

Enumerate the concrete review actions performed (e.g., 'checks correctness, security, and test coverage of Go diffs') so specificity moves beyond a single action.

Add natural trigger synonyms such as 'code review', 'PR', 'pull request', or 'patch' to the when-clause so the skill fires on phrasings users actually say.

State what artifacts are handled (e.g., 'uncommitted changes, commits, or pull requests') to sharpen distinctiveness against generic code-review skills.

DimensionReasoningScore

Specificity

The description names a concrete domain and action — 'Review a Go change when the user asks to inspect a diff for correctness' — but covers only a single action (correctness review) with no breadth (no security, test coverage, or other review aspects). It matches 'names domain and 1-2 concrete actions, but not comprehensive' rather than level 4, which requires several specific actions.

3 / 5

Completeness

Both what ('Review a Go change... for correctness') and when ('when the user asks to inspect a diff for correctness') are explicitly present, satisfying the 'Use when...' requirement. It falls short of level 5 because the 'what' is thin — a single brief action with no concrete capability list — and the trigger clause is narrow rather than a set of concrete trigger phrases.

4 / 5

Trigger Term Quality

It includes some natural keywords users would say — 'Go', 'diff', 'correctness', 'review a change' — but misses common variations like 'code review', 'PR', 'pull request', 'patch', or 'git diff'. This fits 'some relevant keywords but missing common variations or synonyms'; it is above level 2 because the included terms (especially 'diff' and 'Go') are genuinely natural phrasing, not generic filler.

3 / 5

Distinctiveness Conflict Risk

The Go + diff + correctness niche is mostly distinct, with only minor overlap risk against generic code-review or language-agnostic review skills. It is below level 5 because 'review a change/diff' is shared vocabulary with any general code-review skill, keeping some conflict risk with closely related skills.

4 / 5

Total

14

/

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

jbaruch/agentic-context-registry

Path

examples/complete/skills/review-change/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.