claude-dev-env 8.53.2 → 8.53.3

package/package.json

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

package/scripts/account_broker.py

@@ -54,6 +54,7 @@ from dev_env_scripts_constants.account_broker_constants import (
 from dev_env_scripts_constants.codex_account_constants import (
     CODEX_HOME_ENVIRONMENT_VARIABLE,
     MAIN_CODEX_HOME_DIRECTORY_NAME,
+    NO_ROSTER_ACCOUNT_NAME,
     TIER_NORMAL,
 )
 
@@ -78,7 +79,7 @@ def choose_from_readings(
     """
     if product is Product.CODEX and not all_readings:
         home = Path(os.environ.get(CODEX_HOME_ENVIRONMENT_VARIABLE) or Path.home() / MAIN_CODEX_HOME_DIRECTORY_NAME).resolve()
-        return Decision("run", Account(product, "default", home, True), None, "no roster is configured", TIER_NORMAL)
+        return Decision("run", Account(product, NO_ROSTER_ACCOUNT_NAME, home, True), None, "no roster is configured", TIER_NORMAL)
     all_available = [each_reading for each_reading in all_readings if each_reading.account not in all_spent_accounts]
     selected = (
         _choose_claude(all_available, preferred_command)
@@ -366,7 +367,8 @@ def _parse_spent_mark(all_accounts_by_name: Mapping[str, Reading], mark: str, pr
         raise BrokerConfigurationError(f"invalid reset for account {name}") from error
     reading = all_accounts_by_name.get(name)
     if reading is None:
-        print(f"warning: spent mark names {name}, an account outside the roster; choose waits until its reset", file=sys.stderr)
+        if all_accounts_by_name or name != NO_ROSTER_ACCOUNT_NAME:
+            print(f"warning: spent mark names {name}, an account outside the roster; choose waits until its reset", file=sys.stderr)
         return Reading(Account(product, name, Path(), False), None), reset
     return reading, reset
 

package/scripts/dev_env_scripts_constants/codex_account_constants.py

@@ -66,6 +66,9 @@ CODEX_PROFILES_ROOT_ENVIRONMENT_VARIABLE: str = "CODEX_PROFILES_ROOT"
 MAIN_CODEX_HOME_DIRECTORY_NAME: str = ".codex"
 """Directory under the user home that holds the shared Codex setup."""
 
+NO_ROSTER_ACCOUNT_NAME: str = "default"
+"""Account name the broker gives the shared Codex home when no roster is configured."""
+
 CODEX_HOME_ENVIRONMENT_VARIABLE: str = "CODEX_HOME"
 """Environment variable that points Codex at one account's home."""
 

package/scripts/test_account_broker.py

@@ -1026,6 +1026,38 @@ def test_should_warn_when_a_spent_mark_names_an_account_outside_the_roster(
     assert "outside the roster" in all_warning_lines[0]
 
 
+def test_should_not_warn_when_the_default_account_is_marked_without_a_roster(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    monkeypatch.setitem(account_broker.all_product_adapters, Product.CODEX, _adapter((), {}))
+    _freeze_clock(monkeypatch, NOW)
+    reset = NOW + timedelta(hours=2)
+
+    account_broker.main(("choose", "--product", "codex", "--spent", f"default:{int(reset.timestamp())}"))
+
+    assert capsys.readouterr().err == ""
+
+
+@pytest.mark.parametrize(
+    ("all_roster_names", "marked_name"),
+    [(("first",), "default"), ((), "visitor")],
+)
+def test_should_warn_when_a_spent_mark_names_an_account_the_roster_lacks(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], all_roster_names: tuple[str, ...], marked_name: str
+) -> None:
+    all_accounts = tuple(_account(each_name) for each_name in all_roster_names)
+    monkeypatch.setitem(account_broker.all_product_adapters, Product.CODEX, _adapter(all_accounts, {each_name: _meters(80, 80) for each_name in all_roster_names}))
+    _freeze_clock(monkeypatch, NOW)
+    reset = NOW + timedelta(hours=2)
+
+    account_broker.main(("choose", "--product", "codex", "--spent", f"{marked_name}:{int(reset.timestamp())}"))
+
+    all_warning_lines = capsys.readouterr().err.splitlines()
+    assert len(all_warning_lines) == 1
+    assert marked_name in all_warning_lines[0]
+    assert "outside the roster" in all_warning_lines[0]
+
+
 @pytest.mark.parametrize("mark", ["", ":1800000000"])
 def test_should_reject_a_spent_mark_without_an_account_name(
     monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], mark: str