Maintenance Notes

Inkonsistenzen, Altlasten und sichere Erweiterungspunkte

Diese Seite trennt bestätigte Codeabweichungen von bewusster Kompatibilität. Sie ist als Checkliste für kommende Patches gedacht.

Bestätigte Abweichungen

Darkweb-Level ist nicht einheitlichAPP_ACCESS.APP_DEFINITIONS.darkweb.level ist 12. DARKWEB.requireAccess() verlangt Level 20. Entscheidung nötig: UI-Gate auf 20 anheben oder Service-Regel auf 12 senken; anschließend Texte und Tests gemeinsam anpassen.

Mehrere Speicher-APIs existieren parallelDer Produktionspfad nutzt IDB, trotzdem enthalten PLANT, LAMP, WEED_INVENTORY, CRYPTO_CURRENCY_API und USER_API direkte LocalStorage-Zugriffe. Das ist überwiegend Kompatibilität, erhöht aber die Gefahr neuer Zwei-Quellen-Fehler.

Dokumentationsstand und Code müssen getrennt bewertet werdenREADME.md, Patchpläne und Audits enthalten Historie. Für aktive Regeln sind Kataloge, Services und Validatoren maßgeblich.

Wahrscheinliche Legacy- oder Zukunftsmodule

Datei	Befund	Empfehlung
`ENGINE.js`	leere Klasse, keine aktive Importkante	entfernen oder klare künftige Rolle dokumentieren
`ECONOMY_API.js`	kleiner Altadapter, nicht im aktiven Modulgraph	gegen ECONOMY_SERVICE prüfen und ausmustern
`ITEM_API.js`	separate ältere Item-API, aktiver Pfad nutzt ITEM_CATALOG + INVENTORY_SERVICE	Importe verbieten oder konsolidieren
`PLANT_VISUAL_STATE.js`	reine alternative Bildzustandslogik ohne aktive Importkante	mit PLANT_IMAGES zusammenführen oder als Testmodul markieren
`SAVE_TRANSFER_TEST.js`	Entwickler-/Testhilfe, kein Seiteneinstieg	in separates tests-Verzeichnis verschieben
`slotshop.html`	nur Guards, kein eigenes produktives Modul im Snapshot	Route entfernen oder vollständigen Einstieg herstellen
`MINING_STORAGE.js`	LocalStorage-Kompatibilitätsanzeige neben IDB-Mining	nur lesend halten; nicht für neue Buchungen verwenden
`CHEMISTRY_TABLE.js`	ältere Fallback-API neben CHEMISTRY_TRANSACTIONS	Browserpfad konsequent auf IDB-Service halten

Kopplungspunkte mit hohem Änderungsrisiko

Billing ↔ Produktion

Änderungen an Perioden, Visibility oder Strom müssen zugleich Lampenzähler, Pflanzenwachstum, Mining und activeClockAt berücksichtigen.

Polizei ↔ Warenbewegung

Neue Verkaufs-, Produktions- oder Inventaraktionen müssen unter offenem Polizeifall von assertPoliceFreeze() erfasst werden.

Verkauf ↔ Reservierungen

Jeder neue Verkaufskanal muss freeSaleStock, reserveSale, pruneSales und assertSales integrieren.

State ↔ Mirror

Neue State-Felder, die alte UI liest, benötigen eine explizite Mirror-Abbildung. Mirrorfelder dürfen umgekehrt nicht zur Commit-Basis werden.

Migration ↔ Backup

Jede State-Versionsänderung braucht vor dem Upgrade ein unverändertes Backup, eine neue Upgrade-Funktion und eine strenge Validierung.

Access ↔ Service

UI-Gates sind Komfort, keine Fachgarantie. Die Service-Schicht muss dieselbe Regel nochmals prüfen – aber mit denselben Konstanten.

Sicheres Muster für neue Mechaniken

Katalog/Config: unveränderliche IDs, Preise, Level und Limits definieren.
Pure Rules: Migration, Validierung und zustandslokale Mutation ohne DOM entwickeln.
State-Version: bei struktureller Änderung erhöhen, Backup und Upgrade implementieren.
Service: Use-Case mit stabiler Transaktions-ID über TRANSACTION_MANAGER.commit().
Globale Invarianten: Validierung in validate() oder einem aufgerufenen Regelvalidator ergänzen.
Mirror: nur falls Legacy-UI das Feld benötigt.
UI: erst nach prepareEconomy() aufbauen; Servicefehler sichtbar behandeln.
Tests: Erfolg, Duplicate, Reload, zweiter Tab, Reset/Epoch, Migration und Grenzwerte prüfen.

Empfohlene Regressionstests

Bereich	Minimaler Test
Idempotenz	denselben eventId zweimal senden; nur ein Bestands-/Geldeffekt
Multi-Tab	zwei Seiten, konkurrierende Käufe/Verkäufe; keine verlorene Aktualisierung
Reset	alter Tab nach neuer Epoch darf nicht committen
Billing	aktive 15 Minuten, Hidden/Resume, Page Cache und Periodengrenze
Rechnungen	Teilzahlung, mehrere offene Belege, Hinweis schließen, Sperre/Entsperrung
Pflanzen	jede Lampe nur für ihre Pflanze; Stromsperre pausiert; Lager voll blockiert Ernte
Mining	Coinwechsel erhält Carry/Balance; keine Offline-Produktion; Claim nur einmal
Chemie	Queue, Cancel-Erstattung, volles Lager, Polizei-Freeze, Upgrade-Dauer
Verkauf	Tür- und Darkweb-Hold dürfen dieselbe Ware nicht überbuchen
Access	jedes Menü/App-Gate gegen Fachservice-Regel vergleichen

Priorisierte Wartungsreihenfolge

P1Darkweb-Level vereinheitlichensichtbarer Funktionsbruch

P2Legacy-Grenzen markierenneue Fehlimporte verhindern

P3Teststruktur trennenProduktgraph übersichtlicher

P4Regelkonstanten teilenAccess und Service synchron