fix: Fehlertext bei Waisendatei-Loeschung enthaelt jetzt Exception-Detail

sync_manager.py:294: Fehlermeldung enthielt nur Dateipfad, nicht den
Grund des Fehlers (z.B. PermissionError bei OneDrive-Locks).
Ergaenzt: {rel_path}: {e} statt nur {rel_path}.

Neuer Regressionstest: os.remove per monkeypatch fehlschlagen lassen,
assert Exception-Text in result.errors.

AUFGABEN.txt: model_combo-Verdrahtungsluecke als bekannte Luecke dokumentiert.

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

Author: Lukas Geiger

src/modules/filemanager/sync_manager.py

@@ -291,7 +291,7 @@ def sync(self, config: SyncConfig) -> SyncResult:
                             os.remove(full_path)
                             result.files_deleted += 1
                         except Exception as e:
-                            result.errors.append(f"Löschen fehlgeschlagen: {rel_path}")
+                            result.errors.append(f"Löschen fehlgeschlagen: {rel_path}: {e}")
 
         # Abschluss
         result.duration = (datetime.now() - start_time).total_seconds()

tests/test_core.py

@@ -384,6 +384,29 @@ def test_delete_orphans_respects_excludes(self):
         self.assertTrue(result.success)
         self.assertTrue(os.path.exists(excluded_file), "Ausgeschlossene Datei wurde fälschlich gelöscht")
 
+    def test_delete_orphan_error_includes_exception_detail(self):
+        """Regression: Fehlertext beim Löschen einer Waisendatei muss die Exception enthalten.
+        Ohne Fix: f'Löschen fehlgeschlagen: {rel_path}' — Exception-Grund fehlt komplett."""
+        from unittest.mock import patch
+        from modules.filemanager import SyncConfig
+
+        # Datei im Ziel anlegen (kein Pendant in der Quelle → Waisendatei)
+        orphan_path = os.path.join(self.target_dir, "orphan.txt")
+        with open(orphan_path, "w") as f:
+            f.write("orphan")
+
+        config = SyncConfig(
+            source_path=self.source_dir,
+            target_path=self.target_dir,
+            delete_orphans=True,
+        )
+
+        with patch("os.remove", side_effect=PermissionError("Zugriff verweigert")):
+            result = self.sm.sync(config)
+
+        self.assertTrue(any("Zugriff verweigert" in e for e in result.errors),
+                        f"Fehlermeldung muss Exception-Text enthalten; errors={result.errors}")
+
 
 class TestEventBusUnsubscribeDuringEmit(unittest.TestCase):
     """EventBus: Callback darf sich während Emission selbst abmelden."""