From a5bf4c2da5fbfc27293cd4596d05b6023bc1e417 Mon Sep 17 00:00:00 2001 From: Gergo Magyar Date: Mon, 3 Aug 2026 17:26:18 +0000 Subject: [PATCH] test(cache): move the SCHEMA_BUMP pin to 40 MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The pin at incremental-parse-cache.test.ts asserts the exact value on purpose — it exists to catch two branches claiming one number, and it has earned that eight times. Bumping the constant to 40 without moving the pin turned it red. Found by the Codex (gpt-5.6-sol) review leg, which flagged it as a deterministic committed-test failure. The Claude lanes could not have caught it: they were dispatched before the bump landed. The comment now records the 39 -> 40 movement and its reason, matching the existing convention in that block. Refs #2807 Co-Authored-By: Claude Opus 5 (1M context) --- gitnexus/test/unit/incremental-parse-cache.test.ts | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/gitnexus/test/unit/incremental-parse-cache.test.ts b/gitnexus/test/unit/incremental-parse-cache.test.ts index e22a301e4..b0b4dbd9e 100644 --- a/gitnexus/test/unit/incremental-parse-cache.test.ts +++ b/gitnexus/test/unit/incremental-parse-cache.test.ts @@ -106,7 +106,10 @@ describe('PARSE_CACHE_VERSION', () => { // conditional-directive parse-semantics change (#2771), 38 -> 39 for // receiver-chain wire format v2: every persisted chain string changed prefix // and a v2 decoder refuses v1 by design, so a stale cache replays chains this - // build silently discards. + // build silently discards. 39 -> 40 for inference-typed field captures in six + // languages (#2807) — all parse-time emission, so a warm cache replays the + // pre-fix capture set for byte-unchanged files and the new receiver edges + // never appear. // // This pin has now earned its keep EIGHT times, and twice it caught an EXACT // clash rather than a near-miss: main took 37 for #2416 while this branch @@ -115,8 +118,8 @@ describe('PARSE_CACHE_VERSION', () => { // second clash was caught — after review, while the branch sat waiting to // merge — which is precisely the window in which `main` allocates. Re-check // against origin/main immediately before merge, not at review time. - it('pins SCHEMA_BUMP to 39 so concurrent bumps cannot silently collide (#2766)', () => { - expect(Number(PARSE_CACHE_VERSION.split('+', 1)[0])).toBe(39); + it('pins SCHEMA_BUMP to 40 so concurrent bumps cannot silently collide (#2766)', () => { + expect(Number(PARSE_CACHE_VERSION.split('+', 1)[0])).toBe(40); }); it('embeds the gitnexus package version (so upgrades invalidate the cache)', () => {