claude-dev-env 4.2.0 → 7.0.0

package/hooks/blocking/code_rules_paired_test.py

@@ -215,12 +215,27 @@ def _collect_referenced_identifiers(source: str) -> set[str]:
     return all_referenced_identifiers
 
 
+def _sibling_test_paths(stem_test_path: Path) -> list[Path]:
+    """Return the capped, sorted test files beside the stem test, stem excluded."""
+    suite_directory = stem_test_path.parent
+    stem_path_key = os.path.normcase(str(stem_test_path.resolve()))
+    all_sibling_paths: list[Path] = []
+    for each_glob in ALL_TEST_FILENAME_GLOBS:
+        all_sibling_paths.extend(
+            each_path
+            for each_path in sorted(suite_directory.glob(each_glob))
+            if os.path.normcase(str(each_path.resolve())) != stem_path_key
+        )
+    return all_sibling_paths[:MAX_TEST_FILES_SCANNED]
+
+
 def _suite_referenced_identifiers(stem_test_path: Path) -> set[str]:
     """Return identifiers referenced across every test file beside the stem test.
 
-    Scans every ``test_*.py`` and ``*_test.py`` file in the directory holding the
-    stem-matched test file, bounded by the scan cap, and unions the identifiers
-    each one references.
+    Reads the stem-matched test file first, then scans every other ``test_*.py``
+    and ``*_test.py`` file in its directory, bounded by the scan cap, and unions
+    the identifiers each one references. The stem test always counts, so a
+    directory with more test files than the cap cannot hide it.
 
     Args:
         stem_test_path: The stem-matched test file whose directory is scanned.
@@ -228,12 +243,9 @@ def _suite_referenced_identifiers(stem_test_path: Path) -> set[str]:
     Returns:
         The union of referenced identifiers across the scanned test files.
     """
-    suite_directory = stem_test_path.parent
-    all_test_file_paths: list[Path] = []
-    for each_glob in ALL_TEST_FILENAME_GLOBS:
-        all_test_file_paths.extend(sorted(suite_directory.glob(each_glob)))
+    all_test_file_paths = [stem_test_path, *_sibling_test_paths(stem_test_path)]
     all_referenced_identifiers: set[str] = set()
-    for each_test_file_path in all_test_file_paths[:MAX_TEST_FILES_SCANNED]:
+    for each_test_file_path in all_test_file_paths:
         try:
             test_source = each_test_file_path.read_text(encoding="utf-8")
         except (OSError, UnicodeDecodeError):
@@ -364,16 +376,12 @@ def _production_stem_from_test_filename(test_filename: str) -> str | None:
     if test_filename.startswith(STEM_TEST_FILENAME_PREFIX) and test_filename.endswith(
         PYTHON_SOURCE_SUFFIX
     ):
-        production_stem = test_filename[
-            len(STEM_TEST_FILENAME_PREFIX) : -len(PYTHON_SOURCE_SUFFIX)
-        ]
+        production_stem = test_filename[len(STEM_TEST_FILENAME_PREFIX) : -len(PYTHON_SOURCE_SUFFIX)]
         return production_stem or None
     return None
 
 
-def _paired_production_module_path(
-    test_path: Path, production_stem: str
-) -> Path | None:
+def _paired_production_module_path(test_path: Path, production_stem: str) -> Path | None:
     """Return the production module a stem-matched test file pairs with, or None.
 
     Mirrors ``_stem_matched_test_path`` in reverse for the two common layouts: a
@@ -515,4 +523,3 @@ def check_test_file_omits_module_public_function(
         if len(all_omission_violations) >= MAX_PAIRED_TEST_COVERAGE_ISSUES:
             break
     return all_omission_violations
-

package/hooks/blocking/pii_prevention_blocker.py

@@ -1,23 +1,18 @@
 #!/usr/bin/env python3
 """PreToolUse hook: block writes, durable posts, and commits that carry PII.
 
-Surfaces guarded:
-
-- Write / Edit / MultiEdit — new content about to land on disk
-- Bash / PowerShell ``gh`` post subcommands and GitHub MCP post tools — durable
-  bodies
-- Bash / PowerShell ``git commit`` (including ``git.exe`` and flag forms) —
-  staged file contents about to become history. Commit message bodies are out
-  of scope; only staged blob text is scanned.
-
-Payload and post-body scanning live in ``pii_payload_scan``; git-commit command
-detection lives in ``pii_commit_command``; both re-export through this entry
-module so the hooks.json wiring and every importer resolve unchanged.
-
-A repository named in ``CLAUDE_PII_EXEMPT_REPOS`` or the ``pii_exempt_repositories``
-list in ``~/.claude/local-identity.json`` skips the staged-commit scan; a
-repository without a readable origin remote is never exempt. Exact values listed
-under ``pii_allowlisted_values`` for a repository's origin slug may pass the
+::
+
+    Write/Edit/MultiEdit  -> scan new disk content
+    gh post / GitHub MCP  -> scan durable bodies
+    git commit            -> scan staged blob text, not the commit message
+
+``pii_payload_scan`` and ``pii_commit_command`` re-export through this
+entry module so hooks.json wiring and importers stay unchanged.
+``CLAUDE_PII_EXEMPT_REPOS`` or ``pii_exempt_repositories`` in
+``~/.claude/local-identity.json`` skip the staged-commit scan. A repo
+without a readable origin remote is never exempt.
+``pii_allowlisted_values`` for a repository origin slug may pass
 Write/Edit and staged-commit scans in that repository only.
 """
 
@@ -56,7 +51,7 @@ try:
     )
     from pii_scanner import is_path_exempt_from_pii_scan, scan_text_for_pii
     from precommit_code_rules_gate import resolve_repository_root
-    from volatile_path_in_post_blocker import (
+    from gh_post_body_texts import (
         extract_gh_post_body_texts_for_privacy_gate,
         extract_mcp_body_texts,
     )
@@ -124,7 +119,8 @@ def list_staged_file_paths(
     try:
         completed_process = subprocess.run(
             list(ALL_STAGED_FILES_COMMAND),
-            check=False, capture_output=True,
+            check=False,
+            capture_output=True,
             text=True,
             timeout=GIT_COMMAND_TIMEOUT_SECONDS,
             cwd=str(repository_root),
@@ -148,7 +144,8 @@ def _git_show_staged_blob(
     try:
         completed_process = subprocess.run(
             list(ALL_STAGED_BLOB_SHOW_COMMAND_PREFIX) + [staged_blob_reference],
-            check=False, capture_output=True,
+            check=False,
+            capture_output=True,
             timeout=GIT_COMMAND_TIMEOUT_SECONDS,
             cwd=str(repository_root),
         )
@@ -197,9 +194,7 @@ def _scan_staged_path(
     repository_root: Path, relative_path: str, all_allowlisted_values: frozenset[str]
 ) -> str | None:
     """Return a deny reason for one staged path, or None when it is clean."""
-    staged_text, unscannable_reason = read_staged_file_text(
-        repository_root, relative_path
-    )
+    staged_text, unscannable_reason = read_staged_file_text(repository_root, relative_path)
     if unscannable_reason is not None:
         return unscannable_reason
     if staged_text is None:
@@ -217,9 +212,9 @@ def _scan_staged_path(
 def evaluate_staged_commit(repository_root: Path) -> str | None:
     """Return a deny reason when staged content carries PII or is unscannable.
 
-    Fail-closed: git list/show failures and unscannable blobs deny the commit
-    rather than treating unread content as clean. A value in the repository's
-    PII allowlist is dropped from the findings, so its commits may carry it.
+    Git list or show failures and unscannable blobs deny the commit. A value
+    in the repository's PII allowlist is dropped from the findings, so its
+    commits may carry it.
 
     Args:
         repository_root: Repository whose index is about to be committed.
@@ -234,22 +229,18 @@ def evaluate_staged_commit(repository_root: Path) -> str | None:
     for each_relative_path in all_relative_paths:
         if is_path_exempt_from_pii_scan(each_relative_path):
             continue
-        deny_reason = _scan_staged_path(
-            repository_root, each_relative_path, all_allowlisted_values
-        )
+        deny_reason = _scan_staged_path(repository_root, each_relative_path, all_allowlisted_values)
         if deny_reason is not None:
             return deny_reason
     return None
 
 
-def _evaluate_commit_staged_content(
-    bash_command: str, working_directory: str | None
-) -> str | None:
+def _evaluate_commit_staged_content(bash_command: str, working_directory: str | None) -> str | None:
     """Scan the staged content of the repository the commit command targets.
 
-    Resolves the repository from the command's ``-C``/``cd`` path, falling back
-    to the session working directory only when the command names none, so a
-    removed cwd never blocks a commit that targets a valid repo.
+    Resolves the repository from the command's ``-C`` or ``cd`` path. Falls
+    back to the session working directory only when the command names none. A
+    removed cwd does not block a commit that targets a valid repo.
 
     Args:
         bash_command: The Bash or PowerShell tool command string.
@@ -259,9 +250,7 @@ def _evaluate_commit_staged_content(
         Deny reason text, or None when the staged content is clean or exempt.
     """
     command_directory = compose_command_working_directory(bash_command)
-    attempted_directory = (
-        command_directory if command_directory is not None else working_directory
-    )
+    attempted_directory = command_directory if command_directory is not None else working_directory
     expanded_directory = expand_user_directory(attempted_directory)
     repository_root = resolve_repository_root(expanded_directory)
     if repository_root is None:
@@ -271,9 +260,7 @@ def _evaluate_commit_staged_content(
     return evaluate_staged_commit(repository_root)
 
 
-def evaluate_bash_command(
-    bash_command: str, working_directory: str | None
-) -> str | None:
+def evaluate_bash_command(bash_command: str, working_directory: str | None) -> str | None:
     """Return a deny reason for a shell gh post or git commit with PII.
 
     Args:
@@ -284,10 +271,8 @@ def evaluate_bash_command(
     Returns:
         Deny reason text, or None when the command is clean or out of scope.
     """
-    all_post_bodies, body_file_failure_reason = (
-        extract_gh_post_body_texts_for_privacy_gate(
-            bash_command, working_directory=working_directory
-        )
+    all_post_bodies, body_file_failure_reason = extract_gh_post_body_texts_for_privacy_gate(
+        bash_command, working_directory=working_directory
     )
     if body_file_failure_reason is not None:
         return body_file_failure_reason
@@ -308,9 +293,7 @@ def _evaluate_shell_tool(
         return None
     working_directory_value = all_tool_input.get("working_directory")
     working_directory = (
-        working_directory_value
-        if isinstance(working_directory_value, str)
-        else None
+        working_directory_value if isinstance(working_directory_value, str) else None
     )
     if working_directory is None:
         cwd_value = payload_by_key.get("cwd")
@@ -332,9 +315,7 @@ def evaluate(payload_by_key: dict[str, object]) -> str | None:
     raw_tool_input = payload_by_key.get("tool_input", {})
     all_tool_input = raw_tool_input if isinstance(raw_tool_input, dict) else {}
     if tool_name in ALL_WRITE_EDIT_MULTI_EDIT_TOOL_NAMES:
-        return evaluate_write_edit_payload(
-            tool_name, all_tool_input, hook_payload=payload_by_key
-        )
+        return evaluate_write_edit_payload(tool_name, all_tool_input, hook_payload=payload_by_key)
     if tool_name == APPLY_PATCH_TOOL_NAME:
         return evaluate_apply_patch_payload(all_tool_input, hook_payload=payload_by_key)
     if tool_name in ALL_SHELL_TOOL_NAMES:

package/hooks/blocking/test_bash_post_call_dispatcher.py

@@ -1,7 +1,7 @@
 """Behavior tests for the Bash PostToolUse dispatcher.
 
 One interpreter start runs every hosted PostToolUse hook a Bash call fires.
-The unit tests pin selection, call order, and crash isolation with a fake
+The unit tests pin selection and call order with a fake
 runner; the end-to-end test drives the real hosted hooks through a real
 payload and confirms the recorder's own real side effect lands.
 ``hooks/conftest.py`` already puts the hooks and blocking directories on
@@ -15,19 +15,26 @@ from pathlib import Path
 
 import pytest
 
-from bash_post_call_dispatcher import dispatch, main, select_applicable_entries
+from bash_post_call_dispatcher import (
+    additional_context_from_hook_output,
+    dispatch,
+    main,
+    select_applicable_entries,
+)
 from hooks_constants.bash_pre_tool_use_dispatcher_constants import BASH_TOOL_NAME
 from hooks_constants.hosted_hook_runner import HostedHookRun
 from tdd_enforcer_parts import content_hash_store
 
+_ALL_EXPECTED_ROSTER_PATHS = [
+    "observability/test_failure_recorder.py",
+    "advisory/pr_done_reminder.py",
+]
 
-def test_select_applicable_entries_returns_both_hosted_hooks_for_bash() -> None:
+
+def test_select_applicable_entries_returns_the_hosted_hooks_for_bash() -> None:
     all_entries = select_applicable_entries(BASH_TOOL_NAME)
     all_script_paths = [each_entry.script_relative_path for each_entry in all_entries]
-    assert all_script_paths == [
-        "blocking/gh_pr_author_restore.py",
-        "observability/test_failure_recorder.py",
-    ]
+    assert all_script_paths == _ALL_EXPECTED_ROSTER_PATHS
 
 
 def test_select_applicable_entries_returns_none_for_a_write_tool() -> None:
@@ -48,27 +55,58 @@ def test_dispatch_runs_every_hosted_hook_in_roster_order(
 
     dispatch('{"tool_name": "Bash"}', BASH_TOOL_NAME)
 
-    assert all_call_paths == ["gh_pr_author_restore.py", "test_failure_recorder.py"]
+    assert all_call_paths == [Path(each_path).name for each_path in _ALL_EXPECTED_ROSTER_PATHS]
 
 
-def test_dispatch_runs_the_later_hook_after_an_earlier_one_crashes(
-    monkeypatch: pytest.MonkeyPatch,
+def test_dispatch_forwards_hosted_hook_additional_context_as_one_payload(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
 ) -> None:
-    all_call_paths: list[str] = []
+    context_by_hook = {
+        "test_failure_recorder.py": "",
+        "pr_done_reminder.py": json.dumps(
+            {"hookSpecificOutput": {"hookEventName": "PostToolUse", "additionalContext": "PR #42"}}
+        ),
+    }
 
     def _fake_run_hook(script_path: str, payload_text: str) -> HostedHookRun:
         del payload_text
-        script_name = Path(script_path).name
-        all_call_paths.append(script_name)
-        if script_name == "gh_pr_author_restore.py":
-            return HostedHookRun(captured_stdout="", did_crash=True)
+        return HostedHookRun(
+            captured_stdout=context_by_hook[Path(script_path).name], did_crash=False
+        )
+
+    monkeypatch.setattr("bash_post_call_dispatcher.run_hook_capturing_output", _fake_run_hook)
+
+    dispatch('{"tool_name": "Bash"}', BASH_TOOL_NAME)
+
+    emitted = json.loads(capsys.readouterr().out)
+    assert emitted == {
+        "hookSpecificOutput": {"hookEventName": "PostToolUse", "additionalContext": "PR #42"}
+    }
+    assert "decision" not in emitted
+
+
+def test_dispatch_writes_nothing_when_no_hosted_hook_printed_context(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    def _fake_run_hook(script_path: str, payload_text: str) -> HostedHookRun:
+        del script_path, payload_text
         return HostedHookRun(captured_stdout="", did_crash=False)
 
     monkeypatch.setattr("bash_post_call_dispatcher.run_hook_capturing_output", _fake_run_hook)
 
     dispatch('{"tool_name": "Bash"}', BASH_TOOL_NAME)
 
-    assert all_call_paths == ["gh_pr_author_restore.py", "test_failure_recorder.py"]
+    assert capsys.readouterr().out == ""
+
+
+@pytest.mark.parametrize(
+    "captured_stdout",
+    ["", "   ", "not json", "[]", '{"decision": "block"}', '{"hookSpecificOutput": {}}'],
+)
+def test_additional_context_from_hook_output_ignores_non_context_output(
+    captured_stdout: str,
+) -> None:
+    assert additional_context_from_hook_output(captured_stdout) is None
 
 
 def test_main_exits_zero_and_writes_nothing_to_stdout(
@@ -85,7 +123,7 @@ def test_main_exits_zero_and_writes_nothing_to_stdout(
 
 
 def test_end_to_end_real_hosted_hooks_record_a_real_pytest_failure(tmp_path: Path) -> None:
-    """Drives the real, unfaked roster: both scripts run in one process."""
+    """Drive the real recorder through the dispatcher."""
     test_file = tmp_path / "test_orders.py"
     test_file.write_text("def test_fulfill(): assert False\n")
     session_id = "post-dispatcher-session"

package/hooks/blocking/test_bash_pre_tool_use_dispatcher.py

@@ -222,7 +222,6 @@ def test_powershell_selects_the_shared_hooks_in_registration_order() -> None:
         "blocking/cursor_cli_python_misfire_blocker.py",
         "blocking/unscoped_search_blocker.py",
         "blocking/pii_prevention_blocker.py",
-        "blocking/pr_description_writer_gate.py",
     ]
 
 
@@ -270,9 +269,7 @@ def test_dispatcher_denies_subshell_disguised_as_arithmetic_expansion() -> None:
     the live Bash tool path.
     """
     payload_text = _bash_payload("echo $((cd /tmp) && pwd)")
-    decision, _reason = _decision_from_stdout(
-        _run_process(_DISPATCHER_SCRIPT, payload_text).stdout
-    )
+    decision, _reason = _decision_from_stdout(_run_process(_DISPATCHER_SCRIPT, payload_text).stdout)
     assert decision == "deny"
 
 

package/hooks/blocking/test_code_rules_command_dispatch.py

@@ -11,7 +11,7 @@ from code_rules_command_dispatch import (  # noqa: E402
     check_unanchored_command_dispatch,
 )
 
-GATE_PATH = "packages/claude-dev-env/hooks/blocking/conventional_pr_title_gate.py"
+GATE_PATH = "packages/claude-dev-env/hooks/blocking/block_main_commit.py"
 
 UNANCHORED_SOURCE = (
     "import re\n"
@@ -84,7 +84,7 @@ def test_ignores_file_outside_hooks_blocking() -> None:
 
 
 def test_ignores_test_file() -> None:
-    test_path = "packages/claude-dev-env/hooks/blocking/test_conventional_pr_title_gate.py"
+    test_path = "packages/claude-dev-env/hooks/blocking/test_block_main_commit.py"
     assert check_unanchored_command_dispatch(UNANCHORED_SOURCE, test_path) == []
 
 

package/hooks/blocking/test_code_rules_enforcer_paired_test.py

@@ -19,6 +19,10 @@ _BLOCKING_DIRECTORY = Path(__file__).resolve().parent
 if str(_BLOCKING_DIRECTORY) not in sys.path:
     sys.path.insert(0, str(_BLOCKING_DIRECTORY))
 
+from hooks_constants.paired_test_coverage_constants import (  # noqa: E402
+    MAX_TEST_FILES_SCANNED,
+)
+
 from code_rules_paired_test import (  # noqa: E402
     check_public_function_missing_paired_test,
     check_test_file_omits_module_public_function,
@@ -26,9 +30,7 @@ from code_rules_paired_test import (  # noqa: E402
 
 _TWO_PUBLIC_FUNCTIONS = "def alpha():\n    return 1\n\n\ndef beta():\n    return 2\n"
 _THREE_PUBLIC_FUNCTIONS = (
-    "def alpha():\n    return 1\n\n\n"
-    "def beta():\n    return 2\n\n\n"
-    "def gamma():\n    return 3\n"
+    "def alpha():\n    return 1\n\n\ndef beta():\n    return 2\n\n\ndef gamma():\n    return 3\n"
 )
 
 
@@ -145,6 +147,28 @@ def test_counts_coverage_across_sibling_test_files(
     assert all_issues == []
 
 
+def test_reads_the_stem_test_when_the_suite_exceeds_the_scan_cap(
+    neutral_package_directory: Path,
+) -> None:
+    filler_body = "def test_nothing():\n    assert True\n"
+    _write_test_file(
+        neutral_package_directory,
+        "test_a0000.py",
+        "from pkg.mod import alpha\n\ndef test_alpha():\n    assert alpha() == 1\n",
+    )
+    for each_index in range(1, MAX_TEST_FILES_SCANNED + 1):
+        _write_test_file(neutral_package_directory, f"test_a{each_index:04d}.py", filler_body)
+    _write_test_file(
+        neutral_package_directory,
+        "test_mod.py",
+        "from pkg.mod import beta\n\ndef test_beta():\n    assert beta() == 2\n",
+    )
+    all_issues = check_public_function_missing_paired_test(
+        _TWO_PUBLIC_FUNCTIONS, str(neutral_package_directory / "mod.py")
+    )
+    assert all_issues == []
+
+
 def test_main_and_private_functions_are_never_required(
     neutral_package_directory: Path,
 ) -> None:
@@ -183,8 +207,7 @@ def _write_module(package_directory: Path, body: str) -> None:
 
 
 _SUITE_COVERS_ALPHA_BETA = (
-    "from pkg.mod import alpha, beta\n\n"
-    "def test_pair():\n    assert alpha() == 1 and beta() == 2\n"
+    "from pkg.mod import alpha, beta\n\ndef test_pair():\n    assert alpha() == 1 and beta() == 2\n"
 )
 
 
@@ -193,9 +216,7 @@ def test_flags_module_public_function_when_test_suite_omits_it(
 ) -> None:
     _write_module(neutral_package_directory, _THREE_PUBLIC_FUNCTIONS)
     test_path = str(neutral_package_directory / "tests" / "test_mod.py")
-    all_issues = check_test_file_omits_module_public_function(
-        _SUITE_COVERS_ALPHA_BETA, test_path
-    )
+    all_issues = check_test_file_omits_module_public_function(_SUITE_COVERS_ALPHA_BETA, test_path)
     assert len(all_issues) == 1
     assert "gamma" in all_issues[0]
     assert "mod.py" in all_issues[0]
@@ -231,9 +252,7 @@ def test_skips_test_file_with_no_paired_production_module(
     neutral_package_directory: Path,
 ) -> None:
     test_path = str(neutral_package_directory / "tests" / "test_mod.py")
-    all_issues = check_test_file_omits_module_public_function(
-        _SUITE_COVERS_ALPHA_BETA, test_path
-    )
+    all_issues = check_test_file_omits_module_public_function(_SUITE_COVERS_ALPHA_BETA, test_path)
     assert all_issues == []
 
 
@@ -242,9 +261,7 @@ def test_ignores_a_non_stem_matched_written_file(
 ) -> None:
     _write_module(neutral_package_directory, _THREE_PUBLIC_FUNCTIONS)
     helper_path = str(neutral_package_directory / "helper.py")
-    all_issues = check_test_file_omits_module_public_function(
-        _SUITE_COVERS_ALPHA_BETA, helper_path
-    )
+    all_issues = check_test_file_omits_module_public_function(_SUITE_COVERS_ALPHA_BETA, helper_path)
     assert all_issues == []
 
 
@@ -274,9 +291,7 @@ def test_judges_post_edit_content_over_stale_on_disk_test(
         "from pkg.mod import alpha, beta, gamma\n\n"
         "def test_all():\n    assert alpha() and beta() and gamma()\n"
     )
-    all_issues = check_test_file_omits_module_public_function(
-        post_edit_covers_all, test_path
-    )
+    all_issues = check_test_file_omits_module_public_function(post_edit_covers_all, test_path)
     assert all_issues == []
 
 
@@ -290,9 +305,7 @@ def test_counts_coverage_across_sibling_test_files_on_test_write(
         "from pkg.mod import gamma\n\ndef test_gamma():\n    assert gamma() == 3\n",
     )
     test_path = str(neutral_package_directory / "tests" / "test_mod.py")
-    all_issues = check_test_file_omits_module_public_function(
-        _SUITE_COVERS_ALPHA_BETA, test_path
-    )
+    all_issues = check_test_file_omits_module_public_function(_SUITE_COVERS_ALPHA_BETA, test_path)
     assert all_issues == []
 
 
@@ -301,9 +314,7 @@ def test_test_side_message_names_module_and_function_without_line_prefix(
 ) -> None:
     _write_module(neutral_package_directory, _THREE_PUBLIC_FUNCTIONS)
     test_path = str(neutral_package_directory / "tests" / "test_mod.py")
-    all_issues = check_test_file_omits_module_public_function(
-        _SUITE_COVERS_ALPHA_BETA, test_path
-    )
+    all_issues = check_test_file_omits_module_public_function(_SUITE_COVERS_ALPHA_BETA, test_path)
     assert len(all_issues) == 1
     assert not all_issues[0].startswith("Line ")
     assert "mod.py" in all_issues[0]
@@ -315,9 +326,7 @@ def test_test_side_omission_blocks_on_any_test_file_edit(
 ) -> None:
     _write_module(neutral_package_directory, _THREE_PUBLIC_FUNCTIONS)
     test_path = str(neutral_package_directory / "tests" / "test_mod.py")
-    all_issues = check_test_file_omits_module_public_function(
-        _SUITE_COVERS_ALPHA_BETA, test_path
-    )
+    all_issues = check_test_file_omits_module_public_function(_SUITE_COVERS_ALPHA_BETA, test_path)
     assert len(all_issues) == 1
     assert "gamma" in all_issues[0]
 
@@ -333,7 +342,5 @@ def test_skips_when_paired_production_module_is_exempt(
         "from pkg import alpha, beta, gamma\n\n"
         "def test_all():\n    assert alpha() and beta() and gamma()\n"
     )
-    all_issues = check_test_file_omits_module_public_function(
-        suite_covers_all, beside_test_path
-    )
+    all_issues = check_test_file_omits_module_public_function(suite_covers_all, beside_test_path)
     assert all_issues == []

package/hooks/blocking/test_code_rules_js_resume_task_enumeration.py

@@ -17,8 +17,8 @@ import pathlib
 import sys
 
 _PACKAGE_ROOT = pathlib.Path(__file__).resolve().parents[2]
-_SHIPPED_CONVERGE_MJS = (
-    _PACKAGE_ROOT / ".agents" / "skills" / "autoconverge" / "workflow" / "converge.mjs"
+_ARCHIVED_CONVERGE_MJS = (
+    _PACKAGE_ROOT.parents[1] / "skill-archive" / "autoconverge" / "workflow" / "converge.mjs"
 )
 
 _HOOK_DIRECTORY = pathlib.Path(__file__).parent
@@ -750,8 +750,8 @@ def should_still_flag_real_dispatch_alongside_task_substring_identifier() -> Non
     assert "spawnVerifierAgent" in issues[0]
 
 
-def should_not_flag_shipped_converge_workflow() -> None:
-    converge_source = _SHIPPED_CONVERGE_MJS.read_text(encoding="utf-8")
+def should_not_flag_archived_converge_regression_fixture() -> None:
+    converge_source = _ARCHIVED_CONVERGE_MJS.read_text(encoding="utf-8")
     issues = check_js_resume_task_enumeration_coverage(
         converge_source, "skills/autoconverge/workflow/converge.mjs"
     )

package/hooks/blocking/test_code_rules_js_returns_object_schemaless.py

@@ -34,8 +34,8 @@ check_js_returns_object_schemaless_branch = (
 
 _MJS_PATH = "skills/autoconverge/workflow/converge.mjs"
 
-_SHIPPED_CONVERGE_MJS = (
-    _HOOK_DIRECTORY.parents[1] / ".agents" / "skills" / "autoconverge" / "workflow" / "converge.mjs"
+_ARCHIVED_CONVERGE_MJS = (
+    _HOOK_DIRECTORY.parents[3] / "skill-archive" / "autoconverge" / "workflow" / "converge.mjs"
 )
 
 
@@ -161,7 +161,7 @@ def test_skips_python_files() -> None:
     assert issues == []
 
 
-def test_shipped_converge_mjs_passes_its_own_check() -> None:
-    shipped_source = _SHIPPED_CONVERGE_MJS.read_text(encoding="utf-8")
-    issues = check_js_returns_object_schemaless_branch(shipped_source, _MJS_PATH)
+def test_archived_converge_regression_fixture_passes_the_check() -> None:
+    archived_source = _ARCHIVED_CONVERGE_MJS.read_text(encoding="utf-8")
+    issues = check_js_returns_object_schemaless_branch(archived_source, _MJS_PATH)
     assert issues == []