From 44d9edcf2f520a08f593f1df030b5001a9c7c340 Mon Sep 17 00:00:00 2001 From: npub1cc3ha7z055mu0rwwu7806t2wt8mj3pvu0uv5mfp2c50dahaqhczshdalg6 Date: Thu, 9 Jul 2026 00:49:20 -0400 Subject: [PATCH] chore(desktop): raise useAnchoredScroll size limit 1140->1180 for W4a chase MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The reading-anchor hook is the sole cross-engine scroll writer and the survivor of the GUI-latency sweep's upscroll-reversal chase. The W4a instrumentation legitimately inflated it past 1140: the signed-shift grow/shrink probe (classifier arm) and the renderedScroll gate-clock rekey (w4a-gate-1 arm), both carrying load-bearing provenance inline. Per Eva's ruling, take the scoped override NOW and hold the structural split until the arms in flight against this file settle — splitting a hook whose provenance comments are themselves load-bearing (the anchor-margin lesson lives in it) mid-chase is how a guard gets stripped. Exit condition recorded at the override site: once #1662's fate is decided, the diagnostic emits get stripped or gated and the mid-history corrector + band walk split into a sibling module. This also retires the --no-verify size bypass the classifier commit (9df6d92e, 1156 lines) shipped under: the limit now covers it honestly, and Sami independently flagged the same blocker on the branch. Co-authored-by: Tyler Longwell Signed-off-by: Tyler Longwell --- desktop/scripts/check-file-sizes.mjs | 14 ++++++++++---- 1 file changed, 10 insertions(+), 4 deletions(-) diff --git a/desktop/scripts/check-file-sizes.mjs b/desktop/scripts/check-file-sizes.mjs index a21d68061..d90af7722 100644 --- a/desktop/scripts/check-file-sizes.mjs +++ b/desktop/scripts/check-file-sizes.mjs @@ -189,10 +189,16 @@ const overrides = new Map([ // (Quinn's merge-bar checklist #4), the one-clock/single-writer reasoning, and // the build-stamp/probe contract are all documented AT their sites. The file // was 888 lines at HEAD; this fix's code + load-bearing provenance comments - // crossed 1000. Comment-dominated overage on a correctness fix, not generic - // debt growth. Approved override; queued to split (extract the mid-history - // corrector + band walk into a sibling module) with the rest of this list. - ["src/features/messages/ui/useAnchoredScroll.ts", 1140], + // crossed 1000. The 1140→1180 bump is the W4a chase instrumentation inflating + // a load-bearing hook mid-investigation: the signed-shift grow/shrink probe + // (classifier arm) and the renderedScroll gate-clock rekey (w4a-gate-1 arm), + // both carrying their provenance inline. EXIT CONDITION: once #1662's fate is + // decided, the diagnostic emits get stripped or gated and the mid-history + // corrector + band walk split into a sibling module — Eva's ruling holds the + // structural split until the arms in flight against this file settle, so it is + // NOT done mid-chase. Comment-dominated overage on a correctness fix, not + // generic debt growth. Approved override; queued to split with the rest. + ["src/features/messages/ui/useAnchoredScroll.ts", 1180], // AgentConfigPanel footer fold into ProfileFieldGroup for the config-bridge // panel — a small overage from load-bearing UI plumbing, not generic debt // growth. Approved override; still queued to split with the rest of this list.