Content
75%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, information-dense instruction skill: the contract bullets are highly actionable with specific parameters, fallbacks for host limitations, and an explicit per-scene QC feedback loop. Its main costs are redundancy — the intro restates the description and the Contract section duplicates the README's assembly detail — and the absence of any inline executable command, leaving the run sequence dependent on the bundle files.
Suggestions
Deduplicate the Contract section against scripts/README.md: keep the contract-level rules inline and point to the README for the assembly detail (captions, cut-to-bar, logo bug/end card, ffmpeg composite) instead of restating them in both files.
Trim the opening paragraphs, which restate the frontmatter description nearly verbatim, and remove the duplicated parenthetical in the QC bullet ('off the chosen look — in the felt demo, felt → smooth-3D' appears twice).
Inline the ordered run sequence (or the single key ffmpeg composite command) in the Run section rather than delegating the full step ordering to scripts/PIPELINE.md, so the body alone gives a copy-paste-ready path to the master.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and assumes competence — no explanation of librosa, ffmpeg, or captioning concepts; nearly every sentence carries a project-specific rule. Minor trimming is needed: the opening paragraphs restate the frontmatter description almost verbatim, and the QC bullet duplicates the same parenthetical twice ('off the chosen look — in the felt demo, felt → smooth-3D'). Not 5 because of that redundancy; not 3 because there is no concept over-explanation and almost all tokens earn their place. | 4 / 5 |
Actionability | Concrete guidance throughout: named inputs ('word-timestamps.json', 'bars.json', the brand wordmark SVG), exact filter strings ('overlay=…:enable=\'between(t,st,en)\'', 'loudnorm I=-14'), a fallback path for hosts lacking libass, and specific QC artifacts (2 fps contact sheet, per-bar FACE and HAND crops). Falls short of anchor 5 because the body itself contains no executable commands — the real command-level detail (stitch.py, ffmpeg invocations) lives in the bundle files — and short of a copy-paste-ready standard. | 4 / 5 |
Workflow Clarity | The sequence is clear (song → librosa bar grid → keyframes → one clip per bar → assembly → mux) and validation is explicitly present: per-scene QC with the two named failure modes, an error-recovery loop ('A drifted bar means regenerating that bar's KEYFRAME (not re-cutting)'), and a post-publish re-check of the served bytes. Anchor 4 fits: most checkpoints present with minor gaps, since the fully ordered step-by-step map is delegated to scripts/PIPELINE.md rather than sequenced inline. | 4 / 5 |
Progressive Disclosure | All three referenced bundle files (scripts/config.example.json, scripts/PIPELINE.md, scripts/README.md) exist, are one level deep, and are introduced with a stated purpose ('PIPELINE.md maps every config block to its source step and README.md documents the free assembly'). Not 5 because the Contract section substantially duplicates scripts/README.md sections 1–5 (captions, cut-to-bar, logo bug/end card, ffmpeg composite, QC) rather than pointing to them, an organization gap; not 3 because the split between overview and worked example is real and well-signaled. | 4 / 5 |
Total | 16 / 20 Passed |