fix(init): skip hook question when no installer is present

If neither pre-commit nor prek is on PATH, skip the hook-type
question instead of failing init. Users who want hooks can
install a tool and retry.

Author: DSeaStar

commitizen/commands/init.py

@@ -109,13 +109,7 @@ def __call__(self) -> None:
             tag_format = self._ask_tag_format(tag)  # confirm & text
             update_changelog_on_bump = self._ask_update_changelog_on_bump()  # confirm
             major_version_zero = self._ask_major_version_zero(version)  # confirm
-            hook_types: list[str] | None = questionary.checkbox(
-                "What types of pre-commit hook you want to install? (Leave blank if you don't want to install)",
-                choices=[
-                    questionary.Choice("commit-msg", checked=False),
-                    questionary.Choice("pre-push", checked=False),
-                ],
-            ).unsafe_ask()
+            hook_types = self._ask_hook_types()
         except KeyboardInterrupt:
             raise InitFailedError("Stopped by user")
 
@@ -160,6 +154,25 @@ def __call__(self) -> None:
         out.info("\tcz bump\n")
         out.success("Configuration complete 🚀")
 
+    def _ask_hook_types(self) -> list[str] | None:
+        """Ask which pre-commit hook types to install.
+
+        Skip the question when neither ``pre-commit`` nor ``prek`` is
+        installed, so users who do not use those tools are not prompted.
+        """
+        if not project_info.available_hook_installers():
+            out.info("No pre-commit hook detected, skipping question")
+            return None
+
+        hook_types: list[str] | None = questionary.checkbox(
+            "What types of pre-commit hook you want to install? (Leave blank if you don't want to install)",
+            choices=[
+                questionary.Choice("commit-msg", checked=False),
+                questionary.Choice("pre-push", checked=False),
+            ],
+        ).unsafe_ask()
+        return hook_types
+
     def _ask_hook_installer(self) -> str:
         """Choose ``pre-commit`` or ``prek`` when installing Git hooks.
 

docs/commands/init.md

@@ -43,7 +43,7 @@ During the initialization process, you'll be prompted to configure the following
     - `pep440`: Python Package Versioning
 6. **Changelog Generation**: Configure whether to automatically generate changelog during version bumps
 7. **Alpha Versioning**: Option to keep major version at 0 for alpha/beta software
-8. **Pre-commit Hooks**: Set up Git hooks for automated commit message validation. If you choose to install hooks, Commitizen uses `pre-commit` or `prek` (whichever is on PATH). If both are installed, you are asked which one to use.
+8. **Pre-commit Hooks**: Set up Git hooks for automated commit message validation. If neither `pre-commit` nor `prek` is on PATH, the hook question is skipped. If you choose to install hooks, Commitizen uses whichever of those tools is available. If both are installed, you are asked which one to use.
 
 See [Configuration Options][configuration_options] for more details.
 

tests/commands/test_init_command.py

@@ -9,7 +9,7 @@
 
 from commitizen import cmd, commands
 from commitizen.__version__ import __version__
-from commitizen.exceptions import InitFailedError, NoAnswersError
+from commitizen.exceptions import NoAnswersError
 
 if TYPE_CHECKING:
     from pytest_mock import MockFixture
@@ -228,18 +228,34 @@ def test_cz_hook_exists_in_pre_commit_config(
 
 
 class TestNoPreCommitInstalled:
-    @pytest.mark.usefixtures("default_choice")
-    def test_pre_commit_not_installed(
-        self, mocker: MockFixture, config: BaseConfig, tmp_path, monkeypatch
+    def test_skips_hook_question_when_neither_installer_is_installed(
+        self, mocker: MockFixture, config: BaseConfig, tmp_path, monkeypatch, capsys
     ):
-        # Assume neither `pre-commit` nor `prek` is installed
+        mocker.patch(
+            "questionary.select",
+            side_effect=[
+                FakeQuestion("pyproject.toml"),
+                FakeQuestion("cz_conventional_commits"),
+                FakeQuestion("commitizen"),
+                FakeQuestion("semver"),
+            ],
+        )
+        mocker.patch("questionary.confirm", return_value=FakeQuestion(True))
+        mocker.patch("questionary.text", return_value=FakeQuestion("$version"))
+        checkbox = mocker.patch("questionary.checkbox")
         mocker.patch(
             "commitizen.project_info.available_hook_installers",
             return_value=[],
         )
         monkeypatch.chdir(tmp_path)
-        with pytest.raises(InitFailedError):
-            commands.Init(config)()
+
+        commands.Init(config)()
+
+        checkbox.assert_not_called()
+        captured = capsys.readouterr()
+        assert "No pre-commit hook detected, skipping question" in captured.out
+        assert Path("pyproject.toml").read_text(encoding="utf-8") == expected_config
+        assert not Path(pre_commit_config_filename).exists()
 
 
 def _init_hook_answers(mocker: MockFixture) -> None: