claude-dev-env 2.11.0 → 2.13.0

package/hooks/git-hooks/test_pre_push.py

@@ -17,8 +17,6 @@ SCRIPT_DIRECTORY = Path(__file__).resolve().parent
 REAL_RUN_GIT_TEXT_COMMAND = pre_push.run_git_text_command
 
 
-CODE_REVIEW_ENFORCEMENT_ENV_VAR: str = "CLAUDE_CODE_REVIEW_ENFORCEMENT"
-ENFORCEMENT_CONSTANTS_MODULE_NAME: str = "config.code_review_enforcement_constants"
 ALL_ZEROS_OBJECT_NAME: str = "0" * 40
 NON_ZERO_LOCAL_SHA: str = "a" * 40
 NON_ZERO_REMOTE_SHA_ONE: str = "1" * 40
@@ -111,7 +109,6 @@ def _write_gate_that_allows_every_push(
     passing_gate_path = tmp_path / "code_rules_gate.py"
     passing_gate_path.write_text("import sys\nsys.exit(0)\n", encoding="utf-8")
     monkeypatch.setenv("CODE_RULES_GATE_PATH", str(passing_gate_path))
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.setattr(sys, "stdin", io.StringIO(NEW_BRANCH_PUSH_STDIN))
 
 
@@ -244,11 +241,6 @@ def resolve_remote_head_reference(monkeypatch: pytest.MonkeyPatch) -> None:
     monkeypatch.setattr(pre_push, "run_git_text_command", fake_run_git_text_command)
 
 
-def _isolate_code_review_gate(tmp_path: Path, monkeypatch: pytest.MonkeyPatch) -> None:
-    monkeypatch.setenv(
-        "CODE_REVIEW_PUSH_GATE_PATH", str(tmp_path / "no_code_review_gate.py")
-    )
-
 
 def test_resolve_base_reference_from_stdin_uses_remote_object_when_non_zero() -> None:
     stdin_text = (
@@ -295,7 +287,6 @@ def test_main_exits_zero_when_gate_script_missing(
         "CODE_RULES_GATE_PATH",
         str(tmp_path / "does_not_exist.py"),
     )
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.setattr(sys, "stdin", io.StringIO(""))
 
     exit_code = pre_push.main()
@@ -317,7 +308,6 @@ def test_main_invokes_gate_with_resolved_base_reference(
         encoding="utf-8",
     )
     monkeypatch.setenv("CODE_RULES_GATE_PATH", str(recording_gate_script_path))
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     _isolate_from_default_branch_refs(tmp_path, monkeypatch)
     remote_sha = "9" * 40
     monkeypatch.setattr(
@@ -345,7 +335,6 @@ def test_main_passes_the_pushed_remote_name_to_base_resolution(
     # No default-branch ref so the gate base is the stdin remote object, not a
     # merge-base skip; usable-base resolution then sees the pushed remote name.
     _isolate_repository_and_write_passing_code_rules_gate(tmp_path, monkeypatch)
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     all_recorded_remote_names: list[str] = []
 
     def recording_resolve_usable_base_reference(
@@ -566,7 +555,6 @@ def test_invoke_gate_uses_resolved_path(
     symlink_gate_path.symlink_to(real_gate_path)
     resolved_path = symlink_gate_path.resolve()
     monkeypatch.setenv("CODE_RULES_GATE_PATH", str(symlink_gate_path))
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     _isolate_from_default_branch_refs(tmp_path, monkeypatch)
     monkeypatch.setattr(sys, "stdin", io.StringIO(
         f"refs/heads/feature {NON_ZERO_LOCAL_SHA} refs/heads/feature {NON_ZERO_REMOTE_SHA_ONE}\n"
@@ -684,7 +672,6 @@ def test_main_allows_main_onto_main_push(
     passing_gate_path = tmp_path / "gate.py"
     passing_gate_path.write_text("import sys\nsys.exit(0)\n", encoding="utf-8")
     monkeypatch.setenv("CODE_RULES_GATE_PATH", str(passing_gate_path))
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.setattr(
         sys,
         "stdin",
@@ -698,8 +685,6 @@ def test_main_allows_main_onto_main_push(
     assert exit_code == 0
 
 
-CODE_REVIEW_STUB_BLOCK_REASON: str = "CODE_REVIEW_STUB_BLOCK_REASON"
-
 
 def _write_passing_code_rules_gate(
     tmp_path: Path, monkeypatch: pytest.MonkeyPatch
@@ -725,83 +710,6 @@ def _isolate_repository_and_write_passing_code_rules_gate(
     _write_passing_code_rules_gate(tmp_path, monkeypatch)
 
 
-def _write_code_review_gate_stub(tmp_path: Path, returned_reason_literal: str) -> Path:
-    stub_gate_path = tmp_path / "code_review_push_gate.py"
-    stub_gate_path.write_text(
-        "def deny_reason_for_directory(target_directory):\n"
-        f"    return {returned_reason_literal}\n",
-        encoding="utf-8",
-    )
-    return stub_gate_path
-
-
-def test_main_blocks_when_code_review_gate_returns_reason(
-    tmp_path: Path,
-    monkeypatch: pytest.MonkeyPatch,
-    capsys: pytest.CaptureFixture[str],
-) -> None:
-    _isolate_repository_and_write_passing_code_rules_gate(tmp_path, monkeypatch)
-    stub_gate_path = _write_code_review_gate_stub(
-        tmp_path, repr(CODE_REVIEW_STUB_BLOCK_REASON)
-    )
-    monkeypatch.setenv("CODE_REVIEW_PUSH_GATE_PATH", str(stub_gate_path))
-
-    exit_code = pre_push.main()
-
-    assert exit_code == git_hooks_constants.CODE_REVIEW_STAMP_BLOCK_EXIT_CODE
-    captured = capsys.readouterr()
-    assert CODE_REVIEW_STUB_BLOCK_REASON in captured.err
-
-
-def test_main_allows_when_code_review_gate_returns_none(
-    tmp_path: Path,
-    monkeypatch: pytest.MonkeyPatch,
-) -> None:
-    _isolate_repository_and_write_passing_code_rules_gate(tmp_path, monkeypatch)
-    stub_gate_path = _write_code_review_gate_stub(tmp_path, "None")
-    monkeypatch.setenv("CODE_REVIEW_PUSH_GATE_PATH", str(stub_gate_path))
-
-    exit_code = pre_push.main()
-
-    assert exit_code == 0
-
-
-def test_main_code_review_check_fails_open_when_gate_module_absent(
-    tmp_path: Path,
-    monkeypatch: pytest.MonkeyPatch,
-) -> None:
-    _isolate_repository_and_write_passing_code_rules_gate(tmp_path, monkeypatch)
-    monkeypatch.setenv(
-        "CODE_REVIEW_PUSH_GATE_PATH", str(tmp_path / "does_not_exist.py")
-    )
-
-    exit_code = pre_push.main()
-
-    assert exit_code == 0
-
-
-def test_main_allows_deletion_push_even_when_code_review_gate_would_block(
-    tmp_path: Path,
-    monkeypatch: pytest.MonkeyPatch,
-) -> None:
-    passing_gate_path = tmp_path / "code_rules_gate.py"
-    passing_gate_path.write_text("import sys\nsys.exit(0)\n", encoding="utf-8")
-    monkeypatch.setenv("CODE_RULES_GATE_PATH", str(passing_gate_path))
-    stub_gate_path = _write_code_review_gate_stub(
-        tmp_path, repr(CODE_REVIEW_STUB_BLOCK_REASON)
-    )
-    monkeypatch.setenv("CODE_REVIEW_PUSH_GATE_PATH", str(stub_gate_path))
-    deletion_stdin = (
-        f"refs/heads/feature {ALL_ZEROS_OBJECT_NAME} "
-        f"refs/heads/feature {ALL_ZEROS_OBJECT_NAME}\n"
-    )
-    monkeypatch.setattr(sys, "stdin", io.StringIO(deletion_stdin))
-
-    exit_code = pre_push.main()
-
-    assert exit_code == 0
-
-
 DEFAULT_BRANCH_NAME: str = "main"
 FEATURE_BRANCH_NAME: str = "feature/example"
 UNPROTECTED_DEFAULT_BRANCH_NAME: str = "develop"
@@ -1267,7 +1175,6 @@ def test_main_allows_the_push_when_origin_head_names_an_absent_reference(
         encoding="utf-8",
     )
     monkeypatch.setenv("CODE_RULES_GATE_PATH", str(recording_gate_script_path))
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1307,7 +1214,6 @@ def test_main_skips_the_gate_when_no_merge_base_resolves(
         encoding="utf-8",
     )
     monkeypatch.setenv("CODE_RULES_GATE_PATH", str(recording_gate_script_path))
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1342,7 +1248,6 @@ def test_main_allows_default_branch_file_the_rebase_replayed_onto(
     )
     _set_origin_head(repository, ORIGIN_DEFAULT_BRANCH_REFERENCE)
     _write_diff_scoped_gate(tmp_path, monkeypatch)
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1370,7 +1275,6 @@ def test_main_blocks_violation_the_branch_itself_adds(
     )
     _set_origin_head(repository, ORIGIN_DEFAULT_BRANCH_REFERENCE)
     _write_diff_scoped_gate(tmp_path, monkeypatch)
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1407,7 +1311,6 @@ def test_main_uses_remote_object_when_no_default_branch_reference_resolves(
         encoding="utf-8",
     )
     monkeypatch.setenv("CODE_RULES_GATE_PATH", str(recording_gate_script_path))
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1446,7 +1349,6 @@ def test_main_uses_remote_object_when_the_default_branch_is_pushed(
     )
     _set_origin_head(repository, ORIGIN_DEFAULT_BRANCH_REFERENCE)
     _write_diff_scoped_gate(tmp_path, monkeypatch)
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1481,7 +1383,6 @@ def test_main_uses_merge_base_when_a_default_named_local_branch_updates_a_topic_
     )
     _set_origin_head(repository, ORIGIN_UNPROTECTED_DEFAULT_BRANCH_REFERENCE)
     _write_diff_scoped_gate(tmp_path, monkeypatch)
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1517,7 +1418,6 @@ def test_main_uses_remote_object_when_a_renamed_local_branch_updates_the_default
     )
     _set_origin_head(repository, ORIGIN_UNPROTECTED_DEFAULT_BRANCH_REFERENCE)
     _write_diff_scoped_gate(tmp_path, monkeypatch)
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1547,7 +1447,6 @@ def test_main_reads_default_branch_tracking_reference_when_origin_head_is_unset(
         branch_file_name=BRANCH_FILE_NAME,
     )
     _write_diff_scoped_gate(tmp_path, monkeypatch)
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     monkeypatch.setattr(
         sys,
@@ -1575,7 +1474,6 @@ def test_main_skips_a_deletion_line_to_reach_the_branch_update(
     )
     _set_origin_head(repository, ORIGIN_DEFAULT_BRANCH_REFERENCE)
     _write_diff_scoped_gate(tmp_path, monkeypatch)
-    _isolate_code_review_gate(tmp_path, monkeypatch)
     monkeypatch.chdir(repository)
     deletion_line = (
         f"(delete) {ALL_ZEROS_OBJECT_NAME} refs/heads/retired {NON_ZERO_REMOTE_SHA_ONE}\n"
@@ -1594,31 +1492,3 @@ def test_main_skips_a_deletion_line_to_reach_the_branch_update(
     exit_code = pre_push.main()
 
     assert exit_code == 0
-
-
-def test_main_allows_when_code_review_enforcement_flag_is_off(
-    tmp_path: Path,
-    monkeypatch: pytest.MonkeyPatch,
-) -> None:
-    """Native pre-push honors the real gate with enforcement off.
-
-    Points at the production push gate (not a reason stub). The gate resolves
-    its flag when its config module first executes, so this test clears the
-    environment variable and evicts that one module if a sibling suite cached
-    it earlier in the same session. The gate then returns no deny reason and
-    the backstop allows the push.
-    """
-    monkeypatch.delenv(CODE_REVIEW_ENFORCEMENT_ENV_VAR, raising=False)
-    monkeypatch.delitem(sys.modules, ENFORCEMENT_CONSTANTS_MODULE_NAME, raising=False)
-    _isolate_repository_and_write_passing_code_rules_gate(tmp_path, monkeypatch)
-    real_gate_path = (
-        Path(__file__).resolve().parent.parent
-        / "blocking"
-        / "code_review_push_gate.py"
-    )
-    assert real_gate_path.is_file()
-    monkeypatch.setenv("CODE_REVIEW_PUSH_GATE_PATH", str(real_gate_path))
-
-    exit_code = pre_push.main()
-
-    assert exit_code == 0

package/hooks/hooks.json

@@ -42,16 +42,6 @@
           }
         ]
       },
-      {
-        "matcher": "mcp__plugin_github_github__create_pull_request",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "python3 ${CLAUDE_PLUGIN_ROOT}/hooks/blocking/code_review_pr_create_gate.py",
-            "timeout": 20
-          }
-        ]
-      },
       {
         "matcher": "mcp__plugin_github_github__.*",
         "hooks": [
@@ -67,24 +57,9 @@
           }
         ]
       },
-      {
-        "matcher": "Agent",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "python3 ${CLAUDE_PLUGIN_ROOT}/hooks/blocking/pr_converge_bugteam_enforcer.py",
-            "timeout": 10
-          }
-        ]
-      },
       {
         "matcher": "Agent|Task",
         "hooks": [
-          {
-            "type": "command",
-            "command": "python3 ${CLAUDE_PLUGIN_ROOT}/hooks/blocking/code_verifier_spawn_preflight_gate.py",
-            "timeout": 30
-          },
           {
             "type": "command",
             "command": "python3 ${CLAUDE_PLUGIN_ROOT}/hooks/blocking/fable_spawn_gate.py",
@@ -102,16 +77,6 @@
           }
         ]
       },
-      {
-        "matcher": "Skill",
-        "hooks": [
-          {
-            "type": "command",
-            "command": "python3 ${CLAUDE_PLUGIN_ROOT}/hooks/lifecycle/pr_converge_bugteam_skill_tracker.py",
-            "timeout": 10
-          }
-        ]
-      },
       {
         "matcher": "AskUserQuestion",
         "hooks": [

package/hooks/hooks_constants/CLAUDE.md

@@ -18,10 +18,7 @@ Shared constant modules imported by hooks throughout the `hooks/` tree. Each fil
 | `enter_worktree_prefetch_constants.py` | Tool name, origin-remote ref names, and git-command timeouts for the EnterWorktree origin-prefetch hook |
 | `env_var_table_code_drift_constants.py` | Table patterns, env-var-name and code-file recognizers, scan budget, and block-message text for the env-var-table code-drift blocker |
 | `code_rules_path_utils_constants.py` | Path-matching helpers used by the code-rules check modules |
-| `code_verifier_spawn_preflight_gate_constants.py` | Subagent type, merge-tree command flags, timeouts, and deny-message text for the code-verifier spawn pre-flight gate |
 | `command_dispatch_constants.py` | Command-word regex, command-key access pattern, tokenization pattern, and anchors for the unanchored command-dispatch meta-gate |
-| `convergence_branch_constants.py` | Branch and worktree naming patterns for the convergence gate |
-| `convergence_gate_blocker_constants.py` | The full-PR-URL and `--repo`/`-R` flag parse patterns that key the convergence gate to the PR its command names |
 | `conventional_pr_title_gate_constants.py` | Bash tool name, gh executable basenames and pr create/edit subcommand tokens, title/repo flag names, semantic-title-CI workflow markers, the Conventional Commits type list and title pattern, the semantic-pull-request action `types:` input pattern, and block-message text for the conventional-PR-title gate |
 | `dead_argparse_argument_constants.py` | Patterns for detecting unused argparse arguments |
 | `dead_config_field_constants.py` | Patterns for detecting unused `*Config` / `*Selectors` dataclass fields |
@@ -59,10 +56,6 @@ Shared constant modules imported by hooks throughout the `hooks/` tree. Each fil
 | `piped_pytest_blocker_constants.py` | Bash tool name, pytest program basenames and the python-interpreter basename pattern, the pipe and segment-reset operator token sets, and the deny message for the piped-pytest blocker; classification that consumes those program and wrapper constants lives in `pytest_invocation.py` |
 | `pytest_invocation.py` | Pure parsed-segment classifier for pytest runs through interpreters, shims, wrappers, and shell string-exec forms; no pipeline deny decision |
 | `plain_language_blocker_constants.py` | The list of heavy words and their everyday replacements |
-| `pr_converge_bugteam_enforcer_constants.py` | State keys and timing config for the bugteam-parallel enforcer |
-| `pr_converge_bugteam_enforcer_state.py` | State-file helpers for the bugteam enforcer |
-| `pr_description_enforcer_constants.py` | PR-description shape rules and command patterns |
-| `pr_description_proof_of_work_constants.py` | Proof-part detection patterns, gh command tokens, and corrective messages for the proof-of-work comment audit |
 | `prose_matcher_precision_constants.py` | Sample floor, keep/narrow precision floors, matcher ids, label decisions, advisory log path, emit caps, and context fingerprint length for OP-07B prose-matcher advisory telemetry |
 | `pre_tool_use_stdin.py` | `read_hook_input_dictionary_from_stdin()` — shared stdin parser for PreToolUse hooks |
 | `precommit_code_rules_gate_constants.py` | Scope argument and exit-code constants for the precommit gate |
@@ -70,7 +63,6 @@ Shared constant modules imported by hooks throughout the `hooks/` tree. Each fil
 | `pyproject_config_discovery_constants.py` | Table names (``tool`` key, ``mypy``, ``ruff``) for resolving a validator's config from a pyproject.toml `[tool.<name>]` table |
 | `pytest_testpaths_orphan_blocker_constants.py` | Marker filename, section and key names, test-file pattern, search budget, and block-message text for the pytest unregistered-test-directory blocker |
 | `python_style_checks_constants.py` | Command-line argument count and blank-line count between top-level functions for the style validator |
-| `reviewer_spawn_gate_constants.py` | Bash tool name, the sentinel marker, the Copilot and Bugbot trigger markers, the availability-script relative path and override env-var name, and the deny-message template for the reviewer-spawn gate |
 | `ruff_integration_constants.py` | ``NO_COLOR`` / ``FORCE_COLOR`` environment variable names for plain ruff diagnostics |
 | `send_user_file_open_locally_blocker_constants.py` | Tool name, proactive status, and the block message for the open-locally attach blocker |
 | `sensitive_file_protector_constants.py` | Sensitive filename patterns, the committed-template suffixes that earn an exemption, the write/edit tool names, and the deny decision and message template for `sensitive_file_protector` |
@@ -110,4 +102,3 @@ Shared constant modules imported by hooks throughout the `hooks/` tree. Each fil
 - Tests for these modules live beside them as `test_<module>.py`. Run with `python -m pytest hooks_constants/test_<name>.py`.
 - `dynamic_stderr_handler.py`, `pre_tool_use_stdin.py`, `multi_edit_reconstruction.py`, `hosted_hook_runner.py`, and `text_stripping.py` are utility modules (not pure constants) but live here because they hold logic shared across hook surfaces.
 - `local_identity.py` is a loader: it reads the environment or `~/.claude/local-identity.json` (its path overridable via `CLAUDE_LOCAL_IDENTITY_PATH`) to resolve private NAS values (with committed placeholder defaults), the PII commit-scan exempt-repo slug set (`CLAUDE_PII_EXEMPT_REPOS` / `pii_exempt_repositories`), and the per-repository allowlisted-values mapping (`pii_allowlisted_values`).
-

package/hooks/hooks_constants/bash_pre_tool_use_dispatcher_constants.py

@@ -67,21 +67,10 @@ ALL_BASH_HOSTED_HOOK_ENTRIES: tuple[BashHostedHookEntry, ...] = (
         "blocking/pii_prevention_blocker.py", ALL_BASH_AND_POWERSHELL_TOOL_NAMES
     ),
     BashHostedHookEntry("blocking/conventional_pr_title_gate.py", ALL_BASH_ONLY_TOOL_NAMES),
-    BashHostedHookEntry("blocking/reviewer_spawn_gate.py", ALL_BASH_ONLY_TOOL_NAMES),
     BashHostedHookEntry("blocking/block_main_commit.py", ALL_BASH_ONLY_TOOL_NAMES),
     BashHostedHookEntry("blocking/precommit_code_rules_gate.py", ALL_BASH_ONLY_TOOL_NAMES),
     BashHostedHookEntry("blocking/session_edit_stage_gate.py", ALL_BASH_ONLY_TOOL_NAMES),
-    BashHostedHookEntry("blocking/pr_description_enforcer.py", ALL_BASH_ONLY_TOOL_NAMES),
     BashHostedHookEntry("blocking/test_preflight_check.py", ALL_BASH_ONLY_TOOL_NAMES),
-    BashHostedHookEntry("blocking/convergence_gate_blocker.py", ALL_BASH_ONLY_TOOL_NAMES),
     BashHostedHookEntry("blocking/windows_rmtree_blocker.py", ALL_BASH_ONLY_TOOL_NAMES),
     BashHostedHookEntry("blocking/gh_pr_author_enforcer.py", ALL_BASH_ONLY_TOOL_NAMES),
-    BashHostedHookEntry("blocking/code_review_push_gate.py", ALL_BASH_AND_POWERSHELL_TOOL_NAMES),
-    BashHostedHookEntry(
-        "blocking/code_review_pr_create_gate.py", ALL_BASH_AND_POWERSHELL_TOOL_NAMES
-    ),
-    BashHostedHookEntry(
-        "blocking/code_review_stamp_directory_write_blocker.py",
-        ALL_BASH_AND_POWERSHELL_TOOL_NAMES,
-    ),
 )

package/hooks/hooks_constants/convergence_branch_constants.py

@@ -1,7 +1,6 @@
 """Convergence branch naming policy constants.
 
 Shared by destructive_command_blocker convergence-branch exemptions
-and convergence_gate_blocker pre-flight checks.
 """
 
 ALL_CONVERGENCE_BRANCH_PREFIXES: tuple[str, ...] = ("claude/", "worktree-")

package/hooks/hooks_constants/fable_spawn_gate_constants.py

@@ -11,16 +11,15 @@ into a retry prompt authorizes nothing.
     claude-sonnet-4 -> ['claude', 'sonnet', '4']  ok:   another tier
 
 Every literal the gate body reads lives here. ``AGENT_TOOL_NAME`` and
-``TASK_TOOL_NAME`` come from the sibling spawn-gate constants, so the spawn
-gates share one source for both tool names.
+``TASK_TOOL_NAME`` name the spawn tools the gate evaluates.
 """
 
 from __future__ import annotations
 
 import re
 
-from hooks_constants.code_verifier_spawn_preflight_gate_constants import TASK_TOOL_NAME
-from hooks_constants.pr_converge_bugteam_enforcer_constants import AGENT_TOOL_NAME
+AGENT_TOOL_NAME: str = "Agent"
+TASK_TOOL_NAME: str = "Task"
 
 ALL_SPAWN_TOOL_NAMES: frozenset[str] = frozenset({AGENT_TOOL_NAME, TASK_TOOL_NAME})
 

package/hooks/hooks_constants/pre_tool_use_dispatcher_constants.py

@@ -86,10 +86,6 @@ ALL_HOSTED_HOOK_ENTRIES: tuple[HostedHookEntry, ...] = (
         script_relative_path="blocking/sensitive_file_protector.py",
         applicable_tool_names=ALL_WRITE_AND_EDIT_TOOL_NAMES,
     ),
-    HostedHookEntry(
-        script_relative_path="blocking/code_review_stamp_directory_write_blocker.py",
-        applicable_tool_names=ALL_WRITE_EDIT_MULTI_EDIT_TOOL_NAMES,
-    ),
     HostedHookEntry(
         script_relative_path="blocking/pii_prevention_blocker.py",
         applicable_tool_names=ALL_WRITE_EDIT_MULTI_EDIT_TOOL_NAMES,
@@ -131,10 +127,6 @@ ALL_HOSTED_HOOK_ENTRIES: tuple[HostedHookEntry, ...] = (
         script_relative_path="blocking/hook_prose_detector_consistency.py",
         applicable_tool_names=ALL_WRITE_AND_EDIT_TOOL_NAMES,
     ),
-    HostedHookEntry(
-        script_relative_path="blocking/verified_commit_message_accuracy_blocker.py",
-        applicable_tool_names=ALL_WRITE_AND_EDIT_TOOL_NAMES,
-    ),
     HostedHookEntry(
         script_relative_path="blocking/workflow_substitution_slot_blocker.py",
         applicable_tool_names=ALL_WRITE_EDIT_MULTI_EDIT_TOOL_NAMES,

package/hooks/hooks_constants/send_user_file_open_locally_blocker_constants.py

@@ -7,12 +7,7 @@ PROACTIVE_STATUS: str = "proactive"
 CORRECTIVE_MESSAGE: str = (
     "BLOCKED [open-locally]: SendUserFile attaches a file to the session, which "
     "does not let the user see it while they are at the terminal. Open the file on "
-    "screen in its own viewer:\n"
-    "  Start-Process pwsh -WindowStyle Hidden -ArgumentList "
-    "'-NoProfile','-File',\"$HOME\\.claude\\scripts\\Show-Asset.ps1\","
-    "'<path 1>','<path 2>'\n"
-    "Show-Asset.ps1 sizes each image window to the image and opens every other file "
-    "type in its default app. Pass every path the user named.\n"
+    "screen with its native Windows app. Open every path the user named.\n"
     "The one allowed attach is a phone push: when the user has stepped away and you "
     'want the file to reach their phone, call SendUserFile with status "proactive".'
 )

package/hooks/hooks_constants/stop_dispatcher_constants.py

@@ -23,5 +23,4 @@ ALL_STOP_HOSTED_HOOK_PATHS: tuple[str, ...] = (
     "blocking/intent_only_ending_blocker.py",
     "blocking/session_handoff_blocker.py",
     "blocking/eli11_reply_enforcer.py",
-    "diagnostic/hook_log_stop_wrapper.py",
 )

package/hooks/hooks_constants/test_bash_pre_tool_use_dispatcher_constants.py

@@ -32,26 +32,17 @@ _EXPECTED_BASH_ORDER = (
     "blocking/volatile_path_in_post_blocker.py",
     "blocking/pii_prevention_blocker.py",
     "blocking/conventional_pr_title_gate.py",
-    "blocking/reviewer_spawn_gate.py",
     "blocking/block_main_commit.py",
     "blocking/precommit_code_rules_gate.py",
     "blocking/session_edit_stage_gate.py",
-    "blocking/pr_description_enforcer.py",
     "blocking/test_preflight_check.py",
-    "blocking/convergence_gate_blocker.py",
     "blocking/windows_rmtree_blocker.py",
     "blocking/gh_pr_author_enforcer.py",
-    "blocking/code_review_push_gate.py",
-    "blocking/code_review_pr_create_gate.py",
-    "blocking/code_review_stamp_directory_write_blocker.py",
 )
 
 _POWERSHELL_APPLICABLE = (
     "blocking/unscoped_search_blocker.py",
     "blocking/pii_prevention_blocker.py",
-    "blocking/code_review_push_gate.py",
-    "blocking/code_review_pr_create_gate.py",
-    "blocking/code_review_stamp_directory_write_blocker.py",
 )
 
 
@@ -70,7 +61,7 @@ def test_every_hook_applies_to_the_bash_tool() -> None:
 
 
 def test_powershell_applicable_hooks_include_the_shared_gates() -> None:
-    """PowerShell runs the unscoped-search and PII gates plus the review family."""
+    """PowerShell runs the shared unscoped-search and PII gates."""
     powershell_hooks = tuple(
         each_entry.script_relative_path
         for each_entry in ALL_BASH_HOSTED_HOOK_ENTRIES
@@ -86,22 +77,6 @@ def test_powershell_hooks_carry_the_shared_tool_set() -> None:
             assert each_entry.applicable_tool_names == ALL_BASH_AND_POWERSHELL_TOOL_NAMES
 
 
-def test_code_review_gate_family_registered_for_bash_and_powershell() -> None:
-    """The three code-review shell gates run on both Bash and PowerShell."""
-    code_review_gate_paths = (
-        "blocking/code_review_push_gate.py",
-        "blocking/code_review_pr_create_gate.py",
-        "blocking/code_review_stamp_directory_write_blocker.py",
-    )
-    entry_by_path = {
-        each_entry.script_relative_path: each_entry
-        for each_entry in ALL_BASH_HOSTED_HOOK_ENTRIES
-    }
-    for each_gate_path in code_review_gate_paths:
-        assert each_gate_path in entry_by_path
-        assert entry_by_path[each_gate_path].applicable_tool_names == (
-            ALL_BASH_AND_POWERSHELL_TOOL_NAMES
-        )
 
 
 def test_every_roster_path_points_at_an_existing_hook() -> None:

package/hooks/hooks_constants/test_pre_tool_use_dispatcher_constants.py

@@ -63,11 +63,6 @@ def test_duplicate_rmtree_helper_blocker_runs_via_runpy() -> None:
     assert entry.native_module_name is None
 
 
-def test_code_review_stamp_write_blocker_registered_for_all_file_tools() -> None:
-    entry = _entry_for("blocking/code_review_stamp_directory_write_blocker.py")
-    assert entry is not None
-    assert {"Write", "Edit", "MultiEdit"} <= entry.applicable_tool_names
-
 
 def test_windows_rmtree_blocker_still_registered() -> None:
     entry = _entry_for("blocking/windows_rmtree_blocker.py")

package/hooks/hooks_constants/test_stop_dispatcher_constants.py

@@ -17,7 +17,6 @@ _EXPECTED_STOP_ORDER = (
     "blocking/intent_only_ending_blocker.py",
     "blocking/session_handoff_blocker.py",
     "blocking/eli11_reply_enforcer.py",
-    "diagnostic/hook_log_stop_wrapper.py",
 )
 
 

package/hooks/lifecycle/CLAUDE.md

@@ -8,13 +8,11 @@ Hooks that run at session or config-change boundaries rather than on individual
 |---|---|---|
 | `config_change_guard.py` | PostToolUse (Write/Edit on `settings.json`) | Counts hooks in the edited `settings.json` and logs any change to `~/.claude/cache/config-change-audit.log`; alerts when the hook count drops below the last known value |
 | `enter_worktree_origin_prefetch.py` | PreToolUse (EnterWorktree) | Fetches origin's default branch before a worktree creation call, keeping the `refs/remotes/origin/<default-branch>` ref that `fresh` mode reads current; never blocks on fetch failure |
-| `pr_converge_bugteam_skill_tracker.py` | PreToolUse (Skill) | Tracks which bugteam skill runs have completed within a pr-converge loop, so the enforcer can verify parallel execution |
 | `session_end_cleanup.py` | SessionEnd | Purges stale cache entries from `~/.claude/cache/` (entries older than the configured threshold) and old backup files |
 | `test_config_change_guard.py` | — | Tests for `config_change_guard.py` |
 | `test_enter_worktree_origin_prefetch.py` | — | Tests for `enter_worktree_origin_prefetch.py` |
-| `test_pr_converge_bugteam_skill_tracker.py` | — | Tests for `pr_converge_bugteam_skill_tracker.py` |
 
 ## Conventions
 
-- Constants for these hooks (stale-age threshold, cache directory, known-hook-count file, EnterWorktree prefetch tuning) live in `hooks_constants/session_env_cleanup_constants.py`, `hooks_constants/pr_converge_bugteam_enforcer_constants.py`, and `hooks_constants/enter_worktree_prefetch_constants.py`.
+- Constants for these hooks (stale-age threshold, cache directory, known-hook-count file, EnterWorktree prefetch tuning) live in `hooks_constants/session_env_cleanup_constants.py` and `hooks_constants/enter_worktree_prefetch_constants.py`.
 - Tests run with `python -m pytest lifecycle/test_<name>.py`.

package/package.json

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

package/rules/CLAUDE.md

@@ -45,13 +45,11 @@ Rule files installed into `~/.claude/rules/` by `bin/install.mjs`. A rule withou
 | `plain-illustrative-docstrings.md` | Public docstring narrative reads plainly and shows behavior with a diagram block (a `::` example or a doctest), painting a concrete scene a general developer follows on first read; a run-on backstop hook, a prose-wall backstop hook, and Category O9 audit enforce it |
 | `plain-language.md` | Everyday words, short active sentences, lead with the answer |
 | `prompt-workflow-context-controls.md` | Keep prompt-workflow instruction layers small and stable; load heavy skills on demand |
-| `proof-of-work-pr-comments.md` | Every PR carries one five-part proof-of-work comment before it leaves draft; the `pr_description_enforcer` hook audits proof-shaped comments and gates `gh pr ready` |
 | `re-stage-before-commit.md` | Stage the files edited this session before `git commit`; the session edit stage gate denies a commit that leaves a tracked session edit unstaged, with `-a`, a pathspec, a preceding `git add`, and `# partial-commit` as escapes |
 | `research-mode.md` | Three anti-hallucination constraints: say "I don't know", verify with citations, quote for factual grounding |
 | `shell-invocation.md` | Windows shell commands run through `pwsh`; no `$(...)`, backtick, or process substitution in a Bash tool command |
 | `testing.md` | Test quality and infrastructure standards |
 | `vault-context.md` | Search Obsidian vault for prior sessions and decisions before substantive project work |
-| `verified-commit-gate-skip.md` | Code-verifier review is optional; commits and pushes use the checks selected for the task |
 | `verify-before-asking.md` | Answer questions by inspecting files or running tools before asking; recalled facts expire until re-checked this session |
 | `verify-runtime-state.md` | A "component is fine / not at fault" verdict rests on a live probe this session, never code reading or prior-session memory |
 | `windows-filesystem-safe.md` | Use safe `rmtree` patterns on Windows; `mkdirSync` with `recursive: true` on possibly-existing paths |

package/rules/gh-cli-conventions.md

@@ -24,4 +24,3 @@ Single-object endpoints (`pulls/<n>`, `issues/<n>`) skip pagination and may use
 
 - [`destructive-commands.md`](destructive-commands.md) — why a body describing `rm -rf` must travel by file path.
 - [`durable-post-artifacts.md`](durable-post-artifacts.md) — what a post body may reference.
-- [`proof-of-work-pr-comments.md`](proof-of-work-pr-comments.md) — what the proof comment must contain.

package/rules/git-workflow.md

@@ -41,7 +41,7 @@ User-level rule: applies to **every** git repo that uses GitHub with `gh` (no ex
 | `SESSION_STATE.md` | Local session state |
 | `*.png *.jpg *.jpeg *.gif *.webp *.avif *.svg *.ico` | Images go to external storage, not GitHub |
 
-An image a PR needs as visual evidence is not an exception to that row. Upload it to the repository's durable `artifacts` release with `python3 ~/.claude/scripts/gh_artifact_upload.py <file> <owner/repo>` and embed the permanent URL in the proof comment. The image lives on GitHub without entering the repository tree.
+An image a PR needs as visual evidence is not an exception to that row. Upload it to the repository's durable `artifacts` release with `python3 ~/.claude/scripts/gh_artifact_upload.py <file> <owner/repo>` and embed the permanent URL in the PR comment. The image lives on GitHub without entering the repository tree.
 
 ## Responding to review feedback
 
@@ -60,9 +60,7 @@ Repair only reported findings, then re-verify after every repair.
 | Rule | Covers |
 |---|---|
 | [`gh-cli-conventions.md`](gh-cli-conventions.md) | `--body-file` for post bodies; `--paginate --slurp` for list reads |
-| [`proof-of-work-pr-comments.md`](proof-of-work-pr-comments.md) | The five-part proof comment every PR carries before leaving draft |
 | [`re-stage-before-commit.md`](re-stage-before-commit.md) | Staging session edits so a commit does not drop them |
-| [`verified-commit-gate-skip.md`](verified-commit-gate-skip.md) | Optional code-verifier review guidance |
 | [`durable-post-artifacts.md`](durable-post-artifacts.md) | Keeping volatile scratch paths out of a post body |
 | [`destructive-commands.md`](destructive-commands.md) | Allowed removal forms; destructive literals in commit and post bodies |
 | [`code-standards.md`](code-standards.md) | The code standards a PR's diff is reviewed against |

package/scripts/CLAUDE.md

@@ -28,7 +28,6 @@ Utility scripts installed into `~/.claude/scripts/` by `bin/install.mjs`. Each s
 | `Install-SweepEmptyDirs.ps1` | Registers `sweep_empty_dirs.py` as a scheduled task on Windows |
 | `Sync-RepoMain.ps1` | Fast-forwards a main-tracking mirror (`-RepoPath`, required) to its remote main using `merge --ff-only`; refuses a different checkout or a diverged branch, logs each run to `-LogPath`, and exits non-zero so a scheduled-task history row surfaces the problem. `-StashDirty` stashes tracked changes right before the fast-forward under a dated `pre-sync stash` message and leaves the stash in the list |
 | `check.ps1` | Runs the full code-quality check suite |
-| `Show-Asset.ps1` | Opens files on screen, sizing each image window to the image's pixel dimensions (scaled to fit the screen); non-image files open in their default application |
 | `Get-SessionAccount.ps1` | Reports which Claude account the current session is actually logged into by comparing `~/.claude.json`'s CLI login against a `CLAUDE_USER_DATA_DIR` desktop profile's `lastKnownAccountUuid`, recovering the desktop account's email from profile storage when the two accounts differ |
 | `Capture-PoolHealth.ps1` | Captures Windows memory pool counters, high-handle processes, and kernel pool tags (via `NtQuerySystemInformation` class 22), prints a threshold verdict with a remediation map, and exits non-zero when any alert threshold fires |