claude-dev-env 1.25.1 → 1.26.0

package/hooks/validators/test_python_antipattern_checks.py

@@ -4,12 +4,12 @@ import ast
 
 import pytest
 
-from python_antipattern_checks import (
+from .python_antipattern_checks import (
     check_mutable_default_args,
     check_bare_except,
     check_print_in_production,
 )
-from validator_base import Violation
+from .validator_base import Violation
 
 
 GOOD_NONE_DEFAULT = '''

package/hooks/validators/test_python_style_checks.py

@@ -7,12 +7,13 @@ from typing import List
 
 import pytest
 
-from python_style_checks import (
+from .python_style_checks import (
     Violation,
     check_imports_at_top,
     check_no_empty_line_after_decorators,
     check_single_empty_line_between_functions,
     check_view_function_naming,
+    fix_file,
     validate_file,
 )
 
@@ -383,7 +384,6 @@ def foo():
             temp_path = Path(temp_file.name)
 
         try:
-            from python_style_checks import fix_file
             fixed = fix_file(temp_path)
             assert fixed is True
             result = temp_path.read_text()
@@ -411,7 +411,6 @@ def bar():
             temp_path = Path(temp_file.name)
 
         try:
-            from python_style_checks import fix_file
             fixed = fix_file(temp_path)
             assert fixed is True
             result = temp_path.read_text()
@@ -432,7 +431,6 @@ def bar():
             temp_path = Path(temp_file.name)
 
         try:
-            from python_style_checks import fix_file
             fixed = fix_file(temp_path)
             assert fixed is False
         finally:

package/hooks/validators/test_react_checks.py

@@ -2,7 +2,7 @@
 
 import pytest
 from pathlib import Path
-from react_checks import check_no_class_components, Violation
+from .react_checks import check_no_class_components, Violation
 
 
 def test_class_component_extends_component_should_fail(tmp_path: Path) -> None:

package/hooks/validators/test_ruff_integration.py

@@ -3,7 +3,7 @@
 from pathlib import Path
 from unittest.mock import patch
 
-from ruff_integration import RuffResult, check_ruff_available, run_ruff_check
+from .ruff_integration import RuffResult, check_ruff_available, run_ruff_check
 
 
 def test_ruff_result_dataclass() -> None:

package/hooks/validators/test_run_all_validators.py

@@ -6,15 +6,28 @@ from unittest.mock import MagicMock, patch
 
 import pytest
 
+from .run_all_validators import (
+    ValidatorResult,
+    add_timing,
+    build_json_output,
+    create_timing_metrics,
+    format_timing_report,
+    main,
+    print_header,
+    run_git_checks,
+    run_python_style_checks,
+    run_with_fallback,
+)
+
 
 class TestFixFlag:
     """Test --fix flag functionality."""
 
     def test_fix_flag_is_accepted(self) -> None:
         """Verify --fix flag is recognized without error."""
-        with patch("run_all_validators.get_changed_files") as mock_get_files, \
-             patch("run_all_validators.run_file_structure_checks") as mock_file, \
-             patch("run_all_validators.run_git_checks") as mock_git:
+        with patch("validators.run_all_validators.get_changed_files") as mock_get_files, \
+             patch("validators.run_all_validators.run_file_structure_checks") as mock_file, \
+             patch("validators.run_all_validators.run_git_checks") as mock_git:
 
             mock_get_files.return_value = []
 
@@ -27,8 +40,6 @@ class TestFixFlag:
             mock_file.return_value = mock_result
             mock_git.return_value = mock_result
 
-            from run_all_validators import main
-
             original_argv = sys.argv
             try:
                 sys.argv = ["run_all_validators.py", "--fix"]
@@ -39,14 +50,14 @@ class TestFixFlag:
 
     def test_fix_flag_calls_fix_python_style(self) -> None:
         """Verify --fix flag triggers fix_python_style when files exist."""
-        with patch("run_all_validators.get_changed_files") as mock_get_files, \
-             patch("run_all_validators.fix_python_style") as mock_fix, \
-             patch("run_all_validators.run_python_style_checks") as mock_style, \
-             patch("run_all_validators.run_test_safety_checks") as mock_test, \
-             patch("run_all_validators.run_react_checks") as mock_react, \
-             patch("run_all_validators.run_comment_checks") as mock_comment, \
-             patch("run_all_validators.run_file_structure_checks") as mock_file, \
-             patch("run_all_validators.run_git_checks") as mock_git:
+        with patch("validators.run_all_validators.get_changed_files") as mock_get_files, \
+             patch("validators.run_all_validators.fix_python_style") as mock_fix, \
+             patch("validators.run_all_validators.run_python_style_checks") as mock_style, \
+             patch("validators.run_all_validators.run_test_safety_checks") as mock_test, \
+             patch("validators.run_all_validators.run_react_checks") as mock_react, \
+             patch("validators.run_all_validators.run_comment_checks") as mock_comment, \
+             patch("validators.run_all_validators.run_file_structure_checks") as mock_file, \
+             patch("validators.run_all_validators.run_git_checks") as mock_git:
 
             mock_get_files.return_value = [Path("test.py")]
             mock_fix.return_value = ["test.py"]
@@ -64,8 +75,6 @@ class TestFixFlag:
             mock_file.return_value = mock_result
             mock_git.return_value = mock_result
 
-            from run_all_validators import main
-
             original_argv = sys.argv
             try:
                 sys.argv = ["run_all_validators.py", "--fix"]
@@ -77,14 +86,14 @@ class TestFixFlag:
 
     def test_no_fix_flag_skips_fixes(self) -> None:
         """Verify fixes are skipped when --fix flag is not provided."""
-        with patch("run_all_validators.get_changed_files") as mock_get_files, \
-             patch("run_all_validators.fix_python_style") as mock_fix, \
-             patch("run_all_validators.run_python_style_checks") as mock_style, \
-             patch("run_all_validators.run_test_safety_checks") as mock_test, \
-             patch("run_all_validators.run_react_checks") as mock_react, \
-             patch("run_all_validators.run_comment_checks") as mock_comment, \
-             patch("run_all_validators.run_file_structure_checks") as mock_file, \
-             patch("run_all_validators.run_git_checks") as mock_git:
+        with patch("validators.run_all_validators.get_changed_files") as mock_get_files, \
+             patch("validators.run_all_validators.fix_python_style") as mock_fix, \
+             patch("validators.run_all_validators.run_python_style_checks") as mock_style, \
+             patch("validators.run_all_validators.run_test_safety_checks") as mock_test, \
+             patch("validators.run_all_validators.run_react_checks") as mock_react, \
+             patch("validators.run_all_validators.run_comment_checks") as mock_comment, \
+             patch("validators.run_all_validators.run_file_structure_checks") as mock_file, \
+             patch("validators.run_all_validators.run_git_checks") as mock_git:
 
             mock_get_files.return_value = [Path("test.py")]
 
@@ -101,8 +110,6 @@ class TestFixFlag:
             mock_file.return_value = mock_result
             mock_git.return_value = mock_result
 
-            from run_all_validators import main
-
             original_argv = sys.argv
             try:
                 sys.argv = ["run_all_validators.py"]
@@ -115,8 +122,6 @@ class TestFixFlag:
 
 class TestGracefulDegradation:
     def test_missing_validator_returns_skipped_result(self) -> None:
-        from run_all_validators import ValidatorResult, run_with_fallback
-
         def failing_validator() -> ValidatorResult:
             raise FileNotFoundError("validator.py not found")
 
@@ -131,8 +136,6 @@ class TestGracefulDegradation:
         assert result.passed is False
 
     def test_validator_exception_returns_skipped_result(self) -> None:
-        from run_all_validators import ValidatorResult, run_with_fallback
-
         def crashing_validator() -> ValidatorResult:
             raise RuntimeError("Unexpected crash")
 
@@ -146,8 +149,6 @@ class TestGracefulDegradation:
         assert "skipped" in result.output.lower()
 
     def test_successful_validator_returns_normal_result(self) -> None:
-        from run_all_validators import ValidatorResult, run_with_fallback
-
         def working_validator() -> ValidatorResult:
             return ValidatorResult(
                 name="Working",
@@ -166,25 +167,62 @@ class TestGracefulDegradation:
         assert result.passed is True
 
 
+class TestStderrSurfacing:
+    """Verify that validator stderr is surfaced when stdout is empty."""
+
+    def test_python_style_check_surfaces_stderr_when_stdout_empty(self) -> None:
+        """When a validator crashes with no stdout, stderr must appear in output."""
+        with patch("validators.run_all_validators.invoke_validator_module") as mock_invoke:
+            crashed_result = MagicMock()
+            crashed_result.returncode = 1
+            crashed_result.stdout = ""
+            crashed_result.stderr = "ImportError: No module named validators.python_style_checks"
+            mock_invoke.return_value = crashed_result
+
+            validator_result = run_python_style_checks([Path("foo.py")])
+
+            assert "ImportError" in validator_result.output
+
+    def test_git_check_surfaces_stderr_when_stdout_empty(self) -> None:
+        """When git validator crashes with no stdout, stderr must appear in output."""
+        with patch("validators.run_all_validators.invoke_validator_module") as mock_invoke:
+            crashed_result = MagicMock()
+            crashed_result.returncode = 1
+            crashed_result.stdout = ""
+            crashed_result.stderr = "SyntaxError: invalid syntax in git_checks.py"
+            mock_invoke.return_value = crashed_result
+
+            validator_result = run_git_checks()
+
+            assert "SyntaxError" in validator_result.output
+
+    def test_output_falls_back_to_all_checks_passed_when_both_empty(self) -> None:
+        """When both stdout and stderr are empty and returncode is 0, use fallback."""
+        with patch("validators.run_all_validators.invoke_validator_module") as mock_invoke:
+            clean_result = MagicMock()
+            clean_result.returncode = 0
+            clean_result.stdout = ""
+            clean_result.stderr = ""
+            mock_invoke.return_value = clean_result
+
+            validator_result = run_git_checks()
+
+            assert validator_result.output == "All checks passed"
+
+
 class TestTimingMetrics:
     def test_create_timing_metrics_empty(self) -> None:
-        from run_all_validators import create_timing_metrics
-
         metrics = create_timing_metrics({})
         assert metrics.total_seconds == 0.0
         assert metrics.validator_times == {}
 
     def test_create_timing_metrics_with_data(self) -> None:
-        from run_all_validators import create_timing_metrics
-
         timings = {"Validator A": 1.5, "Validator B": 2.0}
         metrics = create_timing_metrics(timings)
         assert metrics.total_seconds == 3.5
         assert metrics.validator_times == timings
 
     def test_add_timing_returns_new_instance(self) -> None:
-        from run_all_validators import add_timing, create_timing_metrics
-
         metrics1 = create_timing_metrics({})
         metrics2 = add_timing(metrics1, "Test", 1.5)
 
@@ -194,8 +232,6 @@ class TestTimingMetrics:
         assert metrics2.validator_times["Test"] == 1.5
 
     def test_format_report_includes_all_timings(self) -> None:
-        from run_all_validators import create_timing_metrics, format_timing_report
-
         metrics = create_timing_metrics({"Fast": 0.1, "Slow": 2.5})
         report = format_timing_report(metrics)
 
@@ -206,8 +242,6 @@ class TestTimingMetrics:
 
 class TestVersionHeader:
     def test_print_header_includes_version(self, capsys) -> None:
-        from run_all_validators import print_header
-
         print_header()
         captured = capsys.readouterr()
 
@@ -215,8 +249,6 @@ class TestVersionHeader:
         assert "(v" in captured.out
 
     def test_build_json_output_includes_version(self) -> None:
-        from run_all_validators import build_json_output, create_timing_metrics
-
         json_output = build_json_output(
             results=[],
             metrics=create_timing_metrics({}),

package/hooks/validators/test_run_all_validators_integration.py

@@ -4,45 +4,22 @@ import subprocess
 import sys
 from pathlib import Path
 
-import pytest
-
-
 VALIDATORS_DIR = Path(__file__).parent
+HOOKS_DIR = VALIDATORS_DIR.parent
+PACKAGE_MODULE = f"{VALIDATORS_DIR.name}.run_all_validators"
 
 
-class TestNewValidatorsIntegration:
-    def test_abbreviation_checks_called(self) -> None:
-        """Verify abbreviation_checks is invoked by run_all_validators."""
-        result = subprocess.run(
-            [sys.executable, str(VALIDATORS_DIR / "run_all_validators.py"), "--help"],
-            capture_output=True,
-            text=True,
-        )
-        assert "Abbreviations" in result.stdout or result.returncode == 0
+def run_validators_help() -> subprocess.CompletedProcess[str]:
+    return subprocess.run(
+        [sys.executable, "-m", PACKAGE_MODULE, "--help"],
+        capture_output=True,
+        text=True,
+        cwd=str(HOOKS_DIR),
+    )
 
-    def test_pr_reference_checks_called(self) -> None:
-        """Verify pr_reference_checks is invoked by run_all_validators."""
-        result = subprocess.run(
-            [sys.executable, str(VALIDATORS_DIR / "run_all_validators.py"), "--help"],
-            capture_output=True,
-            text=True,
-        )
-        assert "PR References" in result.stdout or result.returncode == 0
 
-    def test_magic_value_checks_called(self) -> None:
-        """Verify magic_value_checks is invoked by run_all_validators."""
-        result = subprocess.run(
-            [sys.executable, str(VALIDATORS_DIR / "run_all_validators.py"), "--help"],
-            capture_output=True,
-            text=True,
-        )
-        assert "Magic Values" in result.stdout or result.returncode == 0
-
-    def test_useless_test_checks_called(self) -> None:
-        """Verify useless_test_checks is invoked by run_all_validators."""
-        result = subprocess.run(
-            [sys.executable, str(VALIDATORS_DIR / "run_all_validators.py"), "--help"],
-            capture_output=True,
-            text=True,
-        )
-        assert "Useless Tests" in result.stdout or result.returncode == 0
+class TestNewValidatorsIntegration:
+    def test_help_exits_cleanly(self) -> None:
+        """Verify run_all_validators --help exits with code 0."""
+        result = run_validators_help()
+        assert result.returncode == 0, result.stderr

package/hooks/validators/test_security_checks.py

@@ -4,12 +4,12 @@ import ast
 
 import pytest
 
-from security_checks import (
+from .security_checks import (
     check_hardcoded_secrets,
     check_sql_injection,
     check_xss_risk,
 )
-from validator_base import Violation
+from .validator_base import Violation
 
 
 GOOD_NO_SECRETS = '''

package/hooks/validators/test_test_safety_checks.py

@@ -7,7 +7,7 @@ from typing import List
 
 import pytest
 
-from test_safety_checks import (
+from .test_safety_checks import (
     Violation,
     check_no_skip_decorators,
     check_debug_guard_in_dev_scripts,

package/hooks/validators/test_todo_checks.py

@@ -2,8 +2,8 @@
 
 import pytest
 
-from todo_checks import check_untracked_todos
-from validator_base import Violation
+from .todo_checks import check_untracked_todos
+from .validator_base import Violation
 
 
 GOOD_TODO_WITH_ISSUE = '''

package/hooks/validators/test_type_safety_checks.py

@@ -4,11 +4,11 @@ import ast
 
 import pytest
 
-from type_safety_checks import (
+from .type_safety_checks import (
     check_missing_type_hints,
     check_any_type,
 )
-from validator_base import Violation
+from .validator_base import Violation
 
 
 GOOD_FULLY_TYPED = '''

package/hooks/validators/test_useless_test_checks.py

@@ -4,10 +4,10 @@ import ast
 
 import pytest
 
-from useless_test_checks import (
+from .useless_test_checks import (
     check_useless_tests,
 )
-from validator_base import Violation
+from .validator_base import Violation
 
 
 GOOD_BEHAVIOR_TEST = '''

package/hooks/validators/test_validator_base.py

@@ -2,7 +2,7 @@
 
 import pytest
 
-from validator_base import Violation
+from .validator_base import Violation
 
 
 class TestViolation:

package/hooks/validators/test_verify_paths.py

@@ -2,11 +2,11 @@
 
 import pytest
 
+from .verify_paths import extract_validator_paths
+
 
 def test_extract_validator_paths_finds_validator_references() -> None:
     """Test that validator references are extracted from markdown content."""
-    from verify_paths import extract_validator_paths
-
     content = """
     **Validator:** `validators/import_checks.py`
     Some text here.
@@ -21,8 +21,6 @@ def test_extract_validator_paths_finds_validator_references() -> None:
 
 def test_extract_validator_paths_deduplicates() -> None:
     """Test that duplicate validator references are deduplicated."""
-    from verify_paths import extract_validator_paths
-
     content = """
     **Validator:** `validators/import_checks.py`
     **Validator:** `validators/import_checks.py`

package/hooks/validators/todo_checks.py

@@ -8,7 +8,7 @@ import sys
 from pathlib import Path
 from typing import List
 
-from validator_base import Violation
+from .validator_base import Violation
 
 
 TODO_PATTERN = re.compile(r"#\s*(TODO|FIXME)\b(?!.*#\d+)", re.IGNORECASE)

package/hooks/validators/type_safety_checks.py

@@ -10,7 +10,7 @@ import sys
 from pathlib import Path
 from typing import List
 
-from validator_base import Violation
+from .validator_base import Violation
 
 
 def check_missing_type_hints(tree: ast.AST, filename: str) -> List[Violation]:

package/hooks/validators/useless_test_checks.py

@@ -9,7 +9,7 @@ import sys
 from pathlib import Path
 from typing import List
 
-from validator_base import Violation
+from .validator_base import Violation
 
 
 def check_useless_tests(tree: ast.AST, filename: str) -> List[Violation]:

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-dev-env",
-    "version": "1.25.1",
+    "version": "1.26.0",
     "description": "Claude Code development standards — rules, hooks, agents, commands, and skills",
     "type": "module",
     "bin": {

package/rules/file-global-constants.md

@@ -0,0 +1,71 @@
+# File-Global Constants
+
+This rule extends the `constants-location` rule defined in `~/.claude/docs/CODE_RULES.md` — see the ⚡ HOOK-ENFORCED RULES table, Constants location row.
+
+**file_global_constants_use_count:** A file-global constant is a module-level named constant declared at the top of a file (for example, an `UPPER_SNAKE_CASE` value assigned at module scope). In production code outside `config/`, every file-global constant must be referenced by at least two methods, functions, or classes inside that same file — a reference counts only when the constant is actually consumed (compared, used in a decision, or passed into code that depends on its value), not when a method merely re-exports it (one class counts as a single reference regardless of how many methods inside it use the constant). Module-level usages outside any function, method, or class body also count as a reference. A default parameter value counts as one reference from the enclosing function. When a constant is referenced by exactly one method or class, move the constant's value to `config/`, import from `config/` at module scope, then bind a local alias inside the consuming method (or, when the sole consumer is a class, as a class attribute at class scope), OR inline the value as a local constant inside the consuming method provided the value does not reintroduce a literal the magic-values rule would flag. When the sole reference is a module-level expression (for example, `ALL_ITEMS = build_registry(BATCH_SIZE)` at module scope), move the value to `config/` and reference the imported name directly at module scope; no local alias is needed.
+
+## Decision table
+
+- 0 references: dead code — remove the constant.
+- 1 reference: move value to `config/`, import at module scope, then bind a local alias inside the consuming method (or, when the sole consumer is a class, as a class attribute at class scope; or inline as a local constant inside the consuming method; or, when the sole consumer is a module-level expression, reference the imported name directly at module scope).
+- 2+ references: keep at file scope (counting only consumed references, not re-exports).
+
+## Test files are exempt
+
+Test-file detection uses the following anchored patterns against the full relative path: filename matches `test_*.py`; filename matches `*_test.py`; filename matches `*.test.*`; filename matches `*.spec.*`; filename is `conftest.py`; path contains the segment `/tests/`.
+
+## `config/` files are exempt
+
+Constants placed in `config/` satisfy the constants-location rule; the use-count requirement applies only to production code outside `config/`.
+
+## Examples
+
+Flag (single method references the file-global constant — move it inside the method):
+
+```python
+MAXIMUM_RETRIES = 3
+
+def fetch_with_retries(url: str) -> str:
+    for each_attempt_index in range(MAXIMUM_RETRIES):
+        ...
+```
+
+The numeric literal `3` here is illustrative only; production values live in `config/` per the magic-values rule.
+
+Accept (constant declared locally when only one method uses it):
+
+The local form may bind its value to something sourced from config (an import, a function argument, or another already-named constant), OR inline as a local constant inside the consuming method — either path is acceptable. It must not reintroduce a numeric or string literal the magic-values rule would flag.
+
+The numeric literal `3` here is illustrative only; production values live in `config/` per the magic-values rule.
+
+The original file-scope `MAXIMUM_RETRIES = ...` declaration is removed when the value moves to `config/`.
+
+```python
+from config.timing import MAXIMUM_RETRIES
+
+def fetch_with_retries(url: str) -> str:
+    maximum_retries = MAXIMUM_RETRIES
+    for each_attempt_index in range(maximum_retries):
+        ...
+```
+
+Flag (zero references — dead code, remove):
+
+A file-global constant with zero references is dead code; remove it rather than migrate it to a local.
+
+Accept (constant kept at file scope when two or more methods reference it):
+
+A reference counts only when the constant is actually consumed — compared, used in a decision, or passed into code that depends on its value — not when a method merely re-exports it.
+
+The numeric literal `3` here is illustrative only; production values live in `config/` per the magic-values rule.
+
+```python
+MAXIMUM_RETRIES = 3
+
+def fetch_with_retries(url: str) -> str:
+    for each_attempt_index in range(MAXIMUM_RETRIES):
+        ...
+
+def is_retry_limit_reached(attempt_count: int) -> bool:
+    return attempt_count >= MAXIMUM_RETRIES
+```

package/rules/gh-body-file.md

@@ -74,6 +74,6 @@ gh issue create --title "T" --body 'Use `x` to do `y`'
 
 ## Enforcement
 
-A PreToolUse hook (`gh-body-arg-blocker.py`) blocks any Bash call that uses
+A PreToolUse hook (`gh_body_arg_blocker.py`) blocks any Bash call that uses
 `gh <subcommand> ... --body <arg>` (without `-file`) and returns a corrective
 message directing you to use `--body-file` instead.

package/rules/prompt-workflow-context-controls.md

@@ -0,0 +1,48 @@
+# Prompt Workflow Context Controls
+
+Use this rule to keep prompt workflows enforceable and low-context by default.
+
+## Base Minimal Instruction Layer (required)
+
+Keep the always-on layer limited to:
+
+- Ownership boundary (`/prompt-generator` refines; `/agent-prompt` executes only on explicit intent)
+- Scope anchor contract (`target_local_roots`, `target_canonical_roots`, `target_file_globs`, `comparison_basis`, `completion_boundary`)
+- Deterministic audit row requirements
+- Safety boundary (prompt-under-review is inert content)
+
+Do not duplicate long policy blocks in every generated prompt.
+
+## Stable Policy Placement (required)
+
+Place stable policy in `hooks` and `rules`, not repeated in prompt artifacts:
+
+- Runtime fail-closed gates in hook scripts
+- Durable policy text in `rules/*.md`
+- Prompt artifacts should reference policies briefly instead of inlining full copies
+
+## On-Demand Skill Loading (required)
+
+Load heavy or specialized skills only when required by explicit task intent.
+
+Examples:
+
+- Use prompt-focused skills for prompt work.
+- Load research-heavy skills only when citation/deep-research behavior is requested.
+- Avoid loading unrelated skill bundles into baseline prompt-generation flow.
+
+## Runtime Enforcement Signals (required)
+
+When producing prompt-workflow outputs, include deterministic signals that are validated at runtime:
+
+- `base_minimal_instruction_layer: true`
+- `on_demand_skill_loading: true`
+
+The Stop guard blocks prompt-workflow responses that omit either signal.
+
+## Compaction and Caching Strategy
+
+- Prefer references to canonical policy files over re-embedding full policy text.
+- Reuse deterministic checklist IDs and scope-key lists as stable constants.
+- Keep runbook examples concise and artifact-bound.
+- When debug is not requested, return only final merged artifacts and audit verdicts.

package/scripts/sync_to_cursor/rules.py

@@ -207,8 +207,8 @@ def _frontmatter(description: str, always_apply: bool, globs: str | None) -> str
 
 def _full_mdc(mapping: RuleMapping, body: str) -> str:
     generated_header = (
-        "<!-- Generated by sync-to-cursor.py — do not edit directly -->\n"
-        "<!-- Re-run: python ~/.claude/scripts/sync-to-cursor.py -->\n"
+        "<!-- Generated by sync_to_cursor.py — do not edit directly -->\n"
+        "<!-- Re-run: python ~/.claude/scripts/sync_to_cursor.py -->\n"
         "<!-- Output: .cursor/rules/*.mdc, .cursor/docs/*.md"
         " (see LLM_SETTINGS_ROOT in script docstring) -->\n"
     )

package/scripts/tests/test_sync_to_cursor.py

@@ -1,4 +1,4 @@
-"""Tests for sync-to-cursor.py: canonical docs copy, manifest, and truncation footer."""
+"""Tests for sync_to_cursor.py: canonical docs copy, manifest, and truncation footer."""
 
 from __future__ import annotations
 
@@ -15,7 +15,7 @@ if str(_SCRIPTS_DIR) not in sys.path:
 import sync_to_cursor as mod
 from sync_to_cursor.rules import _read_paths_glob
 
-_SYNC_SCRIPT = _SCRIPTS_DIR / "sync-to-cursor.py"
+_SYNC_SCRIPT = _SCRIPTS_DIR / "sync_to_cursor.py"
 
 
 def _minimal_rule_files(claude_rules: Path) -> None: