cli/sync(refactor[typing]): Fix update_repo return type

why: Return type was GitSync but function returns GitSync | HgSync | SvnSync.
what:
- Add HgSync and SvnSync imports from libvcs
- Change return type to union GitSync | HgSync | SvnSync
- Remove incorrect t.cast("GitSync", ...) wrappers
- Remove resolved TODO comment
- Add casts in tests that access GitSync-specific methods

Author: tony

src/vcspull/cli/sync.py

@@ -23,6 +23,8 @@
 from libvcs._internal.shortcuts import create_project
 from libvcs._internal.types import VCSLiteral
 from libvcs.sync.git import GitSync
+from libvcs.sync.hg import HgSync
+from libvcs.sync.svn import SvnSync
 from libvcs.url import registry as url_tools
 
 from vcspull import exc
@@ -874,7 +876,7 @@ def update_repo(
     repo_dict: ConfigDict,
     progress_callback: ProgressCallback | None = None,
     # repo_dict: Dict[str, Union[str, Dict[str, GitRemote], pathlib.Path]]
-) -> GitSync:
+) -> GitSync | HgSync | SvnSync:
     """Synchronize a single repository."""
     repo_payload = t.cast("dict[str, object]", deepcopy(repo_dict))
     if "pip_url" not in repo_payload:
@@ -896,14 +898,14 @@ def update_repo(
 
     assert repo_vcs is not None
 
+    r: GitSync | HgSync | SvnSync
     if repo_vcs == "git":
         r = create_project(**t.cast("GitRepoPayload", repo_payload))
     elif repo_vcs == "svn":
-        r = t.cast("GitSync", create_project(**t.cast("SvnRepoPayload", repo_payload)))
+        r = create_project(**t.cast("SvnRepoPayload", repo_payload))
     else:
-        r = t.cast("GitSync", create_project(**t.cast("HgRepoPayload", repo_payload)))
+        r = create_project(**t.cast("HgRepoPayload", repo_payload))
 
     r.update_repo(set_remotes=True)  # Creates repo if not exists and fetches
 
-    # TODO: Fix this
     return r

tests/test_sync.py

@@ -166,7 +166,7 @@ def test_config_variations(
     assert len(repos) == 1
 
     for repo_dict in repos:
-        repo: GitSync = update_repo(repo_dict)
+        repo = t.cast("GitSync", update_repo(repo_dict))
         remotes = repo.remotes() or {}
         remote_names = set(remotes.keys())
         assert set(remote_list).issubset(remote_names) or {"origin"}.issubset(
@@ -281,7 +281,7 @@ def test_updating_remote(
     for repo_dict in filter_repos(
         [initial_config],
     ):
-        local_git_remotes = update_repo(repo_dict).remotes()
+        local_git_remotes = t.cast("GitSync", update_repo(repo_dict)).remotes()
         assert "origin" in local_git_remotes
 
     expected_remote_url = f"git+file://{mirror_repo}"
@@ -296,7 +296,7 @@ def test_updating_remote(
 
     repo_dict = filter_repos([expected_config], name="myclone")[0]
     assert isinstance(repo_dict, dict)
-    repo = update_repo(repo_dict)
+    repo = t.cast("GitSync", update_repo(repo_dict))
     for remote_name in repo.remotes():
         remote = repo.remote(remote_name)
         if remote is not None: