fix: harden desktop cloud provider and autostart regressions

Author: lukisch

CHANGELOG.md

@@ -6,9 +6,15 @@ Format basiert auf [Keep a Changelog](https://keepachangelog.com/de/1.1.0/).
 ## [Unreleased]
 
 ### Hinzugefügt / Added
+- **Box provider (Windows):** root auto-discovery via `~/Box` plus optional `CustomBoxLocation` registry path, process detection via `Box.exe`, and pause/resume support.
+- **Nextcloud provider (Windows):** root auto-discovery via `%APPDATA%\\Nextcloud\\nextcloud.cfg` plus default `~/Nextcloud`, process detection via `nextcloud.exe`, and pause/resume support.
 - README.md, README.de.md and `llms.txt`: added discovery/search context for
   OneDrive access-denied, `cldflt.sys`, Windows Cloud Files filter, error
   `0x8007016A`, and cloud-sync locked-folder retry workflows.
+
+### Behoben / Fixed
+- **Autostart in packaged builds:** PyInstaller/Frozen builds now register the
+  packaged executable instead of the source-tree `clf_launcher.pyw`.
 - `tests/source_platform_smoke.py`: headless Smoke-Tests für Linux und macOS — prüft Modul-Import, Version, `ops`-Operationen (rename/move/delete), `models.Queue`-Persistenz, `paths.data_dir()` und `worker.run_once()` ohne Cloud-Client oder GUI.
 - `.github/workflows/source-platform-smoke.yml`: CI-Matrix für `ubuntu-latest` und `macos-latest`, die die Smoke-Tests bei jedem Push/PR auf `main` ausführt.
 

README.de.md

@@ -38,9 +38,11 @@ des Sync-Clients.
   **Tray-Dialog** und **Explorer-Rechtsklick**-Kontextmenü
 - Auto-Retry in einstellbarem Intervall (Default 2 h) und auf Abruf
 - Optionales Pausieren/Neustarten des Sync-Clients während einer Operation
-  (OneDrive-Provider)
+  für unterstützte ordnerbasierte Provider
 - Optionaler Präventiv-Wächter, der den Sync-Client je nach Ordner-Aktivität
   pausiert/fortsetzt
+- Unterstützte Windows-Provider aktuell: OneDrive, Google Drive, Dropbox, Box,
+  iCloud und Nextcloud
 - Autostart mit Windows; Single-Instance-Tray-App
 
 ## Installation
@@ -111,12 +113,12 @@ stehen, aber temporär durch einen Cloud-Sync-Provider blockiert werden.
 - **P3 (fertig):** Präventiv-Wächter (Änderungsrate *konfigurierter* Ordner
   beobachten → Sync-Client automatisch pausieren/fortsetzen; bounded + stat-only,
   hydratisiert keine Online-only-Placeholder; opt-in).
-- **Tests:** `pytest`, **92 grün** (Core + P2/P3 + i18n + Multicloud + Leerordner-Eigen-Handle-Lock-Regressionen).
+- **Tests:** `pytest`, **110 grün** (Core + P2/P3 + i18n + Multicloud + Box + Nextcloud + Leerordner-Eigen-Handle-Lock-Regressionen).
 - **Im Lifetest gehärtet (2026-05-29):** `is_running()/pause()` robust gegen
   nicht-UTF-8-`tasklist`-Ausgabe; `delete` entfernt read-only-Attribute statt an
   WinError 5 zu scheitern. Erster echter Einsatz: ein Ordner-Rename, den manuelle
   Versuche/`cldflt` zuvor blockierten, gelang per copy+delete.
-- **Offen/künftig:** weitere Provider-Adapter (Dropbox/Google Drive/iCloud);
+- **Offen/künftig:** weitere Provider-Adapter (pCloud/Synology Drive);
   optional Relaunch-Unterdrückung des Sync-Clients während langer Operationen.
 
 Windows-only (`cldflt` ist Windows-spezifisch); der Kern ist plattformneutral

README.md

@@ -38,9 +38,12 @@ sync client.
 - Multiple input paths: **CLI** (for LLMs/scripts), human-readable
   **`queue.txt`**, **tray dialog**, and an **Explorer right-click** context menu
 - Auto-retry on a configurable interval (default 2 h) and on demand
-- Optional sync-client pause/restart during an operation (OneDrive provider)
+- Optional sync-client pause/restart during an operation for supported
+  folder-based providers
 - Optional preventive watcher that pauses/resumes the sync client based on
   folder activity
+- Supported Windows providers today: OneDrive, Google Drive, Dropbox, Box,
+  iCloud, and Nextcloud
 - Autostart with Windows; single-instance tray app
 
 ## Installation
@@ -112,8 +115,8 @@ cloud-sync provider.
 - **P3 (done):** Preventive watcher (observes the change rate of *configured*
   folders → pauses/resumes the sync client; bounded, stat-only, does not
   hydrate online-only placeholders; opt-in).
-- **Tests:** `pytest`, 92 passing (core + P2/P3 + i18n + multicloud + empty-dir own-handle-lock regressions).
-- **Open/future:** more provider adapters (Dropbox / Google Drive / iCloud);
+- **Tests:** `pytest`, 110 passing (core + P2/P3 + i18n + multicloud + Box + Nextcloud + empty-dir own-handle-lock regressions).
+- **Open/future:** more provider adapters (pCloud / Synology Drive);
   optional suppression of sync-client relaunch during long operations.
 
 Windows-only; the core is platform-neutral for later ports.

ROADMAP.md

@@ -39,7 +39,10 @@
 
 ## Langfristig
 
-- [ ] Weitere Cloud-Provider (Box, Nextcloud, pCloud, Synology Drive)
+- [~] Weitere Cloud-Provider (Box, Nextcloud, pCloud, Synology Drive)
+  - Box erledigt 2026-06-17
+  - Nextcloud erledigt 2026-06-16
+  - Offen: pCloud, Synology Drive
 - [ ] Konfigurierbares Retry-Verhalten (Exponential Backoff, max Retries)
 - [ ] Benachrichtigungen (System-Toast bei Dauerfehler)
 - [ ] Web-Dashboard / Remote-Status

TODO.md

@@ -55,4 +55,7 @@ Beim Port der copy+delete-Logik in den ellmos-filecommander-mcp-Server (TypeScri
 - [ ] Test-CI beobachten und bei Bedarf Windows-spezifische Runtime-Abhängigkeiten ergänzen
 - [ ] Cross-Platform: Linux-Support (siehe PORTIERUNGSPLAN.md)
 - [ ] Cross-Platform: macOS-Support (siehe PORTIERUNGSPLAN.md)
-- [ ] Weitere Provider: Box, Nextcloud, pCloud, Synology Drive
+- [~] Weitere Provider: Box, Nextcloud, pCloud, Synology Drive
+      Box erledigt 2026-06-17 (`~/Box` plus `CustomBoxLocation`-Registry-Pfad, `Box.exe`-Prozesssteuerung).
+      Nextcloud erledigt 2026-06-16 (`nextcloud.cfg`-Root-Erkennung + Prozesssteuerung).
+      Offen bleiben pCloud und Synology Drive.

llms.txt

@@ -2,7 +2,7 @@
 
 CloudLockFixer is a Windows tray and CLI tool for delayed rename, move and delete
 operations in cloud-sync folders. It is designed for OneDrive, Dropbox, Google
-Drive and iCloud folders where Windows Cloud Files locks can make normal file
+Drive, Box, iCloud and Nextcloud folders where Windows Cloud Files locks can make normal file
 operations fail.
 
 Version: 0.2.2

src/cloudlockfixer/autostart.py

@@ -9,6 +9,8 @@
 
 def _launch_command() -> str:
     """Befehl, der die Tray-App startet (pythonw, kein Konsolenfenster)."""
+    if getattr(sys, "frozen", False):
+        return f'"{sys.executable}"'
     from .paths import launcher, pythonw
     return f'"{pythonw()}" "{launcher()}"'
 

src/cloudlockfixer/providers.py

@@ -111,6 +111,30 @@ def _get_volume_label(drive_letter: str) -> str:
         return ""
 
 
+def _read_box_custom_location() -> Path | None:
+    """Read Box Drive's optional custom root parent from the Windows registry."""
+    if sys.platform != "win32":
+        return None
+    try:
+        import winreg
+    except ImportError:
+        return None
+
+    try:
+        with winreg.OpenKey(winreg.HKEY_LOCAL_MACHINE, r"SOFTWARE\Box\Box") as key:
+            value, _ = winreg.QueryValueEx(key, "CustomBoxLocation")
+    except (OSError, ValueError):
+        return None
+
+    if not isinstance(value, str):
+        return None
+    normalized = value.strip().strip('"')
+    if not normalized:
+        return None
+    base = Path(normalized)
+    return base if base.name.lower() == "box" else base / "Box"
+
+
 # ── OneDrive ───────────────────────────────────────────────────────
 
 
@@ -245,6 +269,99 @@ def resume(self) -> bool:
         return False
 
 
+# ── Box ────────────────────────────────────────────────────────────
+
+
+class BoxProvider(SyncProvider):
+    name = "Box"
+    mount_type = "folder"
+
+    def _detect_roots(self) -> list[Path]:
+        roots: list[Path] = []
+        default_root = Path.home() / "Box"
+        if default_root.exists():
+            roots.append(default_root)
+
+        custom_root = _read_box_custom_location()
+        if custom_root and custom_root.exists():
+            roots.append(custom_root)
+        return _dedup_paths(roots)
+
+    def is_running(self) -> bool:
+        return _check_process("Box.exe")
+
+    def pause(self) -> bool:
+        return _kill_process("Box.exe")
+
+    def resume(self) -> bool:
+        if sys.platform != "win32":
+            return False
+        candidates = [
+            Path(r"C:\Program Files\Box\Box\Box.exe"),
+            Path(r"C:\Program Files (x86)\Box\Box\Box.exe"),
+        ]
+        for exe in candidates:
+            if exe.exists():
+                try:
+                    subprocess.Popen([str(exe)])
+                    return True
+                except OSError:
+                    continue
+        return False
+
+
+# ── Nextcloud ───────────────────────────────────────────────────────
+
+
+class NextcloudProvider(SyncProvider):
+    name = "Nextcloud"
+    mount_type = "folder"
+
+    def _detect_roots(self) -> list[Path]:
+        roots: list[Path] = []
+        default_root = Path.home() / "Nextcloud"
+        if default_root.exists():
+            roots.append(default_root)
+
+        cfg_path = Path(os.environ.get("APPDATA", "")) / "Nextcloud" / "nextcloud.cfg"
+        if cfg_path.exists():
+            try:
+                for raw_line in cfg_path.read_text(encoding="utf-8", errors="ignore").splitlines():
+                    line = raw_line.strip()
+                    if not line or line.startswith(("#", ";")) or "localPath=" not in line:
+                        continue
+                    _, raw_path = line.split("localPath=", 1)
+                    normalized = raw_path.strip().strip('"').replace("/", os.sep)
+                    if normalized:
+                        roots.append(Path(normalized))
+            except OSError:
+                pass
+        return _dedup_paths(roots)
+
+    def is_running(self) -> bool:
+        return _check_process("nextcloud.exe")
+
+    def pause(self) -> bool:
+        return _kill_process("nextcloud.exe")
+
+    def resume(self) -> bool:
+        if sys.platform != "win32":
+            return False
+        candidates = [
+            Path(r"C:\Program Files\Nextcloud\nextcloud.exe"),
+            Path(r"C:\Program Files (x86)\Nextcloud\nextcloud.exe"),
+            Path(os.environ.get("LOCALAPPDATA", "")) / "Programs" / "Nextcloud" / "nextcloud.exe",
+        ]
+        for exe in candidates:
+            if exe.exists():
+                try:
+                    subprocess.Popen([str(exe)])
+                    return True
+                except OSError:
+                    continue
+        return False
+
+
 # ── iCloud ─────────────────────────────────────────────────────────
 
 
@@ -292,7 +409,7 @@ def resume(self) -> bool:
 
 def _discover_providers() -> list[SyncProvider]:
     candidates = [OneDriveProvider(), GoogleDriveProvider(),
-                  DropboxProvider(), ICloudProvider()]
+                  DropboxProvider(), BoxProvider(), NextcloudProvider(), ICloudProvider()]
     active: list[SyncProvider] = []
     for prov in candidates:
         try:

src/cloudlockfixer/tray.py

@@ -44,11 +44,11 @@ def _make_icon(color: str = "#2e9e44") -> QIcon:
             if not ic.isNull():
                 return ic
     pm = QPixmap(64, 64)
-    pm.fill(Qt.transparent)
+    pm.fill(Qt.GlobalColor.transparent)
     p = QPainter(pm)
-    p.setRenderHint(QPainter.Antialiasing)
+    p.setRenderHint(QPainter.RenderHint.Antialiasing)
     p.setBrush(QColor(color))
-    p.setPen(Qt.NoPen)
+    p.setPen(Qt.PenStyle.NoPen)
     p.drawEllipse(8, 26, 28, 26)
     p.drawEllipse(26, 18, 30, 28)
     p.drawRoundedRect(10, 38, 46, 16, 8, 8)
@@ -57,7 +57,7 @@ def _make_icon(color: str = "#2e9e44") -> QIcon:
     f.setBold(True)
     f.setPointSize(13)
     p.setFont(f)
-    p.drawText(pm.rect(), Qt.AlignCenter, "↻")
+    p.drawText(pm.rect(), Qt.AlignmentFlag.AlignCenter, "↻")
     p.end()
     return QIcon(pm)
 

tests/test_bug_regressions.py

@@ -0,0 +1,45 @@
+"""Regressionstests — bugfix-library-transfer 2026-06-21."""
+import unittest
+from pathlib import Path
+
+SRC = Path(__file__).resolve().parents[1] / "src" / "cloudlockfixer"
+TRAY = SRC / "tray.py"
+
+
+class TestD2NoDeprecatedEnums(unittest.TestCase):
+    """BUG-D2: Deprecated Qt/QPainter-Enums in PySide6 6.4+."""
+
+    def _src(self):
+        return TRAY.read_text(encoding="utf-8")
+
+    def test_no_qt_transparent_bare(self):
+        self.assertNotIn(
+            "Qt.transparent",
+            self._src(),
+            "Qt.transparent (deprecated bare) in tray.py — BUG-D2",
+        )
+
+    def test_no_qpainter_antialiasing_bare(self):
+        self.assertNotIn(
+            "QPainter.Antialiasing",
+            self._src(),
+            "QPainter.Antialiasing (deprecated bare) in tray.py — BUG-D2",
+        )
+
+    def test_no_qt_nopen_bare(self):
+        self.assertNotIn(
+            "Qt.NoPen",
+            self._src(),
+            "Qt.NoPen (deprecated bare) in tray.py — BUG-D2",
+        )
+
+    def test_no_qt_aligncenter_bare(self):
+        self.assertNotIn(
+            "Qt.AlignCenter",
+            self._src(),
+            "Qt.AlignCenter (deprecated bare) in tray.py — BUG-D2",
+        )
+
+
+if __name__ == "__main__":
+    unittest.main()