claude-dev-env 8.46.23 → 8.46.25

package/docs/CODE_RULES.md

@@ -96,6 +96,8 @@ Open [details](code-rules/design-and-structure.md) when renaming symbols.
 
 Never swallow a failure into a default unless the caller explicitly opted in at the boundary. Name the specific exception (`except KeyError:`) and propagate the rest. Collapsing every error class to `None` masks programming errors and makes debugging impossible.
 
+A member loop catches its declared `*ItemBlocked` type inside the loop and lets a `*RunFatal` pass through, and an `except Exception` in that loop triggers this rule.
+
 Open [details](code-rules/design-and-structure.md) when handling batch failures.
 
 ## 9.8 REMOVE CODE YOU ORPHAN (Dead Code Elimination)

package/package.json

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