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.
The body is a dense, highly concrete orchestration policy: exact tool invocations, file paths, validation gates, and an explicit phase sequence, with detail properly pushed down to three real one-level-deep references. Its one real weakness is conciseness — the phase-boundary rule is restated several times and one read-order item sprawls into an unparsable run-on paragraph.
Suggestions
State the query-boundary rule once and reference it: the 'do not cross phases / do not continue downstream automatically' idea is restated at least three times across the Query Boundary section and the read-order items.
Split read-order item 6 into separate items (profile selection, release contents and section reporting, `pcb.export`, CAD comparison) so each rule is individually addressable instead of one run-on paragraph.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | Nearly every sentence is custom policy rather than concepts Claude already knows, but the query-boundary rule is restated at least three times ("do not authorize entering a later phase", "do not use `user.ask` for it or continue downstream automatically", "Naming PCB does not authorize doing a missing BOM and PCB in one query") and read-order item 6 is one sprawling paragraph mixing profile choice, release contents, section reporting, `pcb.export`, and CAD comparison. The redundancy and run-on density exceed the 'minor instances of over-explanation' of a 4, but there is no padding or generic explanation that would warrant a 2. | 3 / 5 |
Actionability | The instruction-only guidance is fully concrete and executable: exact parameterized commands ('pcb.run(profile="release")', 'cad.export(intent="handoff")', 'purpose="bom_item_confirmation"'), exact paths ('source/pcb.config.json', 'cad/part-plan.json', 'release-manifest.json'), and exact deliverables ("Gerbers, drill files, `assembly.step`, `assembly.glb`, and the JLC SMT BOM/CPL", "one standard unsliced 3MF per plate"). No gaps in the common cases. | 5 / 5 |
Workflow Clarity | The phase sequence is explicit ("requirements, hardware/BOM, PCB, CAD, then explicitly requested manufacturing/release") with real validation gates: '`pcb.run(profile="check")`' for DRC, blocked release sections that "list why", PRD readiness from "an empty `open_questions` list", and release-manifest comparison before CAD edits, plus the don't-advance-while-blocked feedback rule. It is not a 5 because the manufacturing/release phase's own steps are only gated, never described, and some checkpoints remain implicit. | 4 / 5 |
Progressive Disclosure | The body is a genuine entry-point overview ("Keep the native system prompt small; detailed workflow and safety rules live here and in the linked references") that delegates detail to three real one-level-deep files, each with an explicit when-to-read condition ("Read `references/operating-model.md` before changing project files", "load the `replicator-pcb` skill before PCB source..."). All three referenced files exist and contain no nested reference chains; ~298 lines of detail live in the references while the body stays a routing overview. | 5 / 5 |
Total | 17 / 20 Passed |