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/claude-code/files/.claude/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.

Extremely lean and unambiguous for the single step it describes, with an explicit output contract for the check script. However, the substance of the review promised by the description is absent — the skill ends at 'Finish here.' after one stub check — the non-zero-exit failure path is unaddressed, and the bundled references/REFERENCE.md is orphaned from the body.

Suggestions

Add the actual review procedure after the check step (what to examine in the change and its evidence, and what to report) — currently the workflow ends at 'Finish here.' with no review performed.

State what to do when check.sh exits non-zero (e.g., report the failure output and stop) to close the feedback loop the exit-code contract opens.

Link references/REFERENCE.md from the body (or remove it) so the bundled guidance is discoverable, and prefer the relative path 'scripts/check.sh' over the installation-specific absolute path.

DimensionReasoningScore

Conciseness

The body is ~10 lines with zero padding: 'Process steps in order. Do not skip ahead.' is pure instruction and nothing explains concepts Claude already knows. Every token earns its place, so it matches the top anchor rather than the minor-trim anchor at 4.

5 / 5

Actionability

'Run `.claude/skills/acr__example__all-agents__review-change/scripts/check.sh`' is a concrete executable command with an explicit output contract ('emits `{"ok":true}` on stdout and exits non-zero on failure'), but key details are missing: nothing describes what the actual review of the change and its evidence consists of, or what to do with the check result before 'Finish here.' Not 4 because the gap is major rather than minor; not 2 because the guidance given is concrete and executable, not high-level hints.

3 / 5

Workflow Clarity

The single step is unambiguous with an explicit success/failure checkpoint (expected JSON output plus exit-code semantics) and 'Finish here.' bounds the process. Not 5 because no failure-recovery path is stated for the non-zero exit and 'Process steps in order. Do not skip ahead.' implies additional steps that do not exist; not 3 because the checkpoint is explicit rather than missing or implicit.

4 / 5

Progressive Disclosure

The body is short and sectioned, but the bundle contains references/REFERENCE.md ('Use deterministic evidence.') that the body never mentions or links, so a bundled reference is present but not clearly signaled. Not 4/5 because that reference is undiscoverable; not 2 because no content is inlined that clearly belongs in a separate file.

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 states a clear what and an explicit when in third person without fluff, but it is thin: it names a single concrete action, covers few natural trigger synonyms (no 'diff', 'PR', or 'code review'), and carries real overlap risk with generic code-review skills.

Suggestions

Add concrete trigger phrases such as 'Use when the user asks to review a diff, PR, or staged changes' to raise completeness and trigger coverage together.

Enumerate the specific review actions performed (e.g., verifying claims against the diff, checking that tests were run, flagging untested edge cases) to improve specificity and distinctiveness.

Include the natural synonyms users would say — 'diff', 'PR', 'pull request', 'code review' — to broaden trigger-term coverage.

DimensionReasoningScore

Specificity

'Review a proposed code change and its evidence' names the domain plus one concrete action, matching the '1-2 concrete actions but not comprehensive' anchor. Not 2 because a real action is named rather than a bare domain label; not 4 because only one action is given where several specific actions would be needed.

3 / 5

Completeness

A clear what ('Review a proposed code change and its evidence') is paired with an explicit when ('when asked to assess implementation quality'), in third person throughout. Not 5 because the when clause lacks concrete trigger phrases like 'Use when reviewing a diff or PR'; not 3 because the when is explicit, satisfying the cap condition rather than triggering it.

4 / 5

Trigger Term Quality

Terms 'review', 'code change', and 'assess implementation quality' are relevant natural keywords, but common variations users actually say — 'diff', 'PR', 'pull request', 'code review' — are missing. Fits the 'some relevant keywords but missing common variations or synonyms' anchor; not 4 because the missing terms are among the most natural triggers for this skill.

3 / 5

Distinctiveness Conflict Risk

Scoped to 'a proposed code change and its evidence', but generic code-review skills would trigger on the same phrasing, so overlap with similar skills remains. Not 2 because it is not fully broad; not 4 because no distinct trigger phrase separates it from a standard code-review skill.

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/claude-code/files/.claude/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.