CtrlK
BlogDocsLog inGet started
Tessl Logo

qodo-review

Review local changes with Qodo during substantive coding milestones and before a PR or completed-work handoff. Use light background checkpoints while coding, collect and assess findings with session context, and run a final review before handing off. Also use for "review my local diff", "pre-PR review", or "run qodo review". Use qodo-review-resolver for findings already posted on a PR.

68

Quality

83%

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

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

A dense, highly actionable operational guide with real executable scripts and thorough validation/recovery handling. Its weaknesses are repetition of the depth-selection and lifecycle policy across sections and inlining of protocol detail that belongs in the existing reference structure.

Suggestions

Consolidate the depth-selection policy ('--fast'/'--deep'/auto justification rules) into the 'Choose when and how deeply to review' section and reduce the restatements in 'Checkpoints and final handoff', 'Configuration', and 'Guardrails' to one-line cross-references.

Turn the 'Instructions' section into an actual sequenced workflow (preflight → depth selection → submit → collect → present → act → recover) so the full lifecycle is readable as an ordered checklist instead of being spread across topic sections.

Move the async submission/polling shell recipes and the failure-shape taxonomy ('Recover a review') into a references/ file, keeping only the exit-code table and the decision rules in SKILL.md, mirroring how connected-progress.md is already handled.

DimensionReasoningScore

Conciseness

Nearly all content is non-obvious operational policy rather than concepts Claude already knows, so it is not 'noticeably verbose' at level 2 — but depth-selection rules are restated in 'Choose when and how deeply to review', 'Checkpoints and final handoff', 'Configuration', and 'Guardrails', and async/recovery caveats also repeat. It is mostly efficient but could be meaningfully tightened, matching the level-3 anchor.

3 / 5

Actionability

The body provides copy-paste-ready commands throughout: a quick-start CLI block, an executable async-submission snippet with jq validation, a complete polling loop with traps and exit-code handling, an exit-code decision table, and a concrete context JSON example. Specific examples cover the common cases.

5 / 5

Workflow Clarity

The lifecycle is well covered with explicit validation gates (version probe first, whoami preflight, exit-code classification, bounded retries, verify-then-re-review after fixes). It falls short of level 5 because the 'Instructions' section is only a one-line topic list — the actual end-to-end sequence must be assembled from topic-scattered sections rather than a clearly ordered workflow.

4 / 5

Progressive Disclosure

The three references (skill-updates.md, connected-progress.md, local-triage.md) are real, one level deep, and clearly signaled at their point of use. Not level 5 because roughly 150 lines of async protocol, recovery, and error-taxonomy detail are inlined in SKILL.md where a reference file would keep the overview lean.

4 / 5

Total

16

/

20

Passed

Description

92%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: concrete multi-step capabilities, explicit 'what' and 'when', quoted natural trigger phrases, and clear boundary routing against the sibling resolver skill. The only gap is a few missing common synonyms for triggering a review.

DimensionReasoningScore

Specificity

The description lists multiple concrete actions covering the full workflow: 'use light background checkpoints while coding', 'collect and assess findings with session context', and 'run a final review before handing off'. Coverage is comprehensive rather than leaving minor gaps, so it matches the top anchor.

5 / 5

Completeness

Both 'what' (review local changes via light checkpoints, collect/assess findings, final review) and 'when' ('during substantive coding milestones and before a PR or completed-work handoff', plus the 'Also use for...' trigger phrases) are explicitly and concretely answered.

5 / 5

Trigger Term Quality

Explicitly quotes natural user phrases — "review my local diff", "pre-PR review", "run qodo review" — which are exactly what a user would say. Falls short of the top anchor because common synonyms like "code review", "review my changes", or "pre-commit review" are absent.

4 / 5

Distinctiveness Conflict Risk

It carves a clear niche (local, pre-PR review of unpushed work) and explicitly routes the adjacent case away: 'Use qodo-review-resolver for findings already posted on a PR', minimizing wrong-skill triggering.

5 / 5

Total

19

/

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
qodo-ai/qodo-skills
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.