claude-dev-env 8.53.0 → 8.53.2

package/docs/second-claude-account.md

@@ -43,8 +43,7 @@ room.
 
 | Account | Has room while |
 |---|---|
-| main | under 90% of its week and under 50% of its 5-hour window |
-| second and each extra profile | under 95% of its week and under 90% of its 5-hour window |
+| every account | under 99% of its week and under 95% of its 5-hour window |
 
 When no account has room, the job waits until the next account reset, or for one
 hour when no reset is known. An unreadable meter never picks its account, and

package/hooks/blocking/reply_length_gate.py

@@ -10,6 +10,13 @@ It also denies a pull request number the reader cannot open::
     flag: It merged in #4347.
     ok:   Both land in [PR 5256](https://github.com/owner/repo/pull/5256).
 
+It denies a colon that joins two clauses on one line, and an em dash::
+
+    flag: That second reading matters: one message added 6.7k.
+    ok:   That second reading matters. One message added 6.7k.
+    ok:   Two checks failed at 9:47, then the list follows:
+          (each item on its own line below the colon)
+
 It denies a banned word or phrase, matched whole and case-insensitive::
 
     flag: The cause is likely the cache.
@@ -64,6 +71,8 @@ from hooks_constants.reply_length_gate_constants import (
     CONFIG_FILE_ENCODING,
     DECISION_CARD_OPTIONS_KEY,
     DECISION_CARD_TOOL_NAME,
+    EM_DASH_MESSAGE,
+    EM_DASH_PATTERN,
     FENCED_BLOCK_PATTERN,
     HOOK_EVENT_NAME,
     INLINE_CODE_PATTERN,
@@ -73,6 +82,8 @@ from hooks_constants.reply_length_gate_constants import (
     MARKDOWN_LINK_PATTERN,
     MAXIMUM_SENTENCE_COUNT,
     MAXIMUM_WORDS_PER_SENTENCE,
+    MID_SENTENCE_COLON_MESSAGE,
+    MID_SENTENCE_COLON_PATTERN,
     RETRY_INSTRUCTION,
     SENTENCE_END_PATTERN,
     SENTENCE_PREVIEW_SUFFIX,
@@ -126,6 +137,20 @@ def length_violation(reply_text: str) -> str | None:
     return None
 
 
+def mid_sentence_colon_violation(reply_text: str) -> str | None:
+    """Return the deny reason for a colon followed by prose on its line, or None."""
+    if MID_SENTENCE_COLON_PATTERN.search(countable_text(reply_text)) is None:
+        return None
+    return MID_SENTENCE_COLON_MESSAGE
+
+
+def em_dash_violation(reply_text: str) -> str | None:
+    """Return the deny reason for an em dash in the prose, or None."""
+    if EM_DASH_PATTERN.search(countable_text(reply_text)) is None:
+        return None
+    return EM_DASH_MESSAGE
+
+
 def unlinked_pull_request_violation(reply_text: str) -> str | None:
     """Return the deny reason for a pull request number outside a link, or None."""
     without_fences = FENCED_BLOCK_PATTERN.sub(" ", reply_text)
@@ -203,6 +228,8 @@ def tool_violation(tool_name: object, all_tool_input: dict[str, object]) -> tupl
         return None
     violation = (
         length_violation(reply_text)
+        or mid_sentence_colon_violation(reply_text)
+        or em_dash_violation(reply_text)
         or unlinked_pull_request_violation(reply_text)
         or banned_word_violation(reply_text, configured_banned_words())
     )

package/hooks/blocking/test_reply_length_gate.py

@@ -442,3 +442,48 @@ def test_should_apply_the_configured_list_to_decision_cards(
     )
     assert (probably_exit_code, synergy_exit_code) == (0, 2)
     assert 'Banned word "synergy"' in stderr_text
+
+
+QUOTED_COLON_SENTENCE = "That second reading matters: one message added 6.7k to Messages."
+
+
+def test_should_deny_the_quoted_mid_sentence_colon(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    exit_code, stderr_text = run_gate(
+        monkeypatch, capsys, POST_TOOL_NAME, {"text": QUOTED_COLON_SENTENCE}
+    )
+    assert exit_code == 2
+    assert "colon" in stderr_text
+
+
+@pytest.mark.parametrize(
+    "allowed_text",
+    [
+        "That second reading matters. One message added 6.7k to Messages.",
+        "Two checks failed:\n- lint\n- types",
+        "It fired at 9:47 today.",
+        "The call `a: int` passed.",
+        "See https://example.com/a: b for it.",
+        "Both land in [PR 5256](https://github.com/owner/repo/pull/5256).",
+        "```\nflag: x\n```",
+    ],
+)
+def test_should_allow_a_colon_that_introduces_a_list_or_sits_in_a_span(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], allowed_text: str
+) -> None:
+    exit_code, stderr_text = run_gate(monkeypatch, capsys, REPLY_TOOL_NAME, {"text": allowed_text})
+    assert (exit_code, stderr_text) == (0, "")
+
+
+def test_should_deny_an_em_dash_and_allow_one_in_code(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    denied_code, denied_text = run_gate(
+        monkeypatch, capsys, REPLY_TOOL_NAME, {"text": "The fix \u2014 a rename \u2014 is in."}
+    )
+    allowed_code, _ = run_gate(
+        monkeypatch, capsys, REPLY_TOOL_NAME, {"text": "The fix uses `\u2014` in a label."}
+    )
+    assert (denied_code, allowed_code) == (2, 0)
+    assert "em dash" in denied_text

package/hooks/hooks_constants/reply_length_gate_constants.py

@@ -24,6 +24,8 @@ MARKDOWN_LINK_PATTERN = re.compile(r"\[[^\]\n]*\]\([^)\s]*\)")
 UNLINKED_PULL_REQUEST_PATTERN = re.compile(
     r"\b(?:PRs?|pull requests?)\s*#?\d+|(?<![\w/&])#\d+", re.IGNORECASE
 )
+MID_SENTENCE_COLON_PATTERN = re.compile(r":[ \t]+\S")
+EM_DASH_PATTERN = re.compile("\u2014")
 LINE_BREAK_PATTERN = re.compile(r"\n+")
 SENTENCE_END_PATTERN = re.compile(r"(?<=[.!?])\s+")
 WORD_PATTERN = re.compile(r"[A-Za-z0-9]+(?:['.,-][A-Za-z0-9]+)*")
@@ -33,6 +35,11 @@ SENTENCE_PREVIEW_SUFFIX = "..."
 RETRY_INSTRUCTION = " Cut the text and resend the call."
 TOO_MANY_SENTENCES_MESSAGE = "Reply too long: {sentence_count} sentences, limit {sentence_limit}."
 UNLINKED_PULL_REQUEST_MESSAGE = 'Pull request "{reference}" has no link. Write it as [PR N](https://github.com/<owner>/<repo>/pull/N).'
+MID_SENTENCE_COLON_MESSAGE = (
+    "A colon joins two clauses on one line. Write two sentences, or end the line with the colon"
+    " and put the list on the lines below."
+)
+EM_DASH_MESSAGE = "An em dash is in the text. Use a period or a comma."
 ALL_DEFAULT_BANNED_WORDS = (
     "real",
     "really",

package/hooks/hooks_constants/test_reply_length_gate_constants.py

@@ -45,3 +45,9 @@ def test_gate_reads_only_the_prose_fields_of_a_decision_card() -> None:
         constants.DECISION_CARD_OPTIONS_KEY,
         constants.ALL_DECISION_OPTION_PROSE_KEYS,
     ) == (("question", "context"), "options", ("label", "consequence"))
+
+
+def test_colon_pattern_matches_a_colon_with_prose_after_it_on_the_line() -> None:
+    assert constants.MID_SENTENCE_COLON_PATTERN.search("matters: one message")
+    assert constants.MID_SENTENCE_COLON_PATTERN.search("at 9:47") is None
+    assert constants.MID_SENTENCE_COLON_PATTERN.search("items:\n- one") is None

package/package.json

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

package/scripts/dev_env_scripts_constants/claude_account_constants.py

@@ -130,16 +130,16 @@ WINDOWS_OS_NAME: str = "nt"
 TEXT_ENCODING: str = "utf-8"
 """Encoding for the launcher file and the JSON reports."""
 
-MAIN_WEEKLY_USED_CEILING_PERCENT: float = 90.0
+MAIN_WEEKLY_USED_CEILING_PERCENT: float = 99.0
 """Main takes jobs only while its weekly use is under this percent."""
 
-MAIN_SESSION_USED_CEILING_PERCENT: float = 50.0
+MAIN_SESSION_USED_CEILING_PERCENT: float = 95.0
 """Main takes jobs only while its 5-hour use is under this percent."""
 
-SECOND_WEEKLY_USED_CEILING_PERCENT: float = 95.0
+SECOND_WEEKLY_USED_CEILING_PERCENT: float = 99.0
 """The second account takes jobs while its weekly use is under this percent."""
 
-SECOND_SESSION_USED_CEILING_PERCENT: float = 90.0
+SECOND_SESSION_USED_CEILING_PERCENT: float = 95.0
 """The second account takes jobs while its 5-hour use is under this percent."""
 
 FULL_PERCENT: float = 100.0

package/scripts/dev_env_scripts_constants/test_claude_account_constants.py

@@ -1,4 +1,4 @@
-"""The second-account thresholds keep the main account the protected one."""
+"""The picker report and meter keys stay distinct."""
 
 from __future__ import annotations
 
@@ -11,18 +11,9 @@ from dev_env_scripts_constants.claude_account_constants import (
     JSON_SESSION_USED_PERCENT_KEY,
     JSON_WEEKLY_RESETS_AT_KEY,
     JSON_WEEKLY_USED_PERCENT_KEY,
-    MAIN_SESSION_USED_CEILING_PERCENT,
-    MAIN_WEEKLY_USED_CEILING_PERCENT,
-    SECOND_SESSION_USED_CEILING_PERCENT,
-    SECOND_WEEKLY_USED_CEILING_PERCENT,
 )
 
 
-def test_main_ceilings_sit_below_the_second_account_ceilings() -> None:
-    assert MAIN_WEEKLY_USED_CEILING_PERCENT < SECOND_WEEKLY_USED_CEILING_PERCENT
-    assert MAIN_SESSION_USED_CEILING_PERCENT < SECOND_SESSION_USED_CEILING_PERCENT
-
-
 def test_picker_report_keys_never_collide() -> None:
     all_report_keys = [
         JSON_ACCOUNT_KEY,

package/scripts/test_account_broker.py

@@ -112,8 +112,8 @@ def test_should_keep_list_order_when_tighter_windows_tie() -> None:
 @pytest.mark.parametrize(
     ("weekly_reset", "weekly_left", "short_left"),
     (
-        (NOW + timedelta(hours=4), 10, 80),
-        (NOW + timedelta(hours=4), 20, 50),
+        (NOW + timedelta(hours=4), 1, 80),
+        (NOW + timedelta(hours=4), 20, 5),
     ),
 )
 def test_should_guard_main_at_each_limit(
@@ -158,7 +158,7 @@ def test_should_pick_an_extra_with_more_room_than_main() -> None:
     assert decision.account == extra.account
 
 
-@pytest.mark.parametrize(("main_short_left", "expected_name"), ((51, "main"), (50, "extra")))
+@pytest.mark.parametrize(("main_short_left", "expected_name"), ((6, "main"), (5, "extra")))
 def test_should_pick_main_only_while_under_its_5_hour_ceiling(
     main_short_left: float, expected_name: str
 ) -> None:
@@ -166,7 +166,7 @@ def test_should_pick_main_only_while_under_its_5_hour_ceiling(
         _account("main", Product.CLAUDE, main=True),
         _meters(main_short_left, 95, weekly_reset=NOW + timedelta(days=5)),
     )
-    extra = Reading(_account("extra", Product.CLAUDE), _meters(20, 20))
+    extra = Reading(_account("extra", Product.CLAUDE), _meters(20, 2))
 
     decision = choose_from_readings(Product.CLAUDE, (main, extra), now=NOW)
 
@@ -190,7 +190,7 @@ def test_should_wait_for_main_5_hour_reset_while_its_week_resets_days_away() ->
     short_reset = NOW + timedelta(hours=2)
     main = Reading(
         _account("main", Product.CLAUDE, main=True),
-        _meters(40, 50, short_reset=short_reset, weekly_reset=NOW + timedelta(days=3)),
+        _meters(5, 50, short_reset=short_reset, weekly_reset=NOW + timedelta(days=3)),
     )
 
     decision = choose_from_readings(Product.CLAUDE, (main,), now=NOW)
@@ -202,8 +202,8 @@ def test_should_wait_for_main_5_hour_reset_while_its_week_resets_days_away() ->
 @pytest.mark.parametrize(
     ("short_left", "weekly_left", "weekly_reset", "expected_reset"),
     (
-        (40, 20, NOW + timedelta(hours=4), NOW + timedelta(hours=2)),
-        (80, 5, NOW + timedelta(hours=4), NOW + timedelta(hours=4)),
+        (5, 20, NOW + timedelta(hours=4), NOW + timedelta(hours=2)),
+        (80, 1, NOW + timedelta(hours=4), NOW + timedelta(hours=4)),
     ),
 )
 def test_should_wait_for_the_meter_or_window_that_blocks_main(
@@ -292,7 +292,7 @@ def test_should_wait_until_both_blocking_windows_reset() -> None:
 
 def test_should_use_claude_extra_floors_and_wait_for_unread() -> None:
     blocked = Reading(
-        _account("blocked", Product.CLAUDE), _meters(10, 5)
+        _account("blocked", Product.CLAUDE), _meters(5, 1)
     )
     unread = Reading(_account("unread", Product.CLAUDE), None)
 
@@ -462,14 +462,14 @@ _EV_CLAUDE = _account("ev", Product.CLAUDE)
             id="every-meter-unreadable",
         ),
         pytest.param(
-            (Reading(_MAIN_CLAUDE, _meters(40, 50, short_reset=NOW + timedelta(minutes=30))), Reading(_EV_CLAUDE, None)),
+            (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",
             NOW + timedelta(minutes=30),
             id="unreadable-beside-a-reset-before-the-check",
         ),
         pytest.param(
-            (Reading(_MAIN_CLAUDE, _meters(40, 50, short_reset=NOW + timedelta(hours=2))), Reading(_EV_CLAUDE, None)),
+            (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",
             NOW + timedelta(hours=1),
@@ -484,7 +484,7 @@ _EV_CLAUDE = _account("ev", Product.CLAUDE)
         ),
         pytest.param(
             (
-                Reading(_MAIN_CLAUDE, _meters(40, 50, short_reset=NOW + timedelta(hours=2))),
+                Reading(_MAIN_CLAUDE, _meters(5, 50, short_reset=NOW + timedelta(hours=2))),
                 Reading(_EV_CLAUDE, _meters(5, 80, short_reset=NOW + timedelta(hours=3))),
             ),
             {},
@@ -493,7 +493,7 @@ _EV_CLAUDE = _account("ev", Product.CLAUDE)
             id="every-meter-read-with-known-resets",
         ),
         pytest.param(
-            (Reading(_MAIN_CLAUDE, Meters(40, None, 50, None)),),
+            (Reading(_MAIN_CLAUDE, Meters(5, None, 50, None)),),
             {},
             "no account has room; next check at 2026-10-03T01:00:00+00:00",
             NOW + timedelta(hours=1),