claude-dev-env 8.105.0 → 8.105.2

package/hooks/blocking/test_turn_review_gate.py

@@ -115,6 +115,17 @@ def test_should_block_the_stop_with_the_finding_when_the_turn_leaves_a_bug(
     assert "app" in str(stop_output["reason"])
 
 
+def test_should_tell_the_agent_to_fix_a_finding_unless_the_user_asked_for_it(
+    project: Path, gate_environment: dict[str, str]
+) -> None:
+    run_hook(gate_environment, "UserPromptSubmit")
+    drop_retry(project)
+    block_reason = str(run_hook(gate_environment, "Stop")["reason"])
+    assert "Fix each one before you end the turn." in block_reason
+    assert "is not a request for the problem it causes" in block_reason
+    assert "quote those words in your reply" in block_reason
+
+
 def test_should_review_only_the_edits_made_after_the_prompt(
     project: Path, gate_environment: dict[str, str]
 ) -> None:

package/hooks/hooks_constants/turn_review_gate_constants.py

@@ -149,7 +149,10 @@ FINDING_LINE_SEPARATOR = "\n"
 REPOSITORY_FINDINGS_SEPARATOR = "\n\n"
 BLOCK_REASON_TEMPLATE = (
     "A fresh-context review of this turn's changes found these problems. "
-    "Fix each one, or say in your reply why it is wrong, before you end the turn:\n\n"
+    "Fix each one before you end the turn. Code the user pasted or described is not a "
+    "request for the problem it causes. Keep a finding's behavior only when the user's own "
+    "words ask for that behavior, and quote those words in your reply. When a finding is "
+    "wrong, say in your reply why:\n\n"
     "{all_findings}"
 )
 TOO_MANY_FILES_DIAGNOSTIC_TEMPLATE = (

package/package.json

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

package/scripts/dev_env_scripts_constants/account_broker_constants.py

@@ -392,15 +392,27 @@ def _unreadable_text(reading: Reading) -> str:
     return f"{reading.account.name} ({reading.unread_reason})"
 
 
-def _wait_reason(all_unreadable_names: Sequence[str], is_every_account_unreadable: bool, soonest: _RetryTime) -> str:
+def _room_text(reading: Reading, is_spent: bool) -> str:
+    if is_spent or reading.meters is None:
+        return f"{reading.account.name} spent"
+    percent_left = reading.meters.tightest_percent_left
+    if percent_left is None:
+        return f"{reading.account.name} percent left unread"
+    return f"{reading.account.name} {percent_left:g}% left"
+
+
+def _wait_reason(
+    all_room_texts: Sequence[str], all_unreadable_names: Sequence[str], is_every_account_unreadable: bool, soonest: _RetryTime
+) -> str:
     next_event = "reset" if soonest.is_known_reset else "check"
     next_text = f"next {next_event} at {utc_time_text(soonest.at)}"
+    room_text = f" ({', '.join(all_room_texts)})" if all_room_texts else ""
     if not all_unreadable_names:
-        return f"no account has room; {next_text}"
+        return f"no account has room{room_text}; {next_text}"
     unreadable_text = ", ".join(all_unreadable_names)
     if is_every_account_unreadable:
         return f"no account meter could be read ({unreadable_text}); {next_text}"
-    return f"no readable account has room; meter unreadable for {unreadable_text}; {next_text}"
+    return f"no readable account has room{room_text}; meter unreadable for {unreadable_text}; {next_text}"
 
 
 def _wait_decision(
@@ -423,5 +435,11 @@ def _wait_decision(
         for each_reading in all_readings
         if each_reading.meters is None and each_reading.account not in all_spent_accounts
     ]
+    all_room_texts = [
+        _room_text(each_reading, each_reading.account in all_spent_accounts)
+        for each_reading in all_readings
+        if each_reading.meters is not None or each_reading.account in all_spent_accounts
+    ]
     is_every_account_unreadable = len(all_unreadable_names) == len(all_readings)
-    return Decision("wait", None, soonest.at, _wait_reason(all_unreadable_names, is_every_account_unreadable, soonest), "wait")
+    reason = _wait_reason(all_room_texts, all_unreadable_names, is_every_account_unreadable, soonest)
+    return Decision("wait", None, soonest.at, reason, "wait")

package/scripts/test_account_broker.py

@@ -91,7 +91,7 @@ def test_should_name_the_unread_account_and_its_reason_when_no_account_has_room(
 
     assert decision.action == "wait"
     assert decision.reason == (
-        "no readable account has room; meter unreadable for codex-3 (codex app-server failed: timed out); "
+        "no readable account has room (codex-4 0% left); meter unreadable for codex-3 (codex app-server failed: timed out); "
         "next check at 2026-10-03T01:00:00+00:00"
     )
     assert payload[0]["unread_reason"] == "codex app-server failed: timed out"
@@ -542,21 +542,21 @@ _EV_CLAUDE = _account("ev", Product.CLAUDE)
         pytest.param(
             (Reading(_MAIN_CLAUDE, _meters(5, 50, short_reset=NOW + timedelta(minutes=30))), Reading(_EV_CLAUDE, None)),
             {},
-            "no readable account has room; meter unreadable for ev; next reset at 2026-10-03T00:30:00+00:00",
+            "no readable account has room (main 5% left); meter unreadable for ev; next reset at 2026-10-03T00:30:00+00:00",
             NOW + timedelta(minutes=30),
             id="unreadable-beside-a-reset-before-the-check",
         ),
         pytest.param(
             (Reading(_MAIN_CLAUDE, _meters(5, 50, short_reset=NOW + timedelta(hours=2))), Reading(_EV_CLAUDE, None)),
             {},
-            "no readable account has room; meter unreadable for ev; next check at 2026-10-03T01:00:00+00:00",
+            "no readable account has room (main 5% left); meter unreadable for ev; next check at 2026-10-03T01:00:00+00:00",
             NOW + timedelta(hours=1),
             id="unreadable-beside-a-reset-after-the-check",
         ),
         pytest.param(
             (Reading(_MAIN_CLAUDE, _meters(80, 80)), Reading(_EV_CLAUDE, None)),
             {_MAIN_CLAUDE: NOW + timedelta(minutes=30)},
-            "no readable account has room; meter unreadable for ev; next reset at 2026-10-03T00:30:00+00:00",
+            "no readable account has room (main spent); meter unreadable for ev; next reset at 2026-10-03T00:30:00+00:00",
             NOW + timedelta(minutes=30),
             id="unreadable-beside-a-spent-account",
         ),
@@ -566,17 +566,31 @@ _EV_CLAUDE = _account("ev", Product.CLAUDE)
                 Reading(_EV_CLAUDE, _meters(5, 80, short_reset=NOW + timedelta(hours=3))),
             ),
             {},
-            "no account has room; next reset at 2026-10-03T02:00:00+00:00",
+            "no account has room (main 5% left, ev 5% left); next reset at 2026-10-03T02:00:00+00:00",
             NOW + timedelta(hours=2),
             id="every-meter-read-with-known-resets",
         ),
         pytest.param(
             (Reading(_MAIN_CLAUDE, Meters(5, None, 50, None)),),
             {},
-            "no account has room; next check at 2026-10-03T01:00:00+00:00",
+            "no account has room (main 5% left); next check at 2026-10-03T01:00:00+00:00",
             NOW + timedelta(hours=1),
             id="every-meter-read-without-a-known-reset",
         ),
+        pytest.param(
+            (),
+            {},
+            "no account has room; next check at 2026-10-03T01:00:00+00:00",
+            NOW + timedelta(hours=1),
+            id="no-account-in-the-roster",
+        ),
+        pytest.param(
+            (Reading(_MAIN_CLAUDE, Meters(None, None, None, None)),),
+            {},
+            "no account has room (main percent left unread); next check at 2026-10-03T01:00:00+00:00",
+            NOW + timedelta(hours=1),
+            id="meter-read-without-a-percent",
+        ),
     ),
 )
 def test_should_name_what_the_broker_knows_in_the_wait_reason(