pre-commit-uv 4.2.1__tar.gz → 4.2.2__tar.gz

{pre_commit_uv-4.2.1 → pre_commit_uv-4.2.2}/PKG-INFO

@@ -1,10 +1,10 @@
 Metadata-Version: 2.4
 Name: pre-commit-uv
-Version: 4.2.1
+Version: 4.2.2
 Summary: Run pre-commit with uv
 Project-URL: Bug Tracker, https://github.com/tox-dev/pre-commit-uv/issues
-Project-URL: Changelog, https://github.com/tox-dev/pre-commit-uv/releases
 Project-URL: Source Code, https://github.com/tox-dev/pre-commit-uv
+Project-URL: Changelog, https://github.com/tox-dev/pre-commit-uv/releases
 Project-URL: Documentation, https://github.com/tox-dev/pre-commit-uv/
 Author-email: Bernat Gabor <gaborjbernat@gmail.com>
 License: Permission is hereby granted, free of charge, to any person obtaining a
@@ -39,11 +39,6 @@ Classifier: Programming Language :: Python :: 3.14
 Requires-Python: >=3.10
 Requires-Dist: pre-commit>=4.3
 Requires-Dist: uv>=0.9.1
-Provides-Extra: testing
-Requires-Dist: covdefaults>=2.3; extra == 'testing'
-Requires-Dist: pytest-cov>=7; extra == 'testing'
-Requires-Dist: pytest-mock>=3.15.1; extra == 'testing'
-Requires-Dist: pytest>=8.4.2; extra == 'testing'
 Description-Content-Type: text/markdown
 
 # pre-commit-uv

{pre_commit_uv-4.2.1 → pre_commit_uv-4.2.2}/pyproject.toml

@@ -2,7 +2,7 @@
 build-backend = "hatchling.build"
 requires = [
   "hatch-vcs>=0.5",
-  "hatchling>=1.27",
+  "hatchling>=1.28",
 ]
 
 [project]
@@ -36,18 +36,39 @@ dependencies = [
   "pre-commit>=4.3",
   "uv>=0.9.1",
 ]
-optional-dependencies.testing = [
+urls."Bug Tracker" = "https://github.com/tox-dev/pre-commit-uv/issues"
+urls."Source Code" = "https://github.com/tox-dev/pre-commit-uv"
+urls.Changelog = "https://github.com/tox-dev/pre-commit-uv/releases"
+urls.Documentation = "https://github.com/tox-dev/pre-commit-uv/"
+
+[dependency-groups]
+dev = [
+  { include-group = "lint" },
+  { include-group = "pkg-meta" },
+  { include-group = "test" },
+  { include-group = "type" },
+]
+test = [
   "covdefaults>=2.3",
   "pytest>=8.4.2",
   "pytest-cov>=7",
   "pytest-mock>=3.15.1",
 ]
-urls."Bug Tracker" = "https://github.com/tox-dev/pre-commit-uv/issues"
-urls."Changelog" = "https://github.com/tox-dev/pre-commit-uv/releases"
-urls."Source Code" = "https://github.com/tox-dev/pre-commit-uv"
-urls.Documentation = "https://github.com/tox-dev/pre-commit-uv/"
+type = [
+  "ty>=0.0.17",
+  { include-group = "test" },
+]
+lint = [
+  "pre-commit-uv>=4.2",
+]
+pkg-meta = [
+  "check-wheel-contents>=0.6.3",
+  "twine>=6.2",
+  "uv>=0.10.2",
+]
 
 [tool.hatch]
+version.source = "vcs"
 build.dev-mode-dirs = [
   "src",
 ]
@@ -58,7 +79,6 @@ build.targets.sdist.include = [
 ]
 build.targets.wheel.only-include = [ "src" ]
 build.targets.wheel.sources = [ "src" ]
-version.source = "vcs"
 
 [tool.ruff]
 line-length = 120
@@ -69,7 +89,6 @@ lint.select = [
   "ALL",
 ]
 lint.ignore = [
-  "ANN101", # no type annotation for self
   "ANN401", # allow Any as type annotation
   "COM812", # Conflict with formatter
   "CPY",    # No copyright statements
@@ -105,6 +124,9 @@ count = true
 [tool.pyproject-fmt]
 max_supported_python = "3.14"
 
+[tool.ty]
+environment.python-version = "3.14"
+
 [tool.pytest]
 ini_options.testpaths = [
   "tests",
@@ -127,12 +149,3 @@ paths.source = [
 report.fail_under = 63
 html.show_contexts = true
 html.skip_covered = false
-
-[tool.mypy]
-show_error_codes = true
-strict = true
-overrides = [
-  { module = [
-    "pre_commit.*",
-  ], ignore_missing_imports = true },
-]

{pre_commit_uv-4.2.1 → pre_commit_uv-4.2.2}/src/pre_commit_uv/__init__.py

@@ -4,7 +4,20 @@ from __future__ import annotations
 
 # only import built-ins at top level to avoid interpreter startup overhead
 import os
+import pathlib
 import sys
+from typing import TYPE_CHECKING, cast
+
+if TYPE_CHECKING:
+    from collections.abc import Callable, Sequence
+    from typing import Protocol
+
+    from pre_commit.prefix import Prefix
+
+    class _PatchablePython(Protocol):
+        install_environment: Callable[[Prefix, str, Sequence[str]], None]
+        health_check: Callable[[Prefix, str], str | None]
+
 
 _original_main = None
 
@@ -33,7 +46,7 @@ def _patch() -> None:
     if _is_calling_pre_commit() and os.environ.get("DISABLE_PRE_COMMIT_UV_PATCH") is None:
         from pre_commit import main  # noqa: PLC0415
 
-        _original_main, main.main = main.main, _new_main
+        _original_main, main.main = main.main, _new_main  # ty: ignore[invalid-assignment]
         if "--version" in sys.argv:
             from importlib.metadata import version as _metadata_version  # noqa: PLC0415
 
@@ -47,18 +60,12 @@ def _patch() -> None:
             )
 
 
-def _new_main(argv: list[str] | None = None) -> int:
+def _new_main(argv: Sequence[str] | None = None) -> int:
     # imports applied locally to avoid patching import overhead cost
     from functools import cache  # noqa: PLC0415
-    from typing import TYPE_CHECKING, cast  # noqa: PLC0415
 
     from pre_commit.languages import python  # noqa: PLC0415
 
-    if TYPE_CHECKING:
-        from collections.abc import Sequence  # noqa: PLC0415
-
-        from pre_commit.prefix import Prefix  # noqa: PLC0415
-
     def _install_environment(
         prefix: Prefix,
         version: str,
@@ -120,17 +127,48 @@ def _new_main(argv: list[str] | None = None) -> int:
 
         return _metadata_version("uv")
 
-    @cache
-    def _version_info(exe: str) -> str:
-        from pre_commit.util import CalledProcessError, cmd_output  # noqa: PLC0415
+    patched = cast("_PatchablePython", python)
+    patched.install_environment = _install_environment
+    patched.health_check = _health_check
+    assert _original_main is not None  # noqa: S101
+    return _original_main(argv)
 
-        prog = 'import sys;print(".".join(str(p) for p in sys.version_info[0:3]))'
-        try:
-            return cast("str", cmd_output(exe, "-S", "-c", prog)[1].strip())
-        except CalledProcessError:
-            return f"<<error retrieving version from {exe}>>"
 
-    python.install_environment = _install_environment
-    python._version_info = _version_info  # noqa: SLF001
-    assert _original_main is not None  # noqa: S101
-    return cast("int", _original_main(argv))
+def _version_info(exe: str) -> str:
+    from pre_commit.util import CalledProcessError, cmd_output  # noqa: PLC0415
+
+    prog = 'import sys;print(".".join(str(p) for p in sys.version_info[0:3]))'
+    try:
+        return cmd_output(exe, "-S", "-c", prog)[1].strip()
+    except CalledProcessError:
+        return f"<<error retrieving version from {exe}>>"
+
+
+def _health_check(prefix: Prefix, version: str) -> str | None:
+    # uv may record fewer version components in pyvenv.cfg than pre-commit expects (e.g. "3.14" vs "3.14.6"),
+    # so compare only the components uv actually wrote rather than requiring an exact string match
+    from pre_commit.lang_base import environment_dir  # noqa: PLC0415
+    from pre_commit.languages import python  # noqa: PLC0415
+    from pre_commit.util import win_exe  # noqa: PLC0415
+
+    pyvenv_cfg = pathlib.Path(environment_dir(prefix, python.ENVIRONMENT_DIR, version)) / "pyvenv.cfg"
+    if not pyvenv_cfg.exists():
+        return "pyvenv.cfg does not exist (old virtualenv?)"
+
+    cfg = python._read_pyvenv_cfg(str(pyvenv_cfg))  # noqa: SLF001
+    if "version_info" not in cfg:
+        return "created virtualenv's pyvenv.cfg is missing `version_info`"
+    expected = cfg["version_info"].split(".")
+
+    py_exe = prefix.path(python.bin_dir(str(pyvenv_cfg.parent)), win_exe("python"))
+    targets = [("virtualenv python", py_exe)]
+    if "base-executable" in cfg:
+        targets.append(("base executable", cfg["base-executable"]))
+    for label, exe in targets:
+        if (actual := _version_info(exe)).split(".")[: len(expected)] != expected:
+            return (
+                f"{label} version did not match created version:\n"
+                f"- actual version: {actual}\n"
+                f"- expected version: {cfg['version_info']}\n"
+            )
+    return None

pre_commit_uv-4.2.2/tests/test_health_check.py

@@ -0,0 +1,77 @@
+from __future__ import annotations
+
+import sys
+from typing import TYPE_CHECKING
+
+from pre_commit.languages import python
+from pre_commit.prefix import Prefix
+
+from pre_commit_uv import _health_check
+
+if TYPE_CHECKING:
+    from pathlib import Path
+
+version = "default"
+major_minor = f"{sys.version_info[0]}.{sys.version_info[1]}"
+full_version = ".".join(str(p) for p in sys.version_info[0:3])
+wrong_version = f"{sys.version_info[0]}.{sys.version_info[1] + 1}"
+
+
+def _make_env(tmp_path: Path, pyvenv_cfg: str | None) -> Prefix:
+    envdir = tmp_path / f"py_env-{version}"
+    bin_dir = python.bin_dir(str(envdir))
+    (tmp_path / bin_dir).mkdir(parents=True)
+    (tmp_path / bin_dir / python.win_exe("python")).symlink_to(sys.executable)
+    if pyvenv_cfg is not None:
+        (envdir / "pyvenv.cfg").write_text(pyvenv_cfg)
+    return Prefix(str(tmp_path))
+
+
+def test_health_check_truncated_version_passes(tmp_path: Path) -> None:
+    """uv >=0.11.22 writes only the major.minor into pyvenv.cfg (see issue #152)."""
+    prefix = _make_env(tmp_path, f"version_info = {major_minor}\nbase-executable = {sys.executable}\n")
+
+    assert _health_check(prefix, version) is None
+
+
+def test_health_check_full_version_passes(tmp_path: Path) -> None:
+    prefix = _make_env(tmp_path, f"version_info = {full_version}\nbase-executable = {sys.executable}\n")
+
+    assert _health_check(prefix, version) is None
+
+
+def test_health_check_no_base_executable_passes(tmp_path: Path) -> None:
+    prefix = _make_env(tmp_path, f"version_info = {major_minor}\n")
+
+    assert _health_check(prefix, version) is None
+
+
+def test_health_check_version_mismatch_fails(tmp_path: Path) -> None:
+    prefix = _make_env(tmp_path, f"version_info = {wrong_version}\n")
+
+    result = _health_check(prefix, version)
+
+    assert result is not None
+    assert "virtualenv python version did not match created version" in result
+    assert f"- expected version: {wrong_version}" in result
+
+
+def test_health_check_base_executable_mismatch_fails(tmp_path: Path) -> None:
+    prefix = _make_env(tmp_path, f"version_info = {major_minor}\nbase-executable = /does/not/exist/python\n")
+
+    result = _health_check(prefix, version)
+
+    assert result is not None
+    assert "base executable version did not match created version" in result
+
+
+def test_health_check_missing_pyvenv_cfg(tmp_path: Path) -> None:
+    prefix = _make_env(tmp_path, None)
+
+    assert _health_check(prefix, version) == "pyvenv.cfg does not exist (old virtualenv?)"
+
+
+def test_health_check_missing_version_info(tmp_path: Path) -> None:
+    prefix = _make_env(tmp_path, "home = /usr\n")
+
+    assert _health_check(prefix, version) == "created virtualenv's pyvenv.cfg is missing `version_info`"

pre_commit_uv-4.2.1/tox.ini

@@ -1,70 +0,0 @@
-[tox]
-requires =
-    tox>=4.31
-    tox-uv>=1.28.1
-env_list =
-    fix
-    3.14
-    3.13
-    3.12
-    3.11
-    3.10
-    3.9
-    3.8
-    type
-    pkg_meta
-skip_missing_interpreters = true
-
-[testenv]
-description = run the unit tests with pytest under {base_python}
-package = wheel
-wheel_build_env = .pkg
-extras =
-    testing
-pass_env =
-    PYTEST_*
-set_env =
-    COVERAGE_FILE = {work_dir}/.coverage.{env_name}
-commands =
-    python -m pytest {tty:--color=yes} {posargs: \
-      --cov {env_site_packages_dir}{/}pre_commit_uv --cov {tox_root}{/}tests \
-      --cov-config=pyproject.toml --no-cov-on-fail --cov-report term-missing:skip-covered --cov-context=test \
-      --cov-report html:{env_tmp_dir}{/}htmlcov --cov-report xml:{work_dir}{/}coverage.{env_name}.xml \
-      --junitxml {work_dir}{/}junit.{env_name}.xml \
-      tests}
-
-[testenv:fix]
-description = format the code base to adhere to our styles, and complain about what we cannot do automatically
-skip_install = true
-deps =
-    pre-commit-uv>=4.1.1
-commands =
-    pre-commit run --all-files --show-diff-on-failure
-
-[testenv:type]
-description = run type check on code base
-deps =
-    mypy==1.18.2
-commands =
-    mypy src
-    mypy tests
-
-[testenv:pkg_meta]
-description = check that the long description is valid
-skip_install = true
-deps =
-    check-wheel-contents>=0.6.3
-    twine>=6.2
-    uv>=0.9.1
-commands =
-    uv build --sdist --wheel --out-dir {env_tmp_dir} .
-    twine check {env_tmp_dir}{/}*
-    check-wheel-contents --no-config {env_tmp_dir}
-
-[testenv:dev]
-description = generate a DEV environment
-package = editable
-commands =
-    uv tree
-    python -c 'import sys; print(sys.executable)'
-    python task/dev_pth.py {env_site_packages_dir}