Content
77%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 highly actionable and the five-phase workflow is exceptionally well sequenced with exit-code gates and a fix-and-retry checklist. Its weaknesses are self-inflicted: it repeats its own rules in multiple sections despite mandating 'each concept appears exactly once', and it references bundle files that are not present while inlining two large scripts that belong in scripts/.
Suggestions
Move the Phase D and Phase E scanner scripts into scripts/ files (mirroring test_doc_code_blocks.py) and invoke them by command, removing ~125 lines of inline Python from SKILL.md.
Actually include API_REFERENCE.md, DOCS_STRUCTURE.md, and scripts/test_doc_code_blocks.py in the skill bundle, or remove the 'Reference Files' section — the current references are dead paths.
Consolidate the duplicated rules (no emojis, '1000+ tools', single setup entry point) into the 'Apple-Style Simplification Rules' section and reference them from the checklist instead of restating them.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The scanners, fix tables, and checklists are dense and project-specific, but the body violates its own 'each concept appears exactly once' rule: the no-emoji rule, the '1000+ tools' rule, and the setup entry point are each stated in 2-3 places, and the philosophy is duplicated across 'Two Equal Priorities' and 'Less Is More'. | 3 / 5 |
Actionability | Fully executable throughout: complete heredoc Python scanners, exact commands (e.g., the rg tool-count sweep), a wrong-to-correct method fix table, and a runtime-failure table with causes and fixes. | 5 / 5 |
Workflow Clarity | Phases are explicitly ordered ('D first, then E, then C/A/B') with a summary table, exit-code gates on every scan, and a validation checklist with an explicit feedback loop ('stop, fix it, re-run the relevant phase, confirm it passes'). | 5 / 5 |
Progressive Disclosure | A clearly signaled 'Reference Files' section exists, but API_REFERENCE.md, DOCS_STRUCTURE.md, and scripts/test_doc_code_blocks.py are absent from the bundle, and ~125 lines of scanner Python are inlined for Phases D/E while Phase C is (inconsistently) invoked as a script file. | 3 / 5 |
Total | 16 / 20 Passed |