util(cleanup): Remove deprecated contract_user_home

Author: tony

src/vcspull/util.py

@@ -7,8 +7,6 @@
 import typing as t
 from collections.abc import Mapping
 
-from vcspull._internal.private_path import PrivatePath
-
 LEGACY_CONFIG_DIR = pathlib.Path("~/.vcspull/").expanduser()  # remove dupes of this
 
 
@@ -44,47 +42,6 @@ def get_config_dir() -> pathlib.Path:
     return path
 
 
-def contract_user_home(path: str | pathlib.Path) -> str:
-    """Contract user home directory to ~ for display purposes.
-
-    Parameters
-    ----------
-    path : str | pathlib.Path
-        Path to contract
-
-    Returns
-    -------
-    str
-        Path with $HOME contracted to ~
-
-    Examples
-    --------
-    >>> contract_user_home("/home/user/code/repo")
-    '~/code/repo'
-    >>> contract_user_home("/opt/project")
-    '/opt/project'
-    """
-    path_str = str(path)
-    if path_str == "":
-        return path_str
-
-    if path_str.startswith("~"):
-        return path_str
-
-    home_str = str(pathlib.Path.home())
-    if path_str.startswith(home_str):
-        collapsed = str(PrivatePath(pathlib.Path(path_str)))
-        if (
-            path_str.endswith(os.sep)
-            and not collapsed.endswith(os.sep)
-            and path_str.rstrip(os.sep) != home_str
-        ):
-            collapsed += os.sep
-        return collapsed
-
-    return path_str
-
-
 T = t.TypeVar("T", bound=dict[str, t.Any])
 
 

tests/_internal/test_private_path.py

@@ -29,3 +29,14 @@ def test_absolute_paths_outside_home_are_unmodified() -> None:
 def test_existing_tilde_prefix_is_preserved() -> None:
     path = PrivatePath("~/.config/vcspull")
     assert str(path) == "~/.config/vcspull"
+
+
+def test_trailing_slash_collapses_to_directory_label() -> None:
+    home = Path.home()
+    path = PrivatePath(home / "projects/")
+    assert str(path) == f"~{os.sep}projects"
+
+
+def test_relative_path_unmodified() -> None:
+    path = PrivatePath("relative/path")
+    assert str(path) == "relative/path"

tests/test_utils.py

@@ -5,7 +5,7 @@
 import pathlib
 import typing as t
 
-from vcspull.util import contract_user_home, get_config_dir
+from vcspull.util import get_config_dir
 
 if t.TYPE_CHECKING:
     import pytest
@@ -37,42 +37,3 @@ def test_vcspull_configdir_no_xdg(monkeypatch: pytest.MonkeyPatch) -> None:
     """Test retrieving config directory without XDG_CONFIG_HOME set."""
     monkeypatch.delenv("XDG_CONFIG_HOME")
     assert get_config_dir()
-
-
-def test_contract_user_home_contracts_home_path() -> None:
-    """Test contracting home directory to ~."""
-    home = str(pathlib.Path.home())
-
-    # Test full path
-    assert contract_user_home(f"{home}/code/repo") == "~/code/repo"
-
-    # Test home directory itself
-    assert contract_user_home(home) == "~"
-
-    # Test with pathlib.Path
-    assert contract_user_home(pathlib.Path(home) / "code" / "repo") == "~/code/repo"
-
-
-def test_contract_user_home_preserves_non_home_paths() -> None:
-    """Test that non-home paths are not contracted."""
-    # Test absolute paths outside home
-    assert contract_user_home("/opt/project") == "/opt/project"
-    assert contract_user_home("/usr/local/bin") == "/usr/local/bin"
-
-    # Test relative paths
-    assert contract_user_home("./relative/path") == "./relative/path"
-    assert contract_user_home("relative/path") == "relative/path"
-
-
-def test_contract_user_home_handles_edge_cases() -> None:
-    """Test edge cases in path contraction."""
-    home = str(pathlib.Path.home())
-
-    # Test trailing slashes
-    assert contract_user_home(f"{home}/code/") == "~/code/"
-
-    # Test empty path
-    assert contract_user_home("") == ""
-
-    # Test path with ~ already in it (should pass through)
-    assert contract_user_home("~/code/repo") == "~/code/repo"