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.
This is a high-quality operational skill: every instruction is an executable command, workflows are sequenced with explicit validation gates and timeout-recovery loops, and it encodes genuinely non-obvious domain rules (ID opacity, status semantics, no-focus conventions, safety boundaries). The two areas with headroom are tightening the repeated and edge-case-heavy status-semantics prose and splitting reference-grade detail (status state machine, read-source catalog) into a references file.
Suggestions
Move the idle/done state-machine detail (the three paragraphs of focus-attention edge cases) into a short reference table in a references/ file, keeping only the operational rule ('wait for done in background tabs, idle in watched tabs; treat both as completed') inline.
Deduplicate repeated guidance: state the split geometry rule and 'inspect before waiting' once, and reference them from the two workflows instead of restating the same command and rule in both.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body assumes Claude's competence throughout (no basic explanations) and nearly every line is Herdr-specific knowledge Claude cannot know, e.g. "Treat every ID as an opaque string" and the enumerated read sources. It is not a 5 because sections could be tightened: the idle/done semantics span three dense paragraphs of edge cases ("If the foreground client is explicitly unfocused, completion can become done even in the active tab"), and points are repeated ("Inspect before waiting" appears at lines 150 and 192; the split command is shown twice). It is clearly above anchor 3 since there is no generic padding. | 4 / 5 |
Actionability | Quotes: "test "${HERDR_ENV:-}" = 1", "herdr pane split --current --direction right --no-focus", "herdr wait agent-status <returned-pane-id> --status done --timeout 120000", "herdr pane read <returned-pane-id> --source recent-unwrapped --lines 120". Every workflow step is a copy-paste-ready executable command with real flags, timeouts, and JSON field names ("Read result.pane.pane.pane_id"), plus concrete error-handling decision rules. This matches anchor 5; anchor 4 would imply minor gaps in executable coverage, which are not present. | 5 / 5 |
Workflow Clarity | Sequences are explicit with validation checkpoints and feedback loops: env gate ("If the check fails... stop"), "Inspect the pane after launch. If agent_status is not yet idle, wait for the idle transition", "If a wait times out, inspect herdr pane get... and pane read before deciding what to do", and "Re-read create, split, move, list, or get responses after mutations". The pre-flight guard against accidental mutation ("Do not probe a mutating nested command by omitting arguments") strengthens the safety sequence. This matches anchor 5's explicit validation with error-recovery loops; anchor 4 would require missing checkpoints. | 5 / 5 |
Progressive Disclosure | The body is well-sectioned (Learn the current CLI, IDs and current context, Start agents interactively, Run an ordinary command, Safety rules) and wisely delegates CLI syntax to runtime discovery via "herdr --help" instead of inlining version-sensitive help text, which is appropriate one-level-late disclosure. However, the skill exceeds the simple-skill threshold at ~190 lines with no reference layer: the agent-status state machine and the read-source catalog are reference-grade detail living inline, and the bundle's scripts/ and assets/ are never referenced from the body. This lands at anchor 4 (good structure, minor organization gaps) rather than 5. | 4 / 5 |
Total | 18 / 20 Passed |