Content
27%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
The skill is a thorough but bloated single-file roleplay script: it over-specifies dialogue Claude could generate itself, lacks bundle-file decomposition, and uses conversational waits in place of concrete validation gates. Actionability and sequencing are present but diluted by the verbosity.
Suggestions
Move the scripted party-mode dialogue into a referenced file (e.g. DIALOGUE.md) and keep SKILL.md as a concise overview of the 13-step workflow with the concrete commands and file contracts, fixing the progressive-disclosure and conciseness issues.
Replace conversational 'WAIT for {user_name}' placeholders with explicit validation checkpoints (e.g. confirm epic_number is set before proceeding; validate YAML after writing sprint-status and re-try on parse failure) to lift workflow clarity to 3.
Cut explanatory narrative about retrospectives and facilitation philosophy that Claude already knows; keep only the BMAD-specific contracts (paths, scripts, status keys, output filename) so every token earns its place.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | At ~1520 lines, the body is a monolithic wall of pre-scripted roleplay dialogue and templated placeholders that pads the context with content Claude does not need to be told how to produce; it violates the token-budget principle that skills should add only what Claude doesn't already know. | 1 / 3 |
Actionability | It provides some concrete executable guidance (real commands like 'python3 {project-root}/_bmad/scripts/resolve_customization.py', file path patterns, and a YAML action_items snippet to write), but the bulk of the actionable steps are buried under scripted narrative and placeholder variables rather than crisp instructions. | 2 / 3 |
Workflow Clarity | The 13 steps are clearly numbered and sequenced with branching <check> blocks, but validation checkpoints are largely conversational ('WAIT for {user_name}...') rather than concrete verification gates, and the destructive sprint-status/YAML writes in Step 12 lack an explicit validate-then-retry feedback loop. | 2 / 3 |
Progressive Disclosure | Everything lives in a single monolithic SKILL.md with no bundle files and no one-level-deep reference pointers; the few external paths referenced are framework scripts (resolve_customization.py, config.yaml), not skill-owned reference materials that decompose the content. | 1 / 3 |
Total | 6 / 12 Passed |