Content
88%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-engineered body: concrete commands with expected results, a validated procedure with a stop-and-repair loop, and appropriate offloading of detail to a single one-level-deep reference. The only deductions are mild repetition of the index-invisibility theme across the Mindset and Anti-Patterns sections, and one undocumented bundle script.
Suggestions
Document or remove scripts/check-wave-evidence.sh: it ships in the bundle but is referenced neither in SKILL.md (Quick Commands / Procedure) nor in references/regeneration-reference.md, so an agent cannot discover it.
Merge the three reference-table rows that all point to references/regeneration-reference.md into one row (or link section anchors) so the References table reads as distinct navigation rather than repetition.
Trim the duplicated 'malformed frontmatter makes a file invisible' explanation across Mindset and the second/third Anti-Patterns — state it once in Mindset and let the Anti-Patterns reference it, cutting ~10 lines.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with project-specific operational knowledge (exact commands, expected results, exit codes, env vars) and never explains concepts Claude already knows, but the 'invisible/excluded from the index' point is repeated across the Mindset section and two anti-patterns, and each anti-pattern's Consequence line restates its WHY (e.g. "functionally the same as it not existing" vs. "quietly disappears from every summary"). This is efficient-with-minor-trimming rather than the lean every-token-earns-its-place anchor. | 4 / 5 |
Actionability | Every script has a copy-paste-ready invocation with its expected result and exit semantics, e.g. "./scripts/check-context-filenames.sh ... Expected result: `context filenames OK`, or a list of naming violations", plus the full frontmatter schema, the CONTEXT_STALENESS_THRESHOLD_DAYS knob, and five --split flag combinations. Fully executable guidance covering the common cases. | 5 / 5 |
Workflow Clarity | The Procedure is a numbered sequence with an explicit validation checkpoint in step 1 ("**Verify:** the script prints nothing and exits 0"), a stricter validation step before CI gates, and an explicit feedback loop in step 4 ("**Stop if:** it reports files excluded on stderr; go back to step 2"). For a batch operation over every context file, validation and error-recovery loops are present and explicit, matching the top anchor. | 5 / 5 |
Progressive Disclosure | Good structure overall: the body stays an overview, the single reference (references/regeneration-reference.md) is real, one level deep, and clearly signaled by a topic/when-to-use table, and the typology catalog is correctly delegated to the companion skill. Minor gaps: three of the four reference-table rows point to the same file with overlapping scopes, and the bundled scripts/check-wave-evidence.sh is not referenced anywhere in SKILL.md or the reference, leaving an undocumented script in the bundle. This fits 'good structure with minor organization gaps' rather than the cleanly-navigable score-5 anchor. | 4 / 5 |
Total | 18 / 20 Passed |