Content
62%Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.
This skill is highly actionable and has excellent workflow clarity with comprehensive error handling and validation checkpoints. However, it is severely over-verbose — the sketch-aware promotion logic, repeated preference check patterns, and exhaustive conditional branching inflate the token cost dramatically. Much of this detail (sketch manifest handling, preference lookup chains, false-negative follow-up logic) should be extracted into reference files rather than inlined in the main skill body.
Suggestions
Extract the sketch promotion logic (Step 2c and Step 4's sketch-aware sections) into a dedicated reference file like `references/sketch-promotion.md` — this alone would save ~100 lines from the main body.
Abstract the repeated preference check pattern (used identically for sketch-preview and simplification) into a single reference file like `references/preference-gate.md` with parameterized instructions, then reference it twice with different parameters.
Compress the specialist pre-check and false-negative follow-up sections (Step 2a-2c) — the term lists and boolean logic could be in a reference file, with the main body just saying 'Apply specialist pre-check per references/specialist-pre-check.md'.
Remove the Pipeline Position ASCII diagram and the detailed Artifacts table at the end — these are informational rather than instructional and add tokens without changing Claude's behavior.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | This skill is extremely verbose at ~400+ lines, with extensive procedural detail that could be dramatically compressed. It explains preference lookup chains, sketch manifest status handling, and conditional branching in exhaustive detail that Claude could infer from much shorter instructions. Many sections repeat patterns (preference check → gate → follow-up) that could be abstracted into a shared reference. | 1 / 3 |
Actionability | The skill provides highly specific, concrete guidance at every step: exact file paths, specific JSON field names and values, precise conditional logic, exact user-facing messages, and clear agent dispatch instructions. Every step has unambiguous deliverables and the artifact schemas are fully specified. | 3 / 3 |
Workflow Clarity | The workflow is clearly sequenced with numbered steps, explicit validation checkpoints (spec-lite approval, plan approval, test verification, review-lite with verdict logic), and well-defined feedback loops (self-heal up to 3 attempts, re-review after fixes, adjust cycles). Error handling is comprehensive with specific recovery paths for each failure mode. | 3 / 3 |
Progressive Disclosure | The skill references external files appropriately (step-0-fast-path.md, specialist-pre-check.md, standard-plan-template.md, swift-review-checklist.md, pattern-refresh.md, preferences-schema.md), but the main body contains enormous amounts of inline detail that should be in reference files — particularly the sketch promotion logic (~100 lines), preference check patterns (repeated 2x), and the detailed componentMapping handling. No bundle files were provided to verify references exist. | 2 / 3 |
Total | 9 / 12 Passed |