Content
60%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-structured, instruction-only governance skill with a clear scope boundary and concrete routing rules. Its main weaknesses are redundancy across activation/usage sections and a missing key artifact (the referenced CONTEXT.md template, with no inline example to substitute), which keeps actionability from being fully executable. Sequencing is clear and anti-patterns are well chosen.
Suggestions
Merge 'When to Activate' and 'When to Use' into a single activation section; they restate the same triggers almost verbatim.
State the task/schedule boundary once — it currently appears in the 'Do not include' list, 'How It Works' step 4, and the entire 'Scheduling and Task Boundary' section.
Ship `templates/context.example.md` in the bundle or include a short inline example of a CONTEXT.md entry (term, definition, relationships, a `needs confirmation` ambiguity) so the creation step is executable as written.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Individually the sections are lean and assume Claude's competence (no basic concepts re-explained), but the same rules are repeated: 'When to Activate' and 'When to Use' restate the same triggers, and the keep-schedules-in-the-tracker boundary appears three times ('Keep those in code/MAP, STATUS/Issue Tracker...', 'Keep schedules, blockers, and execution status in the existing issue tracker', and the entire 'Scheduling and Task Boundary' section). This is 'mostly efficient but could be tightened' rather than anchor 2's padded explanations of known concepts. | 3 / 5 |
Actionability | Much of the guidance is concrete (reuse 'the `context` role in `.governance/docs-map.json`', 'lazily create the default `CONTEXT.md` from `templates/context.example.md`', 'Mark unconfirmed items as `needs confirmation`'), but the core creation step depends on `templates/context.example.md`, which is not present in this bundle, and no inline example of a CONTEXT.md entry or its format is given. Per the scoring notes, absent code is not penalized per se, but a key executable detail is missing, matching anchor 3 ('some concrete guidance but incomplete; missing key details') rather than anchor 4's 'minor gaps'. | 3 / 5 |
Workflow Clarity | 'How It Works' and the 'ADR Handoff' section give clear numbered sequences, and there are implicit checkpoints ('Terms must be traceable to code, contracts, or business evidence', 'Mark unconfirmed items as `needs confirmation`; do not guess'). No destructive or batch operation is involved, so no validation cap applies, but the checkpoints are verification rules rather than explicit validate-and-fix loops — anchor 4 ('clear sequence with most checkpoints present; minor validation gaps'), not anchor 5. | 4 / 5 |
Progressive Disclosure | No bundle files exist (no references/, scripts/, assets/), and the body is well organized into focused sections under clear headers, so it scores on its own structure. It is above the simple-skill exception line (~70 lines, not under 50), and it points to external artifacts (`templates/context.example.md`, `architecture-decision-records`) that are inline mentions rather than clearly signaled one-level-deep references, and the referenced template is absent from the bundle — anchor 4 ('good structure; most content appropriately placed; references mostly clear; minor organization gaps') rather than anchor 5. | 4 / 5 |
Total | 14 / 20 Passed |