fix(settings): _apply_settings() uebertraegt ai.model und max_tokens (B-001); roter Rahmen in NewProjectDialog wird bei textChanged zurueckgesetzt (B-002)

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: Lukas Geiger

BUGS.md

@@ -7,21 +7,21 @@ Format: `[Status] Titel — Kurzbeschreibung`
 
 ## Offen
 
+_(Keine offenen Bugs)_
+
+---
+
+## Behoben
+
 ### B-001: Modell-Auswahl nicht funktional
-**Status:** Offen  
+**Status:** Behoben (2026-06-05)  
 **Datei:** `src/gui/main_window.py` — `_apply_settings()`  
-**Schwere:** Mittel (Feature silent broken)  
-**Beschreibung:**  
-`_apply_settings()` liest `ai.model` aus den Einstellungen, ruft aber `ai_service.set_model()` nie auf. Der gespeicherte Wert ist zudem der Anzeigename (z. B. "Claude Haiku") statt der API-ID. Die Modell-Auswahl in den Einstellungen wird daher vollständig ignoriert.  
-**Workaround:** Keiner. Das KI-Modell bleibt beim Standardwert der `AIService`-Instanz.
+**Fix:** `_apply_settings()` liest nun `ai.model` und ruft `set_model()` mit dem korrekten `AIModel`-Enum-Wert auf. Auch `max_tokens` wird übertragen.
 
 ### B-002: Rotes Validierungs-Rahmen in NewProjectDialog wird nicht zurückgesetzt
-**Status:** Offen  
-**Datei:** `src/gui/dialogs/new_project_dialog.py` — `_validate_input()` / `name_edit`  
-**Schwere:** Gering (kosmetischer UX-Defekt)  
-**Beschreibung:**  
-Wenn der Benutzer im `NewProjectDialog` einen ungültigen Projektnamen eingibt und auf "Erstellen" klickt, setzt `_validate_input()` einen roten Rahmen auf `name_edit`. Sobald der Benutzer dann wieder zu tippen beginnt, wird dieser Rahmen nicht automatisch zurückgesetzt. Der rote Rahmen bleibt, bis das Formular neu validiert wird.  
-**Workaround:** Nutzer muss das Formular erneut absenden, um den Rahmen zu entfernen.
+**Status:** Behoben (2026-06-05)  
+**Datei:** `src/gui/dialogs/new_project_dialog.py` — `_reset_name_style()`  
+**Fix:** `name_edit.textChanged` ist nun mit `_reset_name_style()` verbunden, das `setStyleSheet("")` aufruft und so den roten Rahmen beim nächsten Tastendruck löscht.
 
 ---
 

src/gui/dialogs/new_project_dialog.py

@@ -43,6 +43,7 @@ def _setup_ui(self):
         self.name_edit = QLineEdit()
         self.name_edit.setPlaceholderText("Mein Python Projekt")
         self.name_edit.textChanged.connect(self._update_path)
+        self.name_edit.textChanged.connect(self._reset_name_style)
         details_layout.addRow("Name:", self.name_edit)
 
         # Speicherort
@@ -184,6 +185,10 @@ def _apply_styles(self):
             }
         """)
 
+    def _reset_name_style(self):
+        """Setzt den roten Validierungsrahmen zurück wenn der Nutzer zu tippen beginnt."""
+        self.name_edit.setStyleSheet("")
+
     def _browse_path(self):
         """Öffnet den Verzeichnis-Dialog"""
         path = QFileDialog.getExistingDirectory(

src/gui/main_window.py

@@ -1073,7 +1073,21 @@ def _apply_settings(self):
         """Wendet geänderte Einstellungen an"""
         # API-Key aktualisieren
         self.ai_service.set_api_key(self.settings.get('ai.api_key', ''))
-        
+
+        # Modell aktualisieren (Anzeigename → AIModel-Enum)
+        from modules.ai_assistant.ai_service import AIModel
+        _MODEL_MAP = {
+            'Claude Sonnet': AIModel.CLAUDE_SONNET,
+            'Claude Opus': AIModel.CLAUDE_OPUS,
+            'Claude Haiku': AIModel.CLAUDE_HAIKU,
+        }
+        model_name = self.settings.get('ai.model', 'Claude Sonnet')
+        if model_name in _MODEL_MAP:
+            self.ai_service.set_model(_MODEL_MAP[model_name])
+
+        # Max Tokens aktualisieren
+        self.ai_service.max_tokens = self.settings.get('ai.max_tokens', 4096)
+
         # Editor-Einstellungen auf alle offenen Editoren anwenden
         for index in range(self.editor_tabs.count()):
             widget = self.editor_tabs.widget(index)

tests/test_core.py

@@ -1254,6 +1254,50 @@ def test_new_lanczos_api_used(self):
                       "WindowsStorePublisher_3.py muss Image.Resampling.LANCZOS verwenden")
 
 
+class TestApplySettingsUpdatesModel(unittest.TestCase):
+    """Bug B-001: _apply_settings() muss set_model() aufrufen."""
+
+    def test_apply_settings_calls_set_model(self):
+        """_apply_settings() muss ai.model lesen und set_model() aufrufen — ohne Fix wird
+        die Modell-Auswahl in den Einstellungen vollständig ignoriert."""
+        import inspect
+        from gui.main_window import MainWindow
+        source = inspect.getsource(MainWindow._apply_settings)
+        self.assertIn('set_model', source,
+                      "_apply_settings() muss set_model() aufrufen")
+        self.assertIn('ai.model', source,
+                      "_apply_settings() muss ai.model aus den Einstellungen lesen")
+
+    def test_apply_settings_updates_max_tokens(self):
+        """_apply_settings() muss auch ai.max_tokens auf den Service übertragen."""
+        import inspect
+        from gui.main_window import MainWindow
+        source = inspect.getsource(MainWindow._apply_settings)
+        self.assertIn('max_tokens', source,
+                      "_apply_settings() muss ai.max_tokens auf ai_service übertragen")
+
+
+class TestNewProjectDialogBorderReset(unittest.TestCase):
+    """Bug B-002: Roter Validierungsrahmen muss bei textChanged zurückgesetzt werden."""
+
+    def test_name_edit_textchanged_resets_style(self):
+        """name_edit.textChanged muss mit _reset_name_style verbunden sein — ohne Fix
+        bleibt der rote Rahmen auch nach Korrektur des Namens sichtbar."""
+        import inspect
+        from gui.dialogs.new_project_dialog import NewProjectDialog
+        source = inspect.getsource(NewProjectDialog._setup_ui)
+        self.assertIn('_reset_name_style', source,
+                      "name_edit.textChanged muss mit _reset_name_style verbunden sein")
+
+    def test_reset_name_style_clears_stylesheet(self):
+        """_reset_name_style() muss das Stylesheet löschen (leerer String)."""
+        import inspect
+        from gui.dialogs.new_project_dialog import NewProjectDialog
+        source = inspect.getsource(NewProjectDialog._reset_name_style)
+        self.assertIn('setStyleSheet("")', source,
+                      "_reset_name_style() muss setStyleSheet('') aufrufen")
+
+
 if __name__ == "__main__":
     # Verbose Output
     unittest.main(verbosity=2)