wdi-method 0.5.13 → 0.5.14
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
|
@@ -332,7 +332,7 @@ their rows join the catalogue and the three inventories, and **G3 reopens over t
|
|
|
332
332
|
| Agent at G1–G3 | Writes the brief, the PRD, and the blueprint | Decide depth or accept risk |
|
|
333
333
|
| Agent at G4 | Writes one component's behaviour and mechanism | Write code |
|
|
334
334
|
| Builder | One story through to a green PR | Change `.what/`, `.how/`, or an `applied` decision |
|
|
335
|
-
| Review panel |
|
|
335
|
+
| Review panel | Reviews the diff independently of the builder | Give a final verdict — findings are adjudicated by the coordinator |
|
|
336
336
|
|
|
337
337
|
Panel composition and CLI/model selection are governed by the global Agent Rules and MUST NOT be restated
|
|
338
338
|
here.
|
|
@@ -162,7 +162,7 @@ the corpus. No → `_bmad-output/`. In doubt, `../document/corpus-guide.md`.
|
|
|
162
162
|
|---|---|
|
|
163
163
|
| Decisions — proposing a slicing, wording a `DEC-`, preparing a gate | `opus@high` |
|
|
164
164
|
| Writing, derivation, a review-fix pass | `sonnet@high` |
|
|
165
|
-
| Code review panel |
|
|
165
|
+
| Code review panel | Reviewers dispatched separately from the builder — the local Agent Rules govern CLI/model pairing |
|
|
166
166
|
|
|
167
167
|
In a derivation pass, quality comes from the input rather than the model. Running a "find the gap" lens with
|
|
168
168
|
the most careful model produces the most gaps, and each one becomes an open question — a cost nobody sees
|
|
@@ -1,5 +1,5 @@
|
|
|
1
|
-
# Review Panel: two reviewers
|
|
2
|
-
#
|
|
1
|
+
# Review Panel: two reviewers, each dispatched separately from the builder. Which CLIs and models
|
|
2
|
+
# staff the panel is governed by the local Agent Rules, not by this kit.
|
|
3
3
|
#
|
|
4
4
|
# WARNING: the adversarial and edge-case lenses here are read from the bundled copy of
|
|
5
5
|
# review-prompts/, NOT from the bmad-review skill. An override on bmad-review will not be visible
|
|
@@ -124,8 +124,9 @@ Three rules this corpus adds. All three MUST be stated in the dispatch of any st
|
|
|
124
124
|
|
|
125
125
|
Panel composition follows `risk_accepted`: at `low` a two-reviewer panel is **required** on the code; at
|
|
126
126
|
`medium` and `high` it is available and SHOULD be used when the diff touches money, personal data, or a third
|
|
127
|
-
party. The Agent Rules
|
|
128
|
-
the worker's own review layers are
|
|
127
|
+
party. The local Agent Rules govern which CLIs and models staff the panel. A reviewer MUST be a dispatch
|
|
128
|
+
separate from the builder — the worker's own review layers are self-review by construction and never satisfy
|
|
129
|
+
the panel.
|
|
129
130
|
|
|
130
131
|
- MUST adjudicate every contested finding by reading the cited lines. Votes MUST NOT settle a finding. A finding
|
|
131
132
|
neither reviewer can locate in the diff is dismissed with that reason stated.
|
|
@@ -227,7 +228,7 @@ Run in this order and stop at the first failure:
|
|
|
227
228
|
- Opening a PR with an unresolved must-fix, or before the story-closing checklist is answered
|
|
228
229
|
- Editing a guard, a test, or an assertion to turn something green
|
|
229
230
|
- Reporting green without checking the head SHA, or reading green `korpus.yml` as a passing build
|
|
230
|
-
-
|
|
231
|
+
- Counting the builder's own review layers as a panel reviewer
|
|
231
232
|
- Dispatching a worker without its Unattended flag
|
|
232
233
|
- Closing the wave without the registry catch-up in Phase 4 — that is where five checklist items now live
|
|
233
234
|
- Letting `SPEC.md` state something `.what/` and `.how/` do not
|
|
@@ -13,7 +13,7 @@ artifact but of the component's `risk_accepted`, and defaulting to structure + p
|
|
|
13
13
|
one lens that matters for behaviour; and `bmad-review` is class D — it writes nothing, so nothing proves
|
|
14
14
|
it ran.
|
|
15
15
|
|
|
16
|
-
You MUST NOT use this for code or diffs. That is `bmad-code-review` and the
|
|
16
|
+
You MUST NOT use this for code or diffs. That is `bmad-code-review` and the Review Panel.
|
|
17
17
|
|
|
18
18
|
## What it covers
|
|
19
19
|
|