docs(overhaul): close s5 c5

This commit is contained in:
Erik 2026-09-05 05:26:03 +02:00
parent 1b7ee4e581
commit 29c05e48c8
2 changed files with 69 additions and 2 deletions

View file

@ -3196,3 +3196,70 @@ The InstalledDat nonpassing identities must remain the accepted nine failures
plus one look-in skip. Production/gate-honesty review then resumes as pass
3/10; the already-passed retail/deletion lens is not repeated because this
round changes no production or retail claim.
## 35. S5-c5 landing, review closeout, and lead graphical smoke — 2026-09-05
**CLOSED AND LANDED; GRAPHICAL PASS IS PROVISIONAL PENDING OWNER
INSPECTION.** The reviewed implementation stack in `s5-c5-openai-impl` is
`36bea55f337c010683263a2eb315514c47121e9f` ->
`2886091aa92f62ceec8c16d4fd5b2dd6056eb5b8` ->
`158656f0df3edd5e28ac3a9ed743f4c435e321f9`. It maps to the campaign stack
`bf53e2ad6` -> `e625dc4e6` -> `1b7ee4e58` on top of contract
`b77989c32`. The final campaign commit is
`1b7ee4e58100b27a55063e0bd462f54c589b1d7e`.
The deletion removes the callerless portal-BFS research graph and its tests,
the complete Facility-stair instrumentation family, the obsolete renderer
probe/output branches and hidden scratch, and the raw-GL diagnostic reader.
The live `RetailFrameWalk`/`WalkFrameDriver`, `BeginWalkFrame`, outside-view
reassembly, two FIFO lists/router/material/shader paths, typed walk transcript,
frame profiler, terrain diagnostics, and
`UnregisteredRenderMembershipCount` remain. Architecture and inventory now
state one production graph; AP-19 is retired, AD-116 is amended, and
AP-238--AP-243 remain active.
Sequential review closed in three passes. Retail/deletion pass 1/10 returned
PASS with no finding after independently rechecking the named functions and
the paired binary/PDB identity. Production/gate-honesty pass 2/10 found one
MINOR test-only non-vacuity hole: the unique-owner reflection scan omitted
static fields. Fix round 1 added `BindingFlags.Static`. The lead reproduced
both required static-owner mutations and their intended first failures, then
restored them exactly. Production re-review pass 3/10 returned PASS with no
remaining finding; the production source tree was unchanged by the fix.
Exact reviewed-tip evidence is hermetic 16,921 passed / 0 failed / 0 skipped,
canonical InstalledDat 368 passed / the accepted nine failures / one accepted
look-in skip, and both manifests 30/30. Fresh campaign-tip verification passed
Release at 0 warnings / 0 errors, App focused 146/146, and Core focused 8/8.
The campaign tree was clean before the graphical gate.
The first smoke attempt is excluded as environment preflight evidence. It
opened the default machine-local `acdream.pak`, rejected recipe 8 against the
recipe-10 client, emitted no PNG, exited with the .NET unhandled-exception code,
and left no client. Log:
`logs/selfgate-20260905-051948-s5-c5-landed`. No renderer assertion was
reached. The lead then ran the repository's validated transactional bake to a
new path, leaving the recipe-8 package untouched. Recipe-10 output:
2,237,865 keys, zero failures, 597,482,264 bytes, SHA-256
`4E0FA6C589ECE12E13099A56385077964961EC52F888BC9161C6F5D34829B59B`.
The one corrected smoke run began with ACE listening on UDP 9000, zero client
processes, a clean campaign tree, and the already-green Release binary. It ran
`tools/overhaul-selfgate/route.txt` once against the new package, exited 0,
confirmed graceful logout, and left zero client processes. The log contains no
crash, invariant failure, device-loss/screenshot failure, or deleted probe
prefix. Exact evidence root:
`logs/selfgate-20260905-052130-s5-c5-landed-v10`. PNGs:
- `artifacts/screenshots/00-login.png`;
- `artifacts/screenshots/01-cathedral-stairs.png`;
- `artifacts/screenshots/02-facility-stairs.png`;
- `artifacts/screenshots/03-holtburg-house.png`; and
- `artifacts/screenshots/04-holtburg-terrain-leak.png`.
Lead original-resolution inspection and comparison with
`logs/selfgate-20260905-020703-s5c4-onepass-detail` found the same intact
cathedral, Facility, house-portal, and outdoor-terrain geometry; only authored
time/weather changed. Verdict: **PASS, provisional until the owner inspects
the PNGs.** S5-c5 is closed. S5 automated/performance closeout and G4 remain;
never merge to `main` before G4.