From 32141f803a64b791afa21ce795983376d1418379 Mon Sep 17 00:00:00 2001 From: Jakub Zych Date: Tue, 27 Oct 2026 14:31:37 +0100 Subject: [PATCH] docs(32): record code review disposition --- .../32-REVIEW-DISPOSITION.md | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) create mode 100644 .planning/phases/32-seo-foundation/32-REVIEW-DISPOSITION.md diff --git a/.planning/phases/32-seo-foundation/32-REVIEW-DISPOSITION.md b/.planning/phases/32-seo-foundation/32-REVIEW-DISPOSITION.md new file mode 100644 index 00000000..d6729899 --- /dev/null +++ b/.planning/phases/32-seo-foundation/32-REVIEW-DISPOSITION.md @@ -0,0 +1,19 @@ +--- +phase: 32 +review: 32-REVIEW.md +titles: json +findings: +open: 0 +total: 0 +unparsed: 52 +recorded: 2026-10-27T13:31:36.596Z +--- + +# Phase 32: Code Review Disposition + +| Finding | Severity | Disposition | Source | +|---------|----------|-------------|--------| + +Dispositions: `open` (recorded, not yet triaged), `fixed`, `skipped`, `deferred`. +Set `deferred` by hand and put the reason in the Source cell; both are preserved. A `|` in the reason is kept as prose and escaped on the next run. +Re-running the gate keeps every row it can. A row the current review no longer reports is kept and its Source cell flagged, so a finding does not leave this record silently. ONE exception: when a finding id is REUSED by a different finding, the earlier decision cannot keep a row — the id is taken — and it is dropped. A RECORDED decision (anything but `open`) is named on the console when that happens; a row still at `open` is replaced silently, because `open` records no decision to lose.