Generalize frozen pre-commit install instruction for uv users (#762)

nathanjmcdougall · web-flow · commit e8d49e057527 · 2025-06-18T22:05:20.000+12:00
* Generalize frozen pre-commit install instruction for uv users

* Update tests to reflect new behaviour
diff --git a/src/usethis/_integrations/pre_commit/core.py b/src/usethis/_integrations/pre_commit/core.py
@@ -5,6 +5,7 @@
 from usethis._integrations.pre_commit.errors import PreCommitInstallationError
 from usethis._integrations.uv.call import call_uv_subprocess
 from usethis._integrations.uv.errors import UVSubprocessFailedError
+from usethis._integrations.uv.used import is_uv_used
 
 
 def remove_pre_commit_config() -> None:
@@ -24,7 +25,10 @@ def install_pre_commit_hooks() -> None:
     in a git repo.
     """
     if usethis_config.frozen:
-        box_print("Run 'pre-commit install' to register pre-commit.")
+        if is_uv_used():
+            box_print("Run 'uv run pre-commit install' to register pre-commit.")
+        else:
+            box_print("Run 'pre-commit install' to register pre-commit.")
         return
 
     tick_print("Ensuring pre-commit is installed to Git.")
diff --git a/tests/usethis/_core/test_core_tool.py b/tests/usethis/_core/test_core_tool.py
@@ -632,7 +632,7 @@ def test_pre_commit_after(
                 "☐ Install the dependency 'pre-commit'.\n"
                 "✔ Writing '.pre-commit-config.yaml'.\n"
                 "✔ Adding hook 'deptry' to '.pre-commit-config.yaml'.\n"
-                "☐ Run 'pre-commit install' to register pre-commit.\n"
+                "☐ Run 'uv run pre-commit install' to register pre-commit.\n"
                 "☐ Run 'uv run pre-commit run --all-files' to run the hooks manually.\n"
             )
 
@@ -1481,7 +1481,7 @@ def test_fresh(self, uv_init_dir: Path, capfd: pytest.CaptureFixture[str]):
                 "☐ Remove the placeholder hook in '.pre-commit-config.yaml'.\n"
                 "☐ Replace it with your own hooks.\n"
                 "☐ Alternatively, use 'usethis tool' to add other tools and their hooks.\n"
-                "☐ Run 'pre-commit install' to register pre-commit.\n"
+                "☐ Run 'uv run pre-commit install' to register pre-commit.\n"
                 "☐ Run 'uv run pre-commit run --all-files' to run the hooks manually.\n"
             )
             # Config file
diff --git a/tests/usethis/_integrations/pre_commit/test_pre_commit_core.py b/tests/usethis/_integrations/pre_commit/test_pre_commit_core.py
@@ -80,6 +80,26 @@ def test_message(self, uv_env_dir: Path, capfd: pytest.CaptureFixture[str]):
                 "ℹ This may take a minute or so while the hooks are downloaded.\r"  # noqa: RUF001
             )
 
+    @pytest.mark.usefixtures("_vary_network_conn")
+    def test_frozen_message_using_uv(
+        self, uv_init_repo_dir: Path, capfd: pytest.CaptureFixture[str]
+    ):
+        # Arrange
+        with change_cwd(uv_init_repo_dir), PyprojectTOMLManager():
+            add_deps_to_group([Dependency(name="pre-commit")], "dev")
+            add_placeholder_hook()
+            capfd.readouterr()
+
+            # Act
+            install_pre_commit_hooks()
+
+            # Assert
+            out, err = capfd.readouterr()
+            assert not err
+            assert out == (
+                "☐ Run 'uv run pre-commit install' to register pre-commit.\n"
+            )
+
     def test_err(self, tmp_path: Path):
         # Act, Assert
         with change_cwd(tmp_path), pytest.raises(PreCommitInstallationError):
