docs(refactor): §S74 — room_walker.js / compile_rooms.py refactor survey (spec only) - #87
Merged
Conversation
…ty gate vacuous on a dead path, revive-first verdict Spec-only survey of the paired room compiler (build/room_walker.js 1,347 ln / scripts/compile_rooms.py 1,321 ln). Measured, not guessed: the parity witness prints pass=0 fail=0 exit 0 (dead /tmp/wt-fable-livewire path, §S61.3 class B); repointed at ~/bim-ootb/modeller it passes 6/6 byte-identical, so the port is still verbatim and only the lock's front door is dead. witness_geomap_tier3 is RED on a stale exact-lock (1/21 vs measured 3/21). Zero slice-based consumers. Top candidate = revive the witness trio + re-lock the baseline (priority 1); copy-drift consolidation (3 algorithm generations across 4 JS copies) only after. No line-count split recommended. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Spec-only survey — no product code touched
Appends §S74 to
prompts/SCRIPT_LENGTH_REFACTOR_SEAMS.md(base =fable/meshdb-livewire, where the canonical file lives; §S67 was already taken by 4D_GANTT_TM_REFACTOR.md, so the next free number in the shared sequence is §S74).What was measured (all runs logged, Log Mandate)
witness_room_walker_parity.jsas-is: vacuous —SKIP ×6, pass=0 fail=0, exit 0(dead/tmp/wt-fable-livewirepath, the §S61.3 class-B bug). Same dead path inwitness_room_fill.js/witness_room_wellformed.js.~/bim-ootb/modeller: pass=6 fail=0, byte-identical py↔js on all six buildings — the port is still verbatim; only the lock's front door is dead.scripts/witness_geomap_tier3.py: RED today — stale exact-lock (baseline recall claimed 1/21, measures 3/21; the flood baseline improved and the lock was never re-recorded).deploy/dev/lib, +131 in bim-ootbviewer/lib— §CONTAINMENT-ALIAS join never ported to the py "ground truth" despite its own LOCKSTEP comment).Verdict
room_walker_ext.js) is priority 2, blocked on candidate 1 per the file's governing rule (low witness coverage disqualifies).schedule_author.js/ §S60room_graph.jsprecedent).No auto-merge — left for human review.
🤖 Generated with Claude Code