Persist custom processor folder in settings

Remember the processor folder across sessions and use it when initializing the main window. The folder is now saved when browsing, during refresh (after resolving a valid directory), and on close. Processor refresh messaging was updated to show whether processors came from the selected folder or the built-in package. Settings store gained processor-folder get/set helpers that validate and normalize paths, with safe fallback to defaults when paths are missing or invalid.

Author: C-Achard

dlclivegui/gui/main_window.py

@@ -445,7 +445,7 @@ def _build_dlc_group(self) -> QGroupBox:
         # Processor selection
         processor_path_layout = QHBoxLayout()
         self.processor_folder_edit = QLineEdit()
-        self.processor_folder_edit.setText(default_processors_dir())
+        self.processor_folder_edit.setText(self._settings_store.get_processor_folder(default=default_processors_dir()))
         processor_path_layout.addWidget(self.processor_folder_edit)
 
         self.browse_processor_folder_button = QPushButton("Browse...")
@@ -1081,10 +1081,11 @@ def _action_browse_directory(self) -> None:
 
     def _action_browse_processor_folder(self) -> None:
         """Browse for processor folder."""
-        current_path = self.processor_folder_edit.text() or default_processors_dir()
+        current_path = self.processor_folder_edit.text().strip() or default_processors_dir()
         directory = QFileDialog.getExistingDirectory(self, "Select processor folder", current_path)
         if directory:
             self.processor_folder_edit.setText(directory)
+            self._settings_store.set_processor_folder(directory)
             self._refresh_processors()
 
     def _action_open_recording_folder(self) -> None:
@@ -1138,10 +1139,17 @@ def _refresh_processors(self) -> None:
         self.processor_combo.addItem("No Processor", None)
 
         selected_folder = self.processor_folder_edit.text().strip()
-        if Path(selected_folder).exists():
-            self._scanned_processors = scan_processor_folder(selected_folder)
+        selected_path = Path(selected_folder).expanduser() if selected_folder else None
+
+        if selected_path is not None and selected_path.is_dir():
+            resolved_folder = str(selected_path.resolve())
+            self._settings_store.set_processor_folder(resolved_folder)
+            self._scanned_processors = scan_processor_folder(resolved_folder)
+            source_text = resolved_folder
         else:
             self._scanned_processors = scan_processor_package("dlclivegui.processors")
+            source_text = "package dlclivegui.processors"
+
         self._processor_keys = list(self._scanned_processors.keys())
 
         for key in self._processor_keys:
@@ -1150,9 +1158,7 @@ def _refresh_processors(self) -> None:
             self.processor_combo.addItem(display_name, key)
 
         self.processor_combo.update_shrink_width()
-        self.statusBar().showMessage(
-            f"Found {len(self._processor_keys)} processor(s) in package dlclivegui.processors", 3000
-        )
+        self.statusBar().showMessage(f"Found {len(self._processor_keys)} processor(s) in {source_text}", 3000)
 
     # ------------------------------------------------------------------
     # Recording path preview and session name persistence
@@ -2157,6 +2163,9 @@ def closeEvent(self, event: QCloseEvent) -> None:  # pragma: no cover - GUI beha
 
         # Remember model path on exit
         self._model_path_store.save_if_valid(self.model_path_edit.text().strip())
+        # Remember processor folder on exit
+        if hasattr(self, "processor_folder_edit"):
+            self._settings_store.set_processor_folder(self.processor_folder_edit.text().strip())
 
         # Close the window
         super().closeEvent(event)

dlclivegui/utils/settings_store.py

@@ -57,6 +57,42 @@ def get_fast_encoding(self, default: bool = False) -> bool:
             return value
         return str(value).strip().lower() in {"1", "true", "yes", "on"}
 
+    def get_processor_folder(self, default: str = "") -> str:
+        """
+        Return the persisted processor folder if it still exists and is a directory.
+        Otherwise return default.
+        """
+        value = self._s.value("dlc/processor_folder", default)
+        value = str(value).strip() if value is not None else ""
+
+        if not value:
+            return default
+
+        try:
+            path = Path(value).expanduser()
+            if path.is_dir():
+                return str(path.resolve())
+        except Exception:
+            logger.debug("Persisted processor folder is invalid: %s", value, exc_info=True)
+
+        return default
+
+    def set_processor_folder(self, folder: str) -> None:
+        """
+        Persist processor folder only if it exists and is a directory.
+        Invalid folders are ignored.
+        """
+        folder = str(folder).strip() if folder is not None else ""
+        if not folder:
+            return
+
+        try:
+            path = Path(folder).expanduser()
+            if path.is_dir():
+                self._s.setValue("dlc/processor_folder", str(path.resolve()))
+        except Exception:
+            logger.debug("Failed to persist processor folder: %s", folder, exc_info=True)
+
     def set_fast_encoding(self, enabled: bool) -> None:
         self._s.setValue("recording/fast_encoding", bool(enabled))