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.
A highly actionable, gate-driven orchestration workflow with exact commands, explicit validation and error-recovery loops, and well-signaled progressive disclosure into reference files. The weaknesses are modest: some redundant restatement in the opening sections and one or two bundle/reference inconsistencies (missing sub-agents file, incomplete background-scripts list).
Suggestions
Deduplicate the opening: 'You are the orchestrator' and the routing rules are stated twice before Step 0 — fold them into one statement and let Step 0's opening rule carry the routing detail.
Add `sub-agents/frame-worker.md` to the bundle (or fix the path), since Step 5 and the packet builder treat it as the frame-worker role delta.
Include `frame-packets.mjs` in the Quick Reference's 'Background scripts' list so the shipped-scripts inventory matches what Step 5 actually invokes.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense, imperative, and free of concepts Claude already knows, but carries minor redundancy that could be trimmed: "You are the orchestrator" is stated twice in the opening paragraphs, and the front-door/routing rules are restated across the intro and Step 0's opening rule. This matches the 'efficient; minor instances of over-explanation' anchor rather than the fully lean one. | 4 / 5 |
Actionability | Every step carries exact, copy-paste-ready commands with concrete paths and flags (e.g. `npx hyperframes init "videos/<project>" --non-interactive --example=blank --skill=faceless-explainer`, `node <SKILL_DIR>/scripts/audio.mjs sync-durations --audio-meta ./audio_meta.json --storyboard ./STORYBOARD.md`), plus explicit expected artifacts and branch conditions covering the common cases. | 5 / 5 |
Workflow Clarity | Steps 0–6 are strictly sequenced, user-gated steps are named up front, every step ends in an explicit **Gate**, and validation with a feedback loop is present (lint/check/snapshot, "If a command fails, surface stderr and stop — fix it yourself… then rerun the failed check", plus the documented `check` false-positive and the review loop's approve/revise cycle). | 5 / 5 |
Progressive Disclosure | Structure is good: method detail is pushed to clearly signaled one-level-deep references with a Read/When table in the Quick Reference, and the four local `references/` files all exist. Minor gaps keep it below a 5: the locally-referenced `sub-agents/frame-worker.md` is absent from the bundle, and the Quick Reference's "Background scripts" list omits `frame-packets.mjs`, which Step 5 actually invokes. | 4 / 5 |
Total | 18 / 20 Passed |