Content
65%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.
The body is well-organized and actionable with concrete commands and a clear lifecycle, but two of three reference files are unlinked with their content inlined, and the destructive merge workflow lacks an explicit validation checkpoint. Tightening both would lift the content score.
Suggestions
Add a validation checkpoint before /hub:merge — e.g. 'Verify the winning branch passes tests/build in its worktree before merging' — to satisfy the destructive-operation feedback-loop requirement.
Move the inlined DAG Model and coordination-pattern content into references/dag-patterns.md and references/coordination-strategies.md and link them one level deep from the body instead of duplicating.
Drop or trim the "When This Skill Activates" trigger-phrase list since the same triggers already appear in the description's "Use when" clause, to reduce token redundancy.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is mostly lean — command tables, templates, and the coordinator protocol earn their tokens — but the "When This Skill Activates" trigger list duplicates the description's "Use when" clause and a few table descriptions could be trimmed; efficient with minor over-explanation. | 4 / 5 |
Actionability | Provides concrete executable commands with flags (e.g. `python scripts/result_ranker.py --session {id} --eval-cmd "pytest bench.py --json" --metric p50_ms --direction lower`), slash-command and script tables, and a post-format template; minor gaps remain (external `clawhub install`, placeholder `{id}` tokens). | 4 / 5 |
Workflow Clarity | The INIT→DISPATCH→MONITOR→EVALUATE→MERGE lifecycle is clearly sequenced with a state-transition table, but the destructive/batch merge step (merge winner, `git tag hub/archive/{session}/agent-{i}` losers, worktree cleanup) has no explicit validation checkpoint before merge, so workflow clarity is capped at 3 per the destructive-operations rule. | 3 / 5 |
Progressive Disclosure | Templates are cleanly signaled ("Templates are defined in `references/agent-templates.md`"), but `coordination-strategies.md` and `dag-patterns.md` exist in the bundle yet are never linked from the body — their content (DAG model, coordination patterns) is inlined instead, fitting the 'some structure, content that should be separate is inline' anchor. | 3 / 5 |
Total | 14 / 20 Passed |