@opengsd/gsd-core 1.11.0 → 1.12.0

package/hooks/dist/gsd-cursor-subagent-start.js

@@ -56,6 +56,7 @@
 const fs = require('fs');
 const path = require('path');
 const os = require('os');
+const { allow } = require('./lib/hook-exit.js');
 
 // Workspace resolution is shared across the Cursor hooks (#2587) — see
 // hooks/lib/cursor-workspace.js. Staged next to these scripts by
@@ -342,35 +343,39 @@ function resolveIsolationDecision(data, { clock = Date, realpath = fs.realpathSy
   return { action: 'allow' };
 }
 
-// ─── #3566: per-install runtime marker ────────────────────────────────────────
-// Same contract as hooks/gsd-agent-isolation-guard.js's readInstallRuntimeMarker
-// (mirroring src/model-resolver.cts #2297): bin/install.js writes
-// `<install>/gsd-core/.gsd-runtime` beside VERSION for every runtime install;
-// this hook ships at `<install>/hooks/`, so the marker is the `gsd-core` sibling
-// of this file's own directory — the same sibling-layout assumption the
-// require('../gsd-core/bin/lib/…') calls below already make. Epic #3473 B3 owns
-// consolidating every marker reader into one shared seam.
-let _installMarkerCache; // undefined = unread; null = known absent; string = value
-
+// ─── #3897 rung 2: per-install runtime marker, single canonical owner ────────
+// bin/install.js writes `<install>/gsd-core/.gsd-runtime` beside VERSION for
+// every runtime install (#2297); this hook ships at `<install>/hooks/`, so the
+// marker is the `gsd-core` sibling of this file's own directory. Previously
+// this hook held its own private reader/cache (one of four #3897 found); it
+// now delegates to the single canonical owner, `src/runtime-slash.cts`
+// (compiled to gsd-core/bin/lib/runtime-slash.cjs), reached through
+// `ensureRuntimeBuild()` like the other compiled-lib requires in this file
+// (`scripts/lint-hooks-runtime-build-seam.cjs`).
 function readInstallRuntimeMarker() {
-  if (_installMarkerCache !== undefined) return _installMarkerCache;
   try {
-    const markerPath = path.join(__dirname, '..', 'gsd-core', '.gsd-runtime');
-    const raw = fs.readFileSync(markerPath, 'utf-8').trim();
-    _installMarkerCache = raw || null;
+    ensureRuntimeBuild();
+    const runtimeSlash = require('../gsd-core/bin/lib/runtime-slash.cjs');
+    return runtimeSlash.readInstallRuntimeMarker();
   } catch {
-    // No marker: dev/source tree, or an install predating #2297 — "no signal
-    // from this rung", never a resolution failure.
-    _installMarkerCache = null;
+    // Unbuilt runtime library, or any other failure reaching the canonical
+    // owner — "no signal from this rung", never a resolution failure.
+    return null;
   }
-  return _installMarkerCache;
 }
 
-// Test seam — same contract as model-resolver.cts's #2297 seam; the dev/source
-// tree has no marker file, so spawned-hook tests (fresh process, no marker)
-// are unaffected.
+// Test seam — forwards to the canonical owner's seam so this hook and
+// runtime-slash.cjs always share one cache (#3897 rung 2). Spawned-hook tests
+// (fresh process, no marker) are unaffected.
 function _setInstallRuntimeMarkerForTests(value) {
-  _installMarkerCache = value;
+  try {
+    ensureRuntimeBuild();
+    const runtimeSlash = require('../gsd-core/bin/lib/runtime-slash.cjs');
+    runtimeSlash._setInstallRuntimeMarkerForTests(value);
+  } catch {
+    // Test-only seam; an unbuilt library here means the test itself will fail
+    // downstream, which is a louder and more actionable signal than throwing here.
+  }
 }
 
 /**
@@ -564,7 +569,7 @@ function evaluateRootIsolation(root, subagentType, { clock = Date, dispatchIds =
 function main() {
   let raw = '';
   const stdinTimeout = setTimeout(() => {
-    process.exit(0);
+    allow(undefined);
   }, 10000);
 
   process.stdin.setEncoding('utf8');

package/hooks/dist/gsd-cursor-subagent-stop.js

@@ -20,8 +20,10 @@
 
 'use strict';
 
+const { allow } = require('./lib/hook-exit.js');
+
 const stdinTimeout = setTimeout(() => {
-  process.exit(0);
+  allow(undefined);
 }, 10000);
 
 process.stdin.setEncoding('utf8');

package/hooks/dist/gsd-ensure-canonical-path.js

@@ -34,6 +34,7 @@
 const fs = require('fs');
 const path = require('path');
 const os = require('os');
+const { allow } = require('./lib/hook-exit.js');
 
 // Immutable, bundled subdirectories that the canonical path must expose. These
 // are the directories `@~/.claude/gsd-core/<subdir>/...` includes point into.
@@ -301,5 +302,5 @@ if (require.main === module) {
   } catch (_) {
     // Best-effort: a canonical-path failure must never abort a session.
   }
-  process.exit(0);
+  allow(undefined);
 }

package/hooks/dist/gsd-graphify-update.sh

@@ -9,17 +9,20 @@
 # graphify.auto_update defaults to false so existing users see no behavior change.
 #
 # Gates (in fast-fail order — each shaves work off the common non-dispatch path):
+#   0. .planning/config.json exists AND $CI unset/empty (#3729 — both are
+#      parse-free shell tests; running them before the Gate 1 node spawn keeps
+#      non-GSD repositories and CI off the spawn path entirely, which on
+#      Windows otherwise flashes a console window per Bash call)
 #   1. Stdin payload present and tool_name == "Bash"
 #   2. tool_input.command matches a HEAD-advancing git op (shell-direct or
 #      the exact `gsd-tools query commit` command shape; the SDK command invokes
 #      git internally, so the literal "git commit" substring never appears —
 #      see #3653)
-#   3. $CI is unset/empty
-#   4. Inside a git repo
-#   5. Current branch == default branch (git.base_branch override, else main/master/trunk)
-#   6. .planning/config.json sets graphify.enabled=true AND graphify.auto_update=true
-#   7. graphify binary on PATH
-#   8. No rebuild already in flight (PID lock — kill -0 check, stale-tolerant)
+#   3. Inside a git repo
+#   4. Current branch == default branch (git.base_branch override, else main/master/trunk)
+#   5. .planning/config.json sets graphify.enabled=true AND graphify.auto_update=true
+#   6. graphify binary on PATH
+#   7. No rebuild already in flight (PID lock — kill -0 check, stale-tolerant)
 #
 # When all gates pass:
 #   - Writes .planning/graphs/.last-build-status.json with status="running"
@@ -30,6 +33,13 @@
 
 set -uo pipefail
 
+# Gate 0 — GSD project at all, and not CI (#3729). Both are pure shell tests;
+# they must precede the Gate 1 node spawn so the common non-GSD/CI path pays
+# for zero child processes. Hoisting is behavior-preserving: old Gates 3 and 6
+# made the same decisions, just later.
+[ -f .planning/config.json ] || exit 0
+[ -z "${CI:-}" ] || exit 0
+
 # Gate 1 — tool_name == Bash; extract command
 INPUT=$(cat 2>/dev/null || true)
 [ -n "$INPUT" ] || exit 0
@@ -71,22 +81,17 @@ case "$COMMAND" in
   *) exit 0 ;;
 esac
 
-# Gate 3 — not CI
-[ -z "${CI:-}" ] || exit 0
-
-# Gate 4 — inside git repo
+# Gate 3 — inside git repo
 git rev-parse --git-dir >/dev/null 2>&1 || exit 0
 
-# Gate 5 — current branch == default branch
+# Gate 4 — current branch == default branch (config guaranteed by Gate 0)
 DEFAULT_BRANCH=""
-if [ -f .planning/config.json ]; then
-  DEFAULT_BRANCH=$(node -e '
+DEFAULT_BRANCH=$(node -e '
 try {
   const c = require("./.planning/config.json");
   process.stdout.write(c.git?.base_branch || "");
 } catch { process.stdout.write(""); }
 ' 2>/dev/null || echo "")
-fi
 if [ -z "$DEFAULT_BRANCH" ]; then
   for cand in main master trunk; do
     if git rev-parse --verify "$cand" >/dev/null 2>&1; then
@@ -100,8 +105,7 @@ fi
 CURRENT_BRANCH=$(git rev-parse --abbrev-ref HEAD 2>/dev/null || echo "")
 [ "$CURRENT_BRANCH" = "$DEFAULT_BRANCH" ] || exit 0
 
-# Gate 6 — both graphify gates true in config
-[ -f .planning/config.json ] || exit 0
+# Gate 5 — both graphify gates true in config (file existence checked at Gate 0)
 GATES=$(node -e '
 try {
   const c = require("./.planning/config.json");
@@ -111,11 +115,11 @@ try {
 ' 2>/dev/null || echo "0")
 [ "$GATES" = "1" ] || exit 0
 
-# Gate 7 — graphify on PATH
+# Gate 6 — graphify on PATH
 GRAPHIFY_BIN=$(command -v graphify 2>/dev/null || true)
 [ -n "$GRAPHIFY_BIN" ] || exit 0
 
-# Gate 8 — no live rebuild in flight
+# Gate 7 — no live rebuild in flight
 mkdir -p .planning/graphs
 LOCK_FILE=".planning/graphs/.rebuild.lock"
 if [ -f "$LOCK_FILE" ]; then

package/hooks/dist/gsd-node-runner.sh

@@ -0,0 +1,76 @@
+#!/bin/sh
+# gsd-node-runner.sh — GSD portable node resolver (#3662).
+#
+# Managed JS hook commands under --portable-hooks route through this script:
+#
+#   bash "<hooks>/gsd-node-runner.sh" "<baked-node-path>" "<script.js>" [args...]
+#
+# so a config root shared across environments (mounted ~/.claude, shared
+# containers) resolves node at hook-fire time instead of depending on the
+# absolute path of whichever environment ran the installer. Candidates, in
+# order — the first executable one wins:
+#
+#   1. the first argument — the install-time node path, tried FIRST and by
+#      absolute path so the #2979/#3002/#3017/#3022 minimal-PATH guarantee
+#      holds (GUI launches with a stripped PATH still resolve where the
+#      baked path exists);
+#   2. `command -v node`, accepted only when it yields an absolute path;
+#   3. the well-known stable layouts: $HOME-derived mise/volta shims, the
+#      Homebrew prefixes, /usr/local/bin/node, /usr/bin/node.
+#
+# No bare `node` lookup is ever depended on: a candidate is used only after
+# an explicit executable check, and when nothing resolves this script fails
+# visibly (stderr diagnostic + exit 127) rather than emitting a half-resolved
+# invocation.
+#
+# The candidate list below is a SUPERSET of the inline chain token emitted by
+# buildNodeRunnerChainToken (src/runtime-hooks-surface.cts, #3662) — keep the
+# two lists consistent.
+#
+# Diagnostic escape: GSD_NODE_RUNNER_NO_FALLBACKS=1 disables candidates 2-3
+# (first-argument-only resolution) — used by the test suite and useful to
+# pin down which node a given environment picks.
+set -u
+
+preferred=${1:-}
+script=${2:-}
+if [ -n "$script" ]; then
+  shift 2
+elif [ -n "$preferred" ]; then
+  shift 1
+  preferred=
+fi
+
+found=''
+
+# check <path> — record <path> if it is an absolute, executable file.
+# Absolute = POSIX root (/*) or a win32 drive-letter path (C:/…), which is
+# what the installer bakes on Windows; anything else (a relative `command -v`
+# hit under a relative PATH entry, a bare name) is rejected so repo-cwd
+# content can never reach the runner slot.
+check() {
+  case "$1" in
+    /*|[A-Za-z]:/*) if [ -x "$1" ]; then found=$1; fi ;;
+  esac
+  [ -n "$found" ]
+}
+
+check "$preferred" || {
+  if [ "${GSD_NODE_RUNNER_NO_FALLBACKS:-0}" != "1" ]; then
+    path_node=$(command -v node 2>/dev/null || true)
+    check "$path_node" ||
+      check "${HOME:-}/.local/share/mise/shims/node" ||
+      check "${HOME:-}/.volta/bin/node" ||
+      check /opt/homebrew/bin/node ||
+      check /usr/local/bin/node ||
+      check /usr/bin/node ||
+      true
+  fi
+}
+
+if [ -z "$found" ]; then
+  echo "gsd-node-runner: no usable node found (preferred: ${preferred:-<none>})" >&2
+  exit 127
+fi
+
+exec "$found" "$script" "$@"

package/hooks/dist/gsd-phase-boundary.sh

@@ -6,6 +6,7 @@
 #
 # OPT-IN: This hook is a no-op unless config.json has hooks.community: true.
 # Enable with: "hooks": { "community": true } in .planning/config.json
+set -euo pipefail
 
 # Check opt-in config — exit silently if not enabled
 if [ -f .planning/config.json ]; then

package/hooks/dist/gsd-prompt-guard.js

@@ -12,6 +12,13 @@
 // not to create false-positive deadlocks.
 
 const path = require('path');
+const { HOOK_ON_CRASH, allow, crash } = require('./lib/hook-exit.js');
+
+// This guard is advisory-only by design (see header) — it never blocks the
+// Write/Edit it scans, only adds context about it. A crash here must not
+// start blocking now, which is strictly worse than the advisory it exists
+// to add on top of an already-permitted operation (#3911).
+const ON_CRASH = HOOK_ON_CRASH.ALLOW;
 
 // Prompt injection patterns — shared with gsd-read-injection-scanner.js via
 // hooks/lib/injection-patterns.js so the two surfaces cannot drift (#3504).
@@ -115,7 +122,7 @@ function normalizeKimiPayload(data) {
 }
 
 let input = '';
-const stdinTimeout = setTimeout(() => process.exit(0), 3000);
+const stdinTimeout = setTimeout(() => allow(undefined), 3000);
 process.stdin.setEncoding('utf8');
 process.stdin.on('data', chunk => input += chunk);
 process.stdin.on('end', () => {
@@ -126,7 +133,7 @@ process.stdin.on('end', () => {
 
     // Only scan Write and Edit operations
     if (toolName !== 'Write' && toolName !== 'Edit') {
-      process.exit(0);
+      allow(undefined);
     }
 
     // #2595 (review Major 3, sibling sweep): typed read. A non-string
@@ -138,7 +145,7 @@ process.stdin.on('end', () => {
 
     // Only scan files going into .planning/ (agent context files)
     if (!filePath.includes('.planning/') && !filePath.includes('.planning\\')) {
-      process.exit(0);
+      allow(undefined);
     }
 
     // Get the content being written. #3504 (isolated review finding 3): the
@@ -157,7 +164,7 @@ process.stdin.on('end', () => {
       }
     }
     if (!content) {
-      process.exit(0);
+      allow(undefined);
     }
 
     // Scan for injection patterns
@@ -174,7 +181,7 @@ process.stdin.on('end', () => {
     }
 
     if (findings.length === 0) {
-      process.exit(0);
+      allow(undefined);
     }
 
     // Advisory warning — does not block the operation
@@ -191,7 +198,9 @@ process.stdin.on('end', () => {
 
     process.stdout.write(JSON.stringify(output));
   } catch {
-    // Silent fail — never block tool execution
-    process.exit(0);
+    // Silent fail — never block tool execution.
+    // ON_CRASH is declared ALLOW at module top: this preserves today's
+    // exit(0) fail-open behavior exactly (#3911).
+    crash(ON_CRASH, undefined);
   }
 });

package/hooks/dist/gsd-read-guard.js

@@ -20,6 +20,13 @@
 
 const fs = require('fs');
 const path = require('path');
+const { HOOK_ON_CRASH, allow, crash } = require('./lib/hook-exit.js');
+
+// This guard is pure advisory UX — a reminder to Read before Write/Edit on
+// non-Claude-Code runtimes. A crash here must not block a legitimate Write/
+// Edit; the worst outcome of failing open is the model hitting the runtime's
+// own read-before-edit rejection it was trying to help avoid (#3911).
+const ON_CRASH = HOOK_ON_CRASH.ALLOW;
 
 // #2304: Kimi's native hook bus delivers Kimi's tool vocabulary in the payload
 // (Write → WriteFile, Edit/MultiEdit → StrReplaceFile) while the [[hooks]]
@@ -116,7 +123,7 @@ function normalizeKimiPayload(data) {
 }
 
 let input = '';
-const stdinTimeout = setTimeout(() => process.exit(0), 3000);
+const stdinTimeout = setTimeout(() => allow(undefined), 3000);
 process.stdin.setEncoding('utf8');
 process.stdin.on('data', chunk => input += chunk);
 process.stdin.on('end', () => {
@@ -127,7 +134,7 @@ process.stdin.on('end', () => {
 
     // Only intercept Write and Edit tool calls
     if (toolName !== 'Write' && toolName !== 'Edit') {
-      process.exit(0);
+      allow(undefined);
     }
 
     // Claude Code natively enforces read-before-edit — skip the advisory (#1984, #2344, #2520).
@@ -151,7 +158,7 @@ process.stdin.on('end', () => {
       process.env.CLAUDE_SESSION_ID ||
       process.env.CLAUDECODE;
     if (isClaudeCode) {
-      process.exit(0);
+      allow(undefined);
     }
 
     // #2595 (review Major 3, sibling sweep): typed read — same class as the
@@ -160,7 +167,7 @@ process.stdin.on('end', () => {
       ? data.tool_input.file_path
       : '';
     if (!filePath) {
-      process.exit(0);
+      allow(undefined);
     }
 
     // Only inject guidance when the file already exists.
@@ -174,7 +181,7 @@ process.stdin.on('end', () => {
     }
 
     if (!fileExists) {
-      process.exit(0);
+      allow(undefined);
     }
 
     const fileName = path.basename(filePath);
@@ -193,7 +200,9 @@ process.stdin.on('end', () => {
 
     process.stdout.write(JSON.stringify(output));
   } catch {
-    // Silent fail — never block tool execution
-    process.exit(0);
+    // Silent fail — never block tool execution.
+    // ON_CRASH is declared ALLOW at module top: this preserves today's
+    // exit(0) fail-open behavior exactly (#3911).
+    crash(ON_CRASH, undefined);
   }
 });

package/hooks/dist/gsd-read-injection-scanner.js

@@ -23,6 +23,13 @@
 
 const path = require('path');
 const fs = require('fs');
+const { HOOK_ON_CRASH, allow, crash } = require('./lib/hook-exit.js');
+
+// This is a PostToolUse advisory scanner over content the tool call already
+// returned; a crash while scanning must not retroactively block that Read/
+// WebFetch/WebSearch result from reaching the agent — losing the injection
+// check is safer than failing the tool call it is only observing (#3911).
+const ON_CRASH = HOOK_ON_CRASH.ALLOW;
 
 // Summarisation-specific patterns (novel — not in gsd-prompt-guard.js).
 // These target instructions specifically designed to survive context compression.
@@ -213,7 +220,7 @@ function normalizeKimiPayload(data) {
 }
 
 let inputBuf = '';
-const stdinTimeout = setTimeout(() => process.exit(0), 5000);
+const stdinTimeout = setTimeout(() => allow(undefined), 5000);
 process.stdin.setEncoding('utf8');
 process.stdin.on('data', chunk => { inputBuf += chunk; });
 process.stdin.on('end', () => {
@@ -224,7 +231,7 @@ process.stdin.on('end', () => {
     const toolName = data.tool_name;
     const SCANNED_TOOLS = new Set(['Read', 'WebFetch', 'WebSearch']);
     if (!SCANNED_TOOLS.has(toolName)) {
-      process.exit(0);
+      allow(undefined);
     }
 
     // Source label + path-exclusion (path-exclusion applies to file reads only)
@@ -235,8 +242,8 @@ process.stdin.on('end', () => {
       source = typeof data.tool_input?.file_path === 'string'
         ? data.tool_input.file_path
         : '';
-      if (!source) process.exit(0);
-      if (isExcludedPath(source)) process.exit(0);
+      if (!source) allow(undefined);
+      if (isExcludedPath(source)) allow(undefined);
     } else if (toolName === 'WebFetch') {
       source = data.tool_input?.url || 'web';
     } else { // WebSearch
@@ -261,7 +268,7 @@ process.stdin.on('end', () => {
     }
 
     if (!content || content.length < 20) {
-      process.exit(0);
+      allow(undefined);
     }
 
     // Typed findings IR — single source of truth for both the machine-readable
@@ -307,7 +314,7 @@ process.stdin.on('end', () => {
     }
 
     if (findings.length === 0) {
-      process.exit(0);
+      allow(undefined);
     }
 
     // Renders one finding back into the exact prose fragment the advisory has
@@ -349,7 +356,9 @@ process.stdin.on('end', () => {
 
     process.stdout.write(JSON.stringify(output));
   } catch {
-    // Silent fail — never block tool execution
-    process.exit(0);
+    // Silent fail — never block tool execution.
+    // ON_CRASH is declared ALLOW at module top: this preserves today's
+    // exit(0) fail-open behavior exactly (#3911).
+    crash(ON_CRASH, undefined);
   }
 });

package/hooks/dist/gsd-session-state.sh

@@ -5,6 +5,7 @@
 #
 # OPT-IN: This hook is a no-op unless config.json has hooks.community: true.
 # Enable with: "hooks": { "community": true } in .planning/config.json
+set -euo pipefail
 
 # Check opt-in config — exit silently if not enabled
 if [ -f .planning/config.json ]; then