chore(hygiene): fix stale CLAUDE.md code-map; (wip) de-dup steer-firewall + bench stats#183
Conversation
…/Supervisor atom, program.ts/runProgram deleted (#168)
✅ No Blockers —
|
tangletools
left a comment
There was a problem hiding this comment.
✅ Approved — 1 non-blocking finding — 6f736fca
Full multi-shot audit completed 1/1 planned shots over 1 changed files. Global verifier still owns final merge decision.
Full immutable report for this review: trace
Summary comment for this run: full summary
tangletools · 2026-06-06T19:17:51Z · immutable trace
Pays down the architecture debt the self-audit found.
Done: CLAUDE.md code-map was actively misleading — cited
src/loops/,program.ts,runProgram, and adrivers/subdir, none of which exist (op-set deleted #168, loops→runtime rename). Rewritten to reflect reality:runLoop(round-synchronous leaf) +Scope/Supervisor(recursive atom, #151) + the personify combinators, with an honest 'two substrates coexist, prefer Scope/Supervisor' note. It misled the audit itself.Planned on this same branch (next, done carefully — not rushed):
assertTraceDerivedFindingsis implemented twice — a private copy in dynamic.ts + the exported one in personify/analyst.ts; a single correctness invariant must have one enforcement site).bench/src/stats.mts(runPoolre-export +pairedLift+makeRng) +buildRunRecordFromAttempts; route the 6 gate runners through it and convert clbench-codebase/commit0 toopenSandboxRun+openai-compat(kills the duplication the audit flagged).