claude-dev-env 1.42.0 → 1.43.0

package/hooks/blocking/question_to_user_enforcer.py

@@ -15,17 +15,11 @@ import re
 import sys
 from pathlib import Path
 
+_hooks_dir = str(Path(__file__).resolve().parent.parent)
+if _hooks_dir not in sys.path:
+    sys.path.insert(0, _hooks_dir)
 
-def _insert_hooks_tree_for_imports() -> None:
-    hooks_tree = Path(__file__).resolve().parent.parent
-    hooks_tree_string = str(hooks_tree)
-    if hooks_tree_string not in sys.path:
-        sys.path.insert(0, hooks_tree_string)
-
-
-_insert_hooks_tree_for_imports()
-
-from config.messages import USER_FACING_ASKUSERQUESTION_NOTICE
+from hooks_constants.messages import USER_FACING_ASKUSERQUESTION_NOTICE  # noqa: E402
 
 
 def strip_code_and_quotes(text: str) -> str:

package/hooks/blocking/state_description_blocker.py

@@ -6,23 +6,17 @@ Enforces the "describe current state only" rule — no "instead of", "previously
 describe what IS, not what WAS or what CHANGED.
 """
 
-import io
 import json
 import os
 import sys
 from pathlib import Path
+from typing import TextIO
 
+_hooks_dir = str(Path(__file__).resolve().parent.parent)
+if _hooks_dir not in sys.path:
+    sys.path.insert(0, _hooks_dir)
 
-def _insert_hooks_tree_for_imports() -> None:
-    hooks_tree = Path(__file__).absolute().parent.parent
-    hooks_tree_string = str(hooks_tree)
-    if hooks_tree_string not in sys.path:
-        sys.path.insert(0, hooks_tree_string)
-
-
-_insert_hooks_tree_for_imports()
-
-from config.state_description_blocker_constants import (
+from hooks_constants.state_description_blocker_constants import (  # noqa: E402
     ALL_BLOCK_COMMENT_EXTENSIONS,
     ALL_BLOCK_COMMENT_ONLY_EXTENSIONS,
     ALL_COMMENT_BEARING_EXTENSIONS,
@@ -233,7 +227,7 @@ def main() -> None:
     sys.exit(0)
 
 
-def _emit_hook_result(all_hook_data: dict, output_stream: io.TextIOBase) -> None:
+def _emit_hook_result(all_hook_data: dict, output_stream: TextIO) -> None:
     """Write the hook result JSON to the given output stream."""
     output_stream.write(json.dumps(all_hook_data) + "\n")
     output_stream.flush()

package/hooks/blocking/tdd_enforcer.py

@@ -18,7 +18,7 @@ _hooks_root_path_string = str(Path(__file__).resolve().parent.parent)
 if _hooks_root_path_string not in sys.path:
     sys.path.insert(0, _hooks_root_path_string)
 
-from config.messages import USER_FACING_TDD_NOTICE
+from hooks_constants.messages import USER_FACING_TDD_NOTICE
 
 PRODUCTION_EXTENSIONS = {'.py', '.ts', '.tsx', '.js', '.jsx'}
 SKIP_PATTERNS = {

package/hooks/blocking/test_bot_mention_comment_blocker.py

@@ -22,7 +22,7 @@ hook_spec.loader.exec_module(hook_module)
 _detect_bot_mention = hook_module._detect_bot_mention
 _body_contains_token = hook_module._body_contains_token
 
-from config.bot_mention_comment_blocker_constants import (
+from hooks_constants.bot_mention_comment_blocker_constants import (
     CORRECTIVE_MESSAGE_COPILOT,
     CORRECTIVE_MESSAGE_CURSOR,
     CURSOR_MENTION_TOKEN,

package/hooks/blocking/test_code_rules_enforcer.py

@@ -40,18 +40,18 @@ if str(_HOOKS_TREE_DIR) not in sys.path:
     sys.path.insert(0, str(_HOOKS_TREE_DIR))
 
 from code_rules_path_utils import is_config_file as path_utils_is_config_file  # noqa: E402
-from config.banned_identifiers_constants import (  # noqa: E402
+from hooks_constants.banned_identifiers_constants import (  # noqa: E402
     ALL_BANNED_IDENTIFIERS as config_all_banned_identifiers,
     BANNED_IDENTIFIER_MESSAGE_SUFFIX as config_banned_identifier_message_suffix,
     BANNED_IDENTIFIER_SKIP_ADVISORY as config_banned_identifier_skip_advisory,
     MAX_BANNED_IDENTIFIER_ISSUES as config_max_banned_identifier_issues,
 )
-from config.hardcoded_user_path_constants import (  # noqa: E402
+from hooks_constants.hardcoded_user_path_constants import (  # noqa: E402
     HARDCODED_USER_PATH_GUIDANCE as config_hardcoded_user_path_guidance,
     HARDCODED_USER_PATH_PATTERN as config_hardcoded_user_path_pattern,
     MAX_HARDCODED_USER_PATH_ISSUES as config_max_hardcoded_user_path_issues,
 )
-from config.stuttering_check_config import (  # noqa: E402
+from hooks_constants.stuttering_check_config import (  # noqa: E402
     MAX_STUTTERING_PREFIX_ISSUES as config_max_stuttering_prefix_issues,
     STUTTERING_ALL_PREFIX_PATTERN as config_stuttering_all_prefix_pattern,
 )

package/hooks/blocking/test_code_rules_enforcer_any_exempt_files.py

@@ -56,7 +56,7 @@ def test_should_still_flag_in_regular_module() -> None:
 
 
 def test_any_type_config_module_exists_and_exposes_constant() -> None:
-    config_module_path = Path(__file__).parent.parent / "config" / "any_type_config.py"
+    config_module_path = Path(__file__).parent.parent / "hooks_constants" / "any_type_config.py"
     assert config_module_path.is_file(), f"Missing: {config_module_path}"
     spec = importlib.util.spec_from_file_location("any_type_config_under_test", config_module_path)
     assert spec is not None

package/hooks/blocking/test_code_rules_enforcer_cap_meta.py

@@ -48,8 +48,6 @@ KNOWN_UNCAPPED_CHECKS_PENDING_REVIEW: frozenset[str] = frozenset(
         "check_boolean_naming",
         "check_collection_prefix",
         "check_comment_changes",
-        "check_comments_javascript",
-        "check_comments_python",
         "check_constant_equality_tests",
         "check_constants_outside_config",
         "check_constants_outside_config_advisory",

package/hooks/blocking/test_code_rules_enforcer_comment_string_awareness.py

@@ -0,0 +1,184 @@
+"""Tests pinning string-aware ``#`` detection in the Python comment checks.
+
+``#`` characters appear inside string literals in three common shapes:
+hex color codes (``"#FFFFFF"``), URL fragments
+(``"https://x#section"``), and f-string interpolation patterns. None of
+those ``#`` characters belong to a comment token. ``check_comments_python``
+and the Python branch of ``extract_comment_texts`` route their ``#``
+detection through ``tokenize.generate_tokens`` so only true ``COMMENT``
+tokens are considered. These tests pin both halves of that contract:
+``#``-in-strings is exempt; real inline comments that land AFTER such
+a string still flag.
+"""
+
+from __future__ import annotations
+
+import importlib.util
+from pathlib import Path
+from types import ModuleType
+
+
+def _load_enforcer_module() -> ModuleType:
+    module_path = Path(__file__).parent / "code_rules_enforcer.py"
+    spec = importlib.util.spec_from_file_location("code_rules_enforcer", module_path)
+    assert spec is not None
+    assert spec.loader is not None
+    module = importlib.util.module_from_spec(spec)
+    spec.loader.exec_module(module)
+    return module
+
+
+code_rules_enforcer = _load_enforcer_module()
+
+
+def test_python_check_should_not_flag_hex_color_literal() -> None:
+    content = 'palette_primary = "#FFFFFF"\n'
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert issues == []
+
+
+def test_python_check_should_not_flag_url_fragment_in_string() -> None:
+    content = 'docs_link = "https://example.com/guide#installation"\n'
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert issues == []
+
+
+def test_python_check_should_not_flag_hash_inside_fstring_interpolation() -> None:
+    content = 'rendered = f"prefix #{count} suffix"\n'
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert issues == []
+
+
+def test_python_check_should_not_flag_hash_inside_triple_quoted_string() -> None:
+    content = 'message = """use # for inline comments"""\n'
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert issues == []
+
+
+def test_python_check_should_flag_real_inline_comment_after_string_with_hash() -> None:
+    content = 'name = "user"  # this comment must still flag\n'
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert len(issues) == 1
+    assert "Comment found" in issues[0]
+
+
+def test_python_check_should_flag_real_comment_after_hex_color_literal() -> None:
+    content = 'palette_primary = "#FFFFFF"  # accidentally added comment\n'
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert len(issues) == 1
+    assert "Comment found" in issues[0]
+
+
+def test_extract_should_not_classify_hex_color_as_inline_comment() -> None:
+    content = 'palette_primary = "#FFFFFF"\n'
+    inline, standalone = code_rules_enforcer.extract_comment_texts(content, "foo.py")
+    assert inline == set()
+    assert standalone == set()
+
+
+def test_extract_should_classify_real_inline_comment_after_string_with_hash() -> None:
+    content = 'name = "user"  # real comment\n'
+    inline, standalone = code_rules_enforcer.extract_comment_texts(content, "foo.py")
+    assert len(inline) == 1
+    assert "# real comment" in next(iter(inline))
+    assert standalone == set()
+
+
+def test_extract_should_classify_standalone_comment_correctly() -> None:
+    content = "# standalone comment\nx = 1\n"
+    inline, standalone = code_rules_enforcer.extract_comment_texts(content, "foo.py")
+    assert "# standalone comment" in standalone
+    assert inline == set()
+
+
+def test_extract_should_distinguish_inline_from_standalone_in_same_file() -> None:
+    content = '# standalone first\nx = "#FFFFFF"  # inline real comment\n# standalone second\n'
+    inline, standalone = code_rules_enforcer.extract_comment_texts(content, "foo.py")
+    assert "# inline real comment" in next(iter(inline))
+    assert "# standalone first" in standalone
+    assert "# standalone second" in standalone
+
+
+def test_check_comment_changes_skips_removal_when_new_python_un_parseable() -> None:
+    old_content = 'x = 1  # existing comment\n'
+    new_content = '"""unterminated multi-line string\n'
+    issues = code_rules_enforcer.check_comment_changes(old_content, new_content, "foo.py")
+    assert issues == []
+
+
+def test_check_comment_changes_skips_removal_when_old_python_un_parseable() -> None:
+    old_content = '"""unterminated multi-line string\n'
+    new_content = 'x = 1  # newly added comment\n'
+    issues = code_rules_enforcer.check_comment_changes(old_content, new_content, "foo.py")
+    assert issues == []
+
+
+def test_check_comment_changes_still_detects_removal_on_parseable_python() -> None:
+    old_content = 'x = 1  # existing comment\n'
+    new_content = 'x = 1\n'
+    issues = code_rules_enforcer.check_comment_changes(old_content, new_content, "foo.py")
+    assert any("Existing comment removed" in each_issue for each_issue in issues)
+
+
+def test_python_check_should_exempt_directive_without_space_after_hash() -> None:
+    for each_directive in ("#noqa", "#type: ignore", "#pylint: disable", "#pragma: no cover"):
+        content = f"{each_directive}\n"
+        issues = code_rules_enforcer.check_comments_python(content)
+        assert issues == [], f"expected exempt for {each_directive!r}"
+
+
+def test_python_check_should_exempt_directive_with_tab_after_hash() -> None:
+    content = "#\tnoqa: F401\n"
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert issues == []
+
+
+def test_python_check_should_still_flag_unrelated_no_space_comment() -> None:
+    content = "#realcomment text\n"
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert len(issues) == 1
+
+
+def test_extract_should_exempt_directive_without_space_after_hash() -> None:
+    content = "#noqa\nx = 1\n"
+    inline, standalone = code_rules_enforcer.extract_comment_texts(content, "foo.py")
+    assert standalone == set()
+    assert inline == set()
+
+
+def test_python_check_should_exempt_bare_hash_comment() -> None:
+    for each_content in ("#\n", "#  \n", "x = 1  #\n"):
+        issues = code_rules_enforcer.check_comments_python(each_content)
+        assert issues == [], f"expected exempt for bare hash in {each_content!r}"
+
+
+def test_extract_should_not_classify_bare_hash_as_comment() -> None:
+    content = "#\nx = 1  #\n"
+    inline, standalone = code_rules_enforcer.extract_comment_texts(content, "foo.py")
+    assert standalone == set()
+    assert inline == set()
+
+
+def test_python_check_should_exempt_true_shebang_on_line_one() -> None:
+    content = "#!/usr/bin/env python3\nx = 1\n"
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert issues == []
+
+
+def test_python_check_should_flag_inline_shebang_lookalike() -> None:
+    content = "x = 1  #! hidden comment\n"
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert len(issues) == 1
+
+
+def test_python_check_should_flag_shebang_on_later_standalone_line() -> None:
+    content = "x = 1\n#! not a real shebang\n"
+    issues = code_rules_enforcer.check_comments_python(content)
+    assert len(issues) == 1
+
+
+def test_extract_should_classify_inline_shebang_lookalike_as_inline_comment() -> None:
+    content = "x = 1  #! hidden\n"
+    inline, standalone = code_rules_enforcer.extract_comment_texts(content, "foo.py")
+    assert inline != set()
+    assert standalone == set()

package/hooks/blocking/test_code_rules_enforcer_type_checking_scope.py

@@ -110,3 +110,85 @@ def test_should_track_only_innermost_type_checking_block() -> None:
     assert len(issues) == 1
     assert issues[0].startswith("Line 7:")
     assert "Import inside function" in issues[0]
+
+
+def test_should_skip_docstring_lines_starting_with_import_keyword() -> None:
+    """Docstring sentences that incidentally start with ``from `` or ``import `` after
+    stripping must not trigger the import-inside-function check.
+    """
+    content = (
+        "def helper():\n"
+        '    """Apply the priority queue atomically.\n'
+        "\n"
+        "    from a rename within the trailing-revenue window the duplicate\n"
+        "    import the loaders for the cycle so the writer can advance.\n"
+        '    """\n'
+        "    return 42\n"
+    )
+    issues = check_imports_at_top(content)
+    assert issues == []
+
+
+def test_should_still_flag_real_import_after_docstring_closes() -> None:
+    """An actual import statement after a one-line docstring closes must still flag."""
+    content = (
+        "def helper():\n"
+        '    """One-line docstring."""\n'
+        "    import os\n"
+        "    return os\n"
+    )
+    issues = check_imports_at_top(content)
+    assert len(issues) == 1
+    assert "Import inside function" in issues[0]
+
+
+def test_should_skip_triple_single_quoted_docstring_lines() -> None:
+    """Triple-single-quote (''') docstrings exempt their interior lines too."""
+    content = (
+        "def helper():\n"
+        "    '''Apply the cycle reset.\n"
+        "\n"
+        "    from a rename within the cycle window the writer would advance.\n"
+        "    '''\n"
+        "    return 1\n"
+    )
+    issues = check_imports_at_top(content)
+    assert issues == []
+
+
+def test_should_flag_real_import_after_multi_line_docstring_closes() -> None:
+    """Real imports landing AFTER a multi-line docstring closes must still flag,
+    confirming the triple-quote state correctly transitions back to ``None``.
+    """
+    content = (
+        "def helper():\n"
+        '    """Apply the priority queue atomically.\n'
+        "\n"
+        "    from a rename within the cycle window.\n"
+        '    """\n'
+        "    import os\n"
+        "    return os\n"
+    )
+    issues = check_imports_at_top(content)
+    assert len(issues) == 1
+    assert "Import inside function" in issues[0]
+
+
+def test_should_skip_module_level_docstring_text() -> None:
+    """A module-level docstring containing ``from ``/``import `` text must not flag.
+
+    The check ignores top-level lines regardless of triple-quote state because
+    function-tracking is the only path that produces issues, but this exercises
+    the entry-condition path where the docstring opens on line 1.
+    """
+    content = (
+        '"""Module docstring opener.\n'
+        "\n"
+        "from a rename within the trailing-revenue window.\n"
+        '"""\n'
+        "\n"
+        "def helper():\n"
+        "    return 1\n"
+    )
+    issues = check_imports_at_top(content)
+    assert issues == []

package/hooks/blocking/test_code_rules_enforcer_unused_imports.py

@@ -39,7 +39,7 @@ TEST_FILE_PATH = "packages/app/tests/test_loader.py"
 
 def test_should_flag_unused_from_import() -> None:
     source = (
-        "from config.preflight_constants import VENV_DIRECTORY_NAME\n"
+        "from hooks_constants.preflight_constants import VENV_DIRECTORY_NAME\n"
         "\n"
         "def run() -> None:\n"
         "    return None\n"
@@ -52,7 +52,7 @@ def test_should_flag_unused_from_import() -> None:
 
 def test_should_not_flag_used_from_import() -> None:
     source = (
-        "from config.preflight_constants import VENV_DIRECTORY_NAME\n"
+        "from hooks_constants.preflight_constants import VENV_DIRECTORY_NAME\n"
         "\n"
         "def run() -> str:\n"
         "    return VENV_DIRECTORY_NAME\n"
@@ -79,7 +79,7 @@ def test_should_not_flag_when_alias_is_used() -> None:
 
 def test_should_skip_file_with_dunder_all() -> None:
     source = (
-        "from config.preflight_constants import VENV_DIRECTORY_NAME\n"
+        "from hooks_constants.preflight_constants import VENV_DIRECTORY_NAME\n"
         "\n"
         "__all__ = ['something']\n"
     )
@@ -91,7 +91,7 @@ def test_should_skip_file_with_dunder_all() -> None:
 
 def test_should_skip_file_with_dunder_all_annotated_assignment() -> None:
     source = (
-        "from config.preflight_constants import VENV_DIRECTORY_NAME\n"
+        "from hooks_constants.preflight_constants import VENV_DIRECTORY_NAME\n"
         "\n"
         '__all__: list[str] = ["VENV_DIRECTORY_NAME"]\n'
     )
@@ -105,7 +105,7 @@ def test_should_skip_file_with_dunder_all_annotated_assignment() -> None:
 def test_should_skip_file_using_type_checking_block() -> None:
     source = (
         "from typing import TYPE_CHECKING\n"
-        "from config.constants import UNUSED_NAME\n"
+        "from hooks_constants.constants import UNUSED_NAME\n"
         "\n"
         "if TYPE_CHECKING:\n"
         "    from somewhere import OtherName\n"
@@ -121,7 +121,7 @@ def test_should_skip_file_using_type_checking_block() -> None:
 
 def test_should_skip_test_files() -> None:
     source = (
-        "from config.constants import UNUSED_NAME\n"
+        "from hooks_constants.constants import UNUSED_NAME\n"
         "\n"
         "def test_thing() -> None:\n"
         "    assert True\n"
@@ -139,7 +139,7 @@ def test_should_handle_syntax_error_gracefully() -> None:
 def test_should_include_line_number_in_issue() -> None:
     source = (
         "\n"
-        "from config.preflight_constants import VENV_DIRECTORY_NAME\n"
+        "from hooks_constants.preflight_constants import VENV_DIRECTORY_NAME\n"
         "\n"
         "def run() -> None:\n"
         "    return None\n"
@@ -152,7 +152,7 @@ def test_should_include_line_number_in_issue() -> None:
 
 def test_should_flag_each_unused_in_multi_import() -> None:
     source = (
-        "from config.constants import USED_ONE, UNUSED_TWO\n"
+        "from hooks_constants.constants import USED_ONE, UNUSED_TWO\n"
         "\n"
         "def run() -> str:\n"
         "    return USED_ONE\n"
@@ -178,7 +178,7 @@ def test_should_not_flag_when_referenced_in_annotation() -> None:
 
 def test_should_skip_noqa_marked_imports() -> None:
     source = (
-        "from config.constants import UNUSED_BUT_DELIBERATE  # noqa: F401\n"
+        "from hooks_constants.constants import UNUSED_BUT_DELIBERATE  # noqa: F401\n"
         "\n"
         "def run() -> None:\n"
         "    return None\n"
@@ -191,7 +191,7 @@ def test_should_skip_noqa_marked_imports() -> None:
 
 def test_should_skip_noqa_on_from_keyword_line_for_multiline_import() -> None:
     source = (
-        "from config.constants import (  # noqa: F401\n"
+        "from hooks_constants.constants import (  # noqa: F401\n"
         "    SOME_CONSTANT,\n"
         "    ANOTHER_CONSTANT,\n"
         ")\n"
@@ -261,7 +261,7 @@ def test_should_flag_when_name_only_appears_in_comment() -> None:
 
 def test_should_not_skip_when_type_checking_only_in_string_constant() -> None:
     source = (
-        'from config.constants import UNUSED_NAME\n'
+        'from hooks_constants.constants import UNUSED_NAME\n'
         '\n'
         'HELP_TEXT = "See TYPE_CHECKING docs"\n'
         '\n'
@@ -276,7 +276,7 @@ def test_should_not_skip_when_type_checking_only_in_string_constant() -> None:
 
 def test_should_flag_when_noqa_lists_only_non_f401_codes() -> None:
     source = (
-        "from config.constants import UNUSED  # noqa: E402\n"
+        "from hooks_constants.constants import UNUSED  # noqa: E402\n"
         "\n"
         "def run() -> None:\n"
         "    return None\n"
@@ -289,7 +289,7 @@ def test_should_flag_when_noqa_lists_only_non_f401_codes() -> None:
 
 def test_should_skip_when_noqa_is_bare() -> None:
     source = (
-        "from config.constants import UNUSED  # noqa\n"
+        "from hooks_constants.constants import UNUSED  # noqa\n"
         "\n"
         "def run() -> None:\n"
         "    return None\n"
@@ -299,9 +299,9 @@ def test_should_skip_when_noqa_is_bare() -> None:
 
 
 def test_should_not_flag_imports_referenced_only_in_full_file_content() -> None:
-    fragment = "from config.constants import NEW_NAME\n"
+    fragment = "from hooks_constants.constants import NEW_NAME\n"
     full_file = (
-        "from config.constants import NEW_NAME\n"
+        "from hooks_constants.constants import NEW_NAME\n"
         "\n"
         "def existing_function() -> str:\n"
         "    return NEW_NAME\n"
@@ -315,9 +315,9 @@ def test_should_not_flag_imports_referenced_only_in_full_file_content() -> None:
 
 
 def test_should_flag_imports_unused_in_full_file_content() -> None:
-    fragment = "from config.constants import TRULY_UNUSED\n"
+    fragment = "from hooks_constants.constants import TRULY_UNUSED\n"
     full_file = (
-        "from config.constants import TRULY_UNUSED\n"
+        "from hooks_constants.constants import TRULY_UNUSED\n"
         "\n"
         "def existing_function() -> None:\n"
         "    return None\n"
@@ -331,10 +331,10 @@ def test_should_flag_imports_unused_in_full_file_content() -> None:
 
 
 def test_should_only_flag_imports_in_fragment_not_full_file() -> None:
-    fragment = "from config.constants import FRAGMENT_IMPORT\n"
+    fragment = "from hooks_constants.constants import FRAGMENT_IMPORT\n"
     full_file = (
-        "from config.other import PRE_EXISTING_UNUSED\n"
-        "from config.constants import FRAGMENT_IMPORT\n"
+        "from hooks_constants.other import PRE_EXISTING_UNUSED\n"
+        "from hooks_constants.constants import FRAGMENT_IMPORT\n"
         "\n"
         "def existing_function() -> str:\n"
         "    return FRAGMENT_IMPORT\n"
@@ -349,9 +349,9 @@ def test_should_only_flag_imports_in_fragment_not_full_file() -> None:
 
 
 def test_should_skip_when_full_file_declares_dunder_all() -> None:
-    fragment = "from config.constants import NEW_NAME\n"
+    fragment = "from hooks_constants.constants import NEW_NAME\n"
     full_file = (
-        "from config.constants import NEW_NAME\n"
+        "from hooks_constants.constants import NEW_NAME\n"
         "\n"
         "__all__ = ['NEW_NAME']\n"
     )
@@ -365,10 +365,10 @@ def test_should_skip_when_full_file_declares_dunder_all() -> None:
 
 
 def test_should_skip_when_full_file_uses_type_checking_gate() -> None:
-    fragment = "from config.constants import NEW_NAME\n"
+    fragment = "from hooks_constants.constants import NEW_NAME\n"
     full_file = (
         "from typing import TYPE_CHECKING\n"
-        "from config.constants import NEW_NAME\n"
+        "from hooks_constants.constants import NEW_NAME\n"
         "\n"
         "if TYPE_CHECKING:\n"
         "    from somewhere import OtherName\n"
@@ -387,7 +387,7 @@ def test_should_skip_when_full_file_uses_type_checking_gate() -> None:
 
 def test_should_fall_back_to_content_when_full_file_content_is_none() -> None:
     source = (
-        "from config.constants import VENV_DIRECTORY_NAME\n"
+        "from hooks_constants.constants import VENV_DIRECTORY_NAME\n"
         "\n"
         "def run() -> None:\n"
         "    return None\n"
@@ -400,7 +400,7 @@ def test_should_fall_back_to_content_when_full_file_content_is_none() -> None:
 
 
 def test_should_fall_back_when_full_file_content_has_syntax_error() -> None:
-    fragment = "from config.constants import NEW_NAME\n"
+    fragment = "from hooks_constants.constants import NEW_NAME\n"
     full_file_with_syntax_error = "from config import (\n  not python\n"
     issues = check_unused_module_level_imports(
         fragment, PRODUCTION_FILE_PATH, full_file_content=full_file_with_syntax_error,
@@ -471,7 +471,7 @@ def test_should_flag_import_when_only_shadowed_local_name_is_loaded() -> None:
 def test_should_skip_when_type_checking_uses_imported_alias() -> None:
     source = (
         "from typing import TYPE_CHECKING as IS_TYPE_CHECKING\n"
-        "from config.constants import UNUSED_NAME\n"
+        "from hooks_constants.constants import UNUSED_NAME\n"
         "\n"
         "if IS_TYPE_CHECKING:\n"
         "    from somewhere import OtherName\n"
@@ -488,7 +488,7 @@ def test_should_skip_when_type_checking_uses_imported_alias() -> None:
 def test_should_skip_when_type_checking_uses_module_alias() -> None:
     source = (
         "import typing as t\n"
-        "from config.constants import UNUSED_NAME\n"
+        "from hooks_constants.constants import UNUSED_NAME\n"
         "\n"
         "if t.TYPE_CHECKING:\n"
         "    from somewhere import OtherName\n"
@@ -517,7 +517,7 @@ def test_should_not_flag_when_referenced_in_quoted_annotation() -> None:
 
 def test_should_flag_when_noqa_only_appears_inside_string_literal() -> None:
     source = (
-        "from config.constants import UNUSED; MARKER = '# noqa: F401'\n"
+        "from hooks_constants.constants import UNUSED; MARKER = '# noqa: F401'\n"
         "\n"
         "def run() -> None:\n"
         "    return None\n"

package/hooks/blocking/test_gh_body_arg_blocker.py

@@ -19,7 +19,13 @@ hook_spec.loader.exec_module(hook_module)
 _uses_body_string_arg = hook_module._uses_body_string_arg
 _has_backtick = hook_module._has_backtick
 
-from _gh_body_arg_utils import iter_significant_tokens
+from blocking._gh_body_arg_utils import (
+    _all_equals_prefixes_for_skip,
+    _quoted_value_starts_split,
+    all_body_flag_prefixes,
+    count_extra_tokens_to_skip_for_split_quoted_value,
+    iter_significant_tokens,
+)
 
 
 def test_blocks_issue_create_with_body_string() -> None:
@@ -337,40 +343,33 @@ def test_space_form_value_flag_remaining_excludes_consumed_value() -> None:
 
 
 def test_quoted_value_starts_split_unclosed_single_quote() -> None:
-    from _gh_body_arg_utils import _quoted_value_starts_split
     assert _quoted_value_starts_split("'it") is True
 
 
 def test_quoted_value_starts_split_fully_closed() -> None:
-    from _gh_body_arg_utils import _quoted_value_starts_split
     assert _quoted_value_starts_split("'hello'") is False
 
 
 def test_quoted_value_starts_split_double_quote_unclosed() -> None:
-    from _gh_body_arg_utils import _quoted_value_starts_split
     assert _quoted_value_starts_split('"hello') is True
 
 
 def test_count_extra_tokens_returns_none_when_exhausted() -> None:
-    from _gh_body_arg_utils import count_extra_tokens_to_skip_for_split_quoted_value
     result = count_extra_tokens_to_skip_for_split_quoted_value([], "'unclosed")
     assert result is None
 
 
 def test_count_extra_tokens_returns_none_no_closing_in_remaining() -> None:
-    from _gh_body_arg_utils import count_extra_tokens_to_skip_for_split_quoted_value
     result = count_extra_tokens_to_skip_for_split_quoted_value(["word", "another"], "'unclosed")
     assert result is None
 
 
 def test_count_extra_tokens_returns_zero_for_self_contained() -> None:
-    from _gh_body_arg_utils import count_extra_tokens_to_skip_for_split_quoted_value
     result = count_extra_tokens_to_skip_for_split_quoted_value(["next"], "'complete'")
     assert result == 0
 
 
 def test_all_body_flag_prefixes_used_for_equals_skip() -> None:
-    from _gh_body_arg_utils import _all_equals_prefixes_for_skip, all_body_flag_prefixes
     for each_prefix in all_body_flag_prefixes:
         assert each_prefix in _all_equals_prefixes_for_skip