Content
38%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 command-dense and broadly actionable but suffers from excessive length and redundancy, broken/non-existent tool invocations and file paths, missing validation around destructive batch operations, and a monolithic structure with dead cross-references. It functions more as a dumped command catalog than a well-organized skill overview.
Suggestions
Split the monolith: move templates, the full command catalog, and examples into references/ files and keep SKILL.md as a concise overview with one-level-deep links — and create the referenced swarm-pr.md/sync-coordinator.md/workflow-automation.md or remove the links.
Add validation checkpoints to destructive/batch workflows (e.g. dry-run before 'gh issue close', confirm count before batch label edits, verify subtask creation before editing the parent body).
Fix the broken shell and paths: replace '2>$dev$null' with '2>/dev/null' and restore '/' in paths like '.github/swarm-labels.json', 'org/repo#456', and '.github/workflows/issue-swarm.yml'.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | At ~580 lines the body is noticeably verbose, with many redundant sections (Issue Types & Strategies, Examples, Automation Examples, Integration Patterns) repeating near-identical 'npx ruv-swarm github ...' command patterns that could be collapsed, matching the score-2 anchor for padded sections. | 2 / 5 |
Actionability | It provides many concrete, mostly-executable gh/jq commands, but several are non-functional: '2>$dev$null' instead of '2>/dev/null' and corrupted paths like '.github$swarm-labels.json' and 'org$repo#456' where '/' was replaced by '$', leaving guidance incomplete. | 3 / 5 |
Workflow Clarity | Sequences like Initialize-from-Issue and Task Decomposition are commented step-by-step, but destructive/batch operations (auto-closing stale issues, closing duplicates, batch label edits) have no validation or dry-run checkpoints, capping workflow clarity at 3 per the batch/destructive rule. | 3 / 5 |
Progressive Disclosure | Content is a single monolithic file with no bundle files present, yet it inlines ~580 lines of templates, examples, and command reference that belong in separate files, and its only cross-references (swarm-pr.md, sync-coordinator.md, workflow-automation.md) point to non-existent files. | 2 / 5 |
Total | 10 / 20 Passed |