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 strong operational skill body: fully executable commands, explicit validation and recovery loops, and disciplined pointers to external docs. Minor gains available from de-duplicating warnings and pushing deeper troubleshooting detail behind references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense with repo-specific facts (no explaining of concepts Claude already knows), but has minor trim opportunities: the AGENT_DEVICE_STATE_DIR / daemon-kill warning appears in both the Environment paragraph and the Single-replay section, and 'Platform coverage' spends several lines on UI internals that could be tightened. Not a 5 for those redundancies; not a 3 because there is no real padding or filler. | 4 / 5 |
Actionability | Fully executable: the exact command with an argument table, defaults, env-var overrides (APP_ID, REPLAY_TIMEOUT_MS, AGENT_DEVICE_STATE_DIR), a copy-paste single-replay code block, a pre-run checklist, and a symptom→action troubleshooting table. This matches the 'fully executable, copy-paste ready, covers common cases' anchor. | 5 / 5 |
Workflow Clarity | Clear sequence (checklist → command → failure handling) with explicit validation checkpoints: the runner re-checks every @pre/@post around each replay, fails on the first replay that emits no span line and prints the current screen, and the 'If something fails' table gives feedback loops for recovery. Retry discipline ('Stop after the first failed replay... Retry only after an explicit reset') is stated explicitly — matching the top anchor. | 5 / 5 |
Progressive Disclosure | Good structure: a layout tree, clearly signaled one-level-deep pointers to `agent-device/flows/README.md` and `agent-device/SKILL.md` for material this file deliberately does not duplicate, and the referenced `scripts/replay-with-deadline.mjs` exists in the bundle. Not a 5: `measure.sh` and `flows/` are described in the layout but not present in this bundle copy, and some operational detail (e.g. the full 'Platform coverage' diagnostics) sits inline in SKILL.md rather than behind a reference. | 4 / 5 |
Total | 18 / 20 Passed |