Content
78%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 lean, highly actionable skill body: exact commands, concrete encoding/layout specifics, and documented failure modes with remedies. The main gaps are the missing tests/ directory referenced by the regression command and the absence of an explicit final-output verification step.
Suggestions
Ship the tests/ directory (tests/test_*.py) that 'python -m unittest discover -s tests' references, or correct the command to point at wherever the regressions actually live — as written it fails out of the box.
Trim the opening paragraph, which restates the frontmatter description nearly verbatim, and tighten the 'Music coverage' section since references/music.md already carries the detail.
Add an explicit final verification step (e.g., ffprobe master-final.mp4 for duration/resolution and a spot-check that pill text is pixel-crisp) to close the workflow's validation loop.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence (e.g., 'Bold weight and icon-centered-on-pill-middle are load-bearing', 're-encode crf23/maxrate12M'), with no explanation of concepts Claude already knows. It is not a 5 because the opening paragraph restates the frontmatter description almost verbatim and the 'Music coverage' section overlaps references/music.md, so a few tokens could be trimmed. | 4 / 5 |
Actionability | The Run section gives exact commands with flags and paths ('fetch_icons.py --run-dir <run> ; build_overlays.py --config config.json --out-dir <run>/generated/overlays ; compose_master.py --config config.json --run-dir <run>') and the Contract pins concrete details (1080x1920, crf23/maxrate12M, Pillow + ffmpeg). It is not a 5 because the regression instruction 'Run the free regressions with python -m unittest discover -s tests -p test_*.py -v' points at a tests/ directory that does not exist in the bundle. | 4 / 5 |
Workflow Clarity | The three-step pipeline (fetch icons -> build overlays -> compose master) is sequenced with the critical 'Always re-run build_overlays.py before compose_master.py' warning, and failure behavior plus remedies are documented ('composer... stops if copy does not fit; wrap the text or reduce its font size', 'build_overlays.py exits with a clear message if any of it is empty', music checks 'stops for missing, short or prematurely silent music'). It is not a 5 because validation is delegated to tool behavior and there is no explicit verification of the final master output. | 4 / 5 |
Progressive Disclosure | The body is a well-organized overview with a clearly signaled, one-level-deep reference ('Read the [music preparation instructions](references/music.md)') that exists in the bundle, and each script gets a one-line role description. Content is appropriately split between SKILL.md, the reference, and the scripts, so navigation is easy. | 5 / 5 |
Total | 17 / 20 Passed |