fix: Bugsweep Lauf #10 — 4 Bugs (#10-1 bis #10-4) behoben

Bug #10-1: QSystemTrayIcon.Trigger -> ActivationReason.Trigger (PySide6 6.4+)
Bug #10-2: OSError-Guard in models._ingest_txt() write+replace
Bug #10-3: OSError-Guard in settings.save() write+replace
Bug #10-4: ASCII-Digraphe in user-facing return-Strings in ops.py

Style: Docstrings/Kommentare in contextmenu/watcher/worker/paths/models
ebenfalls auf echte Umlaute umgestellt (kein Bug per CLAUDE.md Code-Ausnahme).

4 Regressionstests ergaenzt (tests/test_bugsweep_regressions.py).
105/105 Tests gruen.

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

Author: lukisch

src/cloudlockfixer/contextmenu.py

@@ -1,5 +1,5 @@
-"""Explorer-Rechtsklick-Kontextmenue (Windows, HKCU) als Kaskaden-Untermenue
-'CloudLockFixer' fuer Ordner und Dateien. Eintraege rufen `clf gui-add`."""
+"""Explorer-Rechtsklick-Kontextmenü (Windows, HKCU) als Kaskaden-Untermenü
+'CloudLockFixer' für Ordner und Dateien. Einträge rufen `clf gui-add`."""
 from __future__ import annotations
 
 import sys

src/cloudlockfixer/models.py

@@ -27,7 +27,7 @@ class Step:
     op: StepType
     src: str
     arg: str = ""
-    copied: bool = False  # bei move/rename: Copy erfolgt, nur noch Quelle-Loeschen offen
+    copied: bool = False  # bei move/rename: Copy erfolgt, nur noch Quelle-Löschen offen
 
     def describe(self) -> str:
         if self.op == "rename":
@@ -142,7 +142,7 @@ def _ingest_txt(self) -> None:
         try:
             text = self.txt_path.read_text(encoding="utf-8")
         except (OSError, UnicodeDecodeError):
-            # Analog zu queue.json: korrupte Datei -> Ingest uebersprungen.
+            # Analog zu queue.json: korrupte Datei -> Ingest übersprungen.
             return
         lines = text.splitlines()
         changed = False
@@ -167,10 +167,13 @@ def _ingest_txt(self) -> None:
             else:
                 out.append(line)
         if changed:
-            tmp = self.txt_path.with_suffix(".txt.tmp")
-            tmp.write_text("\n".join(out) + "\n", encoding="utf-8")
-            tmp.replace(self.txt_path)
-            self._save_unlocked()
+            try:
+                tmp = self.txt_path.with_suffix(".txt.tmp")
+                tmp.write_text("\n".join(out) + "\n", encoding="utf-8")
+                tmp.replace(self.txt_path)
+                self._save_unlocked()
+            except OSError:
+                pass  # best-effort; nächster load()-Aufruf wiederholt den Versuch
 
     def add(self, task: Task) -> Task:
         with self._lock:

src/cloudlockfixer/ops.py

@@ -4,9 +4,9 @@
 am Cloud-Files-Filter (cldflt) mit Zugriff-verweigert/EXDEV, automatisch auf
 copy+delete ausweichen (der MS-empfohlene, filter-sichere Workaround).
 
-Idempotenz fuer verzoegerte Retries: Wenn der Copy-Teil gelang, aber das
-Loeschen der Quelle (noch) gesperrt ist, merkt sich der Schritt das via
-`Step.copied` — der naechste Lauf loescht dann nur noch die Quelle und kopiert
+Idempotenz für verzögerte Retries: Wenn der Copy-Teil gelang, aber das
+Löschen der Quelle (noch) gesperrt ist, merkt sich der Schritt das via
+`Step.copied` — der nächste Lauf löscht dann nur noch die Quelle und kopiert
 NICHT erneut (kein Datenverlust, kein Doppel-Copy).
 """
 from __future__ import annotations
@@ -50,9 +50,9 @@ def _force_writable(path: str | os.PathLike) -> None:
 
 def _on_rm_error(func, path, _exc):
     """rmtree-Fehlerbehandler: read-only-Attribut entfernen, dann erneut versuchen.
-    Windows verweigert das Loeschen read-only markierter Dateien/Verzeichnisse mit
+    Windows verweigert das Löschen read-only markierter Dateien/Verzeichnisse mit
     WinError 5 (z.B. .pytest_cache, .git/objects oder von OneDrive 'pinned'). Ohne
-    diesen Handler bricht shutil.rmtree an solchen Eintraegen ab."""
+    diesen Handler bricht shutil.rmtree an solchen Einträgen ab."""
     _force_writable(path)
     func(path)
 
@@ -90,7 +90,7 @@ def _verify_copy(src: Path, dst: Path) -> bool:
 def _delete_path(p: Path) -> tuple[bool, str]:
     try:
         if not p.exists():
-            return True, "bereits geloescht"
+            return True, "bereits gelöscht"
         if p.is_dir():
             try:
                 _rmtree(p)
@@ -99,7 +99,7 @@ def _delete_path(p: Path) -> tuple[bool, str]:
                     # Bug 3: Verzeichnis mit gesperrter Innendatei
                     ok, locked = _delete_dir_skip_locked(p)
                     if ok:
-                        return True, "geloescht (gesperrte Innendateien uebersprungen)"
+                        return True, "gelöscht (gesperrte Innendateien übersprungen)"
                     if locked:
                         names = ", ".join(str(lp.name) for lp in locked[:3])
                         return False, f"Innendatei(en) noch gesperrt (EBUSY): {names}"
@@ -115,9 +115,9 @@ def _delete_path(p: Path) -> tuple[bool, str]:
             except OSError:
                 _force_writable(p)  # read-only Datei -> beschreibbar machen
                 p.unlink()
-        return True, "geloescht"
+        return True, "gelöscht"
     except OSError as e:
-        return False, f"Loeschen fehlgeschlagen: {e}"
+        return False, f"Löschen fehlgeschlagen: {e}"
 
 
 def _delete_dir_skip_locked(p: Path) -> tuple[bool, list[Path]]:
@@ -165,8 +165,8 @@ def _delete_dir_skip_locked(p: Path) -> tuple[bool, list[Path]]:
 
 
 def _do_move(src: Path, dst: Path) -> tuple[bool, str, bool]:
-    """Verschiebt src -> dst. Rueckgabe: (ok, msg, copied).
-    `copied`=True heisst: dst ist vollstaendig erstellt (egal ob Quelle schon weg).
+    """Verschiebt src -> dst. Rückgabe: (ok, msg, copied).
+    `copied`=True heißt: dst ist vollständig erstellt (egal ob Quelle schon weg).
     Ziel-existiert (mit anderem Inhalt als 'schon erledigt') = Konflikt -> Fehler."""
     if not src.exists() and dst.exists():
         return True, "bereits verschoben", True
@@ -199,12 +199,12 @@ def _do_move(src: Path, dst: Path) -> tuple[bool, str, bool]:
 
     if not _verify_copy(src, dst):
         _delete_path(dst)
-        return False, "Copy unvollstaendig (Verify fehlgeschlagen)", False
+        return False, "Copy unvollständig (Verify fehlgeschlagen)", False
 
     ok, msg = _delete_path(src)
     if ok:
         return True, "verschoben (copy+delete)", True
-    return False, f"Kopiert, aber Quelle noch gesperrt — Retry spaeter ({msg})", True
+    return False, f"Kopiert, aber Quelle noch gesperrt — Retry später ({msg})", True
 
 
 # ── Standalone-API (fuer direkte Nutzung/Tests) ─────────────────────
@@ -251,7 +251,7 @@ def execute_step(step: Step) -> tuple[bool, str]:
 
 
 def execute_chain(task: Task) -> bool:
-    """Fuehrt die Kette ab task.step_index aus. Schritt N nur nach Erfolg N-1."""
+    """Führt die Kette ab task.step_index aus. Schritt N nur nach Erfolg N-1."""
     for i in range(task.step_index, len(task.chain)):
         ok, msg = execute_step(task.chain[i])
         if ok:

src/cloudlockfixer/paths.py

@@ -1,4 +1,4 @@
-"""Zentrale Pfade (Datenverzeichnis, Log, Launcher fuer Registry-Befehle)."""
+"""Zentrale Pfade (Datenverzeichnis, Log, Launcher für Registry-Befehle)."""
 from __future__ import annotations
 
 import os

src/cloudlockfixer/settings.py

@@ -28,10 +28,13 @@ def load() -> dict:
 
 def save(settings: dict) -> None:
     p = _path()
-    tmp = p.with_suffix(".json.tmp")
-    tmp.write_text(json.dumps(settings, ensure_ascii=False, indent=2),
-                   encoding="utf-8")
-    tmp.replace(p)
+    try:
+        tmp = p.with_suffix(".json.tmp")
+        tmp.write_text(json.dumps(settings, ensure_ascii=False, indent=2),
+                       encoding="utf-8")
+        tmp.replace(p)
+    except OSError:
+        pass
 
 
 def resolve_language(cfg: dict) -> str:

src/cloudlockfixer/tray.py

@@ -173,7 +173,7 @@ def _build_menu(self) -> None:
                                     triggered=self.app.quit))
 
     def _on_activated(self, reason):
-        if reason == QSystemTrayIcon.Trigger:
+        if reason == QSystemTrayIcon.ActivationReason.Trigger:
             self.menu.popup(self.tray.geometry().center())
 
     # ── Aktionen ────────────────────────────────────────────────────

src/cloudlockfixer/watcher.py

@@ -1,12 +1,12 @@
-"""Praeventiv-Waechter (P3, opt-in).
+"""Präventiv-Wächter (P3, opt-in).
 
-Beobachtet die Aenderungsrate in *konfigurierten* Ordnern (NICHT dem ganzen
-OneDrive-Baum — das wuerde Online-only-Placeholder hydratisieren = ungewollte
-Downloads). Bei hoher Aktivitaet wird der Sync-Client pausiert; nach einer
+Beobachtet die Änderungsrate in *konfigurierten* Ordnern (NICHT dem ganzen
+OneDrive-Baum — das würde Online-only-Placeholder hydratisieren = ungewollte
+Downloads). Bei hoher Aktivität wird der Sync-Client pausiert; nach einer
 Ruhephase (Cooldown) wieder gestartet — so entstehen Locks gar nicht erst.
 
 Die Entscheidungslogik (`decide`) ist rein/zustandsbehaftet und testbar; das
-Zaehlen ist bounded + stat-only (liest nur Metadaten, hydratisiert nicht).
+Zählen ist bounded + stat-only (liest nur Metadaten, hydratisiert nicht).
 """
 from __future__ import annotations
 
@@ -91,7 +91,7 @@ def tick(self) -> str:
                     log.info("Preventive watcher: %s paused (high change rate).",
                              self.provider.name)
                 else:
-                    self._paused_by_us = False  # pause() gescheitert — kein Resume noetig
+                    self._paused_by_us = False  # pause() gescheitert — kein Resume nötig
             else:
                 self._paused_by_us = False
         elif action == "resume":

src/cloudlockfixer/worker.py

@@ -1,8 +1,8 @@
 """Worker: arbeitet die Queue ab.
 
 Standard ist non-disruptiv: copy+delete funktioniert auch ohne den Sync-Client
-zu beenden. Erst wenn ein Task mehrfach haengt (oder force_pause), wird der
-zustaendige Sync-Provider fuer den Lauf pausiert (M2) und danach wieder gestartet.
+zu beenden. Erst wenn ein Task mehrfach hängt (oder force_pause), wird der
+zuständige Sync-Provider für den Lauf pausiert (M2) und danach wieder gestartet.
 """
 from __future__ import annotations
 
@@ -70,7 +70,7 @@ def run_once(queue: Queue, force_pause: bool = False) -> dict:
                 summary["done"] += 1
                 log.info("Task %s completed.", t.id)
             else:
-                t.status = "pending"  # bleibt fuer naechsten Lauf
+                t.status = "pending"  # bleibt für nächsten Lauf
                 summary["failed_again"] += 1
                 log.warning("Task %s still open: %s", t.id, t.last_error)
         queue.save()

tests/test_bugsweep_regressions.py

@@ -0,0 +1,99 @@
+"""Regressionstests: Bugsweep Lauf #10 — CloudLockFixer."""
+from __future__ import annotations
+
+import ast
+from pathlib import Path
+
+SRC = Path(__file__).resolve().parents[1] / "src" / "cloudlockfixer"
+
+
+# ---------------------------------------------------------------------------
+# Bug #10-1: QSystemTrayIcon.Trigger statt QSystemTrayIcon.ActivationReason.Trigger
+# ---------------------------------------------------------------------------
+
+def test_tray_uses_activation_reason_enum():
+    """_on_activated() muss ActivationReason.Trigger nutzen, nicht deprecated .Trigger (Bug #10-1)."""
+    src = (SRC / "tray.py").read_text(encoding="utf-8")
+    tree = ast.parse(src)
+    for node in ast.walk(tree):
+        if not (isinstance(node, ast.FunctionDef) and node.name == "_on_activated"):
+            continue
+        for sub in ast.walk(node):
+            if not isinstance(sub, ast.Compare):
+                continue
+            for comp in sub.comparators:
+                if isinstance(comp, ast.Attribute) and comp.attr == "Trigger":
+                    val = comp.value
+                    if isinstance(val, ast.Attribute) and val.attr == "ActivationReason":
+                        return  # korrekte Form gefunden
+                    raise AssertionError(
+                        "Bug #10-1: QSystemTrayIcon.Trigger verwendet — "
+                        "muss QSystemTrayIcon.ActivationReason.Trigger sein (PySide6 6.4+)"
+                    )
+    # kein Compare mit .Trigger gefunden — ebenfalls OK (kein Vergleich)
+
+
+# ---------------------------------------------------------------------------
+# Bug #10-2: _ingest_txt() kein try/except OSError bei write_text/replace
+# ---------------------------------------------------------------------------
+
+def test_ingest_txt_survives_oserror(tmp_path):
+    """_ingest_txt() darf bei OSError beim Schreiben nicht abstürzen (Bug #10-2)."""
+    import sys
+    sys.path.insert(0, str(Path(__file__).resolve().parents[1] / "src"))
+    from unittest.mock import patch
+    from cloudlockfixer.models import Queue
+
+    q = Queue(tmp_path)
+    txt = tmp_path / "queue.txt"
+    txt.write_text("delete /some/path\n", encoding="utf-8")
+
+    with patch("pathlib.Path.write_text", side_effect=OSError("locked")):
+        try:
+            q.load()
+        except OSError:
+            raise AssertionError("_ingest_txt() muss OSError intern abfangen (Bug #10-2)")
+
+
+# ---------------------------------------------------------------------------
+# Bug #10-3: settings.save() kein try/except OSError
+# ---------------------------------------------------------------------------
+
+def test_settings_save_survives_oserror(tmp_path):
+    """settings.save() darf bei OSError nicht abstürzen (Bug #10-3)."""
+    import sys
+    sys.path.insert(0, str(Path(__file__).resolve().parents[1] / "src"))
+    from unittest.mock import patch
+    import cloudlockfixer.settings as s
+
+    with patch("pathlib.Path.write_text", side_effect=OSError("disk full")):
+        try:
+            s.save({"interval_min": 60})
+        except OSError:
+            raise AssertionError("settings.save() muss OSError intern abfangen (Bug #10-3)")
+
+
+# ---------------------------------------------------------------------------
+# Bug #10-4: ops.py User-facing Strings mit ASCII-Digraphen
+# ---------------------------------------------------------------------------
+
+def test_ops_no_digraphs_in_return_strings():
+    """ops.py return-Strings dürfen keine ASCII-Digraphe enthalten (Bug #10-4)."""
+    import re
+    src = (SRC / "ops.py").read_text(encoding="utf-8")
+    tree = ast.parse(src)
+    digraph_re = re.compile(
+        r"\b(?:geloescht|uebersprungen|loeschen|unvollstaendig|spaeter|"
+        r"fuer|rueckgabe|fuehrt|verzoegert|naechste)\b",
+        re.IGNORECASE,
+    )
+    violations = []
+    for node in ast.walk(tree):
+        if isinstance(node, ast.Constant) and isinstance(node.value, str):
+            m = digraph_re.search(node.value)
+            if m:
+                violations.append(f"Z.{node.lineno}: '{m.group()}'")
+    assert not violations, (
+        "ASCII-Digraphe in ops.py Strings (Bug #10-4) — echte Umlaute verwenden:\n"
+        + "\n".join(violations)
+    )