test: cover backup restore and destructive browser workflows

docs/shared-ui-adoption.md

@@ -1,54 +0,0 @@
-# Shared ComfyAssets UI adoption
-
-Status: proposed; no runtime behavior changes in this document.
-
-Tracking: [PromptManager #186](https://github.com/ComfyAssets/ComfyUI_PromptManager/issues/186)
-and [KikoTools #81](https://github.com/ComfyAssets/ComfyUI-KikoTools/issues/81).
-
-The [KikoTools foundation proposal and mockup PR](https://github.com/ComfyAssets/ComfyUI-KikoTools/pull/82)
-defines the common architecture and acceptance criteria. KikoTools' Local Image
-Loader is the first pilot. PromptManager adoption follows validation of that
-pilot and extraction of a versioned, original ComfyAssets UI core.
-
-## PromptManager requirements
-
-- Preserve existing node names, inputs/outputs, API routes, saved workflows,
-  gallery behavior, and existing test/coverage gates.
-- Use shared control factories, editor slots, settings/help/window behavior and
-  the ComfyUI adapter where applicable. Keep PromptManager controllers and HTTP
-  behavior in this repository.
-- Both projects must look identical: one shared set of controls, typography,
-  spacing, colors and interaction states. Follow ComfyUI light/dark mode with
-  our own styling, a blue primary accent and strong red errors/destructive actions.
-  Use compact buttons, pill switches, segmented selections and responsive rows.
-  Review the pilot mockup before production styling. Existing `--pm-*` tokens
-  are migration context, not an approved design to retain.
-- Theme standalone pages explicitly: `window.open` pages do not inherit the
-  host's CSS variables. Test the agreed palette policy, live changes, fallbacks
-  and persistence, including errors when host settings are unavailable.
-- Ship a pinned, reproducible browser distribution. Installation must work
-  without npm or KikoTools, including when both packs ship different core versions.
-- Attribute [Pixaroma](https://github.com/pixaroma/ComfyUI-Pixaroma) as architectural
-  inspiration. Do not copy/vendor his code, styles, or assets.
-
-## First migration
-
-After the pilot, inventory the gallery dialogs and choose one bounded dialog as
-the second consumer. Record its current behavior with tests before changing it.
-Verify keyboard opening/closing, focus restoration, accessible labels, confirm
-and cancel behavior, network failures, and light/dark appearances. Keep the
-existing gallery and destructive-action browser workflows passing.
-
-Build shared-source extraction and distribution checks before importing the core.
-Test both packs alone and together, scoped styling, isolated settings/storage,
-and independent cleanup. ComfyUI-facing changes require the real-host Classic,
-Nodes 2.0 and live-switching matrix from the foundation proposal. Standalone
-dialogs additionally run through PromptManager's existing browser harness.
-
-## Delivery
-
-Use small issue-linked PRs and TDD: verify each new behavioral test fails before
-implementing it, then run relevant suites and required CI checks. Include visual
-evidence for appearance changes. Migrate additional dialogs/editors only after
-the first consumer validates the package boundaries; do not rewrite all pages
-or change backend schemas as part of this initiative.

tests/test_api_admin.py

@@ -1867,7 +1867,7 @@ class TestScanReadsVideos(AdminAPITestCase):
         (self.output_dir / "bad.png").write_bytes(b"\x89PNG not really")
         GalleryConfig.MONITORING_DIRECTORIES = [self._root(self.output_dir)]
 
-        with self.assertLogs("prompt_manager", level="WARNING") as cm:
+        with self.assertLogs(level="WARNING") as cm:
             body = await (
                 await self.client.request("POST", "/prompt_manager/scan")
             ).text()

tests/test_utils_logging_config.py

@@ -263,35 +263,5 @@ class TestConfiguration(IsolatedLoggerTestCase):
         self.manager.update_config({"console_logging": False})
 
 
-class TestConsoleNoise(IsolatedLoggerTestCase):
-    """ComfyUI's console only sees warnings unless the level is DEBUG."""
-
-    def _console_handler(self) -> logging.StreamHandler:
-        return next(
-            h
-            for h in logging.getLogger("prompt_manager").handlers
-            if type(h) is logging.StreamHandler
-        )
-
-    def test_records_do_not_propagate_to_comfyui_root_logger(self) -> None:
-        self.assertFalse(logging.getLogger("prompt_manager").propagate)
-
-    def test_console_hides_info_at_the_default_level(self) -> None:
-        self.manager.update_config({"console_logging": True})
-        self.assertEqual(self._console_handler().level, logging.WARNING)
-        self.logger.info("startup chatter")
-        self.assertEqual(self.manager.get_recent_logs(limit=1)[0]["level"], "INFO")
-        log_file = pathlib.Path(self.log_dir) / "prompt_manager.log"
-        self.assertIn("startup chatter", log_file.read_text())
-        self.manager.update_config({"console_logging": False})
-
-    def test_debug_level_opens_the_console(self) -> None:
-        self.manager.update_config({"console_logging": True, "level": "DEBUG"})
-        self.assertEqual(self._console_handler().level, logging.DEBUG)
-        self.manager.update_config({"level": "INFO"})
-        self.assertEqual(self._console_handler().level, logging.WARNING)
-        self.manager.update_config({"console_logging": False})
-
-
 if __name__ == "__main__":
     unittest.main()

utils/logging_config.py

@@ -85,8 +85,6 @@ class PromptManagerLogger:
         # Main logger
         self.logger = logging.getLogger("prompt_manager")
         self.logger.setLevel(getattr(logging, self.config["level"]))
-        # ComfyUI's root handler would print every record a second time.
-        self.logger.propagate = False
 
         # Clear existing handlers
         for handler in self.logger.handlers[:]:
@@ -102,7 +100,6 @@ class PromptManagerLogger:
         # Console handler
         if self.config["console_logging"]:
             console_handler = logging.StreamHandler()
-            console_handler.setLevel(self._console_level())
             console_handler.setFormatter(formatter)
             self.logger.addHandler(console_handler)
 
@@ -127,15 +124,6 @@ class PromptManagerLogger:
         # Component-specific loggers
         self._setup_component_loggers()
 
-    def _console_level(self) -> int:
-        """Console threshold: everything at DEBUG, otherwise warnings only.
-
-        INFO still reaches the log file and the web log viewer.
-        """
-        if str(self.config["level"]).upper() == "DEBUG":
-            return logging.DEBUG
-        return logging.WARNING
-
     def _setup_component_loggers(self):
         """Set up loggers for specific components.
 
@@ -375,10 +363,6 @@ class PromptManagerLogger:
                     logger = logging.getLogger(logger_name)
                     logger.setLevel(level)
 
-            for handler in self.logger.handlers:
-                if type(handler) is logging.StreamHandler:
-                    handler.setLevel(self._console_level())
-
         # Re-setup loggers if handlers changed
         if any(
             key in new_config