claude-dev-env 8.43.6 → 8.44.0

package/hooks/blocking/reply_length_gate.py

@@ -18,6 +18,10 @@ It denies a banned word or phrase, matched whole and case-insensitive::
 The gate uses the ``banned_words`` list in ``~/.claude/reply-banned-words.json``,
 or in the file that CLAUDE_REPLY_BANNED_WORDS_PATH names. Without a valid
 list there, it uses ALL_DEFAULT_BANNED_WORDS.
+
+The banned-word check also reads every text field of a decision card.
+A decision card gets no length check.
+
 Each non-empty line counts as its own sentence, so a list counts one
 sentence per item. URLs, markdown link targets, inline code spans, and
 fenced blocks carry no words.
@@ -52,8 +56,10 @@ from hooks_constants.reply_length_gate_constants import (
     BANNED_WORDS_JSON_KEY,
     BANNED_WORDS_PATH_ENV_VAR,
     BLOCK_EXIT_CODE,
+    CARD_TEXT_SEPARATOR,
     CLAUDE_HOME_DIRECTORY_NAME,
     CONFIG_FILE_ENCODING,
+    DECISION_CARD_TOOL_NAME,
     FENCED_BLOCK_PATTERN,
     HOOK_EVENT_NAME,
     INLINE_CODE_PATTERN,
@@ -163,26 +169,52 @@ def banned_word_violation(reply_text: str, all_banned_words: tuple[str, ...]) ->
     return None
 
 
+def all_card_texts(card_value: object) -> list[str]:
+    """Collect every string inside a decision card's input, in order."""
+    if isinstance(card_value, str):
+        return [card_value]
+    if isinstance(card_value, dict):
+        return [
+            each_text
+            for each_value in card_value.values()
+            for each_text in all_card_texts(each_value)
+        ]
+    if isinstance(card_value, list):
+        return [each_text for each_value in card_value for each_text in all_card_texts(each_value)]
+    return []
+
+
+def tool_violation(tool_name: object, all_tool_input: dict[str, object]) -> tuple[str, str] | None:
+    """Return the deny reason and the checked text for one call, or None when it passes."""
+    if tool_name == DECISION_CARD_TOOL_NAME:
+        card_text = CARD_TEXT_SEPARATOR.join(all_card_texts(all_tool_input))
+        card_violation = banned_word_violation(card_text, configured_banned_words())
+        return None if card_violation is None else (card_violation, card_text)
+    if tool_name not in ALL_CHECKED_TOOL_NAMES:
+        return None
+    reply_text = all_tool_input.get(TEXT_KEY)
+    if not isinstance(reply_text, str):
+        return None
+    violation = (
+        length_violation(reply_text)
+        or unlinked_pull_request_violation(reply_text)
+        or banned_word_violation(reply_text, configured_banned_words())
+    )
+    return None if violation is None else (violation, reply_text)
+
+
 def main() -> int:
     hook_input = read_hook_input_dictionary_from_stdin()
     if hook_input is None:
         return ALLOW_EXIT_CODE
     tool_name = hook_input.get(TOOL_NAME_KEY)
-    if tool_name not in ALL_CHECKED_TOOL_NAMES:
-        return ALLOW_EXIT_CODE
     tool_input = hook_input.get(TOOL_INPUT_KEY)
     if not isinstance(tool_input, dict):
         return ALLOW_EXIT_CODE
-    reply_text = tool_input.get(TEXT_KEY)
-    if not isinstance(reply_text, str):
-        return ALLOW_EXIT_CODE
-    violation = (
-        length_violation(reply_text)
-        or unlinked_pull_request_violation(reply_text)
-        or banned_word_violation(reply_text, configured_banned_words())
-    )
-    if violation is None:
+    checked = tool_violation(tool_name, tool_input)
+    if checked is None:
         return ALLOW_EXIT_CODE
+    violation, reply_text = checked
     block_reason = violation + RETRY_INSTRUCTION
     log_hook_block(
         Path(__file__).name,

package/hooks/blocking/test_reply_length_gate.py

@@ -322,3 +322,58 @@ def test_should_keep_the_defaults_when_the_config_file_is_malformed(
     config_path.write_text("{not json", encoding="utf-8")
     exit_code, _ = run_gate(monkeypatch, capsys, REPLY_TOOL_NAME, {"text": "It likely passed."})
     assert exit_code == 2
+
+
+DECISION_TOOL_NAME = "mcp__hearthbot__ask_decision"
+
+
+def test_should_deny_a_decision_card_that_hedges_in_an_option(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    exit_code, stderr_text = run_gate(
+        monkeypatch,
+        capsys,
+        DECISION_TOOL_NAME,
+        {
+            "question": "Exclude the call button?",
+            "context": "Crops show the call button flagged in 7 themes.",
+            "options": [
+                {"label": "Exclude it", "consequence": "Three themes probably pass."},
+                {"label": "Leave it", "consequence": "Three themes keep a warning."},
+            ],
+            "recommended": 0,
+        },
+    )
+    assert exit_code == 2
+    assert '"probably"' in stderr_text
+
+
+def test_should_allow_a_long_decision_card_with_no_hedge(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    exit_code, stderr_text = run_gate(
+        monkeypatch,
+        capsys,
+        DECISION_TOOL_NAME,
+        {
+            "question": "Exclude the call button?",
+            "context": " ".join([FIFTEEN_WORD_SENTENCE] * 5),
+            "options": [
+                {"label": "Exclude it", "consequence": SIXTEEN_WORD_SENTENCE},
+                {"label": "Leave it", "consequence": "Three themes keep a warning."},
+            ],
+            "recommended": 0,
+        },
+    )
+    assert (exit_code, stderr_text) == (0, "")
+
+
+@pytest.mark.parametrize(
+    "reply_text",
+    ["It maybe passed.", "Perhaps it passed.", "It might be the cache.", "I am not sure it passed."],
+)
+def test_should_deny_the_hedges_the_default_list_adds(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], reply_text: str
+) -> None:
+    exit_code, _ = run_gate(monkeypatch, capsys, REPLY_TOOL_NAME, {"text": reply_text})
+    assert exit_code == 2

package/hooks/hooks.json

@@ -83,7 +83,7 @@
         ]
       },
       {
-        "matcher": "mcp__hearthbot__reply|mcp__hearthbot__post_message",
+        "matcher": "mcp__hearthbot__reply|mcp__hearthbot__post_message|mcp__hearthbot__ask_decision",
         "hooks": [
           {
             "type": "command",

package/hooks/hooks_constants/reply_length_gate_constants.py

@@ -5,6 +5,8 @@ import re
 MAXIMUM_SENTENCE_COUNT = 3
 MAXIMUM_WORDS_PER_SENTENCE = 15
 ALL_CHECKED_TOOL_NAMES = frozenset({"mcp__hearthbot__reply", "mcp__hearthbot__post_message"})
+DECISION_CARD_TOOL_NAME = "mcp__hearthbot__ask_decision"
+CARD_TEXT_SEPARATOR = "\n"
 TOOL_NAME_KEY = "tool_name"
 TOOL_INPUT_KEY = "tool_input"
 TEXT_KEY = "text"
@@ -45,6 +47,15 @@ ALL_DEFAULT_BANNED_WORDS = (
     "suspect",
     "guess",
     "my theory",
+    "maybe",
+    "perhaps",
+    "presumably",
+    "might be",
+    "may be",
+    "appears to",
+    "i believe",
+    "not sure",
+    "unsure",
 )
 BANNED_WORD_PART_SEPARATOR = r"\s+"
 BANNED_WORD_PATTERN_TEMPLATE = r"(?<![A-Za-z0-9]){word_pattern}(?![A-Za-z0-9])"

package/package.json

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