fix: harden codebox tab save flows

Author: Lukas Geiger

.gitattributes

@@ -0,0 +1,6 @@
+* text=auto
+*.bat text eol=crlf
+*.md text eol=lf
+*.py text eol=lf
+*.qss text eol=lf
+*.json text eol=lf

.gitignore

@@ -11,6 +11,7 @@ htmlcov/
 dist/
 build/
 *.spec
+!CodeBox.spec
 
 # Virtual Environments
 venv/

CHANGELOG.md

@@ -26,10 +26,19 @@ Format basiert auf [Keep a Changelog](https://keepachangelog.com/de/1.1.0/).
   auf `PATH` korrekt erkannt.
 - LSP-Subprocess-Pipes werden beim Stoppen geschlossen; der Runtime-Test läuft
   dadurch ohne ResourceWarnings.
+- `close_tab()` bricht jetzt ab, wenn das Speichern eines modifizierten Tabs fehlschlägt,
+  statt den Tab trotzdem zu schließen.
+- `run_current()` startet kein Programm mehr, wenn das automatische Speichern vor dem
+  Ausführen fehlschlägt.
+- Tab-Reordering hält die interne Index-Map jetzt synchron; `current_tab()`,
+  `close_tab()` und die offenen-Datei-Prüfungen bleiben nach Drag-and-drop korrekt.
 
 ### Geändert
 - Deutschsprachige Doku sowie Python-Kommentare, Docstrings und naheliegende UI-Texte
   verwenden jetzt echte Umlaute statt `ae/oe/ue`
+- Windows-Build nutzt jetzt die vorhandene PyInstaller-Spec mit lokalem
+  Arbeitsverzeichnis außerhalb von OneDrive; `start.bat` startet bevorzugt
+  `dist\CodeBox.exe` und fällt erst danach auf Release-EXE oder Python zurück.
 - README präzisiert die lokale Privacy-Abgrenzung; `.gitignore` schützt
   zusätzliche Credential-, SSH- und SQLite-Artefakte.
 - `.gitignore` deckt interne Diagnose-/Skill-Dateien, Test-Caches und lokale

CodeBox.spec

@@ -0,0 +1,42 @@
+# -*- mode: python ; coding: utf-8 -*-
+
+
+a = Analysis(
+    ['main.py'],
+    pathex=[],
+    binaries=[],
+    datas=[
+        ('CodeBox.ico', '.'),
+        ('themes/*.qss', 'themes'),
+    ],
+    hiddenimports=[],
+    hookspath=[],
+    hooksconfig={},
+    runtime_hooks=[],
+    excludes=[],
+    noarchive=False,
+    optimize=0,
+)
+pyz = PYZ(a.pure)
+
+exe = EXE(
+    pyz,
+    a.scripts,
+    a.binaries,
+    a.datas,
+    [],
+    name='CodeBox',
+    debug=False,
+    bootloader_ignore_signals=False,
+    strip=False,
+    upx=False,
+    upx_exclude=[],
+    runtime_tmpdir=None,
+    console=False,
+    disable_windowed_traceback=False,
+    argv_emulation=False,
+    target_arch=None,
+    codesign_identity=None,
+    entitlements_file=None,
+    icon=['CodeBox.ico'],
+)

README.md

@@ -16,6 +16,7 @@ Terminal sowie erste LSP- und API-Grundlagen in einer lokalen IDE.
 - Integriertes Terminal mit Shell-Auswahl und History
 - Projekt-Dateibaum mit Filter und Kontextmenü
 - Mehrere Tabs, Suchfunktion und Gehe-zu-Zeile
+- Robuste Tab-Verwaltung mit Drag-and-drop-Reordering und Save-Failure-Guards
 - Theme-System über `features/theme_manager.py`
 - REST-API-/CLI-Grundlage für spätere Fernsteuerung
 - LSP-Diagnostics und Completion-Anbindung für installierte Language Server
@@ -65,8 +66,13 @@ build_exe.bat
 ```
 
 Das Script nutzt PyInstaller und erstellt lokal eine `CodeBox.exe` mit
-`CodeBox.ico`. Build-Ausgaben in `build/`, `dist/` und `releases/` bleiben
-lokale Artefakte und werden nicht versioniert.
+`CodeBox.ico`. Die versionierte `CodeBox.spec` bündelt Icon und Theme-Dateien,
+während temporäre Build-Daten unter `C:\_Local_DEV\codex_build\codebox` liegen.
+Build-Ausgaben in `build/`, `dist/` und `releases/` bleiben lokale Artefakte
+und werden nicht versioniert.
+
+`start.bat` startet bevorzugt `dist\CodeBox.exe`, nutzt danach eine vorhandene
+Release-EXE und fällt erst zuletzt auf `python main.py` zurück.
 
 ## Projektstruktur
 
@@ -90,6 +96,8 @@ Bereits stabil nutzbar:
 - Projektbaum und Terminal im MainWindow
 - Konsistente Fenstertitel über `version.py`
 - Light-/Dark-Theme-Wechsel über den zentralen Theme-Manager
+- Speichern-, Schließen- und Ausführen-Flows behalten Tabs offen, wenn ein
+  Dateisystemfehler das Speichern verhindert.
 
 Noch offen für die nächste größere Ausbaustufe:
 

build_exe.bat

@@ -1,11 +1,16 @@
 @echo off
+chcp 65001 >nul
 cd /d "%~dp0"
 python --version >nul 2>&1
 if errorlevel 1 (
     echo [FEHLER] Python nicht gefunden!
     pause
     exit /b 1
 )
+set "BUILD_ROOT=C:\_Local_DEV\codex_build\codebox"
+set "WORK_DIR=%BUILD_ROOT%\work"
+set "DIST_DIR=%CD%\dist"
+if not exist "%BUILD_ROOT%" mkdir "%BUILD_ROOT%"
 echo Baue CodeBox.exe...
-python -m PyInstaller --noconfirm --clean --windowed --onefile --name CodeBox --icon CodeBox.ico main.py
+python -m PyInstaller --noconfirm --clean --workpath "%WORK_DIR%" --distpath "%DIST_DIR%" CodeBox.spec
 if errorlevel 1 pause

core/tabs.py

@@ -78,6 +78,18 @@ def __init__(self, parent=None):
 
         self.tabCloseRequested.connect(self.close_tab)
         self.currentChanged.connect(self._on_tab_changed)
+        self.tabBar().tabMoved.connect(self._on_tab_moved)
+
+    def _rebuild_tab_map(self):
+        """Baut die Index-Map anhand der aktuellen Tab-Reihenfolge neu auf."""
+        old_tabs = list(self.tabs.values())
+        self.tabs = {}
+        for i in range(self.count()):
+            widget = self.widget(i)
+            for tab in old_tabs:
+                if tab.editor is widget:
+                    self.tabs[i] = tab
+                    break
 
     def open_file(self, file_path: Path) -> EditorTab:
         """Öffnet eine Datei in einem neuen Tab (oder wechselt zu existierendem)"""
@@ -116,20 +128,17 @@ def close_tab(self, index: int):
                 QMessageBox.StandardButton.Save | QMessageBox.StandardButton.Discard | QMessageBox.StandardButton.Cancel
             )
             if reply == QMessageBox.StandardButton.Save:
-                tab.save()
+                if not tab.save():
+                    return
             elif reply == QMessageBox.StandardButton.Cancel:
                 return
 
         self.removeTab(index)
-        # Tab-Dict neu aufbauen
-        old_tabs = self.tabs.copy()
-        self.tabs = {}
-        for i in range(self.count()):
-            widget = self.widget(i)
-            for old_idx, old_tab in old_tabs.items():
-                if old_tab.editor is widget:
-                    self.tabs[i] = old_tab
-                    break
+        self._rebuild_tab_map()
+
+    def _on_tab_moved(self, _from: int, _to: int):
+        """Hält die Tab-Map nach Drag-and-drop im Sync."""
+        self._rebuild_tab_map()
 
     def current_tab(self) -> EditorTab:
         """Gibt den aktuellen EditorTab zurück"""

start.bat

@@ -1,5 +1,21 @@
 @echo off
 chcp 65001 >nul
 cd /d "%~dp0"
+set "DIST_EXE=dist\CodeBox.exe"
+set "RELEASE_EXE=releases\v0.1.0\CodeBox-0.1.0-win64.exe"
+if exist "%DIST_EXE%" (
+    start "" "%DIST_EXE%"
+    exit /b 0
+)
+if exist "%RELEASE_EXE%" (
+    start "" "%RELEASE_EXE%"
+    exit /b 0
+)
+python --version >nul 2>&1
+if errorlevel 1 (
+    echo [FEHLER] Python nicht gefunden und keine EXE vorhanden.
+    pause
+    exit /b 1
+)
 python main.py
 pause

tests/test_save_failure_guards.py

@@ -0,0 +1,111 @@
+from __future__ import annotations
+
+from unittest.mock import Mock, patch
+
+from PySide6.QtWidgets import QApplication, QMessageBox
+
+from core.tabs import TabWidget
+from ui.main_window import MainWindow
+
+
+def _ensure_app():
+    return QApplication.instance() or QApplication([])
+
+
+def test_close_tab_keeps_modified_tab_open_when_save_fails():
+    _ensure_app()
+    widget = TabWidget()
+    tab = widget.new_tab()
+    tab.is_modified = True
+
+    with (
+        patch.object(QMessageBox, "question", return_value=QMessageBox.StandardButton.Save),
+        patch.object(tab, "save", return_value=False),
+    ):
+        widget.close_tab(0)
+
+    assert widget.count() == 1
+    assert widget.tabs[0] is tab
+    assert widget.current_tab() is tab
+
+    widget.close()
+
+
+def test_run_current_does_not_execute_when_save_fails(tmp_path):
+    _ensure_app()
+
+    with patch("features.terminal.TerminalWidget._start_shell", lambda self: None):
+        window = MainWindow()
+
+    tab = window.tab_widget.current_tab()
+    assert tab is not None
+
+    tab.file_path = tmp_path / "script.py"
+    tab.provider = type(
+        "DummyProvider",
+        (),
+        {"get_run_command": staticmethod(lambda file_path: ["python", file_path])},
+    )()
+    window.output.run_command = Mock()
+
+    with patch.object(tab, "save", return_value=False):
+        window.run_current()
+
+    window.output.run_command.assert_not_called()
+
+    window.close()
+
+
+def test_initial_save_updates_tab_title_and_enables_run(tmp_path):
+    _ensure_app()
+
+    with patch("features.terminal.TerminalWidget._start_shell", lambda self: None):
+        window = MainWindow()
+
+    tab = window.tab_widget.current_tab()
+    assert tab is not None
+
+    target = tmp_path / "script.py"
+    tab.editor.setPlainText("print('hi')")
+
+    with patch(
+        "ui.main_window.QFileDialog.getSaveFileName",
+        return_value=(str(target), "Python (*.py)"),
+    ):
+        window.save_file()
+
+    current_index = window.tab_widget.currentIndex()
+    assert tab.file_path == target
+    assert window.tab_widget.tabText(current_index) == "script.py"
+    assert window.output.run_btn.isEnabled()
+
+    window.close()
+
+
+def test_initial_save_failure_restores_untitled_state():
+    _ensure_app()
+
+    with patch("features.terminal.TerminalWidget._start_shell", lambda self: None):
+        window = MainWindow()
+
+    tab = window.tab_widget.current_tab()
+    assert tab is not None
+    original_index = window.tab_widget.currentIndex()
+    original_label = window.lang_label.text()
+
+    with (
+        patch(
+            "ui.main_window.QFileDialog.getSaveFileName",
+            return_value=("C:/tmp/never-written.py", "Python (*.py)"),
+        ),
+        patch("pathlib.Path.write_text", side_effect=OSError("disk full")),
+        patch("PySide6.QtWidgets.QMessageBox.critical", return_value=None),
+    ):
+        window.save_file()
+
+    assert tab.file_path is None
+    assert window.tab_widget.tabText(original_index) == "Unbenannt"
+    assert window.lang_label.text() == original_label
+    assert not window.output.run_btn.isEnabled()
+
+    window.close()

tests/test_tab_reordering.py

@@ -0,0 +1,29 @@
+from pathlib import Path
+
+from PySide6.QtWidgets import QApplication
+
+from core.tabs import TabWidget
+
+
+def _ensure_app():
+    return QApplication.instance() or QApplication([])
+
+
+def test_current_tab_tracks_tab_reordering():
+    _ensure_app()
+    widget = TabWidget()
+
+    first = widget.new_tab()
+    first.file_path = Path("first.py")
+    second = widget.new_tab()
+    second.file_path = Path("second.py")
+
+    widget.tabBar().moveTab(0, 1)
+
+    current_widget = widget.widget(widget.currentIndex())
+    expected = next(tab for tab in widget.tabs.values() if tab.editor is current_widget)
+
+    assert widget.current_tab() is expected
+    assert widget.tabs[widget.currentIndex()] is expected
+
+    widget.close()