Content
88%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 disciplined, metrics-first pipeline skill: concrete commands for every step, explicit validation and regen feedback loops, verbatim customer-facing copy, and clean one-level-deep reference structure. The only real defects are a missing referenced file (lib/queries.mjs) and some repetition of the scope-resolution rules across three sections.
Suggestions
Fix or remove the broken reference to lib/queries.mjs (SKILL.md line 75) — the file does not exist in the bundle; either add it under a lib/ directory or point to the actual location of the metric query registry.
Consolidate the scope-resolution guidance (linking, --scope, unscoped-usage risk) into one section — it currently appears in Prerequisites, the Step 1 prose, and the §1.1 blocker list with overlapping content — and reference it from the other two spots.
Tighten rule passages such as the Workflow-runtime and streaming guidance by stating the rule and its evidence requirement once, dropping the repeated rationale sentences.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational — nearly every line is a rule, command, or verbatim template, with no explanation of concepts Claude already knows. It falls short of a 5 because of minor redundancy: scope-resolution guidance (linking, --scope, unscoped usage risks) is repeated across Prerequisites, the Step 1 prose, and the §1.1 blocker list, and a few rule passages could be tightened without losing meaning. | 4 / 5 |
Actionability | Every pipeline stage has copy-paste-ready bash with real script paths, arguments, and output redirection; blocker checks are given as an exact jq command; user-facing prompts are provided verbatim; and edge cases (unsupported framework, quota exhaustion, regen loop) each carry an exact command or template. This is fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | The pipeline is explicitly sequenced (Collect → Blockers → Gate → Deep-dive/Reconcile → Briefs/Investigate → Collect outputs → Verify → Render) with validation checkpoints at each juncture: a JSON.parse check after collection, a blocker gate before investigation, collector failure handling for missing/duplicate/mismatched outputs, and a verify-and-regen feedback loop that reruns failed briefs and keeps regenerated output only if verification improves. | 5 / 5 |
Progressive Disclosure | The body is a well-organized overview with clearly signaled one-level-deep links (doctrine, data-collection, candidates, recommendations, verification, scoring, scanner-patterns, observability-plus, voice, docs-library.json), and detailed material is appropriately deferred to those files. It is not a 5 because one referenced path, lib/queries.mjs ("The metric registry is lib/queries.mjs"), does not exist in the bundle — a broken reference that undermines navigation. | 4 / 5 |
Total | 18 / 20 Passed |