@@ -6,7 +6,7 @@ This branch is an analysis-only bootstrap for a Mainline-first modernization of
The preserved 8,401-line Mainline source must be analyzed sequentially, in evidence-backed batches, before runtime work begins. All behavior-critical shortened symbols must be resolved and all batches must be complete before any public Lua API, runtime module, modern TOC, packaging, or release work is added.
Batches 01–12 (lines 1–6,000) are documented. The remaining batches are scheduled but not analyzed.
Batches 01–13 (lines 1–6,500) are documented. The remaining batches are scheduled but not analyzed.
@@ -17,7 +17,7 @@ Each row identifies one declaration, not merely one spelling. `chunk` means the
| `O` / chunk | 184 | Written at 50 numeric indices using 10×5 normalized UV cells (188–198), consumed as the hyper-effect atlas (4618–4624), then shadowed by the sound factory at 5049. | `hyperFxAtlasRects` | resolved | High. | 01–11 | UI/Animator | None. |
| `ie` / chunk | 185 | Written at 16 indices using a 4×4 UV grid (211–217). | `atlas4x4Rects` | working | High for shape, medium for texture. | 01–01 | UI/Animations | Identify owning texture. |
| `t` / chunk (second binding) | 186 | Declared nil, then shadowed again at 263; no read. | — | dead | High: no assignment/read before shadow. | 01–01 | Unassigned | Minifier artifact? |
| `i` / chunk | 186 | Atlas scratch indexes `F`,`J`,`ie` (187–216); animator factory later reuses the captured binding for rotation angle radians (4939–4941). | `atlasOrAngleScratch` | resolved | High: both temporal roles explicit. | 01–10 | UI/Animations | Split roles in rewrite. |
| `i` / chunk | 186 | Atlas scratch indexes `F`,`J`,`ie` (187–216), animator construction reuses it for radians (4939–4941), and summary construction reassigns it as the metric Y accumulator (6280–6332) before a local factory shadows it. | `atlasAngleOrLayoutScratch` | resolved | High: all temporal roles explicit. | 01–13 | UI/Animations, UI/Summary | Split roles in rewrite. |
| `e` / first atlas loop | 188 | Loop values 0–4; multiplies row offsets and indices (189–203). | `row` | resolved | High: numeric-for and row arithmetic (188–203). | 01–01 | UI/Animations | None for this scope. |
| `e` / second atlas loop | 211 | Loop values 0–3; computes vertical quarters (212–215). | `row` | resolved | High: loop and `/4` UV math (211–215). | 01–01 | UI/Animations | None for this scope. |
| `E` / chunk | 243 | Receives `math.random`, copied to `m`, then nilled; later resets `gem.fxType` (307–308, 1972) before being shadowed by the minimap factory at 5194. | `nilFxType` after temporary alias | resolved | High: temporal value flow is explicit. | 01–11 | UI/Animations | Preserve nil reset without retaining alias indirection. |
| `S` / chunk | 244 | Constant `-1`; assigned to hidden hint object's `fxType` during level-up reset (527). | `inactiveHintFxType` | working | Medium: reset/hide sequence (526–527). | 01–02 | UI/Animations | Confirm animator interpretation. |
| `y` / chunk | 245 | Constant `1`; assigned to non-hyper gems' effect type during board transition (1990–1992). | `gameOverGemFxType` | working | Medium: function incomplete. | 01–04 | UI/Animations | Complete `Ke` dispatcher evidence. |
| `y` / chunk | 245 | Effect enum `1`; starts gem-atlas cycling for selected and game-over gems, and animator idle branch advances `F` until `fxEnd` (1990–1992, 3223–3225, 4327–4340), then is shadowed by taxi handler at 6153. | `FX_GEM_ANIMATION` | resolved | High. | 01–13 | UI/Animator, UI/GemPool | Preserve numeric effect value. |
| `ge` / chunk | 327 | Populated by `ze` with faction-selected four-region coordinate adjacency graphs (1040–1249); disabled taxi estimator compares its candidate timing table with this graph (6213). | `flightGraph` | resolved | High. | 01–13 | Core/SavedVariables, UI/Flight | Disabled estimator conflates graph/timing-table shapes. |
| `we` / chunk | 327 | Set when max-score level-up begins (529); during level spawning it counts/removes carried hyper and big-star state for later restoration (3890–3906). | `levelSpawnSpecialRecovery` | working | Medium-high. | 01–08 | Engine/Scoring, Engine/BoardSpawn | Find reset and full restoration completion. |
| `r` / chunk | 333 | Six-entry array of four-number direction/offset tuples. | neighbor/offset patterns | working | Medium: signed coordinate-like tuples. | 01–01 | Engine/Matches | Establish tuple field semantics. |
| `i`,`S`,`o` / `_t` declaration | 873 | Minute/remainder locals `i`,`S` are computed but unused; `o` is shadowed at 875 before read. | — | dead | High: complete body. | 02–02 | Unassigned | Minifier/removed display logic. |
@@ -228,10 +228,10 @@ Only direct standard-library aliases, loop counters with complete local bodies,
| `i` / `_t` friend declaration | 923 | Shadowed by the numeric-for variable at 924 before read. | — | dead | High. | 02–02 | Unassigned | Minifier artifact. |
| `n` / `_t` friend declaration | 923 | Receives friend name and is used as whisper target (925–927). | `friendName` | resolved | High for legacy call flow. | 02–02 | Core/Init | Current friend API contract must be verified. |
| `i` / `_t` friend loop | 924 | Iterates friend-list indices `1..GetNumFriends()` and is passed to `GetFriendInfo` (925). | `friendIndex` | resolved | High. | 02–02 | Core/Init | Current friend API contract must be verified. |
| `_` / implicit chunk global | external; writes 925, 1433 | Receives repeated ignored friend-info returns, then a combat-log positional return; each assignment survives globally. | discarded positional return | unresolved | High for accidental global writes; later reads not yet audited. | 02–03 | Core/Init | Does any later code depend on global `_`? |
| `online` / implicit chunk global | external; write/read 925 | Receives the fifth friend-info return and immediately gates whisper publishing. | `friendOnline` | working | High for legacy intended role; API return contract pending. | 02–02 | Core/Init | Must become local and use current API shape. |
| `_` / implicit chunk global | external; writes 925, 1433, 6345 | Receives repeated ignored friend-info returns and a combat-log positional return; each assignment survives globally, and no read appears through line 6500. | discarded positional return | unresolved | High for accidental global writes. | 02–13 | Core/Init, UI/Summary | Must become local discards after API verification. |
| `online` / implicit chunk global | external; write/read 925, 6345 | Receives the fifth friend-info return and immediately gates friend whisper publishing in automatic and manual paths. | `friendOnline` | working | High for intended role; API return contract pending. | 02–13 | Core/Init, UI/Summary | Must become local and use current API shape. |
| `M` / chunk function | 953 | Computes match/combo/special points and skill gains, positions score/rank/skill floating text, and closes at 1037. | `presentMatchScore` | resolved | High: complete body; it presents but does not apply score. | 02–03 | Engine/Matches, Engine/Scoring, UI/Animations | Call sites must establish where score is applied. |
| `M` / chunk function | 953 | Computes match/combo/special points and skill gains and positions score/rank/skill floating text; used through animator/swap processing, then shadowed by event-registration helper at 6149. Earlier closures retain it. | `presentMatchScore` | resolved | High. | 02–13 | Engine/Matches, Engine/Scoring, UI/Animations | Call sites establish score application separately. |
| `R` / chunk renderer | 2059 | Shadows type alias; renders normal/hyper/empty gems and is used by restore/spawn/gravity/animator paths through line 4872, then is shadowed by timed-menu factory at 6019. Earlier closures retain it. | `renderGem` | resolved | High. | 05–13 | UI/GemPool, UI/Animator | Earlier reset closure retains old type alias. |
| `Ne` / shadow function; `t`,`n` params | 3369 | Throttled timer `OnUpdate`; delta parameter `n` is consumed before a later local shadows it. Shadows direction constant for later code. | `updateTimer`; `frame`,`delta` | resolved | High. | 07–07 | UI/Timer | None. |
| `Ne` / shadow function; `t`,`n` params | 3369 | Throttled timer `OnUpdate`; delta parameter `n` is consumed before a later local shadows it. Attached to `BejeweledFlightTimer` at 6014 after shadowing direction constant for later code. | `updateFlightTimer`; `timer`,`delta` | resolved | High. | 07–13 | UI/Flight | None. |
| `n` / `Ne` timed-window local | 3378 | Timed-window object used after delta accumulation. | `timedWindow` | resolved | High. | 07–07 | UI/Timer | None. |
| `i`,`a`,`t`,`o` / `UpdateSavedVariablesDatabase` locals | 3463–3466 | Classic stats, timed stats, saved state, and player identity then checksum seed through completed migration (3463–3614). | `classicStats`,`timedStats`,`savedState`,`playerNameOrChecksum` | resolved | High. | 07–08 | Persistence/Migration | Split `o` roles in rewrite. |
| `n` / score-migration popup scope | 3494 | Popup frame fully constructed, stored as `Bejeweled.updatePopup`, and shown by upgrade controls (3494–3560). | `migrationPopup` | resolved | High. | 07–08 | UI/Migration | Duplicate global No button is preserved defect evidence. |
@@ -744,15 +744,94 @@ Factories `V`, `O`, and `E` close and install their frames in this batch, so the
| `t` / classic `OnHide` param | 5861 | Selector frame whose transient `gameMode` flag distinguishes game start from back navigation. | `classicModeWindow` | resolved | High. | 12–12 | UI/Menu | None. |
| `n` / classic button locals | 5869, 5880 | Continue-action and new-game-action buttons; global frame names are opposite their text/actions. | `continueButton`,`newGameButton` | resolved | High. | 12–12 | UI/Menu | Preserve frame-name compatibility. |
| `t` / classic button callbacks | 5874, 5886 | Distinct clicked-button parameters are never read. | — | dead | High. | 12–12 | UI/Menu | None. |
| `D` / shadow function | 5895 | Begins the Flight Path option factory and shadows`tonumber` retained by earlier closures; remains open after line 6000. | `createFlightOptionWindow` | working | High for ownership. | 12–12 | UI/Flight | Complete in batch 13. |
| `n` / `D` local | 5896 | Top-level flight-option frame with timer/path state, captions, and open Start control; factory remains open. | `flightOptionWindow` | working | High. | 12–12 | UI/Flight | Complete in batch 13. |
| `D` / shadow function | 5895 | Constructs and installs Flight Path option frame, Start action, and `Ne`-driven timer; closes at 6017 after shadowing`tonumber` retained by earlier closures. | `createFlightOptionWindow` | resolved | High. | 12–13 | UI/Flight | None. |
| `n` / `D` local | 5896 | Top-level flight-option frame receives button/timer and is installed as `Bejeweled.flightOptionWindow`. | `flightOptionWindow` | resolved | High. | 12–13 | UI/Flight | None. |
| `t` / flight backdrop local | 5905 | Backdrop descriptor populated and applied to flight-option frame. | `backdropInfo` | resolved | High. | 12–12 | UI/Flight | None. |
| `t` / flight close-button local | 5915 | Inherited close button configured before shadow. | `closeButton` | resolved | High. | 12–12 | UI/Flight | None. |
| `t` / flight informational local | 5920 | Starts as title font string, then is reassigned to remaining-caption, remaining-value, and warning font strings stored where needed. | `infoText` | resolved | High. | 12–12 | UI/Flight | Split temporal roles in rewrite. |
| `t` / flight Start-button local | 5988 | Start button with custom `OnClickScript`; callback and factory continue after line 6000. | `startButton` | working | High. | 12–12 | UI/Flight | Complete in batch 13. |
| `t` / flight Start callback param | 5992 | Callback parameter has no read through line 6000, but its callback remains open. | unknown/likely button | working | Medium. | 12–12 | UI/Flight | Determine whether later lines consume it. |
| `t` / flight Start-button local | 5988 | Start button's custom method launches flight-learning/timed play, manages screens, then is attached as its `OnClick`. | `startButton` | resolved | High. | 12–13 | UI/Flight | None. |
| `t` / flight Start callback param | 5992 | Callback parameter is never read before closure at 6005. | — | dead | High. | 12–13 | UI/Flight | None. |
## Batch 12 resolution policy
Factories `g`, `P`, `N`, `W`, and `F` close in this batch, so their constructed-frame ownership and completed callback roles are resolved. The implicit `obj` read, template-expression result, classic frame-name/label inversion, transition-flag semantics, and fixed flight-duration display are preserved as behavior or defect evidence. Open flight factory `D`, its Start button, and its callback parameter remain working until batch 13.
| `t` / flight-timer local | 6008 | Hidden one-pixel frame driven by `Ne`, stored as `n.timer`. | `flightTimer` | resolved | High. | 13–13 | UI/Flight | None. |
| `R` / shadow function | 6019 | Constructs and installs timed-mode menu; shadows gem renderer retained by earlier closures. | `createTimedModeWindow` | resolved | High. | 13–13 | UI/Flight, UI/Menu | Factory installs globally rather than returning. |
| `o` / `R` local | 6020 | Timed-mode menu configured and installed as `Bejeweled.timedWindow`. | `timedWindow` | resolved | High. | 13–13 | UI/Flight, UI/Menu | None. |
| `t` / timed backdrop local | 6027 | Backdrop descriptor populated and applied to timed menu. | `backdropInfo` | resolved | High. | 13–13 | UI/Menu | None. |
| `t` / timed close-button local | 6036 | Inherited close button configured before shadow. | `closeButton` | resolved | High. | 13–13 | UI/Menu | None. |
| `t` / timed informational local | 6039 | Starts as title, then is reassigned through Time/value/warning/remaining font strings before line-6133 shadow. | `infoText` | resolved | High. | 13–13 | UI/Flight, UI/Menu | Split temporal roles in rewrite. |
| `t` / timed `OnShow` param | 6045 | Timed menu frame whose checkbox and height/remaining UI are reset from flight-timer visibility. | `timedWindow` | resolved | High. | 13–13 | UI/Flight | None. |
| `n` / timed-slider local | 6091 | Options-template slider ranging 2–10 minutes; shadowed by checkbox local after configuration. | `timeSlider` | resolved | High. | 13–13 | UI/Flight | None. |
| `objectText` / implicit chunk global | external; read 6093 | Undeclared value assigned to generated slider label; no declaration appears through line 6500. | unknown slider label | unresolved | High that dependency is implicit; value unknown. | 13–13 | UI/Flight | External injection or accidental nil? |
| `e` / slider-value callback param | 6103 | Slider supplies its generated Value font string name and current minute count. | `timeSlider` | resolved | High. | 13–13 | UI/Flight | None. |
| `n` / flight-checkbox local | 6107 | Checkbox returned by `CreateCheckbox`, styled and stored as `o.flightCheckbox`. | `flightCheckbox` | resolved | High. | 13–13 | UI/Flight | Empty click callback is intentional legacy behavior. |
| `t` / timed-Go local | 6133 | Go button that delegates to flight Start or starts fixed-duration timed play. | `goButton` | resolved | High. | 13–13 | UI/Flight | None. |
| `t` / timed-Go callback param | 6137 | Clicked-button parameter is never read. | — | dead | High. | 13–13 | UI/Flight | None. |
| `M` / shadow function | 6149 | Registers `UNIT_FLAGS` on the main window and shadows score presenter retained by earlier closures. | `registerUnitFlagsEvent` | resolved | High. | 13–13 | Core/Events | Callback/call site remains to locate. |
| `t` / `M` param | 6149 | Function parameter is never read. | — | dead | High. | 13–13 | Core/Events | None. |
| `a`,`s`,`S` / taxi timing locals | 6161–6163 | Total time, per-leg time, and unknown-leg/learning flag in disabled estimator. | `totalTime`,`legTime`,`needsLearning` | resolved | High. | 13–13 | UI/Flight | Branch is forced unreachable. |
| `r` / taxi-path local | 6164 | Alias of `flightOptionWindow.pathArray`, drained live and populated only by disabled estimator. | `pathArray` | resolved | High. | 13–13 | UI/Flight | None. |
| `t` / taxi disabled-table local | 6165 | Nil local whose intended continent graph/timing-table initializer is commented out; all executable reads occur only in forced-false branch. | — | dead | High. | 13–13 | UI/Flight | Removed map API left initializer absent. |
| `e` / path-drain loop | 6166 | Controls one head removal per original path-array entry; value is otherwise unread. | `entryIndex` | resolved | High. | 13–13 | UI/Flight | None. |
| `m` / taxi-node-type local | 6177 | Live `TaxiNodeGetType` result selects forced-disabled reachable or empty current branch. | `nodeType` | resolved | High. | 13–13 | UI/Flight | API requires verification. |
| `l` / disabled route loop | 6185 | Route index 1 through global `NUM_TAXI_ROUTES`; loop is unreachable. | `routeIndex` | resolved | High. | 13–13 | UI/Flight | None. |
| `B` / shadow function | 6252 | Begins summary-screen and brag UI construction, shadowing `table.remove` captured by earlier closures; remains open after line 6500. | `createSummaryScreen` | working | High for ownership. | 13–13 | UI/Summary | Complete in batch 14. |
| `n` / summary-window local | 6253 | Main window used only to derive summary/inner-panel frame levels before line-6281 shadow. | `window` | resolved | High. | 13–13 | UI/Summary | None. |
| `o` / summary-frame local | 6254 | Summary screen receives metrics/actions/brag child and is installed globally; shadowed by dropdown local at 6485 after earlier closures capture it. | `summaryScreen` | working | High. | 13–13 | UI/Summary | Complete factory in batch 14. |
| `t` / summary content local | 6267 | Starts as title/metric captions and values, then is reassigned to brag screen at 6446; factory remains open. | `contentObject` | working | High. | 13–13 | UI/Summary | Split temporal roles in rewrite. |
| `n` / summary-action local | 6334 | Publish button, then reassigned to See High Scores, Brag, and nested brag-submit buttons through line 6487. | `actionButton` | working | High. | 13–13 | UI/Summary | Callback remains open in batch 14. |
| `o` / publish friend pre-loop local | 6343 | Nil binding immediately hidden by numeric-for variable. | — | dead | High. | 13–13 | Network/Publishing | None. |
| `n` / publish friend local | 6343 | Friend name used as whisper target when implicit `online` is truthy. | `friendName` | resolved | High. | 13–13 | Network/Publishing | Verify current friend API tuple. |
| `t` / See High Scores callback param | 6359 | Clicked-button parameter is never read. | — | dead | High. | 13–13 | UI/Summary | None. |
| `t` / Brag-open callback param | 6369 | Clicked-button parameter is never read. | — | dead | High. | 13–13 | UI/Summary | None. |
| `e` / Brag-open local | 6370 | Summary alias used to show brag screen and disable three action buttons. | `summaryScreen` | resolved | High. | 13–13 | UI/Summary | None. |
| `t` / dropdown-item callback param | 6377 | Selected dropdown item supplies text and value for open menu and profile setting. | `menuItem` | resolved | High. | 13–13 | UI/Summary | None. |
| `e` / dropdown-item local | 6378 | Open dropdown or default-publish dropdown fallback updated by selection. | `dropdown` | resolved | High. | 13–13 | UI/Summary | Legacy global dropdown API requires replacement. |
| `i` / dropdown-factory local | 6383 | Local function constructs, initializes, shows, and returns one legacy dropdown. | `createDropdown` | resolved | High. | 13–13 | UI/Summary | None. |
| `i`,`o`,`r`,`n`,`l`,`a`,`d` / dropdown-factory params | 6383 | X, Y, menu width, name suffix, label, parent, and stored update function. | role-specific names | resolved | High. | 13–13 | UI/Summary | Update function is stored but not invoked here. |
| first `s` / dropdown-factory params | 6383 | Hidden by final same-spelling ninth parameter. | — | shadowed | High. | 13–13 | UI/Summary | None. |
| final `s` / dropdown-factory params | 6383 | Surviving ninth parameter is never read. | — | dead | High. | 13–13 | UI/Summary | None. |
| `n` / dropdown local | 6384 | Dropdown frame; shadows name-suffix parameter after RHS initializer uses it. | `dropdown` | resolved | High. | 13–13 | UI/Summary | Lua 5.1 initializer scope is required. |
| `t` / first two initialize locals | 6403, 6404 | Each nil declaration is shadowed by the next same-spelling declaration without a read. | — | shadowed | High. | 13–13 | UI/Summary | Minifier artifact. |
| `t` / initialize info local | 6405 | Shared mutable `dropInfo` record repeatedly wiped/populated for each menu item. | `dropInfo` | resolved | High. | 13–13 | UI/Summary | Dropdown API must copy item data. |
| `o` / initialize selected local | 6408 | Selected/fallback distribution value, initialized to first configured channel. | `selectedChannel` | resolved | High. | 13–13 | UI/Summary | None. |
| `t` / refresh pre-loop local | 6461 | Nil declaration immediately hidden by numeric-for variable. | — | dead | High. | 13–13 | UI/Summary | None. |
| `t` / vararg loop | 6462 | Iterates 1 through `#(...)` and selects that position into system-channel array. | `argumentIndex` | resolved | High for syntax; bound is defective. | 13–13 | UI/Summary | Replace with explicit vararg count. |
| `t`,`n` / channel-scan locals | 6466 | Channel name and channel ID/index used across scan/filter. | `channelName`,`channelIndex` | resolved | High. | 13–13 | UI/Summary | None. |
| `text` / implicit chunk global | external; write/read 6482–6484 | Undeclared brag prompt caption assigned and immediately configured. | `bragPrompt` | working | High for role; accidental global. | 13–13 | UI/Summary | Must become local. |
| `o` / brag-dropdown local | 6485 | Dropdown returned by local factory and marked `publish`; shadows summary-frame local after all required captures. | `publishDropdown` | resolved | High. | 13–13 | UI/Summary | None. |
| `t` / brag-submit callback param | 6492 | Clicked-button parameter is unused before a local shadows its spelling at 6494. | — | dead | High. | 13–13 | UI/Summary | None. |
| `e` / brag-submit local | 6493 | Summary alias supplying `bragString`; callback remains open after line 6500. | `summaryScreen` | working | High. | 13–13 | UI/Summary | Complete in batch 14. |
| `t` / brag-channel local | 6494 | Numeric result of `GetChannelName(defaultPublish)` selecting channel-ID versus distribution send. | `channelId` | resolved | High. | 13–13 | UI/Summary, Network/Publishing | API accepts configured string input in legacy flow. |
## Batch 13 resolution policy
Factories `D` and `R`, helper `M`, and taxi handler `y` close in this batch, supporting resolved ownership even where taxi estimation is deliberately unreachable. Implicit globals, forced-false API code, chunk scratch/key reuse, and the `#(...)` vararg defect are preserved as evidence. Open summary factory `B`, its temporally reused locals, and the Brag callback remain working until batch 14.
Source: `Legacy/Bejeweled_Mainline.lua` at commit `6faec1c`
Coverage: lines 6001–6500 inclusive
Blank: 6018, 6148, 6152, 6251
Status: complete; flight-option factory `D`, timed-menu factory `R`, event-registration helper `M`, and taxi-node handler `y` close, while summary-screen factory `B` and its Brag callback continue into batch 14
## Line-indexed coverage audit
| Lines | Evidence represented |
| ---: | --- |
| 6001–6017 | Finishes the flight Start callback by hiding mode/menu/skills screens and showing the main window. It installs the custom callback as `OnClick`, creates the hidden one-pixel `BejeweledFlightTimer` driven by `Ne`, attaches button/timer to the flight window, installs `Bejeweled.flightOptionWindow`, and closes `D`. |
| 6018 | Blank. |
| 6019–6044 | New `R` shadows gem renderer `R` and constructs the centered Timed Mode menu with inherited close button, backdrop, and title. Successive `t` declarations configure those objects before shadowing. |
| 6045–6078 | Timed-menu show pauses and resets the flight checkbox, then selects a tall layout with remaining-flight UI when the flight timer is visible or a shorter layout without it. Hide with `newGame` clears the flag/unpauses and stops a timed-mode timer or resets flight-learning countdown/movement; ordinary closing returns to game-type selection. |
| 6079–6106 | Adds Time/value captions and an options-template slider spanning 2–10 in unit steps, initialized to 5. It writes undeclared `objectText` into the generated slider label and updates the generated Value font string to `N Minute(s)` on value changes. |
| 6107–6147 | Adds the flight-time checkbox with an empty callback, too-short warning, remaining-flight caption/value, and Go button. Go delegates to flight Start when a visible checkbox is checked; otherwise it starts timed mode through `j(ae,sliderMinutes×60)`. It marks `newGame`, hides the timed menu, installs it as `Bejeweled.timedWindow`, and closes `R`. |
| 6148 | Blank. |
| 6149–6151 | New `M` shadows the completed match-score presenter, ignores its parameter, registers `UNIT_FLAGS` on the main window, and closes. |
| 6152 | Blank. |
| 6153–6168 | New `y` shadows gem-animation effect enum 1 and begins a taxi-node handler. It derives node ID, route count, route-map dimensions, initializes coordinate/time/key temporaries, aliases and fully drains `flightOptionWindow.pathArray`, and retains a nil flight-table local. |
| 6169–6180 | Comments document removed map APIs and disable flight-time estimation. Live code still reads `TaxiNodeGetType`; the reachable branch is forced false by `and 1 == 0`, leaving all subsequent route estimation unreachable. |
| 6181–6238 | Disabled route estimation would select the taxi node, grow global `NUM_TAXI_ROUTES`, derive pixel source/destination keys from normalized coordinates, initialize/fallback continent flight-time tables, accumulate leg times with a 1.5-second adjustment after route two, flag unknown legs, append key/key/time triples to `pathArray`, and publish total time/learning state. |
| 6239–6250 | Still-disabled tooltip logic would show unknown or formatted travel time according to the learning flag. The `CURRENT` node branch is empty, then taxi handler `y` closes. |
| 6251 | Blank. |
| 6252–6279 | New `B` shadows captured `table.remove` and begins the summary-screen factory. It creates the board-sized hidden Game Over frame, backdrop and inner panel, stores `_t` as `FillData`, and adds the title. |
| 6280–6333 | Reuses chunk scratch `i` as a vertical layout accumulator and assigns chunk `I` from lowercase `Final Score` suffix (`score`). Alternating caption/value increments of 20/42 place Final Score, Time, Level, Largest Cascade, and Largest Combo fields and store each on the summary. |
| 6334–6354 | Creates Publish Scores with tooltip. When `dataDump` exists, click sends guild `HSPub`, enumerates friends and whispers online names, opens Feats of Skill/high-score tab, then hides the popup. Friend-info discard slots write implicit global `_`, and online state uses implicit global `online`. |
| 6355–6382 | See High Scores directly opens the same screen/tab. Brag opens a nested brag screen and disables all three summary actions. Global `Bejeweled.Dropdown_Item_OnClick` updates an open/default dropdown's text, selected value, and persisted `defaultPublish`. |
| 6383–6400 | Defines local dropdown factory `i` with duplicate `s` parameters. It constructs a named dropdown, optionally creates a label (empty string is truthy), stores update/width/selection fields, sets menu mode, and styles generated text from the label font. The first duplicate `s` is inaccessible and the surviving one is unused. |
| 6401–6436 | Dropdown initialization uses shared `dropInfo` to add three configured distribution choices and then custom server channels from the brag-screen parent. It retains whichever configured/custom value case-insensitively matches `defaultPublish`, falling back to the first configured channel, then rewrites the persisted setting. Two preliminary `t` declarations are shadowed before use. |
| 6437–6445 | Anchors the dropdown, initializes/reanchors/sets selection and width on show, stores the current selected value, and returns it from the local factory. |
| 6446–6481 | Reassigns caption local `t` to the nested brag screen, styles it, and initializes system/custom channel arrays. `refreshChannels` rebuilds system names by looping to `#(...)`—the length of the first adjusted vararg value rather than the number of varargs—then scans channel IDs 1–15 and excludes names containing any system-channel string before storing custom names. |
| 6482–6500 | Creates the brag prompt through undeclared global `text`, creates/publishes the dropdown, and reuses `n` for the Brag button. Its callback aliases the summary, shadows its unused button parameter with channel ID, sends `bragString` to a numeric channel or configured distribution, and clears the string at line 6500. The callback and factory remain open. |
## Findings
- Flight option startup is now closed: its custom method is also the actual click script, and the separate one-pixel timer delegates all updates to previously resolved `Ne`.
- Timed-menu `newGame` is another transition guard. Cleanup depends on current game mode: ordinary timed mode stops the bar timer, while flight-learning mode resets animator countdown and movement permission.
- Flight estimation is not fully inert. The route-walking/storage branch is forced unreachable, but `GetNumRoutes`, `TaxiRouteMap` dimensions, `TaxiNodeGetType`, and path-array draining execute before that guard.
- Summary construction proves chunk `I` is the literal stats key `score`. Chunk scratch `i` is temporally reused for summary vertical layout before a local dropdown factory shadows it.
- `refreshChannels` uses `#(...)`, which in Lua 5.1 measures the first adjusted vararg value rather than counting arguments. This is behavior-critical defect evidence; a future fix must preserve channel filtering intent without treating it as valid vararg enumeration.
- `UI/Summary.lua`: open summary factory `B`, metric layout, score publishing, navigation actions, and brag-screen/dropdown construction.
- `Network/Publishing.lua`: manual guild/friend publication and channel-based brag sends, including current implicit-global defects.
## Forward dependencies
Batch 14 must resume at line 6501 inside the Brag button callback, close that callback and summary factory `B` only when the source does, and continue sequentially through line 7000 without assuming later factory call order.
Assert-True($lineCount-eq8401)"Legacy source has $lineCount lines, expected 8401"
Write-Output"Verified: forbidden files absent; legacy and $($assetPaths.Count) asset hashes match $SourceCommit."
Write-Output"Verified: strict UTF-8 text, exact 8,401-line schedule, and complete batch-01/02/03/04/05/06/07/08/09/10/11/12 line coverage."
Write-Output"Verified: strict UTF-8 text, exact 8,401-line schedule, and complete batch-01/02/03/04/05/06/07/08/09/10/11/12/13 line coverage."
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Nighthawk42