CtrlK
BlogDocsLog inGet started
Tessl Logo

review

Review changed code for correctness, security, code quality, and performance. Use when the user asks to review code changes, a PR, or specific files. Invoke with `/review`, `/review <pr-number>`, `/review <file-path>`, or `/review <pr-number> --comment` to post inline comments on the PR. Add `--effort low|medium|high` to trade depth for speed (defaults to high for PRs, medium for local changes).

71

Quality

88%

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

SKILL.md
Quality
Evals
Security

Quality

Content

77%

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

The content is highly actionable with a rigorous, validation-gated workflow, but it pays for that rigor in conciseness — extensive rationale anecdotes inflate the file — and in progressive disclosure, since everything lives in one monolithic SKILL.md with no external reference files.

Suggestions

Move the long dogfooding rationale ('Measured:', 'Dogfooded against its own PR #…') into a separate references/ file or condense to one line per rule; the actionable rule and its command earn their place, the war-story justification does not.

Extract the stable reference material — the agent dimension table, the presubmit JSON schema, the review/state JSON formats, and the severity definitions — into reference files under references/ and link to them one level deep, turning the monolithic file into an overview.

Trim restated rules: several constraints (worktree flow, verbatim-copy, no todo_write) are repeated across steps; state each once and reference it.

DimensionReasoningScore

Conciseness

The ~1000-line body is substantively efficient but heavily padded with dogfooding war-stories and repeated rationale ('Measured:', 'Dogfooded against its own PR'), matching the score-2 anchor of useful content that could be tightened rather than the lean score-3.

2 / 3

Actionability

Provides fully executable bash commands (parse-args, fetch-pr, agent-prompt --roster, submit, resolve-anchors), exact JSON schemas, and copy-paste-ready flag grammar throughout.

3 / 3

Workflow Clarity

A clearly sequenced 9-step pipeline with explicit validation checkpoints (check-coverage exit 3, Step 4 verification, Step 5 iterative audit termination rules, Step 7 anchor validation and 422 recovery) and fail-closed feedback loops.

3 / 3

Progressive Disclosure

No references/scripts/assets bundle exists; the skill is a single ~1000-line monolithic file with schemas, dimension tables, and severity definitions inline, matching the score-2 anchor of content that should be split but is kept inline rather than a well-signaled one-level reference structure.

2 / 3

Total

10

/

12

Passed

Description

100%

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 is specific, trigger-rich, complete with an explicit 'Use when' clause, and clearly distinguished from other skills. It is one of the stronger reference examples and needs no changes.

DimensionReasoningScore

Specificity

Lists multiple concrete actions — 'review changed code for correctness, security, code quality, and performance' and 'post inline comments on the PR' — matching the score-3 anchor for specific concrete actions.

3 / 3

Completeness

Explicitly answers both what (review changes, post inline comments) and when via the explicit 'Use when the user asks to review code changes, a PR, or specific files' trigger clause.

3 / 3

Trigger Term Quality

Covers natural terms a user would say — 'review code changes', 'a PR', 'specific files' — plus the /review invocation forms, matching the good-coverage score-3 anchor.

3 / 3

Distinctiveness Conflict Risk

Occupies a clear code-review niche with distinct /review triggers and is unlikely to fire for an unrelated skill.

3 / 3

Total

12

/

12

Passed

Validation

87%

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

Validation14 / 16 Passed

Validation for skill structure

CriteriaDescriptionResult

skill_md_line_count

SKILL.md is long (1008 lines); consider splitting into references/ and linking

Warning

frontmatter_unknown_keys

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

Warning

Total

14

/

16

Passed

Repository
QwenLM/qwen-code
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.