Content
82%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 high-quality, densely written instruction body: an unambiguous five-step workflow, a concrete suite-selection table, executable validation commands, and well-split bundle files. The remaining gaps are a missing error-recovery loop after validation, a few trimmable rationale sentences, and out-of-bundle guide links that break the one-level-deep navigation.
Suggestions
After the `cfs validate` commands, add one explicit recovery line (e.g. 'If validation fails: fix the artifact or TOC entry and re-run all three before checking any DoD item') to close the workflow's feedback loop.
Trim the parenthetical justifications Claude can infer — e.g. why a real-traffic percentile beats a synthetic run — and let scripts/counts.sh's own header carry the MOVED rationale instead of restating it in the body.
Copy the essential vector definitions (or a summary table) into references/ and keep the `../../../.cf-studio/...` guide link only as a provenance note, so navigation from SKILL.md stays one level deep and fully within the bundle.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and assumes competence — it never explains what pytest, vitest, dbt or CI is, and nearly every sentence is an instruction or a rule. A few rationale sentences could be trimmed (e.g. "a synthetic run at an invented scale is weaker evidence than a percentile from real use", and restating counts.sh's MOVED rationale in the body when the script's header already documents it), which keeps it at anchor 4's 'minor instances that could be trimmed' rather than anchor 5's every-token-earns-its-place. It is far above anchor 3, which requires actual unnecessary explanation. | 4 / 5 |
Actionability | Guidance is fully executable: a suite-selection table with concrete suite tags and directory paths, the exact `-m` marker mechanism, a literal docstring example (`Verifies cpt-ir-dod-manual-resolution-idempotency`), instructions to take counts from `scripts/counts.sh` rather than quote them, and copy-paste `cfs validate` commands. This matches anchor 5: specific, ready-to-run commands covering the common cases. | 5 / 5 |
Workflow Clarity | Five clearly sequenced sections (resolve → ground → choose suite → link → validate) end in explicit validation commands plus a two-direction chain-review checklist ('each DoD item has a test naming it, or an open decision with an owner'). What is missing is an explicit error-recovery loop — what to do when `cfs validate` or `validate-toc` fails — which keeps it at anchor 4 ('most checkpoints present; minor validation gaps') rather than anchor 5's feedback loops. | 4 / 5 |
Progressive Disclosure | Against the actual bundle: the vector taxonomy is correctly split into references/vector-mapping.md and the counting denominators into scripts/counts.sh, both real files, clearly signaled and one level deep, with the body keeping only the overview-level rules. The gap is that two links point out-of-bundle to `../../../.cf-studio/config/kits/sdlc/guides/quality-vectors.md`, an unverifiable external path that adds a second hop for the definitions — anchor 4's 'minor organization gaps' rather than anchor 5's clean one-level-deep navigation. | 4 / 5 |
Total | 17 / 20 Passed |