CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

code-review

AI-driven code review workflow that reviews uncommitted changes using a discoverable reviewer profile, presents findings for human decision, and iterates until approved. Supports --unattended for automated iteration. Use when reviewing code before commit or PR. Activated by commands: /start, /continue, /clean.

Quality

81%

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

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.

The body is impeccably lean and its dispatch-and-escalate logic is clearly written, but as a bundle it is broken: every path it delegates to (commands/{command}.md, skills/controller.md, guidelines.md) is missing, leaving the skill with no executable substance. The structure is right in intent yet undelivered in fact.

Suggestions

Ship the referenced files (commands/*.md, skills/controller.md, guidelines.md) in the bundle, or inline the minimal dispatch logic in SKILL.md so the skill is self-contained — currently every instruction resolves to a nonexistent file.

Add a fallback for when commands/{command}.md does not exist (e.g., list available commands or route to the controller) so the primary dispatch branch cannot dead-end.

Briefly name the workflow phases in SKILL.md so the instruction "Do not advance to the next phase" has concrete referents.

DimensionReasoningScore

Conciseness

The ~90-word body is a pure dispatcher — "read `commands/{command}.md` and follow it", "read `skills/controller.md`", "see `guidelines.md`" — with zero padding and no explanations of concepts Claude already knows; every token earns its place, matching anchor 5 exactly.

5 / 5

Actionability

The directions are concrete (exact file paths, explicit two-branch dispatch, "If a step fails or produces unexpected output, stop and report the error"), but the referenced files — commands/{command}.md, skills/controller.md, guidelines.md — do not exist anywhere in the bundle, so the guidance cannot actually be executed; the missing targets are a key-detail gap beyond anchor 4's "minor gaps", while the specificity of the directions keeps it above anchor 2's high-level hints.

3 / 5

Workflow Clarity

A clear conditional sequence (command dispatch vs. controller fallback) with an explicit failure checkpoint — "If a step fails or produces unexpected output, stop and report the error to the user. Do not advance to the next phase. Offer to retry or escalate." — matches anchor 4; it is not anchor 5 because the "phases" the checkpoint guards are never named and the retry/escalate feedback loop is stated but not concretized.

4 / 5

Progressive Disclosure

In form the references are well-signaled and one level deep ("see `guidelines.md`", "read `skills/controller.md`"), but scored against the actual bundle structure per the rubric guideline, none of the referenced files exist — no commands/, skills/, references/, scripts/, or assets/ directories are present — so the disclosure structure leads nowhere; it is not anchor 4 because "most content is appropriately placed" fails when the detail layer is absent, and not anchor 2 because the body inlines nothing that belongs elsewhere and signals its references clearly.

3 / 5

Total

15

/

20

Passed

Description

83%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 strong description: explicit what and when clauses, natural trigger terms, third-person voice, and distinctive command-based activation. The only weaknesses are minor — a couple of vague terms ("AI-driven", "discoverable reviewer profile") and a few missing synonyms like "diff" and "pull request".

DimensionReasoningScore

Specificity

Lists several concrete actions — "reviews uncommitted changes using a discoverable reviewer profile", "presents findings for human decision", "iterates until approved", "Supports --unattended for automated iteration" — but coverage has minor gaps ("AI-driven" is filler and "discoverable reviewer profile" is unexplained), matching anchor 4 rather than the comprehensive anchor 5 and clearly above the 1-2-action anchor 3.

4 / 5

Completeness

Explicitly answers both: what ("reviews uncommitted changes... presents findings for human decision, and iterates until approved") and when ("Use when reviewing code before commit or PR") with concrete trigger phrases and command activation, matching anchor 5; it is not anchor 4 because the when-clause is already explicit and specific rather than needing more specificity.

5 / 5

Trigger Term Quality

Good natural keyword coverage — "code review", "reviewing code", "before commit or PR", "uncommitted changes", plus command names /start, /continue, /clean — but common variations like "diff", "pull request" (spelled out), and "staged changes" are missing, so it falls just short of anchor 5's comprehensive synonym coverage.

4 / 5

Distinctiveness Conflict Risk

The framing — uncommitted-changes review with a human decision loop and explicit command activation — carves a mostly distinct niche with only minor overlap risk against generic code-review/PR-review skills; it is not anchor 5 because "reviewing code" and "PR" are broad triggers shared by other review skills, and not anchor 3 because the workflow specifics narrow it considerably.

4 / 5

Total

17

/

20

Passed

Validation

93%

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

Validation — 15 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

frontmatter_unknown_keys

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

Warning

Total

15

/

16

Passed

Repository

flightctl/ai-workflows

Path

code-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.