fix(types) Eliminate runtime dependency on ConfigDict

Author: tony

src/vcspull/cli/_workspaces.py

@@ -4,9 +4,12 @@
 
 import fnmatch
 import pathlib
+import typing as t
 
 from vcspull.config import canonicalize_workspace_path, workspace_root_label
-from vcspull.types import ConfigDict
+
+if t.TYPE_CHECKING:
+    from vcspull.types import ConfigDict
 
 
 def _normalize_workspace_label(

src/vcspull/cli/sync.py

@@ -24,7 +24,6 @@
 
 from vcspull import exc
 from vcspull.config import filter_repos, find_config_files, load_configs
-from vcspull.types import ConfigDict
 from vcspull.util import contract_user_home
 
 from ._colors import Colors, get_color_mode
@@ -48,6 +47,8 @@
     from libvcs._internal.types import VCSLiteral
     from libvcs.sync.git import GitSync
 
+    from vcspull.types import ConfigDict
+
 log = logging.getLogger(__name__)
 
 ProgressCallback = Callable[[str, datetime], None]

src/vcspull/config.py

@@ -287,13 +287,10 @@ def load_configs(
     return repos
 
 
-ConfigDictTuple = tuple["ConfigDict", "ConfigDict"]
-
-
 def detect_duplicate_repos(
     config1: list[ConfigDict],
     config2: list[ConfigDict],
-) -> list[ConfigDictTuple]:
+) -> list[tuple[ConfigDict, ConfigDict]]:
     """Return duplicate repos dict if repo_dir same and vcs different.
 
     Parameters
@@ -304,13 +301,13 @@ def detect_duplicate_repos(
 
     Returns
     -------
-    list[ConfigDictTuple]
+    list[tuple[ConfigDict, ConfigDict]]
         List of duplicate tuples
     """
     if not config1:
         return []
 
-    dupes: list[ConfigDictTuple] = []
+    dupes: list[tuple[ConfigDict, ConfigDict]] = []
 
     repo_dirs = {
         pathlib.Path(repo["path"]).parent / repo["name"]: repo for repo in config1

tests/cli/test_status.py

@@ -16,6 +16,7 @@
     check_repo_status,
     status_repos,
 )
+from vcspull.types import ConfigDict
 
 if t.TYPE_CHECKING:
     from _pytest.monkeypatch import MonkeyPatch
@@ -561,8 +562,6 @@ async def test_check_repos_status_async_basic(
     tmp_path: pathlib.Path,
 ) -> None:
     """Test basic async concurrent status checking."""
-    from vcspull.types import ConfigDict
-
     # Create test repos
     repo1_path = tmp_path / "repo1"
     repo2_path = tmp_path / "repo2"
@@ -573,7 +572,7 @@ async def test_check_repos_status_async_basic(
     # repo3 intentionally not created (missing)
 
     repos = t.cast(
-        list[ConfigDict],
+        list["ConfigDict"],
         [
             {"name": "repo1", "path": str(repo1_path)},
             {"name": "repo2", "path": str(repo2_path)},
@@ -601,8 +600,6 @@ async def test_check_repos_status_async_with_detailed(
     tmp_path: pathlib.Path,
 ) -> None:
     """Test async status checking with detailed mode."""
-    from vcspull.types import ConfigDict
-
     repo_path, _remote_path = setup_repo_with_remote(tmp_path)
 
     repos = t.cast(
@@ -630,8 +627,6 @@ async def test_check_repos_status_async_concurrency_limit(
     monkeypatch: pytest.MonkeyPatch,
 ) -> None:
     """Test that semaphore limits concurrent operations."""
-    from vcspull.types import ConfigDict
-
     # Create multiple repos
     repos_list = []
     for i in range(10):