@iceinvein/agent-skills 0.16.0 → 0.18.0

package/dist/cli/index.js

@@ -299,6 +299,23 @@ 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" };
+    }
+    if (a.claudeStatuslineScript !== undefined && typeof a.claudeStatuslineScript !== "string") {
+      return { ok: false, error: "'activation.claudeStatuslineScript' must be a string" };
+    }
+    const claudeBundleRoot = d.install.claude?.bundleRoot;
+    if (claudeBundleRoot === undefined) {
+      for (const field of ["claudeHookScript", "claudeStopScript", "claudeStatuslineScript"]) {
+        if (a[field] !== undefined) {
+          return {
+            ok: false,
+            error: `'activation.${field}' is resolved against 'install.claude.bundleRoot', which is not set`
+          };
+        }
+      }
+    }
   }
   return { ok: true, manifest: d };
 }
@@ -522,6 +539,94 @@ 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) + `
+`);
+}
+function statusLineCommand(scriptPath) {
+  return `if [ -f ${shq(scriptPath)} ]; then bash ${shq(scriptPath)}; fi`;
+}
+function ownsStatusLine(command, skillName, scriptPath) {
+  if (typeof command !== "string")
+    return false;
+  if (scriptPath !== undefined && command === statusLineCommand(scriptPath))
+    return true;
+  const shape = /^if \[ -f '(.*)' \]; then bash '(.*)'; fi$/.exec(command);
+  if (shape === null || shape[1] !== shape[2])
+    return false;
+  return shape[1].includes(`/skills/${skillName}/`);
+}
+async function wireStatusLine(settingsPath, skillName, scriptPath) {
+  let settings = {};
+  if (existsSync2(settingsPath)) {
+    settings = await Bun.file(settingsPath).json();
+  }
+  const existing = settings.statusLine;
+  if (existing !== undefined && existing !== null) {
+    if (!ownsStatusLine(existing?.command, skillName, scriptPath))
+      return;
+  }
+  const command = statusLineCommand(scriptPath);
+  if (existing?.type === "command" && existing?.command === command)
+    return;
+  settings.statusLine = { type: "command", command };
+  mkdirSync(dirname(settingsPath), { recursive: true });
+  await Bun.write(settingsPath, JSON.stringify(settings, null, 2) + `
+`);
+}
+async function unwireStatusLine(settingsPath, skillName, scriptPath) {
+  if (!existsSync2(settingsPath))
+    return;
+  const settings = await Bun.file(settingsPath).json();
+  if (settings.statusLine === undefined || settings.statusLine === null)
+    return;
+  if (!ownsStatusLine(settings.statusLine?.command, skillName, scriptPath))
+    return;
+  delete settings.statusLine;
+  await Bun.write(settingsPath, JSON.stringify(settings, null, 2) + `
+`);
+}
 var claudeAdapter = {
   name: "claude",
   async install(cwd, manifest, files, activation) {
@@ -584,6 +689,20 @@ 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 (activation === "global" && manifest.activation?.claudeStatuslineScript && config.bundleRoot) {
+      const settingsPath = join2(cwd, ".claude/settings.json");
+      await wireStatusLine(settingsPath, manifest.name, join2(cwd, config.bundleRoot, manifest.activation.claudeStatuslineScript));
+      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 +765,12 @@ 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);
+    }
+    if (manifest.activation?.claudeStatuslineScript && config.bundleRoot) {
+      await unwireStatusLine(join2(cwd, ".claude/settings.json"), manifest.name, join2(cwd, config.bundleRoot, manifest.activation.claudeStatuslineScript));
+    }
   }
 };
 
@@ -1085,8 +1210,9 @@ async function removeSkill(cwd, skillName) {
       const fullPath = join7(cwd, file);
       if (existsSync6(fullPath)) {
         await unwireSessionStartHook(fullPath, skillName);
+        await unwireStatusLine(fullPath, skillName);
       }
-      warnings.push(`Manifest unavailable (${manifestResult.error}). Left '${file}' in place, only removing '${skillName}'s SessionStart hook from it; any MCP server entries it owns were not touched.`);
+      warnings.push(`Manifest unavailable (${manifestResult.error}). Left '${file}' in place, removing '${skillName}'s SessionStart hook and status line from it; any MCP server entries it owns were not touched.`);
       continue;
     }
     const fullPath = join7(cwd, file);
@@ -1178,8 +1304,9 @@ async function removeSkill2(cwd, skillName) {
       const fullPath = join8(cwd, file);
       if (existsSync7(fullPath)) {
         await unwireSessionStartHook(fullPath, skillName);
+        await unwireStatusLine(fullPath, skillName);
       }
-      warnings.push(`Manifest unavailable (${manifestResult.error}). Left '${file}' in place, only removing '${skillName}'s SessionStart hook from it; any MCP server entries it owns were not touched.`);
+      warnings.push(`Manifest unavailable (${manifestResult.error}). Left '${file}' in place, removing '${skillName}'s SessionStart hook and status line from it; any MCP server entries it owns were not touched.`);
       continue;
     }
     const fullPath = join8(cwd, file);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@iceinvein/agent-skills",
-  "version": "0.16.0",
+  "version": "0.18.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.19.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,51 @@ 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.
+
+## Free text
+
+Every value a command stores is later drawn onto a terminal: by the statusline,
+by `show`, by the SessionStart hook and by the stop guard's refusal. A control
+character in one is not text there, it is an instruction the terminal obeys, and
+task names are model-written and routinely pasted out of issue titles. `ESC[2J`
+in a name clears the screen on every render for as long as the run is open; a
+carriage return walks the cursor back over the row just drawn.
+
+So every flag value is refused if it carries one, with exit 4 naming the flag,
+rather than being quietly stripped: a name that is not the name the caller
+passed is its own surprise, and no legitimate value has ever needed a control
+byte. `plan.sh import` writes task names through the same door and inherits it.
+
+The renders drop them as well. State written before the check existed, or edited
+by hand, is an input like any other, and the only control bytes a render should
+emit are the colour sequences it writes itself.
+
 ## Statusline
 
 This is the part that makes a run visible without anyone asking. Give it rows of
@@ -250,20 +297,11 @@ its own rather than a segment among the badges: it then costs nothing when no ru
 is live and contends with nothing for width when one is, which is what lets the
 bar be wide and the task carry its name rather than only its number.
 
-Capture it wherever the statusline command builds its other lines, keyed off the
-state file existing so a session with no run spawns no process at all:
+Capture it wherever the statusline command builds its other lines, passing the
+directory the session is in and nothing else:
 
 ```bash
-sluice_line=""
-if [ -n "$cwd" ] && { [ -f "$cwd/.sluice/run.json" ] || [ -f "$cwd/.git" ]; }; then
-  for sluice_sh in "$cwd/.claude/skills/sluice/scripts/status.sh" \
-                   "${CLAUDE_CONFIG_DIR:-$HOME/.claude}/skills/sluice/scripts/status.sh" \
-                   "$HOME/.claude/skills/sluice/scripts/status.sh"; do
-    [ -f "$sluice_sh" ] || continue
-    sluice_line=$(bash "$sluice_sh" line --full --dir "$cwd" 2>/dev/null)
-    break
-  done
-fi
+sluice_line=$(bash "$HOME/.claude/skills/sluice/scripts/statusline.sh" --dir "$cwd" 2>/dev/null)
 ```
 
 then print it last, after whatever else the command emits:
@@ -272,22 +310,46 @@ then print it last, after whatever else the command emits:
 if [ -n "$sluice_line" ]; then printf '%s\n' "$sluice_line"; fi
 ```
 
-The gate is two tests because a linked worktree may hold no state file of its
-own and still belong to a set with a run: there `.git` is a regular file naming
-the tree it was cut from, and the script falls back to that tree. A worktree
-with its own run passes the first test. In a tree with no run and no worktree
-behind it `.git` is a directory, so both tests fail and no process is spawned,
-which is the property the gate is for.
+`$cwd` is `workspace.current_dir` from the JSON the harness sends on stdin.
+Substitute the install path: `${CLAUDE_CONFIG_DIR:-$HOME/.claude}` where a
+session may set one, or `$cwd/.claude/skills/sluice/scripts/statusline.sh` for a
+project-local install.
+
+Those two lines are the whole contract, and they are deliberately empty of
+judgement. Whether a run is visible from a given directory is a question about
+this skill's layout, and the answer has moved twice: once when the run anchored
+on the worktree set, once when a deep run began opening inside the implementer
+worktree. Both times a caller carrying the test went stale and stopped drawing
+without saying so, which is indistinguishable from no run being live. A caller
+that contributes only a path cannot go stale, and every install brings
+`statusline.sh` up to date behind it.
+
+`scripts/statusline.sh` holds what used to sit in the caller: it resolves the
+directory it was given, walks up looking for a state file or a worktree marker,
+stops at an ordinary tree root or at `$HOME` with neither, and dispatches to
+`status.sh line --full` only when there is something to draw. The walk is what
+lets a session sitting in a subdirectory see the run in its tree, which testing
+the session's own directory alone never did, and it resolves symlinks first
+because a linked directory's lexical parents lead away from the tree rather than
+up it.
+
+It answers only whether a run might be visible from here, never where one is.
+`status.sh` stays the single authority on that, and is handed the directory the
+caller passed rather than the one the walk stopped at. A gate that were ever
+narrower than the resolution behind it would blank the bar on a run that
+resolves perfectly well, which is the failure this whole arrangement exists to
+end, so it errs permissive: a wasted spawn is the acceptable direction.
+
+The gate is kept rather than dropped because it costs about 8ms against the 33ms
+an ungated `status.sh` pays to work out there is no run, and the common case on
+any machine is a tree with no run at all. Depth barely moves it, 7.7ms stopping
+at a `.git` directory against 8.4ms walking to the root, because 5.4ms of that
+is the bash spawn and the walk itself forks nothing.
 
 `if` rather than `[ ... ] &&`: as the last command of a statusline script the
 short form makes it exit 1 on every render with no run live, which is the common
 case. `%s` rather than `%b`: the render already carries real escape bytes, and
-`%b` would reinterpret a backslash inside a task name. `$cwd` is
-`workspace.current_dir` from the JSON the harness sends on stdin. The configured
-config dir is read before the default because a session started with
-`CLAUDE_CONFIG_DIR` set installs the skill there, which is the one place a
-`$HOME/.claude` lookup will not find it; the two paths collapse to one when the
-variable is unset, at the price of a second `[ -f ]`. It renders as:
+`%b` would reinterpret a backslash inside a task name. It renders as:
 
 ```
 ⧗ deep · sluice-cross-harness   ◷ 38m
@@ -295,6 +357,25 @@ variable is unset, at the price of a second `[ -f ]`. It renders as:
   2/9 done · !T6 model tiers rather than model names +1 · ⟲ 1 unreviewed
 ```
 
+On a machine with no status line at all there is nothing to paste into, so the
+install claims the empty `statusLine` slot and points it at
+`scripts/statusline-command.sh`, a complete command that reads the harness JSON
+on stdin and draws the run and nothing else. A slot already holding someone
+else's command is never touched, on install or on removal, and the one the
+install claimed is given back when the skill is removed, including on the
+removal path that has no manifest to read.
+
+Three things make claiming a single shared slot safe. The command is written
+guarded by its own script's existence, as the hook commands are, so a bundle
+that moved leaves the bar silent rather than running a path that is gone on
+every keystroke. Ownership is matched on the whole command and never as a
+substring, so a command with ours composed into it belongs to whoever composed
+it and is left whole. And the path it matches carries the skill's own directory,
+so the second skill to declare a statusline cannot take the first one's slot on
+install or delete it on removal. That is why the bundled
+command draws no prompt of its own: it exists for the empty slot, not to compete
+for a full one, so with no run live it prints nothing rather than an empty row.
+
 The colour comes out of the script rather than being applied by the caller,
 because the mapping from state to colour belongs next to the state. A caller that
 coloured the line itself would have to re-derive each cell's meaning from its

package/skills/sluice/scripts/postinstall.sh

@@ -0,0 +1,60 @@
+#!/usr/bin/env bash
+# Postinstall: tell whoever just installed the skill how to put the run on their
+# status bar, and touch nothing.
+#
+# Wiring it automatically would mean finding their statusline script, and the
+# only thing settings.json holds is the shell command that runs it
+# (`bash "$HOME/.claude/statusline-command.sh"`). Recovering a path from that is
+# parsing a shell string and hoping, on a file the installer never created. So
+# this prints the two lines and leaves the editing to a person.
+#
+# Runs with the bundle root as its working directory, which puts settings.json
+# two levels up whether the install went to $HOME/.claude or to a
+# $CLAUDE_CONFIG_DIR somewhere else. Always exits 0: a postinstall that fails
+# reports an error against an install that in fact succeeded.
+
+set -uo pipefail
+
+BUNDLE="$PWD"
+SETTINGS="$BUNDLE/../../settings.json"
+RENDER="$BUNDLE/scripts/statusline.sh"
+
+[ -f "$SETTINGS" ] || exit 0
+[ -f "$RENDER" ] || exit 0
+command -v jq >/dev/null 2>&1 || exit 0
+
+# Nothing is said to someone with no statusline configured. Either the install
+# claimed the empty slot, in which case the branch below reports it, or this is
+# a session-mode install that wires nothing, in which case there is no file to
+# paste into and no instruction worth printing.
+CONFIGURED="$(jq -r '.statusLine.command // empty' "$SETTINGS" 2>/dev/null || true)"
+[ -n "$CONFIGURED" ] || exit 0
+
+RENDER="$(cd "$(dirname "$RENDER")" && pwd)/$(basename "$RENDER")"
+
+# The slot points at the bundled command: the install claimed it and there is
+# nothing for anyone to paste. Say so rather than print instructions that would
+# have them edit a file they do not have.
+case "$CONFIGURED" in
+	*skills/sluice/scripts/statusline-command.sh*)
+		echo
+		echo "sluice: your status line now shows a live run. It draws nothing until one"
+		echo "is open, and uninstalling gives the setting back."
+		exit 0
+		;;
+esac
+
+cat <<EOF
+
+sluice: to show a live run on your status line, add these to your statusline
+command. The first goes wherever it builds its other lines, the second last of
+all, after everything else it prints:
+
+  sluice_line=\$(bash "$RENDER" --dir "\$cwd" 2>/dev/null)
+
+  if [ -n "\$sluice_line" ]; then printf '%s\\n' "\$sluice_line"; fi
+
+\$cwd is workspace.current_dir from the JSON the harness sends on stdin. Those
+two lines carry no knowledge of where a run lives, so they stay correct as this
+skill changes; every install updates the script behind them.
+EOF

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
 #
@@ -58,6 +60,16 @@ need_value() { # <flag> <remaining $#> <candidate>
 	case "$3" in
 		--*) err "$1 needs a value, but the next argument is the flag $3"; exit 4 ;;
 	esac
+	# Every value this script stores is later drawn onto a terminal, by the
+	# statusline, by `show` and by the SessionStart hook. A control byte in one is
+	# not text there, it is a command the terminal obeys: ESC[2J clears the screen
+	# on every render for as long as the run is open, and a carriage return walks
+	# the cursor back over the row just drawn. Refused rather than stripped,
+	# because a name that is not the name the caller passed is its own surprise,
+	# and no legitimate value has ever needed one.
+	case "$3" in
+		*[[:cntrl:]]*) err "$1 value contains a control character, which a terminal would act on rather than print"; exit 4 ;;
+	esac
 }
 
 # A word from a space-separated set. Keeps validation in one place so every
@@ -148,7 +160,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
@@ -168,6 +181,12 @@ if [ "$SUB" = "line" ]; then
 	jq -r \
 		--argjson now "$(date -u +%s)" \
 		--arg esc "$(printf '\033')" '
+		# The state file is an input like any other: it predates the check on the
+		# way in, or was hand-edited, so what it holds is not known to be drawable.
+		# Everything this render emits deliberately is an SGR colour sequence, so any
+		# other control byte reaching the terminal came from the state, and is dropped
+		# here rather than obeyed.
+		def clean: if type == "string" then gsub("[\u0000-\u001f\u007f]"; "") else . end;
 		def paint($c; $t): "\($esc)[\($c)m\($t)\($esc)[0m";
 		def join_parts: map(select(. != null and . != "")) | join(" \($esc)[2m·\($esc)[0m ");
 		# Done splits in two. A task that is done and was owed a review nobody has
@@ -227,10 +246,11 @@ if [ "$SUB" = "line" ]; then
 		     end) as $idle
 		| ( paint("1;96"; "⧗") + " "
 		    + ([ paint("1;96"; (.channel // "?")),
-		         paint("2"; (.topic // ""))
+		         paint("2"; (.topic // "" | clean))
 		       ] | 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
@@ -240,8 +260,8 @@ if [ "$SUB" = "line" ]; then
 		                + cellgroup($w)
 		            ] | join($gap)) ),
 		  ( "  " + ([ paint("1"; "\($done)/\(.tasks | length)") + " done",
-		              (if   $blocked then paint("1;91"; "!T\($blocked.id) \($blocked.name // "")")
-		               elif $active  then paint("96"; "▸T\($active.id)") + " " + ($active.name // "")
+		              (if   $blocked then paint("1;91"; "!T\($blocked.id) \($blocked.name // "" | clean)")
+		               elif $active  then paint("96"; "▸T\($active.id)") + " " + ($active.name // "" | clean)
 		               else "" end)
 		              + (if $attn > 1 then paint("2"; " +\($attn - 1)") else "" end),
 		              (if $debt > 0 then paint("33"; "⟲ \($debt) unreviewed") else "" end)
@@ -558,7 +578,12 @@ case "$SUB" in
 		# than silently reading as the whole value.
 		jq -r --argjson now "$(date -u +%s)" '
 			def dash: if . == null or . == "" then "-" else . end;
-			def cell($w): tostring
+			# Same reason as the statusline render: state written before the check
+			# on the way in, or edited by hand, holds bytes a terminal would act on
+			# rather than print. Every table cell passes through `cell`, so the
+			# table is covered there; the lines built outside it clean their own.
+			def clean: if type == "string" then gsub("[\u0000-\u001f\u007f]"; "") else . end;
+			def cell($w): tostring | clean
 				| if length > $w then .[0:$w - 1] + "…"
 				  else . + (" " * ($w - length))
 				  end;
@@ -566,9 +591,9 @@ case "$SUB" in
 			                      ($c[3] | cell(9)), ($c[4] | cell(9)), ($c[5] | cell(4)),
 			                      $c[6]] | join(" "));
 			([.tasks[]? | select(.status == "done")] | length) as $done
-			| ["sluice \(.channel) · \(.topic) · \($done)/\(.tasks | length) done"]
-			+ ["plan          \(.plan | dash)"]
-			+ ["record        \(.record | dash)"]
+			| ["sluice \(.channel | clean) · \(.topic | clean) · \($done)/\(.tasks | length) done"]
+			+ ["plan          \(.plan | dash | clean)"]
+			+ ["record        \(.record | dash | clean)"]
 			# Past a day since the last write the run is idle, and that is said
 			# here because a stale run blocks the next init and nothing else
 			# would name it.
@@ -581,12 +606,13 @@ case "$SUB" in
 			            else ["idle          \($h / 24 | floor)d\($h % 24)h since the last write"]
 			            end
 			     end)
+			+ (if .paused then ["paused        \(.paused | clean)"] 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)]
 			+ ["pre-flight    " + (
 				if (.preflight // {} | length) == 0 then "not recorded"
-				else [(.preflight | to_entries[] | "\(.key)=\(.value)")] | join("; ")
+				else [(.preflight | to_entries[] | "\(.key | clean)=\(.value | clean)")] | join("; ")
 				end)]
 			+ [""]
 			+ [row(["id", "status", "task", "base", "commit", "tier", "model"])]
@@ -682,6 +708,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/statusline-command.sh

@@ -0,0 +1,42 @@
+#!/usr/bin/env bash
+# A complete statusline command, for a machine that had none.
+#
+# `statusline.sh` takes a directory; the harness delivers one as JSON on stdin.
+# This bridges the two, and it is what settings.json can be pointed at directly:
+#
+#   "statusLine": { "type": "command", "command": "bash '<this file>'" }
+#
+# It draws the live run and nothing else. Anyone who wants a prompt as well
+# already has a statusline of their own, and the install refuses to overwrite
+# one: this file exists for the empty slot, not to compete for a full one. So
+# with no run live it prints nothing at all rather than an empty row.
+#
+# Always exits 0 in silence on anything it cannot render, for the same reason
+# statusline.sh does: a status bar draws on every keystroke and has nowhere to
+# put an error.
+
+set -uo pipefail
+
+here="$(cd "$(dirname "$0")" && pwd)"
+RENDER="$here/statusline.sh"
+
+[ -f "$RENDER" ] || exit 0
+command -v jq >/dev/null 2>&1 || exit 0
+
+# Bounded, and skipped on a terminal. The harness closes stdin after the JSON,
+# so a read to EOF returns, but nothing here enforces that: read from a pipe
+# nobody closes and an unbounded read hangs, which on a status bar is a bar that
+# never draws and on a hand-run check is a frozen terminal with no explanation.
+# `read` is a builtin, so the bound costs no fork against the gate's budget.
+[ -t 0 ] && exit 0
+input=""
+IFS= read -r -d "" -t 2 input || true
+[ -n "$input" ] || exit 0
+
+# `cwd` is the older spelling of the same field and both are still sent, so it
+# is a fallback rather than a bug. Neither present is not an error worth a row:
+# it means this is not the JSON we were written against.
+dir="$(printf '%s' "$input" | jq -r '.workspace.current_dir // .cwd // empty' 2>/dev/null || true)"
+[ -n "$dir" ] || exit 0
+
+bash "$RENDER" --dir "$dir" 2>/dev/null || exit 0

package/skills/sluice/scripts/statusline.sh

@@ -0,0 +1,100 @@
+#!/usr/bin/env bash
+# Statusline render: the whole sluice contribution to a status bar, behind one
+# call. The caller passes the directory the session is in and prints whatever
+# comes back.
+#
+#   statusline.sh [--dir <path>]
+#
+# --dir defaults to $PWD. Always exits 0 in silence on anything it cannot
+# render: a status bar draws on every keystroke and has nowhere to put an error,
+# so a message here would be permanent clutter rather than a report anyone acts
+# on.
+#
+# Why this exists as its own script rather than as a snippet in the caller.
+# Whether a run is visible from a given directory is a question about sluice's
+# own layout, and the answer has changed twice: once when the run anchored on
+# the worktree set, once when a deep run began opening inside the implementer
+# worktree. Each time, a caller carrying the test went stale and simply stopped
+# drawing, silently, because silence is also what it looks like when no run is
+# live. Callers now contribute a path and nothing else, and every install brings
+# this file up to date behind them.
+#
+# The gate is still here rather than dropped: measured on this machine, it costs
+# about 8ms against the 33ms an ungated status.sh pays to work out there is no
+# run, and the common case on any machine is a tree with no run at all. Depth
+# barely moves it -- 7.7ms stopping at a .git directory, 8.4ms walking to the
+# root -- because the bash spawn is 5.4ms of it and the walk forks nothing.
+# It answers only "might a run be visible from here", never "where is it":
+# status.sh stays the single authority on resolution, and this stays a
+# deliberately over-permissive filter in front of it. A filter that were ever
+# narrower than status.sh would blank the bar on a run the code behind it can
+# render, which is the whole class of bug this file exists to end.
+
+set -uo pipefail
+
+here="$(cd "$(dirname "$0")" && pwd)"
+STATUS="$here/status.sh"
+
+DIR="$PWD"
+while [ $# -gt 0 ]; do
+	case "$1" in
+		--dir)
+			# An omitted value would silently make the next flag the directory.
+			[ $# -ge 2 ] && [ "${2#--}" = "$2" ] || exit 0
+			DIR="$2"
+			shift 2
+			;;
+		*) exit 0 ;;
+	esac
+done
+
+[ -d "$DIR" ] || exit 0
+[ -f "$STATUS" ] || exit 0
+
+# Is a run reachable from here at all? Walk up rather than test $DIR alone: a
+# session sitting in a subdirectory of the tree is as entitled to the render as
+# one sitting at its root, and testing only $DIR was a second way for the bar to
+# go blank on a live run.
+#
+# Resolve physically first. A directory reached through a symlink has lexical
+# parents that are not its real ones, and climbing those walks away from the
+# repo instead of up it -- a blank bar on a run status.sh resolves perfectly
+# well, which is this file's own failure mode turned on a new input. `cd`
+# failing here means the path is gone or unreadable, and there is nothing to
+# draw for it.
+d="$(cd "$DIR" 2>/dev/null && pwd -P)" || exit 0
+[ -n "$d" ] || exit 0
+
+# Four answers end the walk. A state file is a run, wherever it was found. A
+# `.git` that is a regular file is a linked worktree or a submodule, which may
+# hold no state of its own and still belong to a set that does, so it is a maybe
+# and status.sh resolves it. A `.git` that is a directory is the top of an
+# ordinary tree with no state beside it, which is a no. So is reaching $HOME:
+# without that stop, one forgotten ~/.sluice/run.json would put a status.sh
+# spawn on every keystroke of every session outside a repo, for a run that
+# resolves to nothing.
+#
+# `${d%/*}` rather than `dirname`, which is not a builtin: a fork per level
+# would put the gate's cost in the same range as the render it is avoiding.
+while :; do
+	[ -f "$d/.sluice/run.json" ] && break
+	[ -f "$d/.git" ] && break
+	[ -d "$d/.git" ] && exit 0
+	[ "$d" = "${HOME:-}" ] && exit 0
+	[ "$d" = "/" ] && exit 0
+	d="${d%/*}"
+	[ -n "$d" ] || d="/"
+done
+
+# $DIR, not the $d the walk stopped at: $d is where the filter gave up looking,
+# which is not the same question as where the run is. status.sh anchors on the
+# worktree set and is the only thing that decides that.
+#
+# `line --full` is silent on everything it cannot read, a missing jq and an
+# unreadable state file included, and exits 0 either way. The render arrives
+# already coloured: the mapping from state to colour belongs next to the state,
+# and a caller that applied it would have to re-derive each cell's meaning from
+# its glyph.
+rendered="$(bash "$STATUS" line --full --dir "$DIR" 2>/dev/null)" || exit 0
+[ -n "$rendered" ] || exit 0
+printf '%s\n' "$rendered"

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

@@ -0,0 +1,101 @@
+#!/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}
+	  # The topic lands in a reason the harness prints, so a control byte in it
+	  # would be acted on by the terminal rather than read. State written before
+	  # status.sh refused those, or edited by hand, can still hold one.
+	  else {block: true, progress: "\($done)/\($t | length)",
+	        topic: (.topic // "run" | gsub("[\u0000-\u001f\u007f]"; ""))}
+	  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.19.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",
@@ -19,7 +19,8 @@
   "install": {
     "claude": {
       "prompt": ".claude/skills/sluice/SKILL.md",
-      "bundleRoot": ".claude/skills/sluice"
+      "bundleRoot": ".claude/skills/sluice",
+      "postinstall": "scripts/postinstall.sh"
     }
   },
   "activation": {
@@ -29,6 +30,8 @@
     ],
     "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",
+    "claudeStatuslineScript": "scripts/statusline-command.sh"
   }
 }