Content
52%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 specifies an elaborate, fail-closed multi-backend review loop with genuinely concrete commands, schemas, and validation gates, but as a ~1,200-line monolith it duplicates prompt templates and routing rules verbatim multiple times and inlines everything rather than splitting it across referenced files (which are not shipped in the bundle). It is actionable and well-validated yet token-inefficient and hard to navigate.
Suggestions
Extract the verbatim-repeated 'SCOPE LIMITS' block (repeated 5 times) and the backend calling conventions into a single referenced file (e.g. references/review-prompts.md) and include each prompt template exactly once.
Trim toward ~500 lines by moving the REVIEW_STATE/ACQUITTAL schemas, the review_gate bash block, and the Stop-Gate State-Transition Tests list into references/ or into the helper scripts they document.
Delete the internal correction history in POSITIVE_THRESHOLD ('Earlier wording here used `or` and a stale verdict set ...') and state only the authoritative rule; also ship or inline the ../shared-references/ files the workflow depends on so the skill is self-contained.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | A ~1,200-line monolith with substantial verbatim duplication: the 25-line 'SCOPE LIMITS' block appears five times across prompt templates, the copilot routing contract is stated twice (Reviewer Calling Convention and Phase A), and the ACQUITTAL_LOG.jsonl receipt schema appears twice; the body also carries self-referential correction history ('Earlier wording here used `or` and a stale verdict set ... that was an internal inconsistency'). Not 1 because it does not explain concepts Claude already knows — the verbosity is duplication and spec sprawl, not padding with known material. | 2 / 5 |
Actionability | Highly concrete: exact MCP call signatures with config JSON, a copy-paste bash block resolving and invoking review_gate.py, exact artifact naming patterns ('<run_id>_r<round>_review_<8-random-hex>'), and exact JSONL schemas. Not 5 because several commands depend on helper scripts (review_gate.py, copilot_native_evidence.py, save_trace.sh) and shared-references files that are not part of this bundle, and the review prompts remain placeholder templates ('<path(s)>', '[paste executor's rebuttal]'). | 4 / 5 |
Workflow Clarity | Explicit sequencing (Initialization -> Step -1/0 -> Phases A-E -> Termination) with genuine validation checkpoints (fail-closed REVIEW_UNAVAILABLE, native-evidence verification, gate recomputation after rebuttals) and feedback loops (review -> fix -> re-review, validate -> fix -> re-validate). Not 5 because the state-transition logic is fragmented across Constants, Step 0 snapshots, Phases B.5.1 and E, with round_backend vs reviewer_backend bookkeeping intricate enough that the document itself acknowledges a prior internal inconsistency in the stop condition. | 4 / 5 |
Progressive Disclosure | No references/, scripts/, or assets/ directories exist in the bundle; full prompt templates, bash scripts, JSON state schemas, and a state-transition test list are all inlined in one 1,200-line file, and the linked files (../shared-references/reviewer-routing.md, integration-contract.md, reviewer-backends.md, review-tracing.md, output-versioning.md, output-manifest.md, output-language.md, external-cadence.md) live outside the skill directory and are not present. Not 1 because section headers and clearly labeled markdown links do provide structure, but the dominant trait is that large blocks which clearly belong in separate reference files are inlined. | 2 / 5 |
Total | 12 / 20 Passed |