[US-228] feat: verdict-first code-review template (security + cost + architecture, 30s budget)#376

Merged

rucka merged 11 commits into

mainfrom

feature/US-228-extended-code-review-template

Jul 26, 2026

Commits on Jul 26, 2026

[#228] feat: verdict-first code-review template (security + cost + architecture, 30s budget)

rucka
and
claude
committed
[#228] chore: resolve review round 1 on #376 (consumer sync, chip fallback, byte-eq guard, self-review path, scope ADL)

rucka
and
claude
committed
[#228] chore: resolve review round 3 on #376 (third consumer sync, chip/AC comments)

rucka
and
claude
committed
[#228] chore: resolve review round 4 on #376 (Positive out of severity grouping, fixture vocab note)

rucka
and
claude
committed
[#228] chore: resolve review round 5 on #376 (Questions severity fallback, <...> placeholders -> [...])

rucka
and
claude
committed
[#228] chore: resolve review round 6 on #376 ([...] placeholders, #226->#227, five Security sections, D22 update)

rucka
and
claude
committed
[#228] chore: resolve review round 7 on #376 (native-review idempotency reconciled, drift-note placeholder parity)

rucka
and
claude
committed
[#228] chore: resolve review round 8 on #376 (in-place/append-only reconciled, comment-path softened, perf/deploy prompt, positive Step-5.3 guard)

rucka
and
claude
committed
[#228] chore: resolve review round 9 on #376 (reviewer.md Output severity list adds | Questions to match template + DEFAULT_SEVERITIES)

rucka
and
claude
committed
[#228] docs: fix contract-generator comment wording (severity labels, not sub-headings)

rucka
and
claude
committed
style: prettier-format code-review-verdict-first.test.ts (long regex line wrap)
rucka
committed