@windyroad/tdd 0.2.3 → 0.3.0

package/.claude-plugin/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "wr-tdd",
-  "version": "0.1.0",
+  "version": "0.3.0",
   "description": "TDD state machine enforcement (IDLE/RED/GREEN/BLOCKED) for Claude Code"
-}
+}

package/hooks/lib/session-marker.sh

@@ -0,0 +1,46 @@
+#!/bin/bash
+# Shared session-announcement marker helpers (P095 / ADR-038).
+#
+# Used by UserPromptSubmit hooks to gate verbose MANDATORY instruction
+# prose behind a once-per-session check. First prompt of a session emits
+# the full block AND calls mark_announced; subsequent prompts see the
+# marker via has_announced and emit only a terse reminder.
+#
+# Why no TTL or drift check (unlike review-gate.sh): announcement is
+# bookkeeping for prose verbosity, not enforcement. PreToolUse gates
+# still block unauthorised edits regardless of announcement state; the
+# delegated agent re-reads policy when it runs. Extending the marker's
+# lifetime across policy changes mid-session is safe — the gate, not
+# the announcement, is load-bearing.
+#
+# Marker path convention: /tmp/${SYSTEM}-announced-${SESSION_ID}
+# (mirrors the /tmp/${SYSTEM}-reviewed-${SESSION_ID} convention from
+# style-guide/voice-tone/risk-scorer review-gate.sh; the -announced-
+# suffix distinguishes announcement markers from clearance markers).
+#
+# Empty SESSION_ID fallback: has_announced returns 1 (not announced,
+# full block emits) and mark_announced is a no-op (no file written).
+# This covers manual hook invocation, test harnesses, and any rare
+# case where Claude Code does not pass a session_id on stdin.
+
+# Returns 0 if the hook for SYSTEM has already announced in SESSION_ID,
+# 1 otherwise. Empty SESSION_ID => returns 1 (never announced).
+#
+# Usage: has_announced "architect" "$SESSION_ID"
+has_announced() {
+  local SYSTEM="$1"
+  local SESSION_ID="$2"
+  [ -n "$SESSION_ID" ] || return 1
+  [ -f "/tmp/${SYSTEM}-announced-${SESSION_ID}" ]
+}
+
+# Writes the announcement marker for SYSTEM in SESSION_ID. Empty
+# SESSION_ID => no-op. Safe to call more than once per session.
+#
+# Usage: mark_announced "architect" "$SESSION_ID"
+mark_announced() {
+  local SYSTEM="$1"
+  local SESSION_ID="$2"
+  [ -n "$SESSION_ID" ] || return 0
+  : > "/tmp/${SYSTEM}-announced-${SESSION_ID}"
+}

package/hooks/tdd-inject.sh

@@ -1,15 +1,22 @@
 #!/bin/bash
-# TDD - UserPromptSubmit hook
+# TDD - UserPromptSubmit hook (P095 / ADR-038)
 # Injects TDD instructions and per-file state into every prompt.
 # Only active when a test script is configured in package.json.
+#
+# Progressive disclosure (ADR-038) with tdd-inject carve-out: static
+# prose (STATE RULES table, WORKFLOW, IMPORTANT blocks) is gated
+# behind the once-per-session announcement marker; dynamic content
+# (current TDD state + tracked test files) emits on every prompt.
 
 SCRIPT_DIR="$(cd "$(dirname "$0")" && pwd)"
 source "$SCRIPT_DIR/lib/tdd-gate.sh"
+# shellcheck source=lib/session-marker.sh
+source "$SCRIPT_DIR/lib/session-marker.sh"
 
 INPUT=$(cat)
 SESSION_ID=$(echo "$INPUT" | jq -r '.session_id // empty') || true
 
-# If no test script configured, inject setup instructions
+# If no test script configured, inject setup instructions (unchanged by ADR-038)
 if ! tdd_has_test_script; then
   cat <<'HOOK_OUTPUT'
 INSTRUCTION: MANDATORY TDD ENFORCEMENT. YOU MUST FOLLOW THIS.
@@ -25,13 +32,13 @@ HOOK_OUTPUT
   exit 0
 fi
 
-# Collect per-file states
+# Collect per-file states (always per-prompt)
 ALL_STATES=""
 if [ -n "$SESSION_ID" ]; then
   ALL_STATES=$(tdd_get_all_states "$SESSION_ID")
 fi
 
-# Determine overall status for the header
+# Determine overall status for the header (always per-prompt)
 OVERALL="IDLE"
 if [ -n "$ALL_STATES" ]; then
   if echo "$ALL_STATES" | grep -q ":BLOCKED$"; then
@@ -43,7 +50,14 @@ if [ -n "$ALL_STATES" ]; then
   fi
 fi
 
-cat <<HOOK_OUTPUT
+if has_announced "tdd" "$SESSION_ID"; then
+  # Subsequent prompt: terse reminder + dynamic state only.
+  cat <<HOOK_OUTPUT
+MANDATORY TDD gate active (test script present). Current TDD state: **${OVERALL}**. Write failing test before implementation; .ts/.tsx/.js/.jsx edits gated per test state. See turn-1 instructions for full rules and workflow.
+HOOK_OUTPUT
+else
+  # First prompt of session: full MANDATORY block + mark announced.
+  cat <<HOOK_OUTPUT
 INSTRUCTION: MANDATORY TDD ENFORCEMENT. YOU MUST FOLLOW THIS.
 
 This project enforces Red-Green-Refactor via hooks. Your current TDD state is: **${OVERALL}**
@@ -72,7 +86,10 @@ IMPORTANT:
 - The hook runs only the relevant test after each file write (not the full suite)
 - To refactor existing code, touch the relevant test file first to enter the cycle
 HOOK_OUTPUT
+  mark_announced "tdd" "$SESSION_ID"
+fi
 
+# Dynamic tracked test files (always per-prompt, both branches)
 if [ -n "$ALL_STATES" ]; then
   echo ""
   echo "TRACKED TEST FILES THIS SESSION:"

package/hooks/test/tdd-inject-once-per-session.bats

@@ -0,0 +1,107 @@
+#!/usr/bin/env bats
+
+# P095 / ADR-038: tdd-inject.sh UserPromptSubmit hook is the special
+# case in the cluster — dynamic TDD state (IDLE/RED/GREEN/BLOCKED +
+# tracked test files list) must be emitted on every prompt regardless
+# of announcement state. Only the static prose (STATE RULES, WORKFLOW,
+# IMPORTANT blocks) is gated by the once-per-session marker.
+#
+# Per ADR-038 tdd-inject carve-out.
+
+setup() {
+  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
+  HOOK="$REPO_ROOT/packages/tdd/hooks/tdd-inject.sh"
+
+  # Stub project working dir with a test script in package.json so the
+  # hook does NOT fall through to the "no test script" branch.
+  WORKDIR="$(mktemp -d)"
+  cat > "$WORKDIR/package.json" <<'JSON'
+{ "name": "test-tdd", "version": "0.0.0", "scripts": { "test": "echo pass" } }
+JSON
+
+  SID="tdd-inject-test-$$-$RANDOM"
+}
+
+teardown() {
+  rm -f "/tmp/tdd-announced-${SID}"
+  rm -f "/tmp/tdd-announced-${SID}-alt"
+  rm -rf "$WORKDIR"
+}
+
+run_hook() {
+  local sid="$1"
+  (cd "$WORKDIR" && echo "{\"session_id\":\"$sid\"}" | bash "$HOOK")
+}
+
+@test "tdd-inject: first invocation emits the full MANDATORY block with STATE RULES" {
+  run run_hook "$SID"
+  [ "$status" -eq 0 ]
+  [ "${#output}" -gt 1000 ]
+  [[ "$output" == *"MANDATORY TDD ENFORCEMENT"* ]]
+  [[ "$output" == *"STATE RULES"* ]]
+  [[ "$output" == *"WORKFLOW:"* ]]
+  [[ "$output" == *"IMPORTANT:"* ]]
+}
+
+@test "tdd-inject: first invocation writes the announcement marker" {
+  run run_hook "$SID"
+  [ -f "/tmp/tdd-announced-${SID}" ]
+}
+
+@test "tdd-inject: subsequent invocations drop the static prose (STATE RULES / WORKFLOW / IMPORTANT)" {
+  run_hook "$SID" >/dev/null
+  run run_hook "$SID"
+  [ "$status" -eq 0 ]
+  [ "${#output}" -lt 500 ]
+  [[ "$output" != *"STATE RULES"* ]]
+  [[ "$output" != *"WORKFLOW:"* ]]
+  [[ "$output" != *"IMPORTANT:"* ]]
+}
+
+@test "tdd-inject: subsequent invocations PRESERVE dynamic state (current TDD state line)" {
+  run_hook "$SID" >/dev/null
+  run run_hook "$SID"
+  [ "$status" -eq 0 ]
+  # Dynamic state line must still appear per-prompt. With no test files
+  # tracked yet, the overall state is IDLE.
+  [[ "$output" == *"IDLE"* ]]
+  # And the terse reminder itself carries the MANDATORY signal + the
+  # delegation affordance.
+  [[ "$output" == *"MANDATORY"* ]] || [[ "$output" == *"REQUIRED"* ]] || [[ "$output" == *"NON-OPTIONAL"* ]]
+  [[ "$output" == *"TDD"* ]]
+}
+
+@test "tdd-inject: terse reminder names the trigger artifact" {
+  run_hook "$SID" >/dev/null
+  run run_hook "$SID"
+  # Trigger for this gate is the test script in package.json.
+  [[ "$output" == *"test script"* ]] || [[ "$output" == *"package.json"* ]]
+}
+
+@test "tdd-inject: different session_id re-emits the full static block" {
+  run_hook "$SID" >/dev/null
+  local SID2="${SID}-alt"
+  run run_hook "$SID2"
+  [ "${#output}" -gt 1000 ]
+  [[ "$output" == *"STATE RULES"* ]]
+  rm -f "/tmp/tdd-announced-${SID2}"
+}
+
+@test "tdd-inject: empty session_id emits the full static block and writes no marker" {
+  run run_hook ""
+  [ "${#output}" -gt 1000 ]
+  [[ "$output" == *"STATE RULES"* ]]
+  [ ! -f "/tmp/tdd-announced-" ]
+}
+
+@test "tdd-inject: no-test-script fallback branch is unchanged by this ADR" {
+  local NO_TEST="$(mktemp -d)"
+  cat > "$NO_TEST/package.json" <<'JSON'
+{ "name": "no-test", "version": "0.0.0" }
+JSON
+  run bash -c "cd '$NO_TEST' && echo '{\"session_id\":\"$SID\"}' | bash '$HOOK'"
+  [ "$status" -eq 0 ]
+  [[ "$output" == *"MANDATORY TDD ENFORCEMENT"* ]]
+  [[ "$output" == *"NO test script"* ]]
+  rm -rf "$NO_TEST"
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/tdd",
-  "version": "0.2.3",
+  "version": "0.3.0",
   "description": "TDD state machine enforcement (Red-Green-Refactor cycle)",
   "bin": {
     "windyroad-tdd": "./bin/install.mjs"