Content
88%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.
An exemplary orchestrator skill body: every step is executable, gated, and validated with feedback loops, and detail is properly deferred to one-level-deep references with a read-when table. The two deductions are a broken reference (`sub-agents/frame-worker.md` is missing from the bundle) and modest redundancy in restating the bpm-trust caveat.
Suggestions
Add the missing `sub-agents/frame-worker.md` to the bundle — it is cited in Step 4, the Quick Reference, and the Directory layout but is absent.
State the bpm/beats_sec trust caveat once (in 'Two ideas') and reference it from Steps 1–2 instead of restating it three times.
Tighten Step 0's auth-status paragraph by deferring the sign-in/offline decision detail to the `/hyperframes-media` reference it already cites.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and purpose-built: every step carries commands, gates, or constraints, and it never explains concepts Claude already knows. Minor trimmable redundancy keeps it off anchor 5 — the bpm/beats_sec trust caveat is restated in 'Two ideas that shape everything', again in Step 1, and again via Step 2's pacing, and Step 0's auth-status paragraph is a long digression. Not anchor 3: there is no padding or gratuitous explanation, only deliberate (if repeated) reinforcement. | 4 / 5 |
Actionability | Fully executable throughout: copy-paste-ready commands for every step (`npx hyperframes init ... --non-interactive --example=blank`, the `pip install librosa numpy soundfile` fallback, `python3 <SKILL_DIR>/scripts/analyze-beatgrid.py ... -o ... --print`, `validate-plan.mjs`, `assemble-index.mjs`, the lint/validate/inspect chain, and the final `render ... -o renders/video.mp4 --fps 30`), plus a fully specified sub-agent dispatch context template with concrete placeholder semantics. This matches anchor 5's copy-paste-ready coverage of the common cases. | 5 / 5 |
Workflow Clarity | Steps 0–6 are clearly sequenced with an explicit **Gate** checkpoint after each, user-approval gates at Steps 3 and 6, and machine validation with error-recovery feedback loops: 'Fix every `✗` (hard errors...)', 'a missing or blank frame file means that worker wrote a partial file; re-dispatch it (Step 4) and re-assemble', and install-on-import-failure at Step 1. The batch operation (one sub-agent per frame) is validated at Steps 4–6, and a Resume table handles mid-pipeline restarts — a full match to anchor 5. | 5 / 5 |
Progressive Disclosure | Good structure overall: the body is an orchestrating overview, detailed material is split one level deep into `references/` and read exactly when needed ('Read [`references/frame-skeleton.md`](references/frame-skeleton.md)' at Step 2; the planning/template/montage references at Step 3), and a Quick Reference table maps each read to its step. It falls short of anchor 5 because `sub-agents/frame-worker.md` — referenced in Step 4, the Quick Reference, and the Directory layout — does not exist in the bundle, a broken in-bundle reference, and several cross-skill paths (`../hyperframes-core/...`, `../hyperframes-creative/...`) are unverifiable here. Not anchor 3: references are clearly signaled, one level deep, and the split is otherwise appropriate. | 4 / 5 |
Total | 18 / 20 Passed |