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 high-quality instruction skill body: lean, dense with non-obvious doctrine, fully sequenced workflows with explicit validation gates and error-recovery fallbacks, and a genuinely one-level-deep bundle. Remaining polish is minor — deduplicating repeated activation fallback text, showing one inline example of an AD/memlog entry, and giving the spine template an explicit path.
Suggestions
Deduplicate the near-identical 'Script not found → offer bmad setup' fallback text in Activation steps 1 and 2 into a single shared fallback rule, freeing tokens in the most token-expensive section.
Include one minimal inline example of a logged memlog decision entry and a resulting AD-n (Binds/Prevents/Rule) so the common create path is executable without loading the template.
Name the spine template explicitly (e.g. 'seed the spine from assets/spine-template.md') alongside the {workflow.spine_template} variable so the asset is discoverable from the body without config resolution.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is exceptionally dense with no explanations of concepts Claude already knows — every section carries skill-specific doctrine (the spine test, altitude rules, memlog mechanics). It falls just short of anchor 5 because of minor redundancy: the 'Script not found… offer to run the bmad skill's setup, installing bmad first' fallback is repeated nearly verbatim in Activation steps 1 and 2, and the Coaching-path default is restated three times across sections. | 4 / 5 |
Actionability | Concrete, copy-paste-ready commands throughout ('uv run {project-root}/_bmad/scripts/memlog.py init --workspace …', 'uv run … resolve_customization.py --skill … --key workflow', 'npx skills add bmad-code-org/BMAD-METHOD --skill bmad') with explicit fallback paths for each failure mode. It misses anchor 5 only because artifact shapes — an AD-n with Binds/Prevents/Rule, a memlog entry — are never exemplified inline, relying on the config-mediated template and bundle files for the common cases. | 4 / 5 |
Workflow Clarity | Multi-step processes are clearly sequenced (numbered Activation, 8-step Finalize, Update, Validate) with explicit validation checkpoints (deterministic lint_spine.py pass, reviewer gate, reconcile-inputs subagents, one-at-a-time blocker triage) and genuine feedback loops ('Resolve before polish', 'If validation fails' style fallbacks like re-running after setup or handoff). This matches the top anchor including its checklist-for-complex-processes criterion, which reviewer-gate.md's good-spine checklist supplies. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned, and the two reference files (references/headless.md, references/reviewer-gate.md) exist, are one level deep, and are clearly signaled with load conditions ('Headless… follow references/headless.md for the whole run', 'full mechanics in references/reviewer-gate.md. Load it when finalizing or validating'). The minor gap keeping it from anchor 5: assets/spine-template.md is reachable only through the {workflow.spine_template} config variable rather than an explicit path, and the long dense body could lean slightly more on the bundle. | 4 / 5 |
Total | 17 / 20 Passed |