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.
The body is a well-structured, mostly actionable overview that points to real bundle files rather than inlining them. Its weaknesses are noticeable redundancy with the frontmatter description, a missing final verification step, and one dangling bundle path (`shared/beat-templates/`).
Suggestions
Trim the opening paragraph to remove facts already stated in the frontmatter description (sound-off legibility, claim word limits, SKU rotation, FREE status) and keep them only in '## Contract', cutting tokens with no loss of information.
Add an explicit post-render validation step, e.g. 'Confirm `<dir>/finals/master.mp4` exists and its duration lands in the 10-20s window before shipping'.
Fix or remove the `shared/beat-templates/` reference (it is not in the bundle), or add the directory to `scripts/` so every referenced path resolves.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with project-specific detail and explains nothing Claude already knows, but the opening paragraph substantially repeats the frontmatter description (sound-off legibility, <=4-word claims, SKU rotation, FREE/no paid calls, music-separate are each restated in '## Contract'), so it could be tightened. Not a 4 because the description/body duplication is more than a minor instance — roughly half the intro re-states facts already in the loaded frontmatter. | 3 / 5 |
Actionability | Concrete guidance throughout: `render_master.py --config config.json --project <dir>` with its output path `master-clean.mp4` and resolution, a named config schema (`ad_sample.recipe.config`) plus `config.example.json`, env-var overrides (`RENDER_PYTHON`, `FFMPEG`), and named optional scripts that all exist in `scripts/`. Not a 5 because the run command is not copy-paste ready (no interpreter/cwd shown) and no example config snippet or invocation of the preview scripts is given inline. | 4 / 5 |
Workflow Clarity | The sequence is clear for a single-command skill: author `config.json` per the schema, optionally gate with `build_storyboard_preview.py`, run `render_master.py`, receive the silent master; the preview gallery acts as a checkpoint. Not a 5 because there is no explicit post-render verification (e.g., check the output file exists/plays, confirm duration in the 10-20s window) — checkpoints are present but the final validation is left implicit. | 4 / 5 |
Progressive Disclosure | Good structure for an under-50-line skill: three well-organized sections (Run, Choices, Contract) with config schema details pushed to `config.example.json` and per-script behavior summarized in one line each; nearly all referenced bundle paths (`render_hyperframe.py`, `render_master.py`, `build_storyboard_preview.py`, `build_text_overlays.py`, `config.example.json`) actually exist. Not a 5 because `shared/beat-templates/` is referenced but does not exist anywhere in the bundle, a dangling reference that breaks otherwise clean one-level-deep navigation. | 4 / 5 |
Total | 15 / 20 Passed |