@windyroad/style-guide 0.2.1 → 0.3.0-preview.176

package/.claude-plugin/plugin.json

@@ -1,5 +1,5 @@
 {
   "name": "wr-style-guide",
-  "version": "0.1.0",
+  "version": "0.3.0",
   "description": "Style guide enforcement 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/style-guide-eval.sh

@@ -1,10 +1,25 @@
 #!/bin/bash
-# Style Guide - UserPromptSubmit hook
+# Style Guide - UserPromptSubmit hook (P095 / ADR-038)
 # Detects STYLE-GUIDE.md in the project and injects delegation instruction.
 # If the file doesn't exist, instructs Claude to create it via the agent.
+#
+# Progressive disclosure (ADR-038): full MANDATORY block on first
+# prompt; terse reminder on subsequent prompts in the same session.
+
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+# shellcheck source=lib/session-marker.sh
+source "$SCRIPT_DIR/lib/session-marker.sh"
+
+INPUT=$(cat)
+SESSION_ID=$(echo "$INPUT" | jq -r '.session_id // empty' 2>/dev/null || echo "")
 
 if [ -f "docs/STYLE-GUIDE.md" ]; then
-  cat <<'HOOK_OUTPUT'
+  if has_announced "style-guide" "$SESSION_ID"; then
+    cat <<'HOOK_OUTPUT'
+MANDATORY style-guide gate active (docs/STYLE-GUIDE.md present). Delegate to wr-style-guide:agent before editing UI files (.css, .html, .jsx, .tsx, .vue, .svelte, .ejs, .hbs). See turn-1 instructions for full scope.
+HOOK_OUTPUT
+  else
+    cat <<'HOOK_OUTPUT'
 INSTRUCTION: MANDATORY STYLE GUIDE CHECK. YOU MUST FOLLOW THIS.
 DETECTED: docs/STYLE-GUIDE.md exists in this project.
 
@@ -22,6 +37,8 @@ REQUIRED ACTIONS:
 SCOPE: All .css and UI component files (.html, .jsx, .tsx, .vue, .svelte, .ejs, .hbs).
 Does NOT apply to: .ts/.js backend files, .md files, config files.
 HOOK_OUTPUT
+    mark_announced "style-guide" "$SESSION_ID"
+  fi
 else
   # Check if this is a web project (has UI files)
   if ls src/**/*.tsx src/**/*.jsx src/**/*.css src/**/*.html 2>/dev/null | head -1 | grep -q .; then

package/hooks/test/style-guide-eval-once-per-session.bats

@@ -0,0 +1,77 @@
+#!/usr/bin/env bats
+
+# P095 / ADR-038: style-guide-eval.sh UserPromptSubmit hook must emit
+# the full MANDATORY block only on the first prompt of a session;
+# subsequent prompts emit a terse reminder.
+
+setup() {
+  REPO_ROOT="$(cd "$(dirname "$BATS_TEST_FILENAME")/../../../.." && pwd)"
+  HOOK="$REPO_ROOT/packages/style-guide/hooks/style-guide-eval.sh"
+
+  WORKDIR="$(mktemp -d)"
+  mkdir -p "$WORKDIR/docs"
+  : > "$WORKDIR/docs/STYLE-GUIDE.md"
+
+  SID="sg-eval-test-$$-$RANDOM"
+}
+
+teardown() {
+  rm -f "/tmp/style-guide-announced-${SID}"
+  rm -f "/tmp/style-guide-announced-${SID}-alt"
+  rm -rf "$WORKDIR"
+}
+
+run_hook() {
+  local sid="$1"
+  (cd "$WORKDIR" && echo "{\"session_id\":\"$sid\"}" | bash "$HOOK")
+}
+
+@test "style-guide-eval: first invocation emits the full MANDATORY block" {
+  run run_hook "$SID"
+  [ "$status" -eq 0 ]
+  [ "${#output}" -gt 400 ]
+  [[ "$output" == *"MANDATORY STYLE GUIDE CHECK"* ]]
+  [[ "$output" == *"wr-style-guide:agent"* ]]
+}
+
+@test "style-guide-eval: first invocation writes the announcement marker" {
+  run run_hook "$SID"
+  [ -f "/tmp/style-guide-announced-${SID}" ]
+}
+
+@test "style-guide-eval: second invocation emits only a terse reminder" {
+  run_hook "$SID" >/dev/null
+  run run_hook "$SID"
+  [ "${#output}" -lt 250 ]
+  [[ "$output" == *"style-guide"* ]] || [[ "$output" == *"STYLE"* ]]
+  [[ "$output" == *"wr-style-guide:agent"* ]]
+  [[ "$output" != *"REQUIRED ACTIONS:"* ]]
+}
+
+@test "style-guide-eval: terse reminder preserves the MANDATORY signal word" {
+  run_hook "$SID" >/dev/null
+  run run_hook "$SID"
+  [[ "$output" == *"MANDATORY"* ]] || [[ "$output" == *"REQUIRED"* ]] || [[ "$output" == *"NON-OPTIONAL"* ]]
+}
+
+@test "style-guide-eval: terse reminder names the trigger artifact" {
+  run_hook "$SID" >/dev/null
+  run run_hook "$SID"
+  [[ "$output" == *"STYLE-GUIDE.md"* ]]
+}
+
+@test "style-guide-eval: different session_id re-emits the full block" {
+  run_hook "$SID" >/dev/null
+  local SID2="${SID}-alt"
+  run run_hook "$SID2"
+  [ "${#output}" -gt 400 ]
+  [[ "$output" == *"MANDATORY STYLE GUIDE CHECK"* ]]
+  rm -f "/tmp/style-guide-announced-${SID2}"
+}
+
+@test "style-guide-eval: empty session_id emits the full block and writes no marker" {
+  run run_hook ""
+  [ "${#output}" -gt 400 ]
+  [[ "$output" == *"MANDATORY STYLE GUIDE CHECK"* ]]
+  [ ! -f "/tmp/style-guide-announced-" ]
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/style-guide",
-  "version": "0.2.1",
+  "version": "0.3.0-preview.176",
   "description": "Style guide enforcement for CSS and UI components",
   "bin": {
     "windyroad-style-guide": "./bin/install.mjs"