Content
77%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 orchestration skill body: an explicit gated six-stage workflow with validation and feedback loops, and highly actionable CLI command tables. Its main defects are dangling bundle references (stages/, checklists/, subagent-prompts/, support/ are missing) and duplicated sections that cost tokens without adding guidance.
Suggestions
Create or remove the dangling references: the six `stages/*.md` files, `checklists/quality.md`, `subagent-prompts/data-explorer.md`, and `support/frontend-design/` are cited as core navigation targets but do not exist in the bundle, breaking the Stage Notes and Subagent Delegation sections.
Deduplicate the Shared References list (`references/harness-contract.md` and `references/reports.md` are each listed twice) and drop the redundant '## CLI Tool' section, which repeats a subset of the Quick Reference table.
Inline one concrete `--payload` JSON example (e.g., a minimal `write-plan` or `record-attestation` payload) so the most common mutating commands are executable without opening `references/json-payloads.md`.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and imperative — command tables, mistake/fix pairs, and hard constraints assume Claude's competence with no basic-concept padding, matching the efficient-with-minor-trimming anchor. Not 5 because the ~20-row Quick Reference table duplicates the later CLI Tool section, and the Shared References list repeats entries (`references/harness-contract.md` and `references/reports.md` each appear twice). | 4 / 5 |
Actionability | Concrete, fully spelled-out commands (`$PYTHON_PATH .agentsociety/bin/ags.py analysis load-context --workspace . --hypothesis-id ID ...`), exact output paths, exit-code semantics, and receipt-inspection instructions are mostly copy-paste ready, matching the 4 anchor. Not 5 because payload examples are only placeholders (`--payload '{...}'`, `--spec FILE`) deferred entirely to a reference file, so common cases are not covered inline. | 4 / 5 |
Workflow Clarity | A clearly sequenced six-stage pipeline (dot graph plus Stage Notes) with explicit validation gates per phase, the four-step post-phase loop, exit-code error handling, single-flight/retry semantics, and an explicit feedback loop ("On REVISE/FAIL, loop producer → reviewer until PASS") — matching the 5 anchor's sequence, validation, and error-recovery checkpoints. | 5 / 5 |
Progressive Disclosure | SKILL.md is structured as an overview with well-labeled, one-level-deep references, but scored against the actual bundle, 10 referenced paths dangle: all six `stages/*.md` files, `checklists/quality.md`, `subagent-prompts/data-explorer.md`, and `support/frontend-design/**` do not exist, so the central Stage Notes and delegation navigation is broken. This is more than the 4 anchor's 'minor organization gaps', while the 3 anchor's 'could be better organized' fits; not 2 because the references that do exist are clearly signaled and one level deep. | 3 / 5 |
Total | 16 / 20 Passed |