Content
78%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 orchestrator body: a crisp sequenced pipeline with genuine error-recovery fallbacks, exact path-resolution and confirmation procedures, and exemplary one-level-deep reference structure. The costs are a compressed prose style with occasional rationale padding and validation details that are named in the body but defined only in the reference layer.
Suggestions
Trim the 'compounds value' rationale sentences (Worth lens and Discoverability Check sections) — they justify the skill's purpose rather than instruct the agent, and the body is otherwise pure rule-per-token.
Inline the two or three most safety-critical validation pre-checks (the auto-delete pre-checks from classify.md) as a short checklist in the Classify section, so the destructive-cap logic is verifiable from the body alone.
Rewrite the Worth lens intent-detection paragraph as a rule list (trigger words → action) matching the style of the other sections; its current nested conditionals cost re-reading.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and rule-per-sentence with no beginner-concept padding — e.g. "Candidates are the .md files under `<root>/solutions/`, excluding `README.md` and anything under `_archived/`" — assuming Claude's competence throughout. It sits below anchor 5 because of trimmable rhetorical rationale ("The store only compounds value if every doc can be trusted" and the similar closing flourish in Discoverability) and a somewhat convoluted Worth-lens paragraph that re-explains the confirm-before-investigate rule. | 4 / 5 |
Actionability | Concrete, executable gating throughout: exact config-resolution steps ("Read `docs_root` from `<repo-root>/.compound-engineering/config.yaml` only (`<repo-root>` = `git rev-parse --show-toplevel`)"), a copy-paste numbered-options question block, and a verbatim subagent clause. It stops short of anchor 5 because the operational substance of most steps (classification criteria, per-doc fix procedures) lives one level deeper in the references — the body is an orchestrator that gates on reading them rather than containing the executable detail itself. | 4 / 5 |
Workflow Clarity | A clearly sequenced pipeline (Mode → Worth lens → Artifact Root → Scope → Investigate → Classify → Execute → Vocabulary Capture → Report → Commit → Discoverability) with explicit error-recovery loops in the body: "A failed write is recorded as **recommended**, and the run continues", the git-failure fallback, and the blocking-question-or-numbered-options fallback. For a batch/destructive skill, anchor 5 requires explicit validation steps, but the body only names the validation machinery ("the auto-delete rule and its pre-checks", "unverifiable-is-not-false") and delegates its content to classify.md — a minor validation gap relative to anchor 5, comfortably above anchor 3. | 4 / 5 |
Progressive Disclosure | Model progressive disclosure: an ~80-line overview that splits all detail across ten reference files, each verified to exist, each clearly signaled at the exact workflow point it is needed ("Read `references/classify.md` before assigning any of them"), with scripts and the assets template pushed out of the overview. Reference cross-links (e.g. per-action-flows.md → classify.md) target files also reachable directly from the body, keeping navigation one level deep and easy — matching anchor 5. | 5 / 5 |
Total | 17 / 20 Passed |