@iceinvein/agent-skills 0.16.0 → 0.17.0

package/dist/cli/index.js

@@ -299,6 +299,9 @@ function validateManifest(data) {
     if (a.claudeHookScript !== undefined && typeof a.claudeHookScript !== "string") {
       return { ok: false, error: "'activation.claudeHookScript' must be a string" };
     }
+    if (a.claudeStopScript !== undefined && typeof a.claudeStopScript !== "string") {
+      return { ok: false, error: "'activation.claudeStopScript' must be a string" };
+    }
   }
   return { ok: true, manifest: d };
 }
@@ -522,6 +525,51 @@ async function unwireSessionStartHook(settingsPath, skillName, directive) {
   await Bun.write(settingsPath, JSON.stringify(settings, null, 2) + `
 `);
 }
+async function wireStopHook(settingsPath, skillName, scriptPath) {
+  let settings = {};
+  if (existsSync2(settingsPath)) {
+    settings = await Bun.file(settingsPath).json();
+  }
+  if (!settings.hooks)
+    settings.hooks = {};
+  if (!settings.hooks.Stop)
+    settings.hooks.Stop = [];
+  const command = `if [ -f ${shq(scriptPath)} ]; then bash ${shq(scriptPath)}; fi`;
+  let found = false;
+  for (const group of settings.hooks.Stop) {
+    for (const hook of group.hooks ?? []) {
+      if (hook.skill !== skillName)
+        continue;
+      found = true;
+      hook.command = command;
+    }
+  }
+  if (!found) {
+    settings.hooks.Stop.push({ hooks: [{ type: "command", command, skill: skillName }] });
+  }
+  mkdirSync(dirname(settingsPath), { recursive: true });
+  await Bun.write(settingsPath, JSON.stringify(settings, null, 2) + `
+`);
+}
+async function unwireStopHook(settingsPath, skillName) {
+  if (!existsSync2(settingsPath))
+    return;
+  const settings = await Bun.file(settingsPath).json();
+  const stop = settings.hooks?.Stop;
+  if (!stop)
+    return;
+  const filtered = stop.map((group) => ({ hooks: (group.hooks ?? []).filter((h) => h.skill !== skillName) })).filter((group) => group.hooks.length > 0);
+  if (filtered.length === 0) {
+    delete settings.hooks.Stop;
+  } else {
+    settings.hooks.Stop = filtered;
+  }
+  if (settings.hooks && Object.keys(settings.hooks).length === 0) {
+    delete settings.hooks;
+  }
+  await Bun.write(settingsPath, JSON.stringify(settings, null, 2) + `
+`);
+}
 var claudeAdapter = {
   name: "claude",
   async install(cwd, manifest, files, activation) {
@@ -584,6 +632,13 @@ var claudeAdapter = {
         installed.push(".claude/settings.json");
       }
     }
+    if (activation === "global" && manifest.activation?.claudeStopScript && config.bundleRoot) {
+      const settingsPath = join2(cwd, ".claude/settings.json");
+      await wireStopHook(settingsPath, manifest.name, join2(cwd, config.bundleRoot, manifest.activation.claudeStopScript));
+      if (!installed.includes(".claude/settings.json")) {
+        installed.push(".claude/settings.json");
+      }
+    }
     if (config.postinstall && config.bundleRoot) {
       const scriptPath = join2(cwd, config.bundleRoot, config.postinstall);
       const result = runScript(scriptPath, join2(cwd, config.bundleRoot));
@@ -646,6 +701,9 @@ var claudeAdapter = {
       const settingsPath = join2(cwd, ".claude/settings.json");
       await unwireSessionStartHook(settingsPath, manifest.name, manifest.activation.claudeHookDirective);
     }
+    if (manifest.activation?.claudeStopScript) {
+      await unwireStopHook(join2(cwd, ".claude/settings.json"), manifest.name);
+    }
   }
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iceinvein/agent-skills",
-  "version": "0.16.0",
+  "version": "0.17.0",
   "description": "Install agent skills into AI coding tools",
   "author": "iceinvein",
   "license": "MIT",

package/skills/index.json

@@ -283,7 +283,7 @@
     "name": "sluice",
     "description": "Routes work by change shape into four channels (bypass, fast, main, deep) and applies only the rules each channel needs, so a one-line fix does not pay the cost of a multi-subsystem build. Carries seven rules as one-liners in the router and the full treatment in references read only on friction. Checks the finished plan with plan.sh validate rather than trusting it to memory, seeds the run state from it, keeps a deep run's task breakdown in .sluice/run.json so a statusline segment, one status command and a SessionStart hook can answer where the run is (the hook prints a live run at every session start, compaction included), and closes each run with a ledger read out of the session transcript: elapsed, tools, tokens, and what each dispatched agent cost where the transcript recorded it. Claude Code only; stands down where the superpowers pipeline governs the repo.",
     "type": "prompt",
-    "version": "0.17.0"
+    "version": "0.18.0"
   },
   {
     "name": "temporal-coupling-detector",

package/skills/sluice/SKILL.md

@@ -171,7 +171,12 @@ yours to act on, after a local merge, when the branch is left as it stands, or
 when an open PR lands, not when it opens. A run left open reads as live to the
 statusline and blocks the next one. The SessionStart hook prints a live run at
 every session start, compaction and resume included, so a run you did not
-start is one you were shown, not one you have to remember.
+start is one you were shown, not one you have to remember. A Stop hook refuses,
+once per turn, to end a turn while a `deep` run in this tree is past pre-flight
+with tasks still to go and nothing `blocked` or paused:
+`references/deep-channel.md` says why a run never ends a turn between
+pre-flight and the handback, and `status.sh pause --reason` is how one stands
+still on purpose.
 
 ## Conflicts
 

package/skills/sluice/references/deep-channel.md

@@ -289,6 +289,22 @@ reads the run state rather than the plan: it sees what has actually landed.
   compaction; your memory doesn't.
 - Each task goes to a fresh agent carrying the brief below and nothing this
   session accumulated. What you hold is yours to hold, not theirs.
+- **The run never ends a turn between pre-flight and the handback.** A
+  message with no tool call in it ends the turn, whatever it says, and a run
+  handed back that way stands still until your partner notices, which
+  overnight is the next morning. So an announcement rides in the same message
+  as the dispatch it announces, a wave's completion is followed in the same
+  message by `status.sh ready` and the next dispatch, and "T4 goes next" is
+  never the last thing a message says. The turns that do end are the two
+  stops, a task marked `blocked` because it genuinely needs your partner, and
+  the handback. Everything else that has to wait on them goes through one of
+  those two doors: a finding still open after three review rounds marks its
+  task `blocked`, and re-dispatching it flips the row back to `active` when
+  your partner has answered; a mid-run request for a dispatch, or a
+  show-or-say offer, is a pause, `status.sh pause --reason "<why>"`, so the reason is on disk and
+  the Stop hook lets you go, with `resume` when it moves again. The hook
+  refuses once per turn and then lets the next attempt through, so it is a
+  nudge with the state in it rather than a wall: the rule is yours to keep.
 - **Label the dispatch `T<n>: <task name>`.** The harness lists running agents
   under whatever label the dispatch gave them, so labelled by task that list
   reads as the plan and labelled anything else it reads as a row of anonymous

package/skills/sluice/references/review.md

@@ -37,7 +37,8 @@ offer, so neither is that pass.
 Send findings back to the agent that wrote the code: it already holds the
 task and its reasoning, memory you would otherwise rebuild. Three rounds is
 the cap, and a finding still open when the third one ends is structural, not
-local, so stop there and hand it to your partner.
+local, so stop there and hand it to your partner; in `deep`, mark the task
+`blocked` first, which is what lets that stop through.
 
 Receiving a finding: check it against the codebase before acting, and argue
 back with specifics when it is wrong. Agreeing just to move things along is

package/skills/sluice/references/show-or-say.md

@@ -3,7 +3,8 @@
 This is never offered at the start. What triggers it is a specific moment in
 the conversation: a question arrives that turns on how something looks rather
 than on what it means. Offer then, in a message carrying nothing else, and
-wait. Plenty of conversations never raise such a question, and in those the
+wait; inside a `deep` run, `status.sh pause --reason` first, since the Stop
+hook otherwise reads that wait as a stalled run. Plenty of conversations never raise such a question, and in those the
 offer is simply never made.
 
 Apply the test to each question rather than deciding once: could you settle

package/skills/sluice/references/status.md

@@ -22,6 +22,8 @@ bash <skill-dir>/scripts/status.sh show
 bash <skill-dir>/scripts/status.sh ready
 bash <skill-dir>/scripts/status.sh final
 bash <skill-dir>/scripts/status.sh move --to <worktree>
+bash <skill-dir>/scripts/status.sh pause --reason "waiting on the API key"
+bash <skill-dir>/scripts/status.sh resume
 bash <skill-dir>/scripts/status.sh line --full
 bash <skill-dir>/scripts/status.sh close
 ```
@@ -98,7 +100,7 @@ submodule anchors on its own checkout, not the superproject's, and a directory
 that is no git work tree keeps its run exactly where it sits.
 
 One file for several writers is one file to contend on, so `init`, `task`,
-`preflight`, `final`, `close` and `move` take a lock first, `move` taking the
+`preflight`, `final`, `pause`, `resume`, `close` and `move` take a lock first, `move` taking the
 destination tree's as well as its own: two flips issued at the same moment
 from different trees would otherwise have the later write built on a snapshot
 taken before the earlier one landed, dropping that row without saying so. The lock
@@ -243,6 +245,33 @@ start, that a run not being continued was left open and wants `close`. A tree
 with no run prints nothing. The reading-back rule above still stands; this is
 the harness doing it at the one moment memory has just been cut.
 
+## On stopping
+
+`scripts/stop-guard.sh` is the Stop hook a global install wires. When the model
+tries to end its turn it reads the run in the session's own tree, the git
+top level of the session's working directory, and refuses, with a reason, when
+a `deep` run there is past pre-flight, has tasks still `todo`, `active` or
+`review`, and nothing is `blocked` or paused. The reason says what to do
+instead: dispatch the next wave in the same message, mark the task that needs
+your partner `blocked`, `pause --reason` and say so, or `close` a run that is
+not this session's work. Every real stop is let through: no run in the
+session's tree (the main-tree fallback other commands use is not taken here,
+since a Stop in a runless worktree may be an unrelated session), a channel
+other than `deep`, pre-flight not yet recorded, a blocked task, a paused run,
+every task done, a run idle for a day, and a turn where the harness says a
+stop hook already fired, which is what keeps it from looping. That last rule
+means it refuses once per turn and lets the next attempt through: a nudge, not
+a wall. The gate keys on the git top level of the session's working
+directory and nothing else, so the run has to live in the tree the session
+works in: open it after the worktree is cut, or `move` it there and then enter
+that worktree, since `move` relocates the run and not the session, and a run
+moved out from under a session still sitting in the main tree leaves that
+session unguarded for the rest of the run.
+
+`pause --reason <text>` records why a run is standing still; `show` and the
+statusline carry it, and `resume` clears it. A pause with no reason is refused,
+since the reason is the only thing that separates a pause from a stall.
+
 ## Statusline
 
 This is the part that makes a run visible without anyone asking. Give it rows of

package/skills/sluice/scripts/status.sh

@@ -17,6 +17,8 @@
 #   status.sh ready
 #   status.sh final
 #   status.sh move --to <tree>
+#   status.sh pause --reason <text>
+#   status.sh resume
 #   status.sh line [--full]
 #   status.sh close
 #
@@ -148,7 +150,8 @@ if [ "$SUB" = "line" ]; then
 			    (.channel // "?"),
 			    "\($done)/\(.tasks | length)",
 			    ([.tasks[]? | select(.status == "active") | "▸T\(.id)"] | first // empty),
-			    ([.tasks[]? | select(.status == "blocked") | "!T\(.id)"] | first // empty)
+			    ([.tasks[]? | select(.status == "blocked") | "!T\(.id)"] | first // empty),
+			    (if .paused then "paused" else empty end)
 			  ] | join(" ")
 		' "$STATE" 2>/dev/null || exit 0
 		exit 0
@@ -231,6 +234,7 @@ if [ "$SUB" = "line" ]; then
 		       ] | join_parts)
 		    + (if $clock == "" then "" else "   " + paint("2"; $clock) end)
 		    + (if $idle == "" then "" else " " + paint("2"; "·") + " " + paint("33"; $idle) end)
+		    + (if .paused then " " + paint("2"; "·") + " " + paint("33"; "paused") else "" end)
 		  ),
 		  # The flip is drawn as a rule before its task: everything left of it is
 		  # inert and safe to leave landed, everything right of it is not. That is
@@ -581,6 +585,7 @@ case "$SUB" in
 			            else ["idle          \($h / 24 | floor)d\($h % 24)h since the last write"]
 			            end
 			     end)
+			+ (if .paused then ["paused        \(.paused)"] else [] end)
 			+ (([.tasks[]? | select(.status == "done" and (.tier // 0) >= 1 and (.reviewed // false) == false)] | length) as $debt
 			   | if $debt == 0 then [] else ["unreviewed    \($debt) done, owed a review the tier table promised"] end)
 			+ ["final review  " + (if .final_review then "done" else "pending" end)]
@@ -682,6 +687,33 @@ case "$SUB" in
 		jq --arg now "$(date -u +%Y-%m-%dT%H:%M:%SZ)" '.final_review = $now' "$STATE" | write_state
 		;;
 
+	pause)
+		REASON=""
+		while [ $# -gt 0 ]; do
+			case "$1" in
+				--reason) need_value --reason $# "${2-}"; REASON="$2"; shift 2 ;;
+				*) err "unknown flag: $1"; exit 4 ;;
+			esac
+		done
+		[ -n "$REASON" ] || { err "pause needs --reason <text>: a pause nobody can read the reason for is a stall"; exit 4; }
+		require_run
+		take_lock
+		require_readable
+
+		# A deliberate handback mid-run, which the stop guard otherwise refuses.
+		# The reason is the whole point: it is what the partner reads in `show`
+		# and what the next session reads to know why the run is standing still.
+		jq --arg reason "$REASON" '.paused = $reason' "$STATE" | write_state
+		;;
+
+	resume)
+		[ $# -eq 0 ] || { err "resume takes no arguments"; exit 4; }
+		require_run
+		take_lock
+		require_readable
+		jq 'del(.paused)' "$STATE" | write_state
+		;;
+
 	move)
 		TO=""
 		while [ $# -gt 0 ]; do

package/skills/sluice/scripts/stop-guard.sh

@@ -0,0 +1,97 @@
+#!/usr/bin/env bash
+# Stop hook: refuse to end the turn in the middle of a deep run.
+#
+# A turn ends the moment a message carries no tool call, and a run handed back
+# that way has nothing in it for the partner to decide: it just stands still
+# until they notice, which overnight is the next morning. So when a deep run is
+# past pre-flight, has tasks still to go, and nothing is marked blocked or
+# paused, the stop is refused with a reason saying what to do instead.
+#
+# Every stop that is a real stop is let through: no run, a channel other than
+# deep, pre-flight not yet answered (that stop is owed), a blocked task, a run
+# paused on purpose with `status.sh pause --reason`, every task done (the
+# handback), a run idle for a day, a run that lives in another tree than the
+# session's, and any attempt where the harness says a stop hook already fired
+# this turn, which is what keeps this from looping. One refusal per turn, then:
+# a nudge with the state in it rather than a wall.
+#
+# Reads the harness's stop JSON on stdin for `cwd` and `stop_hook_active`. To
+# refuse, prints {"decision":"block","reason":...} on stdout. Always exits 0.
+
+set -uo pipefail
+
+here="$(cd "$(dirname "$0")" && pwd)"
+STATUS="$here/status.sh"
+
+# jq first: without it nothing below can run, and the stdin read that follows
+# has a worst case worth not paying for nothing.
+command -v jq >/dev/null 2>&1 || exit 0
+
+# Byte-wise and bounded, for the reason session-start.sh gives: bash 3.2 drops
+# a timed-out partial read, and an inherited open stdin must not hang the stop.
+input=""
+n=0
+if [ ! -t 0 ]; then
+	while IFS= read -r -n 1 -t 2 c; do
+		if [ -z "$c" ]; then input="$input
+"; else input="$input$c"; fi
+		# Appending a byte at a time is quadratic, and -t bounds each byte rather
+		# than the total. Real stop JSON is under a kilobyte; past this the input
+		# is not the harness's and is not worth reading on.
+		n=$((n + 1))
+		[ "$n" -lt 65536 ] || break
+	done
+fi
+
+cwd="$PWD"
+active="false"
+if [ -n "$input" ]; then
+	got="$(printf '%s' "$input" | jq -r '.cwd // empty' 2>/dev/null || true)"
+	[ -n "$got" ] && cwd="$got"
+	active="$(printf '%s' "$input" | jq -r '.stop_hook_active // false' 2>/dev/null || echo false)"
+fi
+[ "$active" = "true" ] && exit 0
+[ -d "$cwd" ] || exit 0
+[ -f "$STATUS" ] || exit 0
+
+# The session's own tree, and only that. status.sh lets a tree with no run of
+# its own read the main worktree's, for a controller that moved after init; a
+# Stop in such a tree may be an unrelated session, and a remedy printed to it
+# would reach into somebody else's run. So the run has to sit in the tree the
+# session's cwd belongs to, or there is nothing here to guard.
+top="$(git -C "$cwd" rev-parse --show-toplevel 2>/dev/null)"
+[ -n "$top" ] || top="$cwd"
+[ -f "$top/.sluice/run.json" ] || exit 0
+
+run="$(bash "$STATUS" show --json --dir "$top" 2>/dev/null)" || exit 0
+[ -n "$run" ] || exit 0
+
+# One JSON object out, read back with jq: the topic is user text, and word
+# splitting it would truncate at the first space and glob on the rest.
+verdict="$(printf '%s' "$run" | jq -c --argjson now "$(date -u +%s)" '
+	(.tasks // []) as $t
+	| ([$t[] | select(.status == "done")] | length) as $done
+	| ([$t[] | select(.status == "blocked")] | length) as $blocked
+	| ([$t[] | select(.status == "todo" or .status == "active" or .status == "review")] | length) as $open
+	| ((.updated // .started // "" | try fromdateiso8601 catch 0) as $u
+	   | if $u == 0 then 0 else (($now - $u) / 3600 | floor) end) as $idle_h
+	| if (.channel // "") != "deep" then {block: false}
+	  elif ((.preflight // {}) | length) == 0 then {block: false}
+	  elif .paused then {block: false}
+	  elif ($t | length) == 0 then {block: false}
+	  elif $blocked > 0 then {block: false}
+	  elif $open == 0 then {block: false}
+	  # A run nobody has written to for a day is a stale run, not a live one;
+	  # refusing its stop would press an abandoned plan on whoever opened here.
+	  elif $idle_h >= 24 then {block: false}
+	  else {block: true, progress: "\($done)/\($t | length)", topic: (.topic // "run")}
+	  end
+' 2>/dev/null)" || exit 0
+
+[ "$(printf '%s' "$verdict" | jq -r '.block' 2>/dev/null)" = "true" ] || exit 0
+
+printf '%s' "$verdict" | jq 2>/dev/null '{
+	decision: "block",
+	reason: ("sluice: the deep run \(.topic) is \(.progress) done with tasks still to go and nothing marked blocked or paused, so ending the turn here hands a live run back with nothing for your partner to decide. Continue: run status.sh ready and dispatch the next wave in this same message. If a task genuinely needs them, mark it: status.sh task <id> --status blocked. If the run has to stand still for a reason, record it: status.sh pause --reason \"<why>\", then say so and stop. If this run is not the work you were asked to do, it was left open: status.sh close.")
+}'
+exit 0

package/skills/sluice/skill.json

@@ -1,6 +1,6 @@
 {
   "name": "sluice",
-  "version": "0.17.0",
+  "version": "0.18.0",
   "description": "Routes work by change shape into four channels (bypass, fast, main, deep) and applies only the rules each channel needs, so a one-line fix does not pay the cost of a multi-subsystem build. Carries seven rules as one-liners in the router and the full treatment in references read only on friction. Checks the finished plan with plan.sh validate rather than trusting it to memory, seeds the run state from it, keeps a deep run's task breakdown in .sluice/run.json so a statusline segment, one status command and a SessionStart hook can answer where the run is (the hook prints a live run at every session start, compaction included), and closes each run with a ledger read out of the session transcript: elapsed, tools, tokens, and what each dispatched agent cost where the transcript recorded it. Claude Code only; stands down where the superpowers pipeline governs the repo.",
   "author": "iceinvein",
   "type": "prompt",
@@ -29,6 +29,7 @@
     ],
     "default": "global",
     "claudeHookDirective": "Before acting on a request that changes code, pick a sluice channel and state which one.",
-    "claudeHookScript": "scripts/session-start.sh"
+    "claudeHookScript": "scripts/session-start.sh",
+    "claudeStopScript": "scripts/stop-guard.sh"
   }
 }