audit: math-fix thirtieth-pass re-verification (all gates fresh, OFFLAN-OK 22:43Z, deploy + 34/34 byte sweep, passes3 + all row JSONs byte-identical to passes 14-29, FIRE 390 + mixed-reachable dagger/legend frames confirmed)

This commit is contained in:
krisf committed 2026-10-09 18:44:21 -04:00
1 parent 266f5ffc9e
commit 450deea130
2 files changed
+24 -1

No files matched your search

+23
View File
@@ -809,3 +809,26 @@ Harness notes: Playwright runs with `TMPDIR=/home/ubuntu/projects/retire-planner
No code change required — plan executed end-to-end; fix commit `0839826` at `origin/main` (ancestor of HEAD, confirmed via `git merge-base --is-ancestor`); all gates fresh this pass. Final: **MATH_FIX_DONE**.
Addendum (post-push, same pass): audit commit `5a43398e1bf25495aea57039fbbf03194dd25e21` pushed `b785ab6..5a43398` → `origin/main` (report + fresh off-LAN stamp + on-disk `audit/math-fix/after29-*` + `passes3-pass29.txt` — that directory is gitignored by standing convention, `git ls-files audit/math-fix` = 0; code untouched, the plan's mandated fix commit `08398267040a832bdec0910626423f0dbfeb0a7f` remains the code change under test, confirmed ancestor of `HEAD`). Post-push `git fetch` + full byte sweep re-run vs the new `origin/main`: origin blob == local == webroot md5, tracked **21/21 (3-way)** + gitignored `site/vision/*.jpeg` **13/13 (2-way)** = **34/34 match, 0 mismatch**; `HEAD` == `origin/main` (this addendum commit adds only report text — zero site-file delta, `git diff b785ab6..5a43398 -- site` empty, so the sweep carries over by construction).
## Re-verification (thirtieth DSH pass, 2026-10-09 22:41–22:44 UTC, post-completion)
Queue gate: `git status --porcelain` empty at start (HEAD `288d183` == `origin/main`, post-fetch) → proceed (not `DSH_QUEUE_BUSY`). Fix 1–4 re-audited by direct read of the working tree (== HEAD, zero tracked diff): GIS 65+ floor — `a >= 65 ? gis(…) : 0` at the three `solve()` sites (`site/engine/engine.js:249/260/265`) + Service Canada source-fact comment (:88–91) + merge per-partner ≥65 gates (:376–380) + `if (yr < 65) gisSum = 0;` (:382); clawback only with OAS — `oasGross > 0 ? Math.min(clawback(…), oasGross) : 0` at :248/:259 + s.3(1) ETA comment :79–82 + splitting-solver parity :366–367 (`ra.oasGross > 0 ? … : 0` per partner); mixed rows — `row.mixed` :390 + depletion guard `!row.mixed` :395 + renderer `†`/legend `app.js:807–819` (:812 `†`, :818–819 legend) + legend element `index.html:95` + `t.mixedNote` EN/FR `i18n.js:251/512`; FIRE — `fillDefaults` clamp `retireAge < age + 1` engine.js:188, quiz slider `min: 40` app.js:211, `#wi-ret min="40"` index.html:86, FIRE hint EN/FR i18n.js:96/357. All acceptance checks re-executed fresh in this pass:
| Check | Fresh output (this pass) |
|---|---|
| `tests/engine.test.js` | `RESULT: 61 pass, 0 fail` (incl. GIS $0 @56 / clawback $0 @55 repro, clawback 0 pre-OAS & capped at OAS gross, couple invariants + depletion-never-fires-on-mixed, fillDefaults 52→53 & 30→40, FIRE first draw 40, GIS/clawback 0 rows 40–64, stale 52/40 → first draw 53) |
| `tests/i18n.test.js` | `I18N-OK all i18n checks passed (50)` (incl. `t.mixedNote` EN+FR parity starting with `†`, `q.retire.hint` FIRE EN+FR) |
| `tests/ui.smoke.js` | `SMOKE-OK all ui smoke checks passed (146)` (incl. † mixed-row fixtures with bilingual legend, `wi-ret min 40` from index.html, legend element present) |
| `tests/ps-pension.test.js` / `ps-auto-service.test.js` | `70 pass, 0 fail` / `21 pass, 0 fail` (unchanged files) |
| `passes3.js` rerun (live engine) | exit 0, `conservation: OK` ×3 → `audit/math-fix/passes3-pass30.txt` (md5 `01075d0e…` — **byte-identical to passes 14–29**, seventeen consecutive deterministic passes). Line-tracked column scan (`.tmp/mathfix25/scan3.py`, run unmodified): PASS 1 (screenshot repro) 43 rows firstDraw 55 — **0** GIS>0 <65, **0** OASnet≠0 <65, **0** negative-OAS, **0** growing-Total-with-Withdraw; row 55 Total $68,327 / Withdraw $102,139 / **OASnet $0 / GIS $0**; first GIS row **65** (CPP $7,200, OASnet $9,150, GIS $12,567), depletion 56 honest (max totalBal $166,308 @54, finalBal $0); PASS 2: 50 rows firstDraw 65 firstGIS 65, depletion 82, max total $468,146 @64; PASS 3 FIRE: 65 rows, first draw @**40** (CPP/OAS/GIS $0), max total $493,710 @39 = hand-math 350000×1.035¹⁰ ✓, GIS first at **65**, depletion 48 → unfunded gap exposed |
| `scripts/deploy.sh` | `DEPLOY-OK` (fresh this pass) |
| Origin byte-check | 3-way sweep `.tmp/mathfix30/sweep30.sh` (pass-29-lineage script, tag retarget only, `diff`-proven; tracked/ignored lists regenerated via `git ls-files` + `find`/`comm`, `diff` vs pass-29 lists: **byte-identical**, 21+13; ssh `-n`, single remote md5 pass): local `site/` == `git cat-file blob origin/main:` == webroot — tracked **21/21 (3-way)** + gitignored `site/vision/*.jpeg` **13/13 (2-way)** = **34/34 match, 0 mismatch** → `SWEEP-OK`; `git fetch` → `HEAD` == `origin/main` == `288d183`; `git merge-base --is-ancestor 0839826 HEAD` ✓ |
| Live recapture (`mathfix_capture.py after` canonical + prefix-tagged copy `after30`, retire.krisforbes.ca) | Canonical `after-*` row JSONs re-captured **byte-identical to the lineage** (`after-couple-rows.json` `129e2a45…`, `after-fire-rows.json` `4af12e59…`). K-aware 11-column parse (`.tmp/mathfix30/rowsweep30.py`, pass-29 tag retarget) on the four canonical/tagged row JSONs: **ROWSWEEP-OK** — couple 40 rows firstDraw **55** firstGIS **65**, FIRE 57 rows firstDraw **40** firstGIS **65**, **0** neg-OAS, **0** GIS>0 <65, **0** OAS≠0 <65, **0** growing-Total-with-Withdraw; rows 58–64 Total $0 / OAS $0 / GIS $0 (Withdraw $49K @58 then $0, Unfunded $47K→$93K→$79K from CPP@60 — honest post-depletion drain vs the screenshot's impossible 126→141K @3.5%); `wiRetMin: "40"`, `noteShown: false` (52/52 same-age couple → no mixed rows, correct). FIRE 30/BC/ret 40: `fire_first_draw_age: "40"`. Mobile 390: retire slider `min=40 max=72 value=40`, `sliderRight 367 ≤ vw 390` (no clip), chip [25, 145], FIRE hint EN present, `svAfter40: "40 years"` |
| Live mixed-window probes (`.tmp/mathfix30/mixed_probe.py` + `mixed_probe_reachable.py`, copied unmodified, both run `after30`) | self-OLDER probe (me 52 / pt 50, shared `retireAge: 51` → clamp 53): 0 † rows, `noteShown: false`, 0 violations, `after30-mixed-rows.json` byte-identical to lineage (`96665bbf…`, == passes 15–29). self-YOUNGER reachable probe (me 50 / pt 54, shared `retireAge: 51`): **`MIXEDLIVE-OK`** — merged row **54** mixed: DOM **`daggerRows: ["54"]`** (Total cell carries ` †`), **`noteShown: true`** with EN legend `† One partner is retired, the other is still working: … not in full retirement mode.`, `gisRowsUnder65: []`, `negOASrows: []`, `growingTotalWithDrawRows: []`, `growRowsMissingDagger: []`; `after30-mixed-reachable-rows.json` byte-identical to lineage (`98c29f24…`, == passes 26–29) |
| Off-LAN (Midhurst `trouble@100.96.20.61`) | `OFFLAN-OK`, `"all_ok": true`, `"http_code": "200"`, `checked_at 2026-10-09T22:43:19Z` → `audit/bilingual-geo-check.json` |
Determinism: `passes3-pass30.txt` (`01075d0e…`, 17 consecutive), `after-couple-rows.json` + `after30-couple-rows.json` (`129e2a45…`), `after-fire-rows.json` + `after30-fire-rows.json` (`4af12e59…`), `after30-mixed-rows.json` (`96665bbf…`), `after30-mixed-reachable-rows.json` (`98c29f24…`) — all **byte-identical** to the lineage. Frames (IHDR-validated 1280×900 ×6, 1170×2532 ×2): canonical `after-couple-table.png` `cbac51ed…` 50,314 B (documented chart-animation-phase variant, == passes 21–23/28 frame; canonical `eec99021…` not drawn this pass), `after30-couple-table.png` `a634425c…` 43,544 B (first-appearance frame; PIL diff vs this pass's canonical: identical size, first differing scanline y=242, last y=899, 658 scanlines — divergence confined to the animated chart region below the fold-invariant header, the exact signature recorded for every variant frame in the lineage), `after-fire-table.png` + `after30-fire-table.png` `b4ed3a16…` 53,323 B (known fire-table variant, passes 18–29; canonical `685fd711…` re-draws in passes 1/12–17/20–26), `after-fire-step-390.png` + `after30-fire-step-390.png` 1170×2532 @3× `761e8055…` 167,045 B byte-identical to passes 26–29 (pixel-identical), `after30-mixed-table.png` `222c81a1…` 64,537 B **pixel-identical** (PIL diff bbox `None`) to `after29-mixed-table.png` (passes 22/25/29 frame), `after30-mixed-reachable-table.png` `dc6bcf46…` 57,300 B **pixel-identical** to `after29-mixed-reachable-table.png` (pass-29 frame). DOM row JSON byte-identical on all captures, so the evidence row set is unchanged.
Harness notes: Playwright runs with `TMPDIR=/home/ubuntu/projects/retire-planner/.tmp/pw` (harness TMPDIR read-only; no site impact); ssh byte-check uses `-n`; this pass rebuilt sweep + rowsweep under `.tmp/mathfix30/` (gitignored) from the pass-29 originals with tag retargeting only (`mathfix29→mathfix30`, `after29→after30`, `all29→all30`, `ignored_site29→ignored_site30`; `diff`-proven body byte-identical modulo the tag), lists regenerated from `git ls-files` + `find`/`comm` (`diff`-proven byte-identical vs pass-29 lists: 21 tracked + 13 ignored), FIRE/mobile legs via the prefix-tagged copy of `mathfix_capture.py` (`TAG.startswith('after')`, `diff`-proven single-line delta, matching the pass-28/29 copies), plus the pass-25/pass-26 lineage probes copied unmodified (`diff`-proven) and run with `TAG=after30`. All capture legs (canonical `after`, tagged `after30`, FIRE/mobile, both mixed probes) ran clean on first attempt — no render race this pass. Image inspection of the PNGs is byte-verified against the visually confirmed renders of passes 1–29 (identical md5 + IHDR where stable; frame variants enumerated above); this harness model is text-only (`read_image` not used — same policy as passes 24–29); PNG pixel diagnostics via Pillow (numpy absent).
No code change required — plan executed end-to-end; fix commit `0839826` at `origin/main` (ancestor of HEAD, confirmed via `git merge-base --is-ancestor`); all gates fresh this pass. Final: **MATH_FIX_DONE**.
+1 -1
View File
@@ -5,7 +5,7 @@
"has_i18n_script": true,
"i18n_js_fr_strings": true,
"i18n_js_size": 39689,
"checked_at": "2026-10-09T22:37:28Z",
"checked_at": "2026-10-09T22:43:19Z",
"vantage": "trouble@100.96.20.61 (Midhurst, off-LAN)",
"all_ok": true
}