Content
70%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.
The body is an exceptionally clear, gate-driven workflow with verbatim scripts and strong feedback loops — top marks for actionability and workflow clarity. It is dragged down by systemic redundancy (the workflow restated four ways plus generic design-principle padding) and by a progressive-disclosure failure: the referenced visual-companion.md is missing from the bundle and the scripts/ files are unreferenced.
Suggestions
Collapse the workflow to a single representation: keep the ordered Checklist and cut the dot digraph and 'Key Principles' recap, which restate it verbatim — roughly 35 lines saved with no information loss.
Trim or remove 'Design for isolation and clarity' and 'Working in existing codebases' down to one-line pointers; modularity and 'follow existing patterns' are knowledge Claude already has.
Fix the dangling `skills/brainstorming/visual-companion.md` reference (ship the file or inline its essentials) and add a short 'Visual companion tooling' section pointing at the actual `scripts/` files (start-server.sh / stop-server.sh) so the bundled infrastructure is discoverable.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The same 9-step workflow is presented four times — the Checklist, the 29-line dot digraph, the 'The Process'/'After the Design' prose, and the 'Key Principles' recap — and 'Design for isolation and clarity' lectures on concepts Claude already knows ('Smaller, well-bounded units are also easier for you to work with - you reason better about code you can hold in context at once'). That is several padded/generic sections, matching anchor 2; it avoids anchor 1 because the core dialogue guidance (offer scripts, gates, review checks) is skill-specific and not background explanation, and doesn't reach anchor 3 because the redundancy is systemic rather than a few stray sentences. | 2 / 5 |
Actionability | Guidance is copy-paste ready throughout: the verbatim Visual Companion offer ("Some of what we're working on might be easier to explain if I can show it to you in a web browser..."), the verbatim spec-review message ("Spec written and committed to `<path>`..."), the exact spec path pattern `docs/superpowers/specs/YYYY-MM-DD-<topic>-design.md`, and the four named self-review checks. Per the rubric's instruction-skill note, absence of code isn't penalized when guidance is this concrete; it exceeds anchor 4 because the specific examples cover the common cases (accept/decline companion, approved/changes-requested spec review). | 5 / 5 |
Workflow Clarity | The 9-step checklist is strictly ordered with a HARD-GATE up front, explicit validation checkpoints (user approval after each design section, spec self-review, a dedicated User Review Gate), and explicit feedback loops ("User approves design?" -> 'no, revise'; "User reviews spec?" -> 'changes requested' loops back). This matches anchor 5 ('clear sequence with explicit validation steps; feedback loops for error recovery; checklists for complex processes'); no destructive or batch operations are involved, so no cap applies. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned and its one reference is clearly signaled ("read the detailed guide before proceeding: `skills/brainstorming/visual-companion.md`"), but scored against the actual bundle that path is dangling — no visual-companion.md exists anywhere in the skill — and the five real files in `scripts/` (server.cjs, start-server.sh, stop-server.sh, helper.js, frame-template.html) are never referenced from SKILL.md, leaving the bundle's infrastructure undiscoverable. This sits between anchor 3 (structure present but organization flawed) and anchor 4 (minor gaps only); a broken primary reference plus orphaned bundle files is more than a minor gap, so 3. | 3 / 5 |
Total | 15 / 20 Passed |