claude-dev-env 8.66.0 → 8.67.0

package/hooks/blocking/reply_length_gate.py

@@ -17,6 +17,13 @@ It denies a colon that joins two clauses on one line, and an em dash::
     ok:   Two checks failed at 9:47, then the list follows:
           (each item on its own line below the colon)
 
+It denies a causal sentence ("so", "because", "caused", "due to",
+"therefore", "means") that carries no inline code, URL, or markdown link
+of its own. A command block in the same reply is no evidence::
+
+    flag: That window was Windows PowerShell without Administrator, so the write did not land.
+    ok:   The write did not land, because the next read printed `Right after write: 1`.
+
 It denies a banned word or phrase, matched whole and case-insensitive::
 
     flag: The cause is likely the cache.
@@ -58,6 +65,7 @@ from hooks_constants.reply_length_gate_constants import (
     ALL_DECISION_CARD_PROSE_KEYS,
     ALL_DECISION_OPTION_PROSE_KEYS,
     ALL_DEFAULT_BANNED_WORDS,
+    ALL_EVIDENCE_PATTERNS,
     ALLOW_EXIT_CODE,
     BANNED_WORD_MESSAGE,
     BANNED_WORD_PART_SEPARATOR,
@@ -66,6 +74,7 @@ from hooks_constants.reply_length_gate_constants import (
     BANNED_WORDS_JSON_KEY,
     BANNED_WORDS_PATH_ENV_VAR,
     BLOCK_EXIT_CODE,
+    CAUSAL_CLAIM_PATTERN,
     CARD_TEXT_SEPARATOR,
     CLAUDE_HOME_DIRECTORY_NAME,
     CONFIG_FILE_ENCODING,
@@ -94,6 +103,7 @@ from hooks_constants.reply_length_gate_constants import (
     TOOL_NAME_KEY,
     UNLINKED_PULL_REQUEST_MESSAGE,
     UNLINKED_PULL_REQUEST_PATTERN,
+    UNSOURCED_CAUSE_MESSAGE,
     URL_PATTERN,
     WORD_PATTERN,
     WORD_SEPARATOR,
@@ -151,6 +161,21 @@ def em_dash_violation(reply_text: str) -> str | None:
     return EM_DASH_MESSAGE
 
 
+def unsourced_cause_violation(reply_text: str) -> str | None:
+    """Return the deny reason for the first causal sentence that cites no evidence itself, or None."""
+    all_sentences = [
+        each_sentence
+        for each_line in LINE_BREAK_PATTERN.split(FENCED_BLOCK_PATTERN.sub(" ", reply_text))
+        for each_sentence in SENTENCE_END_PATTERN.split(each_line)
+    ]
+    for each_sentence in all_sentences:
+        is_causal = CAUSAL_CLAIM_PATTERN.search(INLINE_CODE_PATTERN.sub(" ", each_sentence))
+        has_evidence = any(each_pattern.search(each_sentence) for each_pattern in ALL_EVIDENCE_PATTERNS)
+        if is_causal and not has_evidence:
+            return UNSOURCED_CAUSE_MESSAGE.format(sentence=each_sentence.strip())
+    return None
+
+
 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)
@@ -231,6 +256,7 @@ def tool_violation(tool_name: object, all_tool_input: dict[str, object]) -> tupl
         or mid_sentence_colon_violation(reply_text)
         or em_dash_violation(reply_text)
         or unlinked_pull_request_violation(reply_text)
+        or unsourced_cause_violation(reply_text)
         or banned_word_violation(reply_text, configured_banned_words())
     )
     return None if violation is None else (violation, reply_text)

package/hooks/blocking/test_reply_length_gate.py

@@ -487,3 +487,55 @@ def test_should_deny_an_em_dash_and_allow_one_in_code(
     )
     assert (denied_code, allowed_code) == (2, 0)
     assert "em dash" in denied_text
+
+
+def test_should_deny_the_quoted_cause_with_no_source(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    exit_code, stderr_text = run_gate(
+        monkeypatch,
+        capsys,
+        POST_TOOL_NAME,
+        {"text": "That window was Windows PowerShell without Administrator, so the write did not land."},
+    )
+    assert exit_code == 2
+    assert "Cause with no source" in stderr_text
+
+
+@pytest.mark.parametrize(
+    "reply_text",
+    [
+        "The write did not land, because the next read printed `Right after write: 1`.",
+        "The job failed because of [run 12](https://github.com/owner/repo/actions/runs/12).",
+    ],
+)
+def test_should_allow_a_cause_that_cites_evidence(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], reply_text: str
+) -> None:
+    exit_code, stderr_text = run_gate(monkeypatch, capsys, REPLY_TOOL_NAME, {"text": reply_text})
+    assert (exit_code, stderr_text) == (0, "")
+
+
+@pytest.mark.parametrize(
+    "reply_text",
+    [
+        "That window lacked Administrator, so the write did not land.\n\n```powershell\nRestart-Service x\n```",
+        "The error shows the hook is the only line. So the filter wrote nothing.",
+        "One line survived because its start did not match my filter. Run this.",
+    ],
+)
+def test_should_deny_a_cause_whose_own_sentence_cites_nothing(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str], reply_text: str
+) -> None:
+    exit_code, stderr_text = run_gate(monkeypatch, capsys, REPLY_TOOL_NAME, {"text": reply_text})
+    assert exit_code == 2
+    assert "Cause with no source" in stderr_text
+
+
+def test_should_allow_a_reply_with_no_causal_claim(
+    monkeypatch: pytest.MonkeyPatch, capsys: pytest.CaptureFixture[str]
+) -> None:
+    exit_code, _ = run_gate(
+        monkeypatch, capsys, REPLY_TOOL_NAME, {"text": "The runner restarted. Also, so far so good."}
+    )
+    assert exit_code == 0

package/hooks/features/conduct-gates.md

@@ -4,7 +4,7 @@ This family stops a chat reply, edited message, or tool call when its payload br
 
 ## Checks
 
-- `blocking/reply_length_gate.py` caps chat reply length, denies a configured banned word, and requires a readable link when the text names a pull request number.
+- `blocking/reply_length_gate.py` caps chat reply length, denies a configured banned word, and requires a readable link when the text names a pull request number, and denies a causal claim in a reply that cites no code span, output block, or link.
 - `blocking/edit_marker_gate.py` rejects strikethrough and edit-note markers in replacement chat text or cards.
 - `blocking/issue_close_handoff_gate.py` denies an issue comment or issue write that closes the issue and routes a found defect to another issue or epic.
 - `blocking/session_title_format_gate.py` denies a session title that breaks the `<emoji> <name>` shape or has a name longer than 25 characters.

package/hooks/hooks_constants/reply_length_gate_constants.py

@@ -40,6 +40,15 @@ MID_SENTENCE_COLON_MESSAGE = (
     " and put the list on the lines below."
 )
 EM_DASH_MESSAGE = "An em dash is in the text. Use a period or a comma."
+CAUSAL_CLAIM_PATTERN = re.compile(
+    r"(?:^|,)\s*so\b(?!\s+far\b)|\b(?:because|caused|causes|due to|therefore|which means|that means)\b",
+    re.IGNORECASE,
+)
+ALL_EVIDENCE_PATTERNS = (INLINE_CODE_PATTERN, URL_PATTERN, MARKDOWN_LINK_PATTERN)
+UNSOURCED_CAUSE_MESSAGE = (
+    'Cause with no source: "{sentence}". Quote the log line or output in backticks,'
+    " or link the run or file and line that shows it."
+)
 ALL_DEFAULT_BANNED_WORDS = (
     "real",
     "really",

package/package.json

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