Content
67%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 well-structured, instruction-dense skill body with strong validation checkpoints and appropriately externalized detail guides. Its main costs are redundant parallel workflow sections, repeated resource listings, and unverifiable (absent) bundle files referenced throughout.
Suggestions
Collapse the three overlapping workflow framings (Scenes, Structural Flow, Default Workflow) into one canonical sequence to remove redundancy.
List the resources/ files once (References only) and reference that list from Dependencies and Tools instead of repeating it.
Add one inline concrete example (e.g. a minimal schema-doc snippet or sample expand-contract migration) to ground the abstract ACT/REASON scene descriptions.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and imperative with no concept-explaining fluff, but it carries clear redundancy: the PREPARE–ACQUIRE–...–FINALIZE "Scenes", the Entry/Transitions flow, and the Explore/Design/Optimize "Default Workflow" restate the same process three times, and the resource list is repeated in Dependencies, Tools, and References. Not 4 because several sections could be cut or merged without losing information. | 3 / 5 |
Actionability | Provides executable commands (the rg file/grep invocations), a concrete "run the project's migration, query-plan, or retrieval-quality commands" step, and highly specific instructions (expand-contract with dual-write + backfill + soak window; measure→explain→optimize→re-measure). Not 5 because several scene/action descriptions remain abstract ("Model entities/aggregates", "Produce schema docs") with no inline example or template snippet. | 4 / 5 |
Workflow Clarity | Multi-step processes are clearly sequenced with a VERIFY scene, a pre-handoff self-check (resources/checklist.md), a failure-and-recovery section, and explicit validation loops for destructive operations (verified read switch, re-measure), so the destructive-operation cap does not apply. Not 5 because three overlapping workflow framings (Scenes, Structural Flow, Default Workflow) leave the canonical path slightly ambiguous. | 4 / 5 |
Progressive Disclosure | The References section is well-signaled with one-line purpose labels (e.g. "Self-check (run before handoff): resources/checklist.md") and references are one level deep, with detailed guides split into separate files. Not 5 because no bundle files were present to verify the referenced resources/*.md and ../_shared files actually exist, and the resource list is duplicated across Dependencies, Tools and instruments, and References. | 4 / 5 |
Total | 15 / 20 Passed |