CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

review-change

Review a proposed code change and its evidence when asked to assess implementation quality.

Quality

72%

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 ./internal/adaptertest/testdata/all-agents/want/cursor/files/.cursor/skills/acr__example__all-agents__review-change/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

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

An extremely lean, well-sectioned body whose one instruction is concrete, but it under-delivers on the skill's stated purpose: the review itself has no actionable guidance, failure handling is undefined, and the bundle's reference file is orphaned while the script is referenced via a path that doesn't exist in the bundle. Conciseness is exemplary; actionability and navigation are the weak points.

Suggestions

Reference the script with its bundle-relative path (scripts/check.sh) instead of '.cursor/skills/acr__example__all-agents__review-change/scripts/check.sh', so the command is runnable as bundled.

Add a failure branch, e.g. 'If it exits non-zero, stop and report the failure output — do not proceed to a verdict,' so the non-zero-exit case has defined handling.

Link references/REFERENCE.md from the body (e.g. 'Use deterministic evidence — see references/REFERENCE.md') so the bundled guidance is discoverable, and briefly state what the review should examine and report so the body matches the description's promise.

DimensionReasoningScore

Conciseness

The body is nine lines with zero padding: 'Process steps in order. Do not skip ahead.' and the output contract ('It emits {"ok":true} on stdout and exits non-zero on failure') each earn their place, and no concepts Claude already knows are re-explained. This matches the 'lean and efficient; every token earns its place' anchor exactly.

5 / 5

Actionability

The single command is concrete and executable, but key details are missing: nothing says what to do when check.sh exits non-zero, and the body gives no actionable guidance for the review work itself (what to examine in the change or its evidence, what to report), even though the description promises 'Review a proposed code change and its evidence'. This is concrete-but-incomplete, fitting the score-3 anchor rather than the minor-gaps anchor at 4.

3 / 5

Workflow Clarity

As a simple single-action skill the step is unambiguous — a specific script, its success output, and 'Finish here.' terminate the flow — which supports the simple-skill exception. It falls short of 5 because the failure branch is defined ('exits non-zero on failure') yet no recovery or reporting step follows it, a minor checkpoint gap matching the score-4 anchor.

4 / 5

Progressive Disclosure

Scored against the actual bundle (references/REFERENCE.md, scripts/check.sh): the body references the script via '.cursor/skills/acr__example__all-agents__review-change/scripts/check.sh', a path that does not match the bundle-relative 'scripts/check.sh', and references/REFERENCE.md is never mentioned from the body at all, leaving its guidance undiscoverable. Sections are well organized, but references are present yet not correctly signaled — the score-3 anchor, not 4 (the mis-pathed script and orphaned reference are more than minor organization gaps).

3 / 5

Total

15

/

20

Passed

Description

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

The description answers both what and when in third person without padding, landing at a competent mid-level quality. Its weaknesses are thin trigger coverage (no 'Use when...' with natural variations like PR/diff), a single generic action verb rather than enumerated capabilities, and meaningful conflict risk with ordinary code-review skills.

Suggestions

Expand the when clause into an explicit trigger list, e.g. 'Use when the user asks to review a change, PR, diff, or evaluate implementation quality or its supporting evidence.'

Enumerate the concrete actions — e.g. 'Review a proposed code change, verify its supporting evidence, and report a verdict' — instead of the single generic verb 'Review'.

Sharpen distinctiveness by stating what separates this skill from a normal code review, e.g. that it validates the change against deterministic evidence rather than general code quality.

DimensionReasoningScore

Specificity

'Review a proposed code change and its evidence' names the domain and 1–2 actions (review the change, review its evidence), but never states what the review covers or produces — matching the 'names domain and 1-2 concrete actions, but not comprehensive' anchor. Not 4: there aren't several specific actions; not 2: it goes beyond a bare domain claim like 'Processes PDF files'.

3 / 5

Completeness

Both parts are explicitly stated: what — 'Review a proposed code change and its evidence'; when — 'when asked to assess implementation quality'. The when clause is present but thin and could be more explicit with concrete triggers, exactly the score-4 anchor ('when' could be more explicit or specific); score 5 requires concrete trigger phrases, which are absent.

4 / 5

Trigger Term Quality

Natural terms are present ('review', 'assess', 'implementation quality', 'code change') but common variations users say for this task are missing — 'PR', 'pull request', 'diff', 'evaluate'. This fits 'some relevant keywords but missing common variations or synonyms' rather than the good-coverage anchor at 4.

3 / 5

Distinctiveness Conflict Risk

'Review a proposed code change' overlaps heavily with any generic code-review skill, though 'and its evidence' and 'assess implementation quality' narrow the niche somewhat — 'somewhat specific but could still overlap with similar skills' (anchor 3). Not 4: the overlap with code-review skills is more than minor; not 2: it is not broadly applicable to unrelated skills.

3 / 5

Total

13

/

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

internal/adaptertest/testdata/all-agents/want/cursor/files/.cursor/skills/acr__example__all-agents__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.