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 well-structured, documentation-grade skill: lean body, clearly sequenced free-assembly steps with named validation checks, and accurate one-level-deep references to real bundle files that carry the executable detail. Main room for improvement is trimming the restated separation rules and surfacing the frame-diff failure/recovery loop in the body itself.
Suggestions
Fold the Contract's restated separations ('Text is an overlay, never baked', 'Any multi-SKU grid is PIL...') into the 'two non-negotiable separations' section once, letting the Contract just reference them, to cut duplicate tokens.
In the 30s-cut step, state the frame-diff failure condition and recovery inline (e.g. 'if the diff shows whole-outline glow you sliced a static intermediate — re-point at silent-master.mp4') rather than only 'frame-diff to prove localized motion'.
Add one minimal inline example of the Remotion DOM-overlay pattern (or a pointer to the exact PIPELINE.md section per step) so the overlay step is as immediately actionable as the audio and cut steps.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense but efficient — it assumes Claude knows ffmpeg/Remotion/PIL and delegates detail to scripts/README.md ('see `scripts/README.md` for the ffmpeg/Remotion/PIL detail'). Minor over-explanation remains: the 'two non-negotiable separations' are restated nearly verbatim in the Contract section ('Text is an overlay, never baked', 'Any multi-SKU grid is PIL of the real product webps'). Not 5: that redundancy could be trimmed; not 3: there is no concept-explainer padding. | 4 / 5 |
Actionability | For an instruction-only (documentation-grade) skill the guidance is concrete: specific filters ('sidechaincompress', 'loudnorm I=-15', 'setpts <=1.6x'), output filenames ('finals/master-final.mp4', 'finals/master-final-30s-v1.mp4'), ordering ('burn captions LAST'), and gate checks ('TEST one scene before batching'), with executable detail one level deep in scripts/README.md. Not 5: the body's step summaries occasionally stop at direction ('composite chips... as animated DOM on top') without a copy-paste-ready command or minimal Remotion snippet inline. | 4 / 5 |
Workflow Clarity | The sequence is clear and ordered (paid gen steps -> Remotion overlay -> PIL grid -> captions -> audio mix -> 30s cut), with validation present ('frame-diff to prove localized motion', 'TEST one scene before batching') and the README adding a mandatory QC checklist with a recovery loop ('re-point the cut at silent-master.mp4'). Not 5: the body itself names frame-diff but does not state what to do when it fails (whole-outline glow) — that feedback loop lives only in the README. | 4 / 5 |
Progressive Disclosure | Clean overview-plus-references structure: the body is a lean map and each detail file is clearly signaled and real — 'see `scripts/README.md` for the ffmpeg/Remotion/PIL detail', `scripts/config.example.json` (the worked schema example), and `scripts/PIPELINE.md` (the field-to-script map) all exist in the bundle, one level deep, no nested-reference chains. Not 4: navigation is unambiguous and every referenced path resolves. | 5 / 5 |
Total | 17 / 20 Passed |