Content
76%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 highly actionable, token-efficient body with executable examples for every media type and a clear resolution cascade. Its main weakness is the absence of validation/verification checkpoints for the freeze and batch-adopt operations, which caps workflow clarity, alongside version-sensitive install details embedded in the main flow.
Suggestions
Add explicit validation steps to the workflow: after freeze, verify the file exists and probes cleanly (ffprobe duration/dimensions), and after `--adopt`, verify the manifest records and report any assets that failed to probe, with a fix-and-retry path.
Move the heygen install instructions and the '>= v0.1.6' version requirement into a short 'Setup' or 'Deprecated/old versions' section so time-sensitive details don't sit in the main operational flow.
Briefly document the `scripts/lib/` module layout (one line each) so an agent diving deeper than resolve.mjs can navigate the bundle without reading every file.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is lean and operational — no padding, no explanations of concepts Claude already knows, and every section earns its tokens. It falls short of the level-5 anchor because of unpinned-down time-sensitive details: the 'heygen >= v0.1.6' version requirement and install/version instructions sit in the main flow rather than a deprecated/old-patterns section, which the guidelines say should penalize conciseness. | 4 / 5 |
Actionability | Every command is copy-paste ready: the exact resolve invocation with flags, worked examples for all four types with real output lines ('resolved bgm_001 → .media/audio/bgm/bgm_001.mp3 (bgm, 25s)'), an adopt example, the flags table, and install/auth commands. Specific examples cover the common cases, matching the level-5 anchor. | 5 / 5 |
Workflow Clarity | The 'How it works' section gives a clear 5-step cascade and the missing-CLI failure path is described, but there are no validation or verification checkpoints anywhere — nothing confirming the frozen file exists/probes correctly, and no verify step after the batch `--adopt` import. Per the guidelines, batch operations without validation cap workflow_clarity at 3, and this cap applies despite the otherwise clean sequence. | 3 / 5 |
Progressive Disclosure | The body is a well-organized overview with clearly signaled sections, and the referenced bundle path (`<SKILL_DIR>/scripts/resolve.mjs`) is a real file exactly one level deep, keeping search noise in scripts as designed. Minor gaps keep it at level 4 rather than 5: the ~120-line body inlines some material (install/version instructions, the index sample) that could live in a reference file, and the `scripts/lib/` modules and tests are present in the bundle but undocumented. | 4 / 5 |
Total | 16 / 20 Passed |