claude-dev-env 8.46.22 → 8.46.24

package/package.json

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

package/scripts/account_broker.py

@@ -333,7 +333,8 @@ def _parser() -> argparse.ArgumentParser:
     commands = parser.add_subparsers(dest="action", required=True)
     for each_action in ("choose", "check", "accounts", "limits", "run"):
         command = commands.add_parser(each_action)
-        command.add_argument("--product", choices=[each_product.value for each_product in Product], required=True)
+        all_product_choices = [Product.CODEX.value] if each_action == "limits" else [each_product.value for each_product in Product]
+        command.add_argument("--product", choices=all_product_choices, required=True)
         if each_action == "choose":
             command.add_argument("--spent", action="append", default=[])
         if each_action == "limits":
@@ -373,9 +374,7 @@ def _accounts_cli(product: Product) -> int:
     return 0
 
 
-def _limits_cli(product: Product, parsed: argparse.Namespace) -> int:
-    if product is not Product.CODEX:
-        raise BrokerConfigurationError("limits reads Codex accounts only")
+def _limits_cli(parsed: argparse.Namespace) -> int:
     try:
         rate_limit_records = codex_account_meters.read_rate_limit_records(
             codex_account_meters.resolve_codex_path(parsed.codex), parsed.home
@@ -430,7 +429,7 @@ def main(all_arguments: Sequence[str]) -> int:
         if parsed.action == "accounts":
             return _accounts_cli(product)
         if parsed.action == "limits":
-            return _limits_cli(product, parsed)
+            return _limits_cli(parsed)
         if parsed.action == "run":
             return _run_cli(product, parsed, parser)
         return _choose_cli(product, parsed)

package/scripts/dev_env_scripts_constants/claude_chain_constants.py

@@ -10,5 +10,5 @@ ALL_USAGE_LIMIT_SIGNATURES: tuple[str, ...] = (
 )
 """Case-insensitive substrings that mark a non-zero exit as a usage-limit refusal."""
 
-CHAIN_CONFIG_ERROR_EXIT_CODE: int = 3
+CHAIN_CONFIG_ERROR_EXIT_CODE: int = 5
 """CLI exit code when the chain configuration is missing or invalid."""

package/scripts/dev_env_scripts_constants/grok_worker_constants.py

@@ -493,7 +493,7 @@ SPAWN_SERVED_EXIT_CODE: int = 0
 SPAWN_EXHAUSTED_EXIT_CODE: int = 2
 """CLI exit code when no dispatcher tier served the call."""
 
-SPAWN_CONFIG_ERROR_EXIT_CODE: int = 3
+SPAWN_CONFIG_ERROR_EXIT_CODE: int = 5
 """CLI exit code when the claude chain configuration is missing or invalid."""
 
 RESULT_KEY_TIER_USED: str = "tier_used"

package/scripts/resolve_worker_spawn.py

@@ -607,7 +607,7 @@ def main(all_command_arguments: list[str]) -> int:
 
     A timeout outside the accepted bounds is a config fault: the run prints a
     structured outcome whose attempt reason is ``timeout_out_of_bounds`` and
-    exits ``3``, rather than raising out of the CLI. A broker configuration error,
+    exits ``5``, rather than raising out of the CLI. A broker configuration error,
     or an unknown host profile refused by ``detect_host_profile``, prints the
     same exit code with the refusal text as its output.
 

package/scripts/test_account_broker.py

@@ -805,5 +805,7 @@ def test_should_exit_two_when_the_limits_read_fails(
 
 
 def test_should_refuse_limits_for_claude(tmp_path: Path, capsys: pytest.CaptureFixture[str]) -> None:
-    assert account_broker.main(("limits", "--product", "claude", "--home", str(tmp_path))) == 2
-    assert "Codex accounts only" in capsys.readouterr().err
+    with pytest.raises(SystemExit) as exit_info:
+        account_broker.main(("limits", "--product", "claude", "--home", str(tmp_path)))
+    assert exit_info.value.code == 2
+    assert "invalid choice: 'claude'" in capsys.readouterr().err

package/scripts/test_codex_account_meters.py

@@ -164,7 +164,6 @@ class TestReadCodexMeters:
             read_codex_meters(Path("codex"), Path("/h"), broken_exchange)
 
 
-
 class TestReadRateLimitRecords:
     def should_return_every_limit_record_the_reply_carries(self) -> None:
         rate_limit_records = {
@@ -196,6 +195,7 @@ class TestReadRateLimitRecords:
         with pytest.raises(CodexMeterUnreadError, match="no rate-limit reply"):
             read_rate_limit_records(Path("codex"), Path("/h"), lambda *_: [])
 
+
 FAKE_APP_SERVER = """#!/usr/bin/env python3
 import json, os, sys
 for line in sys.stdin:

package/scripts/test_invoke_code_review_cli.py

@@ -19,7 +19,7 @@ from _code_review_test_support import (
     install_seams,
     run_review_cli,
 )
-from dev_env_scripts_constants.account_broker_constants import BrokerConfigurationError
+from dev_env_scripts_constants.account_broker_constants import BrokerConfigurationError, WAIT_EXIT_CODE
 from dev_env_scripts_constants.claude_chain_constants import (
     CHAIN_CONFIG_ERROR_EXIT_CODE,
 )
@@ -82,6 +82,7 @@ def test_cli_emits_json_on_chain_configuration_error(
     exit_code = run_review_cli(working_directory, session_model=FIXTURE_SESSION_OPUS)
 
     assert exit_code == CHAIN_CONFIG_ERROR_EXIT_CODE
+    assert exit_code != WAIT_EXIT_CODE
     captured = capsys.readouterr()
     parsed_payload = json.loads(captured.out)
     assert parsed_payload == {

package/scripts/test_resolve_worker_spawn.py

@@ -299,12 +299,12 @@ def test_grok_usage_limited_on_claude_host_requires_agent(
 def test_out_of_bounds_timeout_is_reported_as_config_not_worker_failure(
     monkeypatch: pytest.MonkeyPatch, tmp_path: Path, capsys: pytest.CaptureFixture[str]
 ) -> None:
-    """A refused timeout prints a structured outcome and exits 3, never a traceback.
+    """A refused timeout prints a structured outcome and exits 5, never a traceback.
 
     ::
 
-        --timeout-seconds 0     ok: reason timeout_out_of_bounds, exit 3
-        --timeout-seconds 5401  ok: reason timeout_out_of_bounds, exit 3
+        --timeout-seconds 0     ok: reason timeout_out_of_bounds, exit 5
+        --timeout-seconds 5401  ok: reason timeout_out_of_bounds, exit 5
     """
     prompt_file, working_directory, run_state_directory = _paths(tmp_path)
     _install_seams(monkeypatch, grok_outcome=_grok_ok())
@@ -350,7 +350,7 @@ def test_out_of_bounds_timeout_is_refused_when_the_grok_tier_is_unreachable(
     ::
 
         preflight unusable, --timeout-seconds 5401
-            ok: reason timeout_out_of_bounds, exit 3, no tier invoked
+            ok: reason timeout_out_of_bounds, exit 5, no tier invoked
     """
     prompt_file, working_directory, run_state_directory = _paths(tmp_path)
     call_log = _install_seams(
@@ -496,7 +496,7 @@ def test_tier_three_wait_reports_reset_and_stops(
     assert call_log.claude_calls == 1
 
 
-def test_config_error_returns_exit_three(
+def test_config_error_exit_code_differs_from_the_wait_exit_code(
     monkeypatch: pytest.MonkeyPatch, tmp_path: Path, capsys: pytest.CaptureFixture[str]
 ) -> None:
     prompt_file, working_directory, run_state_directory = _paths(tmp_path)
@@ -523,6 +523,7 @@ def test_config_error_returns_exit_three(
     )
 
     assert exit_code == SPAWN_CONFIG_ERROR_EXIT_CODE
+    assert exit_code != WAIT_EXIT_CODE
     captured = capsys.readouterr()
     parsed_payload = json.loads(captured.out)
     assert parsed_payload[RESULT_KEY_OK] is False