@windyroad/architect 0.15.4 → 0.15.5

package/.claude-plugin/plugin.json

@@ -123,5 +123,5 @@
     }
   },
   "name": "wr-architect",
-  "version": "0.15.4"
+  "version": "0.15.5"
 }

package/hooks/architect-slide-marker.sh

@@ -1,16 +1,18 @@
 #!/bin/bash
-# Architecture - PostToolUse:Agent|Bash slide-marker hook (P111).
+# Architecture - PostToolUse:Agent|Bash|Skill slide-marker hook (P111 + P213).
 # Slides the parent session's existing architect-reviewed marker forward on
 # subprocess return, treating subprocess wall-clock as continuous parent-
 # session work for TTL purposes. Only TOUCHES an existing marker — never
 # creates one (creation requires a real architect review parsed from the
 # agent's verdict text in architect-mark-reviewed.sh).
 #
-# This addresses P111 / ADR-009 "Subprocess-boundary refresh": Agent and Bash
-# tool calls that wrap long-running subprocesses (other subagents, `claude
-# -p` iteration subprocesses, run_in_background completions) would otherwise
-# let the parent's marker age past TTL even though the parent is still
-# actively working through the subprocess.
+# This addresses P111 / ADR-009 "Subprocess-boundary refresh": Agent, Bash,
+# and Skill tool calls that wrap long-running subprocesses (other subagents,
+# `claude -p` iteration subprocesses, run_in_background completions, or
+# /wr-*:assess-* sibling-assessor SKILLs run by the AFK orchestrator) would
+# otherwise let the parent's marker age past TTL even though the parent is
+# still actively working through the subprocess. The Skill matcher coverage
+# is the 2026-06-08 P213 amendment (Option D).
 #
 # Failed subprocesses (tool_response.is_error=true) do NOT extend the trust
 # window — see slide_marker_on_subprocess_return in lib/gate-helpers.sh.

package/hooks/hooks.json

@@ -15,7 +15,7 @@
     "PostToolUse": [
       { "matcher": "Agent", "hooks": [{ "type": "command", "command": "${CLAUDE_PLUGIN_ROOT}/hooks/architect-mark-reviewed.sh" }] },
       { "matcher": "Edit|Write", "hooks": [{ "type": "command", "command": "${CLAUDE_PLUGIN_ROOT}/hooks/architect-refresh-hash.sh" }] },
-      { "matcher": "Agent|Bash", "hooks": [{ "type": "command", "command": "${CLAUDE_PLUGIN_ROOT}/hooks/architect-slide-marker.sh" }] }
+      { "matcher": "Agent|Bash|Skill", "hooks": [{ "type": "command", "command": "${CLAUDE_PLUGIN_ROOT}/hooks/architect-slide-marker.sh" }] }
     ]
   }
 }

package/hooks/test/architect-slide-marker.bats

@@ -76,3 +76,17 @@ _backdate() {
   AGE=$((NOW - AFTER))
   [ "$AGE" -lt 5 ]
 }
+
+@test "hook: P213 — slides on Skill tool_name (Agent|Bash|Skill matcher expansion)" {
+  # Confirms hooks.json matcher expansion (Agent|Bash → Agent|Bash|Skill)
+  # composes correctly with the existing hook script. The hook script does
+  # not filter on tool_name; the matcher in hooks.json gates which tool
+  # completions fire the script. This test asserts the script handles the
+  # Skill-shaped tool_response identically to Agent|Bash.
+  touch "$REVIEW_MARKER"
+  _backdate "$REVIEW_MARKER" 60
+  BEFORE=$(stat -c%Y "$REVIEW_MARKER" 2>/dev/null || /usr/bin/stat -f%m "$REVIEW_MARKER")
+  echo '{"session_id":"'"$TEST_SESSION"'","tool_name":"Skill","tool_input":{"skill":"wr-risk-scorer:assess-release"},"tool_response":{"content":[{"type":"text","text":"OK"}]}}' | "$HOOK"
+  AFTER=$(stat -c%Y "$REVIEW_MARKER" 2>/dev/null || /usr/bin/stat -f%m "$REVIEW_MARKER")
+  [ "$AFTER" -gt "$BEFORE" ]
+}

package/hooks/test/slide-marker-on-subprocess-return.bats

@@ -98,3 +98,21 @@ _backdate() {
   # Fail-safe: when the hook input cannot be parsed, treat as error and skip
   [ "$BEFORE" = "$AFTER" ]
 }
+
+@test "slide: triggers correctly on Skill tool_response shape (P213, ADR-009 2026-06-08 amendment)" {
+  # hooks.json matcher expansion Agent|Bash → Agent|Bash|Skill (P213 Option D)
+  # widens slide-marker coverage to PostToolUse:Skill completions (e.g. the
+  # /wr-risk-scorer:assess-* sibling assessor SKILLs run as long subprocesses
+  # by the AFK orchestrator). The Skill tool_response shape is identical to
+  # Agent|Bash (Claude Code's uniform PostToolUse contract), so the matcher-
+  # agnostic helper composes without code changes. This test documents that
+  # contract explicitly so a future hook_input shape divergence regression
+  # surfaces here rather than at the gate-denial site.
+  touch "$MARKER"
+  _backdate "$MARKER" 60
+  BEFORE=$(_mtime "$MARKER")
+  _HOOK_INPUT='{"tool_name":"Skill","tool_response":{"content":[{"type":"text","text":"OK"}]}}'
+  slide_marker_on_subprocess_return "$MARKER"
+  AFTER=$(_mtime "$MARKER")
+  [ "$AFTER" -gt "$BEFORE" ]
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@windyroad/architect",
-  "version": "0.15.4",
+  "version": "0.15.5",
   "description": "Architecture decision enforcement for AI coding agents",
   "bin": {
     "windyroad-architect": "./bin/install.mjs"