Content
81%Weight 40%Scale 1-5Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
A highly actionable, validation-rich skill body: executable commands with exact JSON payloads, clearly sequenced workflows with dry-run/read-back/idempotency safeguards, and well-signaled conditional references. The main weaknesses are redundancy — approval and retry rules restated across many sections — and inlining of the long discovery flow that belongs in a reference file.
Suggestions
Consolidate the repeated approval/dry-run/idempotency rules into a single stated-once policy section (or a reference file) referenced from each workflow instead of restating them in Instructions, Preflight, Preview-and-apply, Error Handling, and Guardrails.
Move the detailed "Discover a source repository" and "Set up or manage imported review skills" flows into a reference file, keeping a short routing summary in SKILL.md so the body acts as an overview.
Trim the explanatory "Where rules come from" section to the operationally essential distinctions (sourceType naming rules) and drop the prose about what each field means.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with operational detail rather than concept explanations, but approval/dry-run/idempotency rules are restated across Instructions, Preflight, Preview-and-apply, Error Handling, and Guardrails, and sections like "Where rules come from" are explanatory prose. It fits "mostly efficient but includes some unnecessary explanation or could be tightened" better than the noticeably-padded score-2 anchor. | 3 / 5 |
Actionability | Copy-paste-ready CLI commands with exact flags and JSON payloads are provided (e.g. `qodo rules set-state --args '{"rule_ids":[123,124],"state":"inactive","dry_run":true}' --json`, the review-skills bulk/update examples), plus a concrete schema-verification command (`qodo read tools rules --json`), covering the common cases per the score-5 anchor. | 5 / 5 |
Workflow Clarity | Multi-step flows (preflight, discovery, preview/apply, read-back) are clearly sequenced with explicit validation checkpoints and feedback loops: version-probe gate, dry_run=true previews with matched-count inspection, read-back of every affected id before and after writes, and a bounded retry rule for uncertain outcomes. The destructive/batch validation cap does not apply since validation is pervasive. | 5 / 5 |
Progressive Disclosure | Three real, one-level-deep reference files are well signaled and conditionally loaded (windows-shells.md "read only for CLI commands in PowerShell or Git Bash", windows-launcher.md "only if invocation fails", skill-updates.md for user-requested updates). However, the ~470-line body inlines the full review-skills discovery/configuration flow where a clearer overview-to-reference split would place it, leaving minor organization gaps per the score-4 anchor. | 4 / 5 |
Total | 17 / 20 Passed |