CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

reviewer

Load when reviewing a diff, classifying findings, or returning a verdict.

Quality

70%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Low

Low-risk findings worth noting

Fix and improve this skill with Tessl

tessl review fix ./skills/reviewer/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

85%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 an exceptionally lean, rule-dense contract: fully executable commands with explicit failure handling, a genuine self-validate feedback loop, and precise scoping rules for re-review and mutation-stability runs. Its weaknesses are structural rather than instructional — four referenced files are missing from the bundle, one reference file is undiscoverable from the entrypoint, and the copy/poll protocols are inlined and repeated instead of being split out.

Suggestions

Ship or fix the referenced files: workflows/review.md, workflows/codebase-review.md, workflows/qa-review.md and schemas/review-finding.md are cited from SKILL.md but absent from the bundle, breaking navigation from the entrypoint (and taking the finding-JSON contract with them).

Link references/perf-qa.md directly from SKILL.md — e.g. in the workflow table's qa-review row or a QA section — so it is discoverable without relying on the missing workflows/qa-review.md.

Extract the repeated worktree-copy/free-space protocol (§ Ethos, § Output Contract, § Mutation-Stability Pairing) and the inline waiter-poll shell one-liner into a single reference file or helper script to remove duplicated tokens from SKILL.md.

DimensionReasoningScore

Conciseness

The body is rule-dense with zero padding and zero explanation of concepts Claude already knows ("Fewer high-conviction findings beat lists of nits", "A zero RESULT is a result"), fully assuming competence. It falls short of anchor 5 because the worktree-copy/free-space protocol is restated across § Ethos, § Output Contract and § Mutation-Stability, and the ~300-word inline waiter-poll shell one-liner (§ Mutation-Stability Pairing) is mechanical detail that could be trimmed into a script or reference.

4 / 5

Actionability

Commands are complete and copy-paste ready with every flag and placeholder: `scripts/review-depth --base REF --head REF --repo DIR`, `review-artifact-check --path/--file`, `mutation-stability --worktree [WORKTREE_PATH] --sha [SHA] --test '[TEST_CMD]' ... --stability [SAMPLE_COUNT]`, plus exact shell (`git archive REV | tar -x -C "$dir"`, `mktemp -d "${TMPDIR:-/tmp}/review-copy.XXXXXX"`, `df -Pk "$dir"`), stated defaults (60s/300s budgets), and failure contracts (rerun on exit 124, exit-127 guard in the poll). Anchor 5 fits; the caller-supplied TEST_CMD/BUILD_CMD parameters are legitimately per-change, not gaps.

5 / 5

Workflow Clarity

The Output Contract is an explicit sequence with a real validate-fix-retry loop — capture starting fields, write the findings JSON to the Artifact path, "Self-validate before returning ... Fix until this prints \"ok\": true\"", then return the filled `<output_format>` block — and Mutation-Stability adds bounded phases, budget/timeout rules, retry-on-124, and never-end-turn conditions. Re-review scoping is precise (Fixed items suppressed unless the defect persists; re-report verbatim with the recorded sha). Anchor 5's explicit validation steps with feedback loops are fully met, including for the batch-like copy/build/test operations.

5 / 5

Progressive Disclosure

The references that exist are well-signaled and one level deep (`references/finding-disposition.md` with a clear ownership statement, `scripts/review-depth`, `scripts/mutation-stability`), but four paths cited from the body — `workflows/review.md`, `workflows/codebase-review.md`, `workflows/qa-review.md`, and `schemas/review-finding.md` — do not exist in the bundle, and `references/perf-qa.md` exists but is unreachable from SKILL.md (it is loaded only via the missing qa-review.md). This is more than anchor 4's "minor organization gaps", matching anchor 3; it is not 2 because the references that do resolve are clearly signaled and never nested.

3 / 5

Total

17

/

20

Passed

Description

45%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 functions as a pure load trigger: it is distinct within its skill family and gives one genuinely natural trigger phrase, but it says nothing about what the skill actually does, and its terminology is internal workflow jargon rather than the phrases a user would naturally say. Adding a 'what' clause and everyday terms like "code review" or "PR review" would move it up two levels.

Suggestions

Add a 'what' clause stating concrete capabilities — e.g. produces finding JSON artifacts, classifies findings per a disposition schema, runs mutation-stability controls, returns a merge verdict — so the description answers both what the skill does and when to load it.

Include natural trigger phrases users would actually say: "code review", "PR review", "review findings", "QA review", rather than only internal terms like "classifying findings" and "returning a verdict".

Name the domain explicitly (code review / QA) so actions like "returning a verdict" have a concrete object instead of floating unanchored.

DimensionReasoningScore

Specificity

"reviewing a diff, classifying findings, or returning a verdict" names three actions but gives them no concrete objects — it never says what is reviewed (code, PRs) or what a finding or verdict is about, and it omits the QA-label and artifact work the skill actually does. It goes beyond anchor 3's 1-2 actions but lacks the object-rich specificity of anchor 4's example, so the gap is more than minor.

3 / 5

Completeness

The description is entirely a load trigger — "Load when reviewing a diff, classifying findings, or returning a verdict" — with no 'what' clause: it never states what the skill does or produces (finding JSON artifacts, verdicts, QA-label workflows). This is exactly anchor 2's 'only when is present without what'; it cannot be 3 because a clear 'what' is absent, and not 1 because the 'when' is explicit and specific.

2 / 5

Trigger Term Quality

"reviewing a diff" is a natural user phrase, but "classifying findings" and "returning a verdict" are the skill's internal workflow vocabulary, and the common variations users actually say — "code review", "PR review", "pull request", "QA" — are all missing. This matches anchor 3 (some relevant keywords, missing common variations) rather than anchor 4's good coverage.

3 / 5

Distinctiveness Conflict Risk

The review/verdict niche ("classifying findings", "returning a verdict") is clearly carved out against its sibling skills (orch, code-quality, docs-writing, project-management), with "reviewing a diff" an unlikely trigger for the wrong skill. It is not 5 because "review" and "findings" are broad words that could still collide with a closely related review skill, matching anchor 4's minor overlap risk.

4 / 5

Total

12

/

20

Passed

Validation

81%

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

Validation — 13 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

metadata_version

'metadata.version' is missing

Warning

frontmatter_unknown_keys

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

Warning

relative_links

Relative link issues: 3 missing, 16 suspicious

Warning

Total

13

/

16

Passed

Repository

vanillagreencom/kendex

Path

skills/reviewer/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.