Content
56%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 presents a well-sequenced, validation-conscious process with explicit failure handling and concrete artifact routes, but it is held back by named-but-undefined contracts that force reference chasing for core behaviors, structural redundancy across Outputs/Gotchas/Anti-Patterns/Constraints sections, and a reference topology with multi-level chains, out-of-bundle targets, and duplicated content in the hot-path reference.
Suggestions
Add a one-line inline definition for each named contract (BLUF review contract, first-principles contract, stage arc boundary, ubiquitous-language contract) so core behaviors are executable without first reading external files.
Merge the redundant "Outputs"/"Output Format" and "Gotchas"/"Anti-Patterns"/"Constraints" sections into single canonical sections to reclaim tokens.
Flatten the reference topology: consolidate the chained `Infrastructure/references/...` and `../../references/...` hops into a one-level-deep, in-bundle structure, and remove the body-content duplication in `references/hot-path-folded-context.md`.
| Dimension | Reasoning | Score |
|---|---|---|
Conciseness | The body is dense policy rather than padded explanation, but there is structural redundancy: "Outputs" and "Output Format" list overlapping field sets, and "Gotchas", "Anti-Patterns", and "Constraints" restate the same guardrails three different ways. It fits anchor 3 (mostly efficient but could be tightened) rather than 4, because the duplication is structural across sections, not minor trims. | 3 / 5 |
Actionability | Concrete anchors exist — artifact routes (`.harness/brainstorm/**.md`), the validation command (`python3 Plugins/harness-engineering/scripts/check_bluf_structure.py ... --json`), and explicit output field names — but core behaviors are invoked by name only ("apply the BLUF review contract", "apply the first-principles contract", "apply the stage arc boundary") with no inline summary, so execution requires chasing external files. This matches anchor 3 (some concrete guidance but incomplete, missing key details) rather than 4, where gaps would be minor. | 3 / 5 |
Workflow Clarity | A 12-step sequenced procedure, an explicit Validation section with pass/fail/blocked statuses and a runnable script, and Failure Handling with recovery loops ("stop with the blocker and smallest recovery step", "ask once; otherwise set `autonomous_assumption`") give clear checkpoints with feedback loops. It fits anchor 4 rather than 5 because validation is conditional ("non-trivial generated artifacts") and is a separate section rather than an explicit step wired into the numbered sequence. | 4 / 5 |
Progressive Disclosure | References are clearly signaled with "Read when..." conditions, but the actual bundle structure reveals problems: local reference files chain onward to `Infrastructure/references/harness-engineering/he-brainstorm-doctrine.md` (2+ levels of nesting), roughly ten `../../references/*.md` targets sit outside the bundle entirely, and `references/hot-path-folded-context.md` duplicates body content (Procedure, Examples, References). This lands at anchor 3 — better signaled than anchor 2's buried references, but the nested chains and duplication exceed anchor 4's "minor organization gaps". | 3 / 5 |
Total | 13 / 20 Passed |