Skip to content

Preserve libraries and backups after failed reads - #2

Merged
lukisch merged 2 commits into
masterfrom
fix/codex-library-load-safety-20261002
Oct 1, 2026
Merged

lukisch merged 2 commits into
masterfrom
fix/codex-library-load-safety-20261002

Conversation

@lukisch

@lukisch lukisch commented Oct 1, 2026 •

Copy link
Copy Markdown
Collaborator

Wenn eine Bibliotheksdatei vorübergehend nicht lesbar ist, behandeln die bisherigen Speicherpfade sie als leer und können vorhandene Prompts oder Boards beim nächsten Speichern überschreiben. Der Fix bricht alle Änderungen und Gesamtexporte bei solchen Ladefehlern ab und erhält bestehende Daten und Sicherungen.

Prompt- und Versionslöschungen lesen beide Bibliotheksdateien vor dem ersten Schreiben. Direkte Save-APIs prüfen ihren vorhandenen Bestand; Änderungen desselben Storage-Objekts verwenden einen gemeinsamen Lock. Speicherdialoge bleiben bei Fehlern offen und ändern geteilte Modelle erst nach erfolgreichem Speichern. Beide Wege zum Entfernen einer Board-Kachel verwenden den ergänzten Storage-Pfad.

Validierung: 273 Python-Tests bestanden; drei bekannte Auslassungen für zwei nicht mitgelieferte Release-Artefakte und echtes Rendering in der offscreen-Umgebung. Alle 46 Node-Web-Tests bestanden. 80 neue Tests prüfen Lesefehler, beschädigte Strukturen, beide Löschdateien, vorhandene Exporte, Wiederherstellung, parallele Änderungen und die tatsächlichen Qt-Speichern-Buttons. Die Originalfehler wurden separat gegen master 775b296 reproduziert; Unabhängige Abnahme für db46765 abgeschlossen; alle fünf CI-Jobs erfolgreich (Python 3.10 bis 3.13 und Web).

Grenzen: Die beiden JSON-Dateien bilden keine gemeinsame Transaktion. Andere Storage-Objekte oder Prozesse sowie Rechnerausfälle sind nicht abgesichert. Diese Änderung erzeugt keine neue EXE und enthält keine Browser-, Geräte- oder Store-Abnahme. Unterstützte Legacy-Defaults bleiben erhalten. Hinweise zur Wiederherstellung: LIBRARY_SAFETY.md.

@github-actions

github-actions Bot commented Oct 1, 2026

Copy link
Copy Markdown

Welcome! Thanks for your first pull request in this repository.

A maintainer will review it soon. Please make sure:

  • Your changes are tested
  • Documentation is updated if needed
  • The PR description explains what and why

Thanks for contributing!

@lukisch
lukisch merged commit 75a8e26 into master Oct 1, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant