KaipaBots — PR-Review

Card c014ac — analyst Rec A. Pause-opened Options/Save windows had no back affordance (Esc-only, undiscoverable). Right-aligned localized hint in the shared header (DrawOptions) and title strip (DrawSaveGames), gated on GLOBALClientStage == "Game" so standalone menus (back button) and title screens stay unchanged. Merge-note: locale-table insert conflicts trivially with #1358 — whichever lands second rebases.

Card c140b3 — label fix per Kai. Option gates only SpaceMap's MenuShader; menus draw StarNest unconditionally. Literal renamed; new entries added to both locale tables (previously had none — DE showed the English literal). Option key GLOBALOptionsTable["MenuBackground"] unchanged — no behavior delta.

Card d78ec9. Custom Color / Changes["WindowColor"] killed the hover/active face swap → frozen-looking buttons. Faces now derived: +25%-to-white hover, ×0.7 press, O preserved. No tint helper existed — inline derive per analyst.

Card d465df. Layer strip was static-looking: hover fired only the tooltip, toggle flipped silently. Now: HoverColor tint (α0.2) on hover, ActiveColor tint (α0.35) while held, standard PlayEffect click on Pressed.Left. Returns unchanged.

Card 7bd409. ppairs hash order decided which overlapping container got the wheel — luck, not Z-order. Start now stamps a monotonic DrawIndex (entries were already frame-refreshed via LastSeen); WheelMoved takes the highest-index eligible hover → last drawn wins.

Card 9b100e — soft-lock: Esc on Options/LoadGame left the stage while a Confirm modal held MouseTable.SetModalCapture(true); on Intro the dialog never draws → mouse dead until restart. Mirrors BlueprintEditor's ConfirmWindow.IsActive() guard. GUI.lua:211 already guarded.

Card fbb70a — real data-loss: Save & Update overwrote live GLOBALOptionsTable + disk with a require-time snapshot (reverted key rebinds/theme/toggles written since load). Now mirrors DrawGameOptions: FILE values written INTO GLOBALOptionsTable, then saved.

Card 9b8ebf. getWireHover/getUnitHover early-returned before any WindowHover check — wires and units under open windows still produced hover data while all five later types (item/wall/building/conveyor/terrain) were gated. Inline not WindowHover on both conditions.

Watch-Pass 5aa56135..3799bc20 (spec-only).

Card e7696a. Headbar draws (MachineName or Title) at :193 but keepMinimumWindowSize measured Title only — longer MachineName (e.g. CombatAlert) under-measured MinWidth and the text ran into the X zone. One-arg fix; dead-code half of the card was already absent on main (verified stale by analyst).

Card f53642. Confirm/Cancel (delete/quit) had no audible or visual click feedback — hover composite only, silent toggle+Stop.

Watch-Pass 4619489a..3144c071 (spec-only + eine Pin-Reparatur).

Fixt die Test-Regressions der letzten Merges + nachgezogene Pins:

Spec-PR fuer den Watch-Delta 610d71b0..4619489a (084e5a9a Freecam).

Card c35a60 (analyst sign-correction confirmed).

Card 2e40bf — Kai's pin: crash loudly + caller sweep.

Card cc9419 (analyst count-correction applied: 2 real leaks, not 6).

Card d82be3 (analyst correction honored: up/down already handled at :493/:501 — only delete/home/end added).

Card c82304 (both analyst corrections honored).

Card eb45ab (analyst: all sites confirmed).

Card 0e588a (analyst mechanism-correction applied).

Card 06683a (analyst impl note honored: hit-rect uses the label's real localized width).

Re-pointed the five live legacy DrawButton users at DrawButton02 (same (Position, Size, Changes) shape — Changes.WindowColor honored, BorderWidth was ignored by the old impl too): DrawAsteroidButton, DrawNoItemButton, DrawTextButton, DrawTilePreviewButtonViaFunction, DrawSplitterCW 'All' button All five now get real hover/active beveled faces instead of the legacy identical-color path drawButtons in DrawSplitterCW ga…

Card c502b3 (analyst corrections applied).

GuiConfig.DisabledOpacity = 0.4 — one constant replacing the sprinkled setColor(1,1,1,0.4) wraps Optional Disabled flag on DrawCanvasButton, DrawRoundedCanvasButton, DrawGuiButton, DrawWideGuiButton, DrawSmallGuiButton: draws Canvas['Disabled'] or Canvas['Idle'] dimmed, returns inert {Pressed={}, Release={}} — no hover/active/sound/click Migrated the three manual-alpha sites: DrawBlueprintEditor craft, DrawUpgradeDet…

FILEActiveSliderID latch (FILEActiveScrollBar pattern): drag starts on press in-box, persists while left is held with pointer X clamped to the track, releases on mouse-up — knob no longer drops when the pointer leaves the box Optional ID param (position-derived fallback key) so sibling sliders can't steal each other's latch Optional OnTop param + active-scissor check: covered/clipped sliders can't start drags or fire…

ToggleInput + DrawCollapsed had zero callers — deleted. getLineHeight/getColorForPriority were duplicated across ChatWindow/DrawChatWindow (identical semantics at font 14+4). DrawChatWindow exports both; ChatWindow now calls the shared versions so they cannot drift (priority color vs GuiConfig.ErrorColor was already drifting post-#1314).

handleSound in DrawButton, DrawButton02 and DrawActionSlot fired COMMANDPlayEffect on the per-frame HELD state — the cached static source refires each time the 50-150ms sample ends, so a held click keeps ticking. All three now read the Pressed edge like DrawCanvasButton already does.

Spec-PR + ein Failspec-Fix (2 Zeilen Source) fuer den Watch-Delta aad0b7fb..610d71b0.

Card a63abc — four small UX fixes on the chat input field.

InfoText.AddLine mirrors every line unconditionally into the chat log — action spam (button mashing, blocked actions) flooded it with identical rows.

/ opens the chat input with the slash already typed - command quick-open as users expect from other games.

The track is drawn at Position.x + Size.x while logSize.x = overlaySize.x - padding2 — with padding=6 and a 12px track it ended +6px past the window edge. Text scissor ran flush into the track (0px gap).

DrawLineOfText gains a RawText flag. Stored log lines are display-final and must not re-run LocaleManager.Get every frame - a stored string that happens to equal a locale key silently swaps at draw time (e.g. a save name matching a key). Both chat draw sites (active DrawChatWindow, collapsed ChatWindow overlay) pass RawText=true.

TextField.Key gains per-field OnTab (global tab would break stage-level handlers); CommandParser.MatchPrefix returns sorted matches + common prefix; chat completes then cycles. Editing resets the cycle.

Collapsed overlay drew bare text on the world - OldSkool's black TextColor unreadable on dark space. Each line gets a dark 8-offset halo via DrawLineOfShadowText (explicit dark shadow for theme-independent contrast).

entry.Time was stored but never read. Lines now hold full alpha for 10s (game time), fade linearly to gone at 15s - matching the InfoText mirror's 8s feel. Loaded saves clamp negative age to fresh.

First Up stashes the in-progress input as 'slot 0' (Down below newest restores it, not ''); identical resubmits no longer burn history slots. Stash clears on submit.

CommandParser returns a second severity value (non-breaking - {string} shape unchanged); all error returns mark 1; parseInput threads to AddLog; priority color now reads the theme ErrorColor instead of hardcoded {1,0.3,0.3}.

Unknown command gains localized '- type /help for a list'; bare '/' runs the help handler; commands with registered Usage print 'Usage: /name <args>' on missing args. EN+DE keys.

GUI.KeyPressed still receives keys while typing (TextField), but handleGameSpeed/OverLays/PlayerController/f12/f7/'#' no longer fire out of a chat field. Rest-gap to #1248.

FrameCompositor shims love.graphics.setCanvas during the GUI.Draw bracket: bare resets in the 135-file GUI call-graph resolve to the active layer canvas; targeted calls pass through - zero factory edits. Game.Draw composites Scene -> GUI -> Pause -> screen. Save thumbnails build from scene+GUI via CaptureShot -> the open save/pause menu no longer lands in the image; F12 captureScreenshot unchanged.

Per-title canvas buffers were unbounded (unique generated save names) -> ClearBuffer exports + outpost-switch clear in SaveNameList. TransitionSave/MapTransferSave rows show localized 'Travel Save' with preserved datetime suffix. Info line reordered Date|Mode|Playtime. Shared OpenSaveFolder lib extracted; 'Open Folder' added to the healthy save-detail pane (4-button row, zoom 3->4).

SaveFolderList stores counts computed during enumeration, exposes GetCount(FolderName); folder labels render 'name (n)' and '> ' for the active outpost. Selection logic still uses raw folder names.

New WarningColor theme key exposed via GuiConfig; unsaved Save Options highlight uses amber instead of error red; localized explainer 'Oldest saves are removed beyond these limits' (EN+DE).

MakeSaveNameButtonCanvas gains a Selected flag + selected-canvas variant (keyed with a suffix); SaveNameList and DrawSaveGameFolders pass selection state; obsolete drawHighlight blocks/constants removed. ClearBuffer() exports on all three row-button modules; buffers cleared on outpost switch.

All save kinds (AutoSave, QuickSave, menu Save, Overwrite) report via localized InfoText.AddLine -> ChatWindow.AddLog. DrawFeedbackWindow widget, its requires and the getShowFeedBackWindow helper deleted.

Shrinking loop could hit fontsize 0 -> newFont(0) crash. Floor at 8px; text still too wide is truncated with '...' so the drawn label always fits.

Renaming saves was impossible; only filesystem support existed.

Save menu shipped mixed-language text for German sessions.

The form needed click-to-activate, showed a hardcoded Input... ghost, failed silently on invalid names, and had no way out.

Legacy saves missing GameMode/PlayTime/TimeStamp crashed the whole save-menu frame (fields healGameData never heals).

Colored Auto/Quick/Transit tags on save rows (healed flags); localized keep-N line under the title reading live MaxAutoSaves cap. Fleet 851f65.

Typed name checked against preview FolderName entries; collision routes through localized ConfirmWindow before FILETriggerSave. Fleet 77b5b9.

Signature (Position,Data,OnTop,SlotGap): BonusInventory call becomes natively correct, DrawContainerWithTooltip forwards OnTop. Hover/highlight/press gated. NOTE: overlaps ready PR #1289 in same file — rebase whichever lands second. Fleet b05605.

Hand x slot-state gesture hints appended to slot tooltips; ShowSlotHints toggle default ON; EN+DE locales; UsedBattery maps to swap. Fleet 206fc2.

Both DrawSlotVisual amount wrappers early-return on amount <= 1. Fleet b0d3e8.

Right-click on a slot no longer unbinds while an action is armed — the armed click only deselects. Fleet 8758a7.

DrawContainer fires COMMANDPlayEffect (buttontest1 @ 0.5) on Pressed Left/Right edges — pattern ported from dead DrawContainerSlot.lua, which is deleted. Covers every container caller. Fleet card dd7347.

24 dupe moveStackToInventory/handleShortcuts blocks deleted; UpdateHandSlot.handleLeftClick handles shift+left via new QuickTransfer.SlotToPlayer; LootOnly gains quick-loot. MERGE-ORDER NOTE: touches Radar/Teleporter DrawEnergyTab which #1283 deletes — whichever lands second needs a rebase. Fleet card a9bfc6.

All 17 outliers normalized to WindowData["Size"] = windowSize: 5 in-place towers, 11 no-write machines (DefenceTower trivially conforming), MagicEnergy delegation now return-assigns. Fleet card f4edb5.

Caller-side cap in UpdateHandSlot + OverPlayerInventory twins; command-side clamp in AddItemToSlot/AddItemToPlayerInventorySlot returning the un-inserted remainder. Fleet card a7e98b.

LootOnly.sameItem gains the UsedBattery exclusion both siblings carry. Fleet card 0f440b.

Tab state moves from module-global FILETabData to MachineData["CurrentTab"] (persisted); canvas tabs; title/grid +TABBARHEIGHT; MachineData threaded into DrawFilterChoice. Fleet card 5a734c.

FirstOpenPosition.ClampToScreen exported; re-applied in drawComplexWindow at-rest block — tab-refit and post-release resize growth can no longer overflow right/bottom. Fleet card 567924.

Adds the toggle after Show Status in Drill, MetalWorker, Smelter, Storage, DarkPyramidB/C — the world-overlay gate honors ShowSlotNumbers for pyramids too (MachineOverlayLayer :482). Fleet card 2611c8.

DrawResources, DrawIOMCW, StorageModule x3 — zero external requires. Fleet card d33566.

BatteryLight, Capacitor, PyramidLeech — 56px dead band under the header. Fleet card f55efc.

SLOTBORDERWIDTH=6 for the slot pitch vs GuiConfig border for paddings; duplicate HEADERHEIGHT decl dropped. Fleet card 88b067.

Renames all nine occurrences (grep-truth: Drill included, card listed 8). Fleet card 9b74bb.

BlockTimer Main gate + Set(0) + ConsumeClick — the button fired on raw Pressed.Left. Fleet card 60b792.

Localized TooltipManager.Trigger on edit hover, OnTop-gated. Fleet card 88433b.

Adds the standard bottom-pinned DrawMachineRenameRow (lazy-require pattern per Storage). Fleet card 78f1a5.

FILELastActiveTab file-global -> MachineData["LastFitTab"] — the shared global made two open crank CWs fight the refit per frame and skipped refits on reopen. Fleet card 73d6d2.

rowAmount mfloor -> math.ceil — the partial item row was dropped from the window-size calc, so the CANCEL/BLOCK/ALL button row overlapped items. Fleet card 819571.

Quatum Prediction -> Quantum Prediction. Fleet card e8d9ed.

Pass leftClick as parameter into drawActiveWindow — it was read as an undeclared global (always nil), so the click edge never reached DrawComplexWindow (X, lock, drag, resize). Fleet card 6fd277.

Item icons and Machines/Icons glyphs show the generic 'flame-in-house' instead of the real machine. Kai-agreed approach: throw-away scenario that renders the actual body canvas rather than screenshots.

With the wire layer visible, a tile carries both Type.Machine and Type.Wire — but getHoverTargetType returned only the priority winner and FILEMarkMap deduped by position alone, so wires under every footprint cell (not just the anchor) survived demolish.

All 5 channels were staged-only until 'Save & Update' — volume was tuned blind. Now a representative sample plays at the STAGED volume × staged master.

Kai-agreed design: the 'Modules' window becomes 'Robot' — smaller landscape format, tabs Modules/Upgrades, structured stats with base/upgrade/module breakdown, list-style upgrade view.

Implements card b09805 (3 Kai-coordinated parts; analyst CONFIRMED with flags, all addressed).