tests/log(feat[sync-logger-tests]): Add comprehensive tests for CLI sync logger

why: Ensure CLI sync logger is properly configured with SimpleLogFormatter
what:
- Add test_setup_logger_creates_cli_sync_logger test case
- Update logger cleanup fixture to include sync logger
- Update all setup_logger tests to clear sync logger handlers
- Verify sync logger uses SimpleLogFormatter for clean output

Author: tony

tests/test_log.py

@@ -31,6 +31,7 @@ def cleanup_loggers() -> t.Iterator[None]:
         "vcspull",
         "vcspull.cli.add",
         "vcspull.cli.add_from_fs",
+        "vcspull.cli.sync",
         "libvcs",
         "test_logger",
         "",  # Root logger
@@ -452,6 +453,7 @@ def test_setup_logger_custom_level(self, caplog: LogCaptureFixture) -> None:
             "vcspull",
             "vcspull.cli.add",
             "vcspull.cli.add_from_fs",
+            "vcspull.cli.sync",
             "libvcs",
         ]:
             logger = logging.getLogger(logger_name)
@@ -474,6 +476,7 @@ def test_setup_logger_creates_vcspull_logger(
             "vcspull",
             "vcspull.cli.add",
             "vcspull.cli.add_from_fs",
+            "vcspull.cli.sync",
             "libvcs",
         ]:
             logger = logging.getLogger(logger_name)
@@ -500,6 +503,7 @@ def test_setup_logger_creates_cli_add_logger(
             "vcspull",
             "vcspull.cli.add",
             "vcspull.cli.add_from_fs",
+            "vcspull.cli.sync",
             "libvcs",
         ]:
             logger = logging.getLogger(logger_name)
@@ -526,6 +530,7 @@ def test_setup_logger_creates_cli_add_fs_logger(
             "vcspull",
             "vcspull.cli.add",
             "vcspull.cli.add_from_fs",
+            "vcspull.cli.sync",
             "libvcs",
         ]:
             logger = logging.getLogger(logger_name)
@@ -542,6 +547,33 @@ def test_setup_logger_creates_cli_add_fs_logger(
         handler = add_fs_logger.handlers[0]
         assert isinstance(handler.formatter, SimpleLogFormatter)
 
+    def test_setup_logger_creates_cli_sync_logger(
+        self, caplog: LogCaptureFixture
+    ) -> None:
+        """Test that setup_logger creates CLI sync logger."""
+        # Clear handlers first to avoid interference
+        root_logger = logging.getLogger()
+        for logger_name in [
+            "vcspull",
+            "vcspull.cli.add",
+            "vcspull.cli.add_from_fs",
+            "vcspull.cli.sync",
+            "libvcs",
+        ]:
+            logger = logging.getLogger(logger_name)
+            logger.handlers.clear()
+        root_logger.handlers.clear()
+
+        setup_logger(level="INFO")
+
+        sync_logger = logging.getLogger("vcspull.cli.sync")
+        assert len(sync_logger.handlers) > 0
+        assert sync_logger.propagate is False
+
+        # Test that it uses SimpleLogFormatter
+        handler = sync_logger.handlers[0]
+        assert isinstance(handler.formatter, SimpleLogFormatter)
+
     def test_setup_logger_creates_libvcs_logger(
         self, caplog: LogCaptureFixture
     ) -> None:
@@ -552,6 +584,7 @@ def test_setup_logger_creates_libvcs_logger(
             "vcspull",
             "vcspull.cli.add",
             "vcspull.cli.add_from_fs",
+            "vcspull.cli.sync",
             "libvcs",
         ]:
             logger = logging.getLogger(logger_name)
@@ -595,6 +628,7 @@ def test_setup_logger_with_none_creates_root_logger(
             "vcspull",
             "vcspull.cli.add",
             "vcspull.cli.add_from_fs",
+            "vcspull.cli.sync",
             "libvcs",
         ]:
             logger = logging.getLogger(logger_name)