Nightly Fleet Report — 2026-09-27

Reviewer-Runde nach dem "All lanes finished"-Kickoff. Alle Angaben gegen den

aktuellen main-Stand verifiziert (Suite lokal ausgeführt, Diffs gesichtet).

Kurzfassung

45 PRs offen aus den Nacht-Lanes, 26 von Kai gemergt tagsüber.
Alle drei Tester-Bugreports gefixt (#199, #200), das große Szenario-Rot

(56 broken) war fast komplett die eine AddItemToPlayerInventorySlot-Assert —

Fix liegt bereit.

Failspec-Pipeline hat funktioniert: 3 echte Source-Defekte gefunden

(#234/#237/#238) und alle drei Fixes liegen bereits vor (#236, #239+#200, #240).

Lane 1 hat die Quarantäne auf 0 gebrannt — lokal verifiziert:

2566 Specs, 0 Fehler (#196).

Merge-Reihenfolge nötig: #200 vor #239 (überlappende Hunks).
Kai-Entscheidungen: #199 Restrisiko Itemverlust, #198 Lane-1-Fixes in Source.

Bug-Reports aus dem Tester-Chat

Ledger: BotLogs/BugReports.md. Drei Reports aus dem Chat, alle erledigt:

Datum	Titel	Status	Fix	Anhang
2026-09-26	Absturz beim Ablegen auf volles Inventarfeld	done	#199 (+Härtung #206/#207)
2026-09-26	Item auf volles Feld abstellen crasht	done	#199 — gleiche Assert-Familie
2026-09-26	Absturz beim Wechsel auf die Weltkarte	done	#200 (Anhang: Logfile)	📎 2026-09-25%20-%201790333372.txt
2026-09-26	Ticket: Blaupause 90° drehen	done → Fixer	#212 (siehe Review)

Nächtliche Failspec-Funde (queued → fixer, Fixes bereits offen):

Fund	Failspec	Fix	Bemerkung
`Notes.lua` mitten in Statement abgeschnitten	#234	#236	verifiziert: Datei endet bei `love.graphics.newCanvas(`
TransportsLayer: Heimkehr-Linie nie gezeichnet + Fallback-Crash	#237	#239	nach #200 mergen
LightMap-Freeze bei Null-Ray	#238	#240	Endlosschleife `vector*2` verifiziert

Review der Fixer-PRs (alle diffs gelesen)

PR	Was	Verdict
#199	`assert`-Stack-Crash → Log + `return false` in beiden AddItemTo*Slot.lua	geprüft, korrekt. ⚠️ `UpdateHandSlot`/`swapHandAndSlot` ignorieren `false` → theoretischer Itemverlust-Pfad (Kommentar gepostet)
#200	`PlayerController.GetPlayerAsteroid()` (nil!) → `GameState.GetCurrentAsteroid()`	korrekt. Methode existierte nie — der Call selbst war der Crash
#206	`machineMap[y][x]` Zeilen-Guard in PlacePyramidB/C + Warnung	korrekt, LogManager importiert
#207	nil-Guard `GameState.GetLastPosition()` in SpaceOverlay	korrekt
#212	Editor-Blaupause per R drehen	sehr sauber: UndoRedo.Push, Dirty-Flag, RecomputeWireNames, KeyMap-Gating (Paste/Stamp/Rename/Confirm), 192-Zeilen-Spec
#236	abgeschnittenes `newCanvas(` in Notes.lua auskommentiert	korrekt — Datei wird nirgends required
#239	`originX/Y` auf Funktions-Scope gehoben, Fallback auf Drohnen-Asteroid, Draw-Guard	korrekt, aber allein unvollständig — ruft noch `PlayerController.GetPlayerAsteroid` → nach #200 mergen (Overlap → Rebase nötig)
#240	Null-Rays in `addEdgesToRays` skippen + Early-Return in `getTestRay`	korrekt, Defense in depth

Lane 2 (Szenarien)

PR	Was	Verdict
#203	Seeding über `GivePlayerItems`-Helper statt `AddItemToPlayerInventorySlot` (57 Dateien)	Root-Cause-Fix — erklärt die 44+ Slot-Kollisionen in der Verifikation
#220	S47 + S124–S129 Blueprint/Wire-Batch	reviewed
#223	S130–S142 + Library-Sweep S410–S467 (400 Dateien, +31k)	Struktur geprüft — 397 Scenarios + 2 Helpers; Stichproben sauber
#233	S294 Szenario Blaupausen-Rotation	reviewed — passt zu #212

Verifikations-Stand (Windows, 2026-09-26): 20 verified / 2 flaky / 56 broken.

Die Broken-Liste ist fast vollständig die eine Fehlersignatur

AddItemToPlayerInventorySlot.lua:74 "cannot stack 2 different items" — #199

fixt die Assert, #203 fixt das Seeding: die nächste Matrix sollte massiv grüner

laufen. Flaky: main.lua:200 Update nil (S02, S05) — Timing, beobachten.

Lane 1 (Unit-Tests)

#196 — Quarantäne auf 0, lokal verifiziert. Per-Spec-Isolation war zu

flach (Referenz-Restore + Key-only package.loaded-Snapshot). Fix kopiert

love zwei Ebenen tief und snapshotet Modul-*Werte*. Ich habe den Branch

ausgecheckt und luajit UnitTests/run.lua selbst laufen lassen:

2566 passed, 0 failed — Behauptung stimmt. Nur Harness-Änderung, keine

Source-/Spec-Hacks.

#197 dedupliziert TARGETS.md um 136 Stale-Zeilen aus den Union-Merges.
#198 batch 152 enthält echte Source-Fixes (RemoveItem(x,y) statt ID

— crashte bei jedem Tunnel-Despawn auf FILE_MapInfo[nil]; LogManager.Log

existiert nicht → AddLine; return true-Parität). Fixes verifiziert korrekt.

⚠️ Boundary: Source-Fixes gehören laut Protokoll in den Fixer-Ledger.

#202–#228 (15 Batches + Mega-Batch #228 mit 106 Dateien): Specs/Gate-Pins,

Stichproben mechanisch sauber.

Failspecs #234/#237/#238: verifiziert echte Defekte, korrekt als

queued → fixer im Ledger (Fixes s.o.).

Lane 3 (Cleanup + L10n)

PR	Scope	Verdict
#201	Source/Update (29f)	reviewed — kein Behavior-Diff; totes `TimeID=saveTime`-Feld entfernt (saveTime war nie definiert → nil → No-op)
#227	Source/Objects (97f)	mechanisch sauber
#229	Source/Libraries (304f)	mechanisch sauber
#230	Repair/Tiles/Themes/Stages (27f)	mechanisch sauber
#231	Source/GuiElements (407f)	mechanisch sauber
#232	Tests/top-level/Tools (124f)	mechanisch sauber
#235	L10n-Funnels + ~180 Keys	sicher verifiziert: `LocaleManager.Get(K)` liefert K zurück bei Missing → Passthrough; Cache-Key erst nach Übersetzung

Konventions-Check über alle Cleanup-Giants: systematischer Scan auf verwaiste

interne Call-Sites nach den lowerCamel-Umbenennungen — 4 Kandidaten, alle

False-Positives (M.ExportName ruft weiter PascalCase-API, nur local function

wurde umbenannt). Öffentliche Modul-APIs unverändert.

Daily-Diff (Coordinator)

BotLogs/DailyDiff_2026-09-27.md: 359 Commits gescannt; Manifest-Aktionen

S20/S21/S22/S81/S82 → re-run; kein needs-rewrite; keine neuen Einträge.

Kai-Handlungsempfehlungen

1. Merge-Reihenfolge: #199 → #200 → #239 (Rebase nötig) → #240 → #236; dann

#196 (Quarantäne-0) — danach Failspecs #234/#237/#238, sie dokumentieren die

gefixten Bugs.

2. #199 Restrisiko: bei verweigertem Stapeln kann ein Item verloren gehen

(Caller prüfen false nicht) — Follow-up-Ticket statt Silent-Path.

3. #198 Boundary: Lane-1-PR mit Source-Fixes — heute OK (verifiziert), aber

Pattern nicht etablieren.

4. Flaky main.lua:200 Update nil (S02/S05) beobachten — kein PR nötig, aber

ein queued-Eintrag wäre sinnvoll falls es wiederkehrt.

— Reviewer-Session, ~04:30 UTC

Erzeugt von der Reviewer-Nacht-Session · Markdown-Quelle