@iceinvein/agent-skills 0.17.0 → 0.18.0

package/dist/cli/index.js

@@ -302,6 +302,20 @@ function validateManifest(data) {
     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 };
 }
@@ -570,6 +584,49 @@ async function unwireStopHook(settingsPath, skillName) {
   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) {
@@ -639,6 +696,13 @@ var claudeAdapter = {
         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));
@@ -704,6 +768,9 @@ var claudeAdapter = {
     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));
+    }
   }
 };
 
@@ -1143,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);
@@ -1236,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.17.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.18.0"
+    "version": "0.19.0"
   },
   {
     "name": "temporal-coupling-detector",

package/skills/sluice/references/status.md

@@ -272,6 +272,24 @@ session unguarded for the rest of the run.
 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
@@ -279,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:
@@ -301,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
@@ -324,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

@@ -60,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
@@ -171,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
@@ -230,7 +246,7 @@ 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)
@@ -244,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)
@@ -562,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;
@@ -570,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.
@@ -585,13 +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)"] else [] 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"])]

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

@@ -84,7 +84,11 @@ verdict="$(printf '%s' "$run" | jq -c --argjson now "$(date -u +%s)" '
 	  # 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")}
+	  # 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
 

package/skills/sluice/skill.json

@@ -1,6 +1,6 @@
 {
   "name": "sluice",
-  "version": "0.18.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": {
@@ -30,6 +31,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",
-    "claudeStopScript": "scripts/stop-guard.sh"
+    "claudeStopScript": "scripts/stop-guard.sh",
+    "claudeStatuslineScript": "scripts/statusline-command.sh"
   }
 }