Content
92%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 expertly structured orchestration skill: fully executable commands, explicit validation gates and feedback loops at every stage boundary, and clean deferral of detail to a single one-level reference and real scripts. The only real cost is token weight — a few duplicated prohibitions and an inline version floor could be consolidated without losing coverage.
Suggestions
Consolidate the repeated no-polling and 'steered prose is not the contract' prohibitions into one statement in the stage-boundary section; the duplication at lines 30/65/71 and 71/88 can be trimmed.
Move the 'Herdr is at least 0.8.2' version floor (and other environment prerequisites) into references/runtime-protocol.md#configuration so version-sensitive detail does not sit in the always-loaded body.
The stage-boundary command list repeats 'node <skill-dir>/scripts/fstate/cli.mjs' verbatim four times; a short prefix variable or a pointer to the runtime-protocol catalog would save tokens without losing executability.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is telegraphic and assumes Claude's competence throughout, with no concept explanations or tutorial padding. Minor trimmable instances remain: the no-polling prohibition is stated three times (lines 30, 65, 71), 'Do not treat steered prose as the contract' repeats (lines 71, 88), and the version-sensitive floor 'Herdr is at least 0.8.2' sits in the main flow rather than a config/reference section. | 4 / 5 |
Actionability | Every phase is backed by exact, executable commands (git rev-parse --show-toplevel, the four stage-boundary commands, fstate task transition edges, pane-naming formats) plus a copy-paste reviewer spawn prompt. Not 4: coverage of the common cases is complete and copy-paste ready, with no pseudocode gaps. | 5 / 5 |
Workflow Clarity | Stages are explicitly sequenced (plan → work → review → wrapup → done) with a numbered boundary checklist, explicit validation gates ('If the exit code is 3 or 4, call subagent_resume once with the stderr line'), a bounded feedback loop (at most three review rounds, resume-once on contract failure), explicit stop conditions, task state-machine edges, and a crash-recovery pointer. Not 4: checkpoints, error-recovery loops, and limit enforcement are all explicit, matching the top anchor. | 5 / 5 |
Progressive Disclosure | The body is an operational overview that defers the command catalog, config shape, state ownership, display metadata, and recovery details to references/runtime-protocol.md via well-signaled one-level anchor links (all anchors verified to exist) and to real, present scripts (fstate/cli.mjs, pi-session-reader.py, review-packet.py, github-related.mjs). Not 4: the split is clean and navigation is easy, with no bulk reference material inlined. | 5 / 5 |
Total | 19 / 20 Passed |