Content
75%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 well-engineered skill body: dense tables, executable commands with realistic flags, a clearly sequenced coordinator lifecycle, and proper offloading of conceptual depth (templates, coordination patterns, DAG theory) to real one-level-deep reference files. Remaining gaps are a missing post-merge verification step and a slightly muddled tail section where a dry-run command is listed as if it were a reference document.
Suggestions
Add an explicit post-merge verification step to the Merge phase (e.g. re-run the eval command or test suite on the base branch after `git merge --no-ff`, and roll back the tag if it fails).
Create the baseline explicitly in the Init step (the "No improvement over baseline" proactive trigger depends on it), e.g. run the eval command on the base branch before spawning.
Move `python3 scripts/dry_run.py --help` out of the "More in this skill" reference list into the Scripts table (or a Validation section) so that section contains only loadable references.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and table-driven with concrete commands and no tutorial-style explanation of concepts Claude already knows; concept material (e.g. "Git's commit history is a DAG") is correctly offloaded to references. Not a 5: a 10-item trigger-phrase list duplicates the description's job, and lines like "Equivalent to AgentHub's 'leaves' query" add little. | 4 / 5 |
Actionability | Concrete, mostly copy-paste-ready guidance throughout: `python scripts/result_ranker.py --session {id} --eval-cmd "pytest bench.py --json" --metric p50_ms --direction lower`, `git merge --no-ff`, `git tag hub/archive/{session}/agent-{i}`, a complete agent dispatch prompt, and a full board post format. Not a 5: slash commands like `/hub-init` and `/agenthub:spawn` are never shown with example arguments, and one status command appears without the `python scripts/` prefix. | 4 / 5 |
Workflow Clarity | The INIT → DISPATCH → MONITOR → EVALUATE → MERGE lifecycle is clearly sequenced with numbered phases, a state-transition table, and a proactive-triggers table providing failure feedback loops (all crashed → retry with different constraints; stuck session → check board, consider timeout). Not a 5: there is no post-merge verification (e.g. run tests on the merged result), and the "No improvement over baseline" trigger references a baseline that the Init step never explicitly establishes. | 4 / 5 |
Progressive Disclosure | The body is an overview with well-signaled one-level-deep references that all resolve to real bundle files (agent-templates.md, coordination-strategies.md, dag-patterns.md) plus a scripts table covering 5 of the 6 scripts. Not a 5: the "More in this skill" list ends with a command (`python3 scripts/dry_run.py --help`) masquerading as a loadable reference, and dry_run.py is missing from the Scripts table — minor navigation confusion. | 4 / 5 |
Total | 16 / 20 Passed |