claude-dev-env 8.48.5 → 8.48.7

package/docs/account-broker.md

@@ -27,7 +27,7 @@ The broker stores cached meters, spent marks, and Claude session bindings in one
 | `0` | `choose` or `check` found an account, or the job exited successfully. |
 | `3` | No account clears its floor. The decision includes `resets_at`. |
 | `4` | A Claude job stopped with `advisor_blocked`. |
-| `127` | A Codex job timed out, or its command could not start while no Codex roster is configured. |
+| `127` | A Codex job timed out, its command could not start while no Codex roster is configured, or the command could not start on any account in the roster. |
 | Other command code | `run` returns the command's exit code after a served response. |
 
 `run` writes a JSON report with `product`, `command`, `events`, and `final_decision`. A wait report has `final_decision.action` equal to `wait` and a UTC `final_decision.resets_at`.

package/hooks/routing/test_subagent_model_pin_hook.py

@@ -31,6 +31,7 @@ def test_should_move_the_spawn_to_opus(requested_model: object) -> None:
     if requested_model is not None:
         tool_input["model"] = requested_model
     decision = json.loads(_run_hook(tool_input))["hookSpecificOutput"]
+    assert decision["hookEventName"] == "PreToolUse"
     assert "permissionDecision" not in decision
     assert decision["updatedInput"] == {**SPAWN_INPUT, "model": "opus"}
 

package/package.json

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

package/scripts/account_broker.py

@@ -234,6 +234,9 @@ def _prepare_run(
 
 def _wait_outcome(context: _RunContext, decision: Decision) -> JobOutcome:
     context.report.final_decision = decision
+    all_attempt_statuses = {each_status for _, each_status in context.all_attempts}
+    if all_attempt_statuses == {"start_failed"}:
+        return JobOutcome(COMMAND_MISSING_EXIT_CODE, "", "", None, tuple(context.all_attempts), "start_failed", None, None)
     status = "exhausted" if context.all_attempts else "wait"
     return JobOutcome(WAIT_EXIT_CODE, "", "", None, tuple(context.all_attempts), status, None, decision.resets_at)
 

package/scripts/codex_compat_materializer.py

@@ -684,6 +684,11 @@ def _is_code_rules_enforcer_hook(all_hook_record: dict[str, object]) -> bool:
     )
 
 
+def _is_windows_host() -> bool:
+    """Report whether hook command paths compare without regard to case."""
+    return os.name == "nt"
+
+
 def _is_managed_codex_enforcer_hook(
     all_hook_record: dict[str, object], managed_command: str
 ) -> bool:
@@ -693,7 +698,7 @@ def _is_managed_codex_enforcer_hook(
         return False
     normalized_command = command.replace("\\", path_separator)
     normalized_managed_command = managed_command.replace("\\", path_separator)
-    if os.name == "nt":
+    if _is_windows_host():
         return normalized_command.casefold() == normalized_managed_command.casefold()
     return normalized_command == normalized_managed_command
 

package/scripts/test_account_broker.py

@@ -418,6 +418,50 @@ def test_should_leave_later_choices_open_after_start_failure(
     assert json.loads(capsys.readouterr().out)["decision"]["account"] == "first"
 
 
+@pytest.mark.parametrize("product", (Product.CLAUDE, Product.CODEX))
+def test_should_exit_127_when_every_account_fails_to_start(
+    monkeypatch: pytest.MonkeyPatch, tmp_path: Path, product: Product
+) -> None:
+    accounts = (_account("first", product), _account("second", product))
+    monkeypatch.setitem(account_broker.all_product_adapters, product, _adapter(accounts, {
+        "first": _meters(80, 80), "second": _meters(70, 70)
+    }))
+    monkeypatch.setattr(account_broker.sys, "stdin", io.TextIOWrapper(io.BytesIO(b""), encoding="utf-8"))
+
+    def runner(argv: object, **options: object) -> subprocess.CompletedProcess[str]:
+        raise OSError("missing command")
+
+    with account_broker.override_subprocess_runner(runner):
+        outcome, _ = account_broker._execute(product, ("job",), now=datetime.now(timezone.utc))
+        code = account_broker.main(("run", "--product", product.value, "--report", str(tmp_path / "report.json"), "--", "job"))
+
+    assert outcome.status == "start_failed"
+    assert outcome.returncode == 127
+    assert outcome.wait_reset_at is None
+    assert code == 127
+
+
+def test_should_report_exhausted_when_start_failure_meets_usage_limit(
+    monkeypatch: pytest.MonkeyPatch,
+) -> None:
+    accounts = (_account("first"), _account("second"))
+    monkeypatch.setitem(account_broker.all_product_adapters, Product.CODEX, _adapter(accounts, {
+        "first": _meters(80, 80), "second": _meters(70, 70)
+    }))
+
+    def runner(argv: object, **options: object) -> subprocess.CompletedProcess[str]:
+        if Path(options["env"]["CODEX_HOME"]).name == "first":
+            raise OSError("missing command")
+        return subprocess.CompletedProcess(argv, 1, "", "rate limit")
+
+    with account_broker.override_subprocess_runner(runner):
+        outcome, _ = account_broker._execute(Product.CODEX, ("job",), now=datetime.now(timezone.utc))
+
+    assert outcome.attempts == (("first", "start_failed"), ("second", "usage_limited"))
+    assert outcome.status == "exhausted"
+    assert outcome.returncode == 3
+
+
 def test_should_stop_after_timeout_without_running_job_again(
     monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
 ) -> None:
@@ -707,8 +751,8 @@ def test_should_record_start_failure_without_launching_a_command_missing_from_pa
     outcome, _ = account_broker._execute(Product.CLAUDE, ("claude", "-p"), now=NOW)
 
     assert outcome.attempts == (("extra", "start_failed"),)
-    assert outcome.status == "exhausted"
-    assert outcome.returncode == WAIT_EXIT_CODE
+    assert outcome.status == "start_failed"
+    assert outcome.returncode == 127
 
 
 @pytest.mark.parametrize("marked_name", ["retired", "first"])

package/scripts/tests/test_codex_compat_materializer.py

@@ -1290,7 +1290,7 @@ def test_posix_case_distinct_enforcer_path_remains_foreign(
     target_manifest = _codex_hook_manifest(
         _codex_hook_group(managed_hook, foreign_hook)
     )
-    monkeypatch.setattr(materializer.os, "name", "posix")
+    monkeypatch.setattr(materializer, "_is_windows_host", lambda: False)
 
     projected_manifest, _, _ = _publish_codex_hook_fixture(
         source, target, target_manifest
@@ -1315,7 +1315,7 @@ def test_windows_case_variant_enforcer_path_is_managed(
     target_manifest = _codex_hook_manifest(
         _codex_hook_group({**managed_hook, "command": case_variant_command})
     )
-    monkeypatch.setattr(materializer.os, "name", "nt")
+    monkeypatch.setattr(materializer, "_is_windows_host", lambda: True)
 
     projected_manifest, _, _ = _publish_codex_hook_fixture(
         source, target, target_manifest