claude-dev-env 8.54.3 → 8.54.5

package/hooks/blocking/bash_post_call_dispatcher.py

@@ -38,6 +38,8 @@ from hooks_constants.bash_post_call_dispatcher_constants import (
     ADDITIONAL_CONTEXT_JOIN_SEPARATOR,
     ADDITIONAL_CONTEXT_KEY,
     ALL_BASH_POST_TOOL_USE_HOSTED_HOOK_ENTRIES,
+)
+from hooks_constants.hook_specific_output_keys import (
     HOOK_SPECIFIC_OUTPUT_KEY,
 )
 from hooks_constants.post_tool_use_context import write_post_tool_use_context_to_stdout

package/hooks/blocking/pr_lifecycle_skill_gate.py

@@ -34,11 +34,9 @@ from hooks_constants.pr_lifecycle_skill_gate_constants import (
     GIT_ACTIONS,
     GIT_GLOBAL_OPTIONS_WITH_VALUE,
     HOOK_EVENT_NAME,
-    HOOK_SPECIFIC_OUTPUT_KEY,
     MERGE_PATH_SUFFIX,
     OPTION_AND_VALUE_WORD_COUNT,
     PATH_SEPARATOR_PATTERN,
-    PERMISSION_DECISION_KEY,
     PERMISSION_DECISION_REASON_KEY,
     PULL_REQUEST_SCRIPT_NAME,
     PYTHON_OPTIONS_WITH_VALUE,
@@ -48,6 +46,10 @@ from hooks_constants.pr_lifecycle_skill_gate_constants import (
     SUBAGENT_TRANSCRIPT_DIRECTORY_NAME,
     SUBAGENT_TRANSCRIPT_FILE_TEMPLATE,
 )
+from hooks_constants.hook_specific_output_keys import (
+    HOOK_SPECIFIC_OUTPUT_KEY,
+    PERMISSION_DECISION_KEY,
+)
 from hooks_constants.pre_tool_use_stdin import read_hook_input_dictionary_from_stdin
 from hooks_constants.setup_project_paths_constants import DECODE_ERRORS_POLICY, UTF8_ENCODING
 from blocking.followup_pr_dedupe import duplicate_followup_reason

package/hooks/blocking/verify_before_acting.py

@@ -42,9 +42,11 @@ if hooks_root_directory not in sys.path:
     sys.path.insert(0, hooks_root_directory)
 
 from hooks_constants.bash_post_call_dispatcher_constants import (
+    POST_TOOL_USE_HOOK_EVENT_NAME,
+)
+from hooks_constants.hook_specific_output_keys import (
     HOOK_EVENT_NAME_KEY,
     HOOK_SPECIFIC_OUTPUT_KEY,
-    POST_TOOL_USE_HOOK_EVENT_NAME,
 )
 from hooks_constants.hook_block_logger import log_hook_block
 from hooks_constants.pre_tool_use_stdin import read_hook_input_dictionary_from_stdin

package/hooks/hooks_constants/bash_post_call_dispatcher_constants.py

@@ -22,8 +22,6 @@ __all__ = [
     "ALL_BASH_POST_TOOL_USE_HOSTED_HOOK_ENTRIES",
     "EXIT_CODE_ERROR_PREFIX",
     "POST_TOOL_USE_HOOK_EVENT_NAME",
-    "HOOK_SPECIFIC_OUTPUT_KEY",
-    "HOOK_EVENT_NAME_KEY",
     "ADDITIONAL_CONTEXT_KEY",
     "ADDITIONAL_CONTEXT_JOIN_SEPARATOR",
 ]
@@ -34,7 +32,5 @@ ALL_BASH_POST_TOOL_USE_HOSTED_HOOK_ENTRIES: tuple[BashHostedHookEntry, ...] = (
 
 EXIT_CODE_ERROR_PREFIX: str = "Error: Exit code "
 POST_TOOL_USE_HOOK_EVENT_NAME: str = "PostToolUse"
-HOOK_SPECIFIC_OUTPUT_KEY: str = "hookSpecificOutput"
-HOOK_EVENT_NAME_KEY: str = "hookEventName"
 ADDITIONAL_CONTEXT_KEY: str = "additionalContext"
 ADDITIONAL_CONTEXT_JOIN_SEPARATOR: str = "\n\n"

package/hooks/hooks_constants/{pre_tool_use_output_keys.py → hook_specific_output_keys.py}

@@ -1,4 +1,4 @@
-"""JSON keys of the hookSpecificOutput object a PreToolUse hook writes to stdout."""
+"""JSON keys of the hookSpecificOutput object a hook writes to stdout."""
 
 from __future__ import annotations
 

package/hooks/hooks_constants/nested_project_hooks_constants.py

@@ -29,9 +29,6 @@ ALL_SHELL_PROGRAM_NAMES: tuple[str, ...] = ("bash", "sh")
 SHELL_COMMAND_FLAG: str = "-c"
 
 BLOCKING_EXIT_CODE: int = 2
-HOOK_SPECIFIC_OUTPUT_KEY: str = "hookSpecificOutput"
-HOOK_EVENT_NAME_OUTPUT_KEY: str = "hookEventName"
-PERMISSION_DECISION_KEY: str = "permissionDecision"
 ALL_FORWARDED_PERMISSION_DECISIONS: frozenset[str] = frozenset({"deny", "ask"})
 ADDITIONAL_CONTEXT_KEY: str = "additionalContext"
 CONTEXT_SECTION_SEPARATOR: str = "\n\n"

package/hooks/hooks_constants/post_tool_use_context.py

@@ -7,9 +7,11 @@ import sys
 
 from hooks_constants.bash_post_call_dispatcher_constants import (
     ADDITIONAL_CONTEXT_KEY,
+    POST_TOOL_USE_HOOK_EVENT_NAME,
+)
+from hooks_constants.hook_specific_output_keys import (
     HOOK_EVENT_NAME_KEY,
     HOOK_SPECIFIC_OUTPUT_KEY,
-    POST_TOOL_USE_HOOK_EVENT_NAME,
 )
 
 

package/hooks/hooks_constants/pr_lifecycle_skill_gate_constants.py

@@ -1,8 +1,6 @@
 """Constants for the pull request lifecycle skill gate."""
 
 HOOK_EVENT_NAME = "PreToolUse"
-HOOK_SPECIFIC_OUTPUT_KEY = "hookSpecificOutput"
-PERMISSION_DECISION_KEY = "permissionDecision"
 PERMISSION_DECISION_REASON_KEY = "permissionDecisionReason"
 DENY_DECISION = "deny"
 DENY_REASON = (

package/hooks/hooks_constants/pre_tool_use_allow_output.py

@@ -9,7 +9,7 @@ from hooks_constants.bash_pre_tool_use_dispatcher_constants import (
     ALLOW_DECISION,
     HOOK_EVENT_NAME,
 )
-from hooks_constants.pre_tool_use_output_keys import (
+from hooks_constants.hook_specific_output_keys import (
     HOOK_EVENT_NAME_KEY,
     HOOK_SPECIFIC_OUTPUT_KEY,
     PERMISSION_DECISION_KEY,

package/hooks/hooks_constants/spawn_readiness_hook_constants.py

@@ -89,8 +89,6 @@ ALL_INTERACTIVE_MCP_ACTIONS = frozenset({"ask_decision", "post_widget"})
 SCOPE_SETTLED_PREFIX = "Scope settled:"
 
 PRE_TOOL_USE_EVENT_NAME = "PreToolUse"
-HOOK_SPECIFIC_OUTPUT_KEY = "hookSpecificOutput"
-HOOK_EVENT_NAME_KEY = "hookEventName"
 ADDITIONAL_CONTEXT_KEY = "additionalContext"
 MISSING_INVESTIGATION_REASON = (
     "This spawn comes before any read of the request. Read the files, threads, "

package/hooks/hooks_constants/test_bash_post_call_dispatcher_constants.py

@@ -24,7 +24,6 @@ def test_pr_done_reminder_also_serves_the_powershell_tool() -> None:
 
 def test_context_forwarding_keys_match_the_hook_output_contract() -> None:
     assert constants.POST_TOOL_USE_HOOK_EVENT_NAME == "PostToolUse"
-    assert constants.HOOK_SPECIFIC_OUTPUT_KEY == "hookSpecificOutput"
     assert constants.ADDITIONAL_CONTEXT_KEY == "additionalContext"
     assert constants.ADDITIONAL_CONTEXT_JOIN_SEPARATOR == "\n\n"
 

package/hooks/hooks_constants/test_hook_specific_output_keys.py

@@ -0,0 +1,17 @@
+"""Tests for the hookSpecificOutput key names."""
+
+from __future__ import annotations
+
+from hooks_constants import hook_specific_output_keys
+
+
+def test_keys_match_the_harness_hook_specific_output_field_names() -> None:
+    assert {
+        each_name: getattr(hook_specific_output_keys, each_name)
+        for each_name in hook_specific_output_keys.__all__
+    } == {
+        "HOOK_SPECIFIC_OUTPUT_KEY": "hookSpecificOutput",
+        "HOOK_EVENT_NAME_KEY": "hookEventName",
+        "PERMISSION_DECISION_KEY": "permissionDecision",
+        "UPDATED_INPUT_KEY": "updatedInput",
+    }

package/hooks/lifecycle/nested_project_hooks.py

@@ -48,13 +48,10 @@ from hooks_constants.nested_project_hooks_constants import (
     CONTEXT_SECTION_SEPARATOR,
     DEFAULT_HOOK_TIMEOUT_SECONDS,
     HOOK_EVENT_NAME_KEY,
-    HOOK_EVENT_NAME_OUTPUT_KEY,
-    HOOK_SPECIFIC_OUTPUT_KEY,
     HOOK_TYPE_KEY,
     HOOKS_KEY,
     ALL_MATCH_TARGET_KEYS_BY_EVENT,
     MATCHER_KEY,
-    PERMISSION_DECISION_KEY,
     PRE_TOOL_USE_EVENT,
     PROJECT_DIRECTORY_ENVIRONMENT_VARIABLE,
     SESSION_START_EVENT,
@@ -65,6 +62,11 @@ from hooks_constants.nested_project_hooks_constants import (
     ALL_SHELL_PROGRAM_NAMES,
     TIMEOUT_KEY,
 )
+from hooks_constants.hook_specific_output_keys import (
+    HOOK_EVENT_NAME_KEY as HOOK_EVENT_NAME_OUTPUT_KEY,
+    HOOK_SPECIFIC_OUTPUT_KEY,
+    PERMISSION_DECISION_KEY,
+)
 from hooks_constants.subprocess_window import hidden_window_creation_flags
 
 

package/hooks/routing/spawn_readiness_hook.py

@@ -48,8 +48,6 @@ from hooks_constants.spawn_readiness_hook_constants import (
     DISPATCH_METHOD_INPUT_KEY,
     DISPATCH_PROMPT_INPUT_KEY,
     DISPATCH_RUN_WORKFLOW_METHOD,
-    HOOK_EVENT_NAME_KEY,
-    HOOK_SPECIFIC_OUTPUT_KEY,
     LOG_APPEND_MODE,
     LOG_LINE_END,
     LOG_OUTCOME_KEY,
@@ -75,6 +73,10 @@ from hooks_constants.spawn_readiness_hook_constants import (
     WORKFLOW_SCRIPT_PATH_INPUT_KEY,
     WORKFLOW_TOOL_NAME,
 )
+from hooks_constants.hook_specific_output_keys import (
+    HOOK_EVENT_NAME_KEY,
+    HOOK_SPECIFIC_OUTPUT_KEY,
+)
 from spawn_readiness_steps import readiness_gaps, session_steps
 
 

package/hooks/routing/subagent_model_pin_hook.py

@@ -25,7 +25,7 @@ if hooks_root_directory not in sys.path:
     sys.path.insert(0, hooks_root_directory)
 
 from hooks_constants.bash_pre_tool_use_dispatcher_constants import HOOK_EVENT_NAME
-from hooks_constants.pre_tool_use_output_keys import (
+from hooks_constants.hook_specific_output_keys import (
     HOOK_EVENT_NAME_KEY,
     HOOK_SPECIFIC_OUTPUT_KEY,
     UPDATED_INPUT_KEY,

package/hooks/routing/thread_spawn_pace_hook.py

@@ -28,7 +28,7 @@ if hooks_root_directory not in sys.path:
     sys.path.insert(0, hooks_root_directory)
 
 from hooks_constants.bash_pre_tool_use_dispatcher_constants import HOOK_EVENT_NAME
-from hooks_constants.pre_tool_use_output_keys import (
+from hooks_constants.hook_specific_output_keys import (
     HOOK_EVENT_NAME_KEY,
     HOOK_SPECIFIC_OUTPUT_KEY,
     PERMISSION_DECISION_KEY,

package/package.json

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

package/scripts/conftest.py

@@ -15,9 +15,32 @@ is in place by the time any test module runs its top-level imports.
 
 from __future__ import annotations
 
+import importlib
 import sys
 from pathlib import Path
 
+import pytest
+
 _scripts_directory = str(Path(__file__).resolve().parent)
 if _scripts_directory not in sys.path:
     sys.path.insert(0, _scripts_directory)
+
+_live_broker_state_resolver = importlib.import_module("account_broker_support").broker_state_path
+
+
+@pytest.fixture(autouse=True)
+def isolated_broker_state_path(monkeypatch: pytest.MonkeyPatch, tmp_path: Path) -> Path:
+    """Point every module's ``broker_state_path`` at a per-test file.
+
+    ``account_broker`` binds ``broker_state_path`` by name from
+    ``account_broker_support``, so each bound copy is replaced. A test that
+    patches only one copy still keeps the other off the home directory.
+
+    Returns:
+        The isolated broker state path for this test.
+    """
+    isolated_path = tmp_path / "account-broker" / "state.json"
+    for each_module in list(sys.modules.values()):
+        if getattr(each_module, "broker_state_path", None) is _live_broker_state_resolver:
+            monkeypatch.setattr(each_module, "broker_state_path", lambda: isolated_path)
+    return isolated_path

package/scripts/test_broker_state_isolation.py

@@ -0,0 +1,48 @@
+"""Broker state written during a test stays out of the home directory."""
+
+from __future__ import annotations
+
+import subprocess
+from datetime import datetime, timezone
+from pathlib import Path
+
+import pytest
+
+import account_broker
+from dev_env_scripts_constants.account_broker_constants import (
+    Account,
+    Meters,
+    Product,
+    ProductAdapter,
+)
+
+
+def test_meter_cache_write_lands_outside_the_home_directory(
+    monkeypatch: pytest.MonkeyPatch,
+    tmp_path: Path,
+    isolated_broker_state_path: Path,
+) -> None:
+    fake_home = tmp_path / "home"
+    monkeypatch.setenv("HOME", str(fake_home))
+    monkeypatch.setenv("USERPROFILE", str(fake_home))
+    account = Account(Product.CLAUDE, "only", tmp_path / "only")
+    reset_at = datetime(2026, 10, 5, tzinfo=timezone.utc)
+    adapter = ProductAdapter(
+        lambda: (account,),
+        lambda each_account: Meters(80, reset_at, 80, reset_at),
+        "CLAUDE_CONFIG_DIR",
+        ("usage limit",),
+    )
+    monkeypatch.setitem(account_broker.all_product_adapters, Product.CLAUDE, adapter)
+
+    def runner(argv: list[str], **options: object) -> subprocess.CompletedProcess[str]:
+        return subprocess.CompletedProcess(argv, 0, "", "")
+
+    with account_broker.override_subprocess_runner(runner):
+        account_broker.run_job(Product.CLAUDE, ["claude"])
+
+    home_state_path = fake_home / ".claude" / "account-broker" / "state.json"
+    assert not home_state_path.exists()
+    assert f"claude:only:{account.home}" in isolated_broker_state_path.read_text(
+        encoding="utf-8"
+    )

package/scripts/test_resolve_worker_spawn.py

@@ -1129,7 +1129,6 @@ def test_usage_limit_fallover_delivers_full_prompt_to_each_account(
         return subprocess.CompletedProcess(argv, 0, FIXTURE_CLAUDE_STDOUT, "")
 
     monkeypatch.setitem(account_broker.all_product_adapters, Product.CLAUDE, adapter)
-    monkeypatch.setattr(account_broker, "broker_state_path", lambda: tmp_path / "state.json")
     monkeypatch.setattr(
         dispatcher,
         "spawn_preflight_runner",

package/hooks/hooks_constants/test_pre_tool_use_output_keys.py

@@ -1,17 +0,0 @@
-"""Tests for the PreToolUse hookSpecificOutput key names."""
-
-from __future__ import annotations
-
-from hooks_constants import pre_tool_use_output_keys
-
-
-def test_keys_match_the_harness_pre_tool_use_output_field_names() -> None:
-    assert {
-        each_name: getattr(pre_tool_use_output_keys, each_name)
-        for each_name in pre_tool_use_output_keys.__all__
-    } == {
-        "HOOK_SPECIFIC_OUTPUT_KEY": "hookSpecificOutput",
-        "HOOK_EVENT_NAME_KEY": "hookEventName",
-        "PERMISSION_DECISION_KEY": "permissionDecision",
-        "UPDATED_INPUT_KEY": "updatedInput",
-    }