Skip to content

Commit b0fb62a

Browse files
author
DavidQ
committed
Fix shared extraction guard baseline drift
BUILD_PR_LEVEL_19_8_FIX_SHARED_EXTRACTION_GUARD_BASELINE
1 parent 9d3563a commit b0fb62a

6 files changed

Lines changed: 1419 additions & 290 deletions

docs/dev/CODEX_COMMANDS.md

Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -2,31 +2,31 @@ MODEL: GPT-5.4-codex
22
REASONING: high
33

44
COMMAND:
5-
1. Open:
6-
tests/samples/SamplesProgramCombinedPass.test.mjs
5+
1. Run:
6+
node tools/dev/checkSharedExtractionGuard.mjs
77

8-
2. Locate phase expectation assertion.
8+
2. Capture output differences.
99

10-
3. Replace hardcoded phase list:
11-
phase-01 → phase-15
10+
3. Locate baseline file (likely in tools/dev or reports).
1211

13-
4. Update to:
14-
include phase-16, phase-17, phase-18, phase-19
12+
4. Determine:
13+
- Are new shared entries valid? If YES → update baseline
14+
- If NO → fix classification logic
1515

16-
Preferred:
17-
derive phases dynamically from filesystem OR
18-
extend static list to 19
16+
5. Apply smallest change:
17+
- update baseline JSON OR
18+
- adjust guard filters
1919

20-
5. Run:
21-
node ./scripts/run-node-tests.mjs
20+
6. Run:
21+
npm test
2222

23-
6. Confirm:
24-
SamplesProgramCombinedPass passes
23+
7. Confirm:
24+
pretest passes (no baseline drift)
2525

26-
7. Package ZIP:
27-
<project folder>/tmp/BUILD_PR_LEVEL_19_7_FIX_SAMPLE_PHASE_EXPECTATION.zip
26+
8. Package:
27+
<project folder>/tmp/BUILD_PR_LEVEL_19_8_FIX_SHARED_EXTRACTION_GUARD_BASELINE.zip
2828

2929
CONSTRAINTS:
30-
- Do not modify unrelated tests
31-
- Do not change engine/runtime code
32-
- Smallest scoped change only
30+
- Do not modify unrelated systems
31+
- Do not remove guard
32+
- Preserve intent of guard

docs/dev/COMMIT_COMMENT.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
1-
Fix sample phase expectation mismatch (phase-16–19)
1+
Fix shared extraction guard baseline drift
22

3-
BUILD_PR_LEVEL_19_7_FIX_SAMPLE_PHASE_EXPECTATION
3+
BUILD_PR_LEVEL_19_8_FIX_SHARED_EXTRACTION_GUARD_BASELINE
Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,10 @@
1+
# Shared Extraction Guard Fix
2+
3+
## Problem
4+
baseline_unexpected=288 indicates drift between expected and actual shared usage.
5+
6+
## Resolution
7+
Baseline aligned or guard logic corrected.
8+
9+
## Result
10+
Pretest passes → validation pipeline unblocked.

docs/dev/reports/launch_smoke_report.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
# Launch Smoke Report
22

3-
Generated: 2026-04-17T15:22:34.189Z
3+
Generated: 2026-04-17T15:32:24.952Z
44

55
Filters: games=true, samples=true, tools=true, sampleRange=all
66

Lines changed: 34 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,34 @@
1+
# BUILD_PR_LEVEL_19_8_FIX_SHARED_EXTRACTION_GUARD_BASELINE
2+
3+
## Purpose
4+
Fix blocking validation failure:
5+
tools/dev/checkSharedExtractionGuard.mjs baseline drift (baseline_unexpected=288)
6+
7+
## Scope
8+
- Single purpose PR
9+
- Align baseline to current repo state OR fix guard logic mismatch
10+
- No feature work
11+
12+
## Required Change
13+
1. Run:
14+
npm test (observe baseline failure)
15+
16+
2. Run guard script directly:
17+
node tools/dev/checkSharedExtractionGuard.mjs
18+
19+
3. Identify:
20+
- newly valid shared extractions OR
21+
- false positives in guard logic
22+
23+
4. Apply ONE of:
24+
A. Update baseline file to match current valid state
25+
OR
26+
B. Fix guard logic to correctly classify entries
27+
28+
## Validation
29+
- npm test passes pretest
30+
- baseline_unexpected = 0
31+
32+
## Acceptance
33+
- Guard passes cleanly
34+
- No unintended shared leaks introduced

0 commit comments

Comments
 (0)