claude-dev-env 8.54.4 → 8.54.5

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "claude-dev-env",
-    "version": "8.54.4",
+    "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",