Content
50%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 SKILL.md is concise and well-sectioned but almost entirely generic: it describes a templated workflow without pointing to the concrete, executable ffmpeg scripts that actually live in ./scripts/. This leaves it non-actionable and without validation checkpoints for a destructive batch media operation.
Suggestions
Replace the generic 'Run the ... script described by this skill' step with the actual commands, e.g. 'bash scripts/composite_final.sh edit_plan.json vo.mp3 subtitles.ass music.wav out.mp4 [duration]' and link normalize_clip.sh for per-segment prep.
Add an inline validation checkpoint between passes (e.g. probe nosubs.mp4 duration before burning subtitles) and a feedback loop on ffmpeg/libass failure, since this is a destructive batch media operation.
Add a '## Scripts' section that one-level-deep references each bundle file (scripts/composite.py, scripts/composite_final.py, scripts/normalize_clip.sh) with a one-line purpose so the bundle is discoverable from SKILL.md.
Document the edit_plan.json shape (duration, aspect, cuts[], audio, subtitles, music) inline or via a reference, since both composite scripts depend on it.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is short and avoids explaining ffmpeg concepts Claude already knows; it could be marginally tighter (the generic Workflow/Output/Quality Checks boilerplate is somewhat padded) but is largely efficient. | 4 / 5 |
Actionability | Despite real, executable bundle scripts existing in ./scripts/ (composite.py, composite_final.py, normalize_clip.sh), the body never names or invokes them — it gives only high-level hints like 'Run the provider, script, or planning workflow described by this skill' with no concrete command, path, or edit_plan schema reference. | 2 / 5 |
Workflow Clarity | A five-step sequence is listed, but for a batch/destructive media operation the validation is only a generic post-hoc Quality Checks list; there are no inline validate-then-proceed checkpoints or feedback loops, capping clarity at 3 per the destructive/batch rule. | 3 / 5 |
Progressive Disclosure | The body has clear section headers but never signals the existing bundle scripts (composite.py, composite_final.sh, normalize_clip.sh, voiceless/composite.py) as references, so the split between SKILL.md and the scripts folder is not surfaced for navigation. | 3 / 5 |
Total | 12 / 20 Passed |