claude-dev-env 8.49.1 → 8.49.3

package/package.json

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

package/scripts/account_broker.py

@@ -211,6 +211,7 @@ class _RunContext:
     all_attempts: list[tuple[str, str]]
     all_spent_accounts: set[Account]
     all_spent_resets: dict[Account, datetime]
+    start_failure_text: str = ""
 
 
 def _prepare_run(
@@ -234,7 +235,7 @@ 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)
+        return JobOutcome(COMMAND_MISSING_EXIT_CODE, "", context.start_failure_text, 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, decision.reason)
 
@@ -287,6 +288,7 @@ def _attempt_once(context: _RunContext, decision: Decision) -> JobOutcome | None
     except (OSError, subprocess.TimeoutExpired) as error:
         status = "timeout" if isinstance(error, subprocess.TimeoutExpired) else "start_failed"
         _record_spent_attempt(context, account, status, COMMAND_MISSING_EXIT_CODE)
+        context.start_failure_text = str(error)
         if isinstance(error, subprocess.TimeoutExpired) or not context.all_readings:
             context.report.final_decision = decision
             final_status = "advisor_blocked" if context.product is Product.CLAUDE else status

package/scripts/test_account_broker.py

@@ -908,6 +908,20 @@ def test_should_record_start_failure_without_launching_a_command_missing_from_pa
     assert outcome.returncode == 127
 
 
+def test_should_keep_the_start_failure_text_when_no_account_can_start(monkeypatch: pytest.MonkeyPatch) -> None:
+    monkeypatch.setitem(account_broker.all_product_adapters, Product.CLAUDE, _adapter(
+        (_account("first", Product.CLAUDE), _account("second", Product.CLAUDE)),
+        {"first": _meters(80, 80), "second": _meters(70, 70)},
+    ))
+    monkeypatch.setattr(shutil, "which", lambda name: None)
+    monkeypatch.setattr(account_broker.support.subprocess, "run", lambda *arguments, **options: pytest.fail("a missing command ran"))
+
+    outcome, _ = account_broker._execute(Product.CLAUDE, ("claude", "-p"), now=NOW)
+
+    assert outcome.status == "start_failed"
+    assert "claude" in outcome.stderr
+
+
 @pytest.mark.parametrize("marked_name", ["retired", "first"])
 def test_should_run_past_a_stored_spent_mark_beyond_the_platform_range(
     monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], marked_name: str

package/scripts/test_agent_merge_check.py

@@ -2,16 +2,10 @@
 
 from __future__ import annotations
 
-import sys
 import urllib.parse
-from pathlib import Path
 
 import pytest
 
-_SCRIPTS_DIRECTORY = Path(__file__).resolve().parent
-if str(_SCRIPTS_DIRECTORY) not in sys.path:
-    sys.path.insert(0, str(_SCRIPTS_DIRECTORY))
-
 import agent_merge_check
 from dev_env_scripts_constants.agent_merge_check_constants import (
     BEHIND_HOLD_REASON,