Content
90%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.
An exemplary lean, fully executable skill: every step is a runnable command, sequencing is clear, and it captures hard-won environment gotchas (Node 24 for CI parity, silent broken-anchor warnings). The only improvements are structural — move the inline Python checker to a script file and add an explicit fix-and-retry loop for reported failures.
Suggestions
Add a short feedback loop after the image and cross-reference checks, e.g. 'fix the src or reST role, then re-run the check until it reports no findings'.
Move the inline Python image checker into `scripts/check_broken_images.py` and reference it with a one-line invocation to keep SKILL.md as a lean overview.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Every section is a copy-paste command plus only genuinely non-obvious rationale (e.g. the `onBrokenAnchors` default-to-warn gotcha). No concepts Claude already knows are explained; every token earns its place. | 5 / 5 |
Actionability | Fully executable throughout: `nvm use 24`, `yarn build`/`yarn start`/`yarn crocodocs:generate`, a complete 24-line Python image checker, and a specific `grep -r` for reST roles. No pseudocode or vague direction. | 5 / 5 |
Workflow Clarity | Sequencing is present ('Run this after `yarn build`') and the broken-links section adds a real validation caveat ('grep the build output for `Broken anchor` rather than trusting the exit code'). Not 5 because there is no explicit fix-and-re-run feedback loop after the image or cross-reference checks report failures. | 4 / 5 |
Progressive Disclosure | Well-organized sections with no dead references and no nested-reference problem. Not 5 because the 26-line inline Python image checker is content that arguably belongs in a `scripts/` bundle file rather than in SKILL.md. | 4 / 5 |
Total | 18 / 20 Passed |