claude-dev-env 1.25.1 → 1.26.0

package/hooks/validators/run_all_validators.py

@@ -3,6 +3,7 @@
 This script orchestrates all automated validators and produces a unified report.
 Exit code 0 = all checks pass, 1 = violations found.
 """
+# pragma: no-tdd-gate
 
 import argparse
 import subprocess
@@ -13,13 +14,31 @@ from datetime import datetime
 from pathlib import Path
 from typing import Any, Callable, Dict, List, Optional, Tuple
 
-from health_check import get_validator_version
-from mypy_integration import check_mypy_available, run_mypy_check
-from output_formatter import OutputFormatter, OutputMode, ValidatorResultDict
-from ruff_integration import check_ruff_available, run_ruff_check
+from .health_check import get_system_health, get_validator_version, print_health_report
+from .mypy_integration import check_mypy_available, run_mypy_check
+from .output_formatter import OutputFormatter, OutputMode, ValidatorResultDict
+from .python_style_checks import fix_file
+from .ruff_integration import check_ruff_available, run_ruff_check
 
 
 VALIDATORS_DIR = Path(__file__).parent
+hooks_dir = VALIDATORS_DIR.parent
+package_name = VALIDATORS_DIR.name
+
+
+def invoke_validator_module(module_stem: str, forwarded_file_paths: List[str]) -> subprocess.CompletedProcess[str]:  # pragma: no-tdd-gate
+    """Run a sibling validator as ``python -m validators.<module_stem>``.
+
+    The subprocess is launched with ``cwd`` set to the hooks directory so the
+    ``validators`` package qualifier resolves without requiring PYTHONPATH.
+    """
+    qualified_module = ".".join([package_name, module_stem])
+    return subprocess.run(
+        [sys.executable, "-m", qualified_module, *forwarded_file_paths],
+        capture_output=True,
+        text=True,
+        cwd=str(hooks_dir),
+    )
 
 
 @dataclass(frozen=True)
@@ -169,18 +188,13 @@ def run_python_style_checks(files: List[Path]) -> ValidatorResult:
             output="No Python files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "python_style_checks.py")]
-        + [str(f) for f in py_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("python_style_checks", [str(f) for f in py_files])
 
     return ValidatorResult(
         name="Python Style",
         checks="1,2,3,4",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -195,18 +209,13 @@ def run_test_safety_checks(files: List[Path]) -> ValidatorResult:
             output="No test files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "test_safety_checks.py")]
-        + [str(f) for f in test_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("test_safety_checks", [str(f) for f in test_files])
 
     return ValidatorResult(
         name="Test Safety",
         checks="11,21",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -235,17 +244,13 @@ def run_file_structure_checks(project_root: Optional[Path] = None) -> ValidatorR
             output="Not in a git repository - skipping",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "file_structure_checks.py"), str(project_root)],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("file_structure_checks", [str(project_root)])
 
     return ValidatorResult(
         name="File Structure",
         checks="14,15",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -260,39 +265,30 @@ def run_react_checks(files: List[Path]) -> ValidatorResult:
             output="No React files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "react_checks.py")]
-        + [str(f) for f in react_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("react_checks", [str(f) for f in react_files])
 
     return ValidatorResult(
         name="React",
         checks="17",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
 def run_git_checks() -> ValidatorResult:
     """Run git/GitHub checks."""
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "git_checks.py")],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("git_checks", [])
 
     return ValidatorResult(
         name="Git/PR Workflow",
         checks="23,24",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
 def run_comment_checks(files: List[Path]) -> ValidatorResult:
-    """Comment preservation is enforced by code-rules-enforcer hook.
+    """Comment preservation is enforced by code_rules_enforcer hook.
 
     The hook compares old vs new content to block NEW comments and
     block DELETION of existing comments. This standalone validator
@@ -303,7 +299,7 @@ def run_comment_checks(files: List[Path]) -> ValidatorResult:
         name="No Comments",
         checks="26",
         passed=True,
-        output="Handled by code-rules-enforcer hook (old vs new comparison)",
+        output="Handled by code_rules_enforcer hook (old vs new comparison)",
     )
 
 
@@ -358,18 +354,13 @@ def run_abbreviation_checks(files: List[Path]) -> ValidatorResult:
             output="No Python files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "abbreviation_checks.py")]
-        + [str(f) for f in py_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("abbreviation_checks", [str(f) for f in py_files])
 
     return ValidatorResult(
         name="Abbreviations",
         checks="5",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -384,18 +375,13 @@ def run_pr_reference_checks(files: List[Path]) -> ValidatorResult:
             output="No code files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "pr_reference_checks.py")]
-        + [str(f) for f in code_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("pr_reference_checks", [str(f) for f in code_files])
 
     return ValidatorResult(
         name="PR References",
         checks="6",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -410,18 +396,13 @@ def run_magic_value_checks(files: List[Path]) -> ValidatorResult:
             output="No Python files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "magic_value_checks.py")]
-        + [str(f) for f in py_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("magic_value_checks", [str(f) for f in py_files])
 
     return ValidatorResult(
         name="Magic Values",
         checks="7",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -436,18 +417,13 @@ def run_useless_test_checks(files: List[Path]) -> ValidatorResult:
             output="No test files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "useless_test_checks.py")]
-        + [str(f) for f in test_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("useless_test_checks", [str(f) for f in test_files])
 
     return ValidatorResult(
         name="Useless Tests",
         checks="12",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -462,18 +438,13 @@ def run_security_checks(files: List[Path]) -> ValidatorResult:
             output="No Python files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "security_checks.py")]
-        + [str(f) for f in py_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("security_checks", [str(f) for f in py_files])
 
     return ValidatorResult(
         name="Security",
         checks="27,28,29",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -488,18 +459,13 @@ def run_code_quality_checks(files: List[Path]) -> ValidatorResult:
             output="No Python files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "code_quality_checks.py")]
-        + [str(f) for f in py_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("code_quality_checks", [str(f) for f in py_files])
 
     return ValidatorResult(
         name="Code Quality",
         checks="30,31,32",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -514,18 +480,13 @@ def run_python_antipattern_checks(files: List[Path]) -> ValidatorResult:
             output="No Python files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "python_antipattern_checks.py")]
-        + [str(f) for f in py_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("python_antipattern_checks", [str(f) for f in py_files])
 
     return ValidatorResult(
         name="Python Anti-patterns",
         checks="33,34,35",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -540,18 +501,13 @@ def run_todo_checks(files: List[Path]) -> ValidatorResult:
             output="No Python files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "todo_checks.py")]
-        + [str(f) for f in py_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("todo_checks", [str(f) for f in py_files])
 
     return ValidatorResult(
         name="TODO Tracking",
         checks="36",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -566,18 +522,13 @@ def run_type_safety_checks(files: List[Path]) -> ValidatorResult:
             output="No Python files to check",
         )
 
-    result = subprocess.run(
-        [sys.executable, str(VALIDATORS_DIR / "type_safety_checks.py")]
-        + [str(f) for f in py_files],
-        capture_output=True,
-        text=True,
-    )
+    result = invoke_validator_module("type_safety_checks", [str(f) for f in py_files])
 
     return ValidatorResult(
         name="Type Safety",
         checks="39,40",
         passed=result.returncode == 0,
-        output=result.stdout or "All checks passed",
+        output=result.stdout or result.stderr or "All checks passed",
     )
 
 
@@ -590,8 +541,6 @@ def fix_python_style(files: List[Path]) -> List[str]:
     Returns:
         List of files that were fixed
     """
-    from python_style_checks import fix_file
-
     fixed_files: List[str] = []
     py_files = [f for f in files if f.suffix == ".py"]
 
@@ -657,7 +606,6 @@ def main() -> int:
     args = parser.parse_args()
 
     if args.health:
-        from health_check import get_system_health, print_health_report
         health = get_system_health()
         print_health_report(health)
         return 0 if health.all_healthy else 1

package/hooks/validators/security_checks.py

@@ -12,7 +12,7 @@ import sys
 from pathlib import Path
 from typing import List
 
-from validator_base import Violation
+from .validator_base import Violation
 
 
 SECRET_PATTERNS: frozenset[str] = frozenset({

package/hooks/validators/test_abbreviation_checks.py

@@ -5,11 +5,11 @@ from pathlib import Path
 
 import pytest
 
-from abbreviation_checks import (
+from .abbreviation_checks import (
     check_single_letter_variables,
     validate_file,
 )
-from validator_base import Violation
+from .validator_base import Violation
 
 
 GOOD_DESCRIPTIVE_NAMES = '''

package/hooks/validators/test_code_quality_checks.py

@@ -6,12 +6,12 @@ from pathlib import Path
 
 import pytest
 
-from code_quality_checks import (
+from .code_quality_checks import (
     check_function_length,
     check_nesting_depth,
     check_file_length,
 )
-from validator_base import Violation
+from .validator_base import Violation
 
 
 GOOD_SHORT_FUNCTION = '''

package/hooks/validators/test_file_structure_checks.py

@@ -6,7 +6,7 @@ from typing import List
 
 import pytest
 
-from file_structure_checks import (
+from .file_structure_checks import (
     Violation,
     check_multiple_requirements_txt,
     check_empty_init_files,

package/hooks/validators/test_git_checks.py

@@ -8,7 +8,7 @@ from unittest.mock import MagicMock, patch
 
 import pytest
 
-from git_checks import (
+from .git_checks import (
     Violation,
     check_single_commit_when_pr_exists,
     check_draft_pr_state,
@@ -22,7 +22,10 @@ class TestSingleCommitWhenPrExists:
     @patch("git_checks.subprocess.run")
     def test_no_pr_returns_empty(self, mock_run: MagicMock) -> None:
         """When no PR exists, check should return empty list."""
-        mock_run.return_value = MagicMock(returncode=0, stdout="[]", stderr="")
+        mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
+            MagicMock(returncode=0, stdout="[]", stderr=""),
+        ]
 
         violations = check_single_commit_when_pr_exists()
 
@@ -32,6 +35,7 @@ class TestSingleCommitWhenPrExists:
     def test_single_commit_ahead_passes(self, mock_run: MagicMock) -> None:
         """Exactly 1 commit ahead should pass."""
         mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
             MagicMock(returncode=0, stdout='[{"baseRefName": "main", "number": 123}]', stderr=""),
             MagicMock(returncode=0, stdout="1", stderr=""),
         ]
@@ -44,6 +48,7 @@ class TestSingleCommitWhenPrExists:
     def test_zero_commits_ahead_fails(self, mock_run: MagicMock) -> None:
         """Zero commits ahead should fail."""
         mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
             MagicMock(returncode=0, stdout='[{"baseRefName": "main", "number": 123}]', stderr=""),
             MagicMock(returncode=0, stdout="0", stderr=""),
         ]
@@ -60,6 +65,7 @@ class TestSingleCommitWhenPrExists:
     def test_multiple_commits_ahead_fails(self, mock_run: MagicMock) -> None:
         """More than 1 commit ahead should fail."""
         mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
             MagicMock(returncode=0, stdout='[{"baseRefName": "main", "number": 123}]', stderr=""),
             MagicMock(returncode=0, stdout="3", stderr=""),
         ]
@@ -73,7 +79,10 @@ class TestSingleCommitWhenPrExists:
     @patch("git_checks.subprocess.run")
     def test_gh_cli_not_available_returns_empty(self, mock_run: MagicMock) -> None:
         """When gh CLI not available, should return empty (warning, not failure)."""
-        mock_run.side_effect = FileNotFoundError("gh not found")
+        mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
+            FileNotFoundError("gh not found"),
+        ]
 
         violations = check_single_commit_when_pr_exists()
 
@@ -83,6 +92,7 @@ class TestSingleCommitWhenPrExists:
     def test_git_not_available_returns_empty(self, mock_run: MagicMock) -> None:
         """When git not available, should return empty."""
         mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
             MagicMock(returncode=0, stdout='[{"baseRefName": "main", "number": 123}]', stderr=""),
             FileNotFoundError("git not found"),
         ]
@@ -93,8 +103,9 @@ class TestSingleCommitWhenPrExists:
 
     @patch("git_checks.subprocess.run")
     def test_extracts_base_branch_from_pr_info(self, mock_run: MagicMock) -> None:
-        """Should extract base branch name from gh pr list JSON output."""
+        """Should extract base branch name from gh pr list JSON output, falling back to main when absent."""
         mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
             MagicMock(returncode=0, stdout='[{"baseRefName": "develop", "number": 123}]', stderr=""),
             MagicMock(returncode=0, stdout="2", stderr=""),
         ]
@@ -110,10 +121,30 @@ class TestSingleCommitWhenPrExists:
             timeout=30,
         )
 
+        mock_run.reset_mock()
+        mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
+            MagicMock(returncode=0, stdout='[{"number": 123}]', stderr=""),
+            MagicMock(returncode=0, stdout="2", stderr=""),
+        ]
+
+        fallback_violations = check_single_commit_when_pr_exists()
+
+        assert len(fallback_violations) == 1
+        assert "main" in fallback_violations[0].message
+        mock_run.assert_any_call(
+            ["git", "rev-list", "--count", "main..HEAD"],
+            capture_output=True,
+            text=True,
+            check=True,
+            timeout=30,
+        )
+
     @patch("git_checks.subprocess.run")
     def test_non_numeric_commit_count_returns_empty(self, mock_run: MagicMock) -> None:
         """When git rev-list returns non-numeric output, should return empty."""
         mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
             MagicMock(returncode=0, stdout='[{"baseRefName": "main", "number": 123}]', stderr=""),
             MagicMock(returncode=0, stdout="not a number\n", stderr=""),
         ]
@@ -125,7 +156,10 @@ class TestSingleCommitWhenPrExists:
     @patch("git_checks.subprocess.run")
     def test_gh_timeout_returns_empty(self, mock_run: MagicMock) -> None:
         """When gh CLI times out, should return empty (warning, not failure)."""
-        mock_run.side_effect = subprocess.TimeoutExpired(cmd=["gh", "pr", "list"], timeout=30)
+        mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
+            subprocess.TimeoutExpired(cmd=["gh", "pr", "list"], timeout=30),
+        ]
 
         violations = check_single_commit_when_pr_exists()
 
@@ -135,6 +169,7 @@ class TestSingleCommitWhenPrExists:
     def test_git_timeout_returns_empty(self, mock_run: MagicMock) -> None:
         """When git times out, should return empty (warning, not failure)."""
         mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
             MagicMock(returncode=0, stdout='[{"baseRefName": "main", "number": 123}]', stderr=""),
             subprocess.TimeoutExpired(cmd=["git", "rev-list"], timeout=30),
         ]
@@ -143,6 +178,37 @@ class TestSingleCommitWhenPrExists:
 
         assert violations == []
 
+    @patch("git_checks.subprocess.run")
+    def test_passes_resolved_branch_name_to_gh(self, mock_run: MagicMock) -> None:
+        """gh pr list must receive the resolved branch name, never the literal 'HEAD'."""
+        mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="feature/my-branch\n", stderr=""),
+            MagicMock(returncode=0, stdout='[{"baseRefName": "main", "number": 123}]', stderr=""),
+            MagicMock(returncode=0, stdout="1", stderr=""),
+        ]
+
+        check_single_commit_when_pr_exists()
+
+        mock_run.assert_any_call(
+            ["gh", "pr", "list", "--head", "feature/my-branch", "--json", "baseRefName,number"],
+            capture_output=True,
+            text=True,
+            check=True,
+            timeout=30,
+        )
+
+    @patch("git_checks.subprocess.run")
+    def test_unresolved_branch_returns_empty(self, mock_run: MagicMock) -> None:
+        """When current branch cannot be resolved, should return empty."""
+        mock_run.side_effect = [
+            MagicMock(returncode=0, stdout="\n", stderr=""),
+        ]
+
+        violations = check_single_commit_when_pr_exists()
+
+        assert violations == []
+        mock_run.assert_called_once()
+
 
 class TestDraftPrState:
     """Test that PR is in draft state when pushing review fixes."""
@@ -208,8 +274,8 @@ class TestDraftPrState:
 class TestMain:
     """Test main function integration."""
 
-    @patch("git_checks.check_single_commit_when_pr_exists")
-    @patch("git_checks.check_draft_pr_state")
+    @patch("validators.git_checks.check_single_commit_when_pr_exists")
+    @patch("validators.git_checks.check_draft_pr_state")
     def test_main_no_violations_exits_zero(
         self,
         mock_draft: MagicMock,
@@ -227,8 +293,8 @@ class TestMain:
         captured = capsys.readouterr()
         assert captured.out == ""
 
-    @patch("git_checks.check_single_commit_when_pr_exists")
-    @patch("git_checks.check_draft_pr_state")
+    @patch("validators.git_checks.check_single_commit_when_pr_exists")
+    @patch("validators.git_checks.check_draft_pr_state")
     def test_main_with_violations_exits_one(
         self,
         mock_draft: MagicMock,
@@ -248,8 +314,8 @@ class TestMain:
         captured = capsys.readouterr()
         assert "Branch has 3 commits ahead" in captured.out
 
-    @patch("git_checks.check_single_commit_when_pr_exists")
-    @patch("git_checks.check_draft_pr_state")
+    @patch("validators.git_checks.check_single_commit_when_pr_exists")
+    @patch("validators.git_checks.check_draft_pr_state")
     def test_main_prints_violations_without_file_line(
         self,
         mock_draft: MagicMock,
@@ -270,8 +336,8 @@ class TestMain:
         assert captured.out == "PR must be in draft state\n"
         assert ":0:" not in captured.out
 
-    @patch("git_checks.check_single_commit_when_pr_exists")
-    @patch("git_checks.check_draft_pr_state")
+    @patch("validators.git_checks.check_single_commit_when_pr_exists")
+    @patch("validators.git_checks.check_draft_pr_state")
     def test_main_prints_all_violations(
         self,
         mock_draft: MagicMock,

package/hooks/validators/test_health_check.py

@@ -5,7 +5,7 @@ from pathlib import Path
 
 import pytest
 
-from health_check import (
+from .health_check import (
     ValidatorHealth,
     check_validator_exists,
     check_all_validators,

package/hooks/validators/test_magic_value_checks.py

@@ -6,11 +6,11 @@ from pathlib import Path
 
 import pytest
 
-from magic_value_checks import (
+from .magic_value_checks import (
     check_magic_values,
     validate_file,
 )
-from validator_base import Violation
+from .validator_base import Violation
 
 
 MAGIC_NUMBER_SOURCE = "x = 42\n"

package/hooks/validators/test_mypy_integration.py

@@ -3,7 +3,7 @@
 from pathlib import Path
 from unittest.mock import patch
 
-from mypy_integration import MypyResult, check_mypy_available, run_mypy_check
+from .mypy_integration import MypyResult, check_mypy_available, run_mypy_check
 
 
 def test_mypy_result_dataclass() -> None:

package/hooks/validators/test_output_formatter.py

@@ -1,8 +1,10 @@
 """Tests for output formatting."""
 
+import os
+
 import pytest
 
-from output_formatter import (
+from .output_formatter import (
     OutputFormatter,
     OutputMode,
     format_violation_with_context,

package/hooks/validators/test_pr_reference_checks.py

@@ -4,11 +4,11 @@ from pathlib import Path
 
 import pytest
 
-from pr_reference_checks import (
+from .pr_reference_checks import (
     check_pr_references,
     validate_file,
 )
-from validator_base import Violation
+from .validator_base import Violation
 
 
 GOOD_NO_REFERENCES = '''