Content
88%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 lean, highly operational skill body with a well-bounded multi-pass workflow, explicit stop conditions, and good use of a single reference file for contracts and schemas. Main gaps: no inline copy-paste workflow script, and the reference path prefix will not resolve against the actual bundle layout.
Suggestions
Fix the reference path: the bundle file lives at `references/pass-contracts.md` relative to SKILL.md, but the body says `skills/council-mode/references/pass-contracts.md` — resolve it relative to the skill's own directory so navigation works.
Include a minimal inline example of the Pass 1 workflow script shape (even 5-10 lines) so the orchestration is copy-paste ready without opening the reference.
State the claim-matrix format concretely (columns/fields for agreements, disputed claims, missing proof) to remove ambiguity in step 5.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense and operational with zero padding — e.g. "Advisors stay read-only and do not see peer transcripts by default" and "A resume needs a retained run id and task; it excludes `agent` and rejects `gate`" — and it never explains concepts Claude already knows. Every token earns its place (anchor 5); anchor 4 would require identifiable over-explanation to trim, and there is none. | 5 / 5 |
Actionability | Concrete, near-executable guidance throughout — `subagent({ action: "list" })`, `subagent({ workflow: true, async: true })` with `runs.all`, `phase: "Council pass 1"`, `output: false`, fallback rules (`context: "fork"` for `oracle`), and a cap of "at most five material relay claims per advisor". It falls short of anchor 5 because the actual workflow script and output schemas are not inline copy-paste ready — they live in the reference file — leaving minor gaps such as the exact claim-matrix format. | 4 / 5 |
Workflow Clarity | The Protocol is a clear 7-step sequence with explicit stop conditions ("Stop at convergence, pass cap, failed fallback, or user interruption"), user checkpoints (steps 2, 4, and 6), error-recovery paths (fallback roster, degraded mode, fresh-context fallback), and a memo checklist. The batch operation of launching multiple advisors is explicitly bounded ("Never use more than four advisors", "Never run an unbounded loop"), matching anchor 5 rather than anchor 4's 'minor validation gaps'. | 5 / 5 |
Progressive Disclosure | The split is appropriate — orchestration in SKILL.md, schemas and the advisor profile template in one one-level-deep reference, clearly signaled ("Before launch, read `skills/council-mode/references/pass-contracts.md`"). However, the referenced path does not match the actual bundle layout (`references/pass-contracts.md` relative to SKILL.md), a minor navigation gap that keeps it at anchor 4 ('minor organization gaps') rather than anchor 5's 'easy navigation'. | 4 / 5 |
Total | 18 / 20 Passed |