diff --git a/docs/branch-review-records/33f8fcf0543655449bf077fbb7595b849f39599f7d54238755ee9a649967f25a.record.md b/docs/branch-review-records/33f8fcf0543655449bf077fbb7595b849f39599f7d54238755ee9a649967f25a.record.md new file mode 100644 index 0000000000..f37abd6a8b --- /dev/null +++ b/docs/branch-review-records/33f8fcf0543655449bf077fbb7595b849f39599f7d54238755ee9a649967f25a.record.md @@ -0,0 +1 @@ +| 2026-08-12 | PR #1888 (claude/filter-contract-factsheets-pr-d) | 95515078031b4bd29d90f05986dc5eb3a36ce660 | factsheets category counts, filter-contract correction, desktop-rail exception decision | Filter contract PR D — corrected the earlier false claim that factsheets' category presets discard the query and need eviction (searchHref preserves q); kept the desktop rail as real Link elements rather than converging to SegmentedControl because /factsheets/search is a genuine server-rendered searchParams route, recorded as a documented exception in filter-contract.md; added per-query category counts to both breakpoints from one shared options array. verify:pr-local full green pre- and post-merge; bundle budget within tolerance; UI proved with pinned-Chromium Playwright (11/11) and the updated ui-smoke.spec.ts factsheets test against a real build. /issues #170 updated to correct its own stale factsheets claim. | verify:pr-local (green), check:bundle-budget (within tolerance), lint, typecheck, full vitest (564/564 post-merge), targeted Playwright (1/1) + manual browser script (11/11) | diff --git a/docs/branch-review-records/770c855f8ddb79707f4ddb49b266ea1f4d27795aa33fcc02f099586e423c15b2.record.md b/docs/branch-review-records/770c855f8ddb79707f4ddb49b266ea1f4d27795aa33fcc02f099586e423c15b2.record.md new file mode 100644 index 0000000000..75e194050a --- /dev/null +++ b/docs/branch-review-records/770c855f8ddb79707f4ddb49b266ea1f4d27795aa33fcc02f099586e423c15b2.record.md @@ -0,0 +1 @@ +| 2026-08-13 | PR #1888 / claude/filter-contract-factsheets-pr-d | f8930c36d11a15820bd0a76aa6b4ff24928db224 | PR #1888 fresh heavy review-and-fix before required main sync | Dispositioned the PR delta as superseded by current main: retained main's shared counted SegmentedControl, selected dead-end handling, dedicated DOM coverage, and current filter contract; no remaining high-confidence PR-introduced P0-P2 defect. | historical exact-head PR required and Production UI green; current-main factsheets component, contract, and tests reviewed; CodeRabbit unavailable (spending cap), distinct manual adversarial pass |