@vibe-agent-toolkit/utils 0.1.39-rc.9 → 0.1.39

package/dist/skill-test/env-scrub.d.ts

@@ -11,4 +11,62 @@ export interface ForwardEnvOptions {
     modelVars?: string[];
 }
 export declare function buildForwardedEnv(source: NodeJS.ProcessEnv, opts: ForwardEnvOptions): NodeJS.ProcessEnv;
+/**
+ * The full set of env var names protected from declared override. A declared
+ * passEnv/injectEnv entry naming one of these is ignored (the protected value
+ * wins) and a warning is surfaced — a test must never clobber PATH, the auth
+ * credentials, or the admin key. `modelVars` (run-specific model env var names)
+ * join the set so a declared var can't shadow them either.
+ */
+export declare function protectedEnvNames(modelVars?: readonly string[]): Set<string>;
+/**
+ * Returns true if `name` is in `protectedSet`.
+ *
+ * On Windows (win32) env names are case-insensitive — `path` and `PATH` are the
+ * same variable — so we upper-case both sides before comparing. On POSIX, names
+ * are genuinely case-distinct, so the comparison stays exact.
+ *
+ * The `platform` parameter defaults to `process.platform` but is exposed so
+ * unit tests can exercise the win32 branch on any host OS.
+ */
+export declare function isProtectedName(name: string, protectedSet: Set<string>, platform?: string): boolean;
+/** Declared test env (Features A + B) to union onto a forwarded env. */
+export interface DeclaredEnvInput {
+    /** Parent env to read Feature-A pass-through values from. */
+    source: NodeJS.ProcessEnv;
+    /** Feature A: names to forward from `source` if present. */
+    passEnv?: readonly string[];
+    /** Feature B: explicit key→value injections (already interpolated). */
+    injectEnv?: Record<string, string>;
+    /** Run-specific model env var names that are also protected. */
+    modelVars?: readonly string[];
+}
+export interface DeclaredEnvResult {
+    /** The forwarded env with declared additions unioned in. */
+    env: NodeJS.ProcessEnv;
+    /** Human-readable warnings (protected-key collisions). */
+    warnings: string[];
+    /** Names injected via Feature B (shown in the transparency line). */
+    injected: string[];
+    /** Names passed through via Feature A (redacted in the transparency line). */
+    passedThrough: string[];
+}
+/**
+ * Union declared test env (Features A + B) onto an already-built forwarded env.
+ * Protected keys always win: a declared name colliding with a process-essential,
+ * auth, model, or admin var is ignored and a warning emitted. Feature-B injection
+ * (explicit value) takes precedence over Feature-A pass-through for the same key.
+ * The input `base` object is never mutated.
+ */
+export declare function applyDeclaredEnv(base: NodeJS.ProcessEnv, input: DeclaredEnvInput): DeclaredEnvResult;
+/**
+ * Render the single-line stderr transparency summary of the forwarded env. Key
+ * names are always shown. Auth/secret values are redacted; Feature-A pass-through
+ * values are redacted (host-sourced, may be a secret); Feature-B injected values
+ * are shown (they come from committed config).
+ */
+export declare function formatForwardedEnvLine(env: NodeJS.ProcessEnv, classified: {
+    injected: readonly string[];
+    passedThrough: readonly string[];
+}): string;
 //# sourceMappingURL=env-scrub.d.ts.map

package/dist/skill-test/env-scrub.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"env-scrub.d.ts","sourceRoot":"","sources":["../../src/skill-test/env-scrub.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,WAAW,iBAAiB;IAChC,sHAAsH;IACtH,iBAAiB,EAAE,OAAO,CAAC;IAC3B,6EAA6E;IAC7E,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;CACtB;AAWD,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,MAAM,CAAC,UAAU,EACzB,IAAI,EAAE,iBAAiB,GACtB,MAAM,CAAC,UAAU,CAuBnB"}
+{"version":3,"file":"env-scrub.d.ts","sourceRoot":"","sources":["../../src/skill-test/env-scrub.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,MAAM,WAAW,iBAAiB;IAChC,sHAAsH;IACtH,iBAAiB,EAAE,OAAO,CAAC;IAC3B,6EAA6E;IAC7E,SAAS,CAAC,EAAE,MAAM,EAAE,CAAC;CACtB;AA2DD,wBAAgB,iBAAiB,CAC/B,MAAM,EAAE,MAAM,CAAC,UAAU,EACzB,IAAI,EAAE,iBAAiB,GACtB,MAAM,CAAC,UAAU,CAuBnB;AAED;;;;;;GAMG;AACH,wBAAgB,iBAAiB,CAAC,SAAS,GAAE,SAAS,MAAM,EAAO,GAAG,GAAG,CAAC,MAAM,CAAC,CAShF;AAED;;;;;;;;;GASG;AACH,wBAAgB,eAAe,CAC7B,IAAI,EAAE,MAAM,EACZ,YAAY,EAAE,GAAG,CAAC,MAAM,CAAC,EACzB,QAAQ,GAAE,MAAyB,GAClC,OAAO,CAST;AAED,wEAAwE;AACxE,MAAM,WAAW,gBAAgB;IAC/B,6DAA6D;IAC7D,MAAM,EAAE,MAAM,CAAC,UAAU,CAAC;IAC1B,4DAA4D;IAC5D,OAAO,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;IAC5B,uEAAuE;IACvE,SAAS,CAAC,EAAE,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;IACnC,gEAAgE;IAChE,SAAS,CAAC,EAAE,SAAS,MAAM,EAAE,CAAC;CAC/B;AAED,MAAM,WAAW,iBAAiB;IAChC,4DAA4D;IAC5D,GAAG,EAAE,MAAM,CAAC,UAAU,CAAC;IACvB,0DAA0D;IAC1D,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,qEAAqE;IACrE,QAAQ,EAAE,MAAM,EAAE,CAAC;IACnB,8EAA8E;IAC9E,aAAa,EAAE,MAAM,EAAE,CAAC;CACzB;AAED;;;;;;GAMG;AACH,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,CAAC,UAAU,EAAE,KAAK,EAAE,gBAAgB,GAAG,iBAAiB,CAiCpG;AAKD;;;;;GAKG;AACH,wBAAgB,sBAAsB,CACpC,GAAG,EAAE,MAAM,CAAC,UAAU,EACtB,UAAU,EAAE;IAAE,QAAQ,EAAE,SAAS,MAAM,EAAE,CAAC;IAAC,aAAa,EAAE,SAAS,MAAM,EAAE,CAAA;CAAE,GAC5E,MAAM,CAUR"}

package/dist/skill-test/env-scrub.js

@@ -4,12 +4,59 @@
  * prefix — that would silently leak ANTHROPIC_ADMIN_API_KEY, proxy overrides,
  * etc. into untrusted skill code. Anything not enumerated is dropped.
  */
-/** Process-essential vars that must survive for the child to even start. */
-const PROCESS_ESSENTIALS = ['PATH', 'Path', 'HOME', 'USERPROFILE', 'SystemRoot', 'TEMP', 'TMP', 'TMPDIR', 'LANG', 'LC_ALL'];
+/**
+ * Process-essential vars that must survive for the child to even start. Includes
+ * the standard Windows essentials (APPDATA/LOCALAPPDATA — Claude's default config
+ * dir derives from these when CLAUDE_CONFIG_DIR is unset; SystemDrive, windir,
+ * PATHEXT, COMSPEC — needed to locate and launch executables). None are
+ * secret-bearing; the allowlist stays strict/exact-name.
+ *
+ * USER/LOGNAME (the POSIX username vars) are load-bearing for subscription auth
+ * on macOS: the `claude` CLI resolves the OAuth/subscription token from the login
+ * Keychain, and that lookup needs $USER — strip it and `claude auth status`
+ * reports loggedIn:false even with an active subscription, so `--auth
+ * subscription` (and inherit's subscription fallback) wrongly fail preflight, and
+ * the experimenter child can't authenticate. Neither is secret-bearing (the
+ * username is already derivable from the forwarded HOME), so they fit the policy.
+ * Not needed for api-key auth (key is self-contained) or Linux subscription auth
+ * (token is a HOME-relative file) — which is why this gap went unnoticed.
+ */
+const PROCESS_ESSENTIALS = [
+    'PATH', 'Path', 'HOME', 'USER', 'LOGNAME', 'USERPROFILE', 'SystemRoot', 'TEMP', 'TMP', 'TMPDIR', 'LANG', 'LC_ALL',
+    'APPDATA', 'LOCALAPPDATA', 'SystemDrive', 'windir', 'PATHEXT', 'COMSPEC',
+];
 /** The only auth/config vars ever forwarded. */
 const AUTH_CONFIG_ALLOWLIST = ['CLAUDE_CONFIG_DIR'];
 /** The active inference credential candidates (forwarded unless scrubbed). */
 const INFERENCE_CREDENTIALS = ['ANTHROPIC_API_KEY', 'ANTHROPIC_AUTH_TOKEN'];
+/**
+ * Vars that a declared env entry (injectEnv / passEnv) may NEVER override.
+ * These are deny-only — they are not in the forward allowlist either.
+ *
+ * WHY: Any of these names let a committed config attack without skill code:
+ *   • ANTHROPIC_*_BASE_URL / ANTHROPIC_API_URL — redirect the inference endpoint
+ *     so the forwarded ANTHROPIC_API_KEY is sent to an attacker-controlled server.
+ *   • *_PROXY / *_proxy — redirect all HTTPS traffic (including API calls) through
+ *     a MITM proxy even when the base URL looks correct.
+ *   • NODE_OPTIONS — inject arbitrary code into the claude child process via
+ *     --require or --import before any userland code runs.
+ *   • NODE_EXTRA_CA_CERTS — add a rogue CA certificate, enabling TLS interception.
+ *   • LD_PRELOAD / LD_LIBRARY_PATH / DYLD_INSERT_LIBRARIES / DYLD_LIBRARY_PATH —
+ *     the OS-linker siblings of NODE_OPTIONS: load an attacker .so/.dylib into the
+ *     child before any userland code runs (native code injection, bundler-agnostic).
+ *   • NODE_PATH — prepend an attacker directory to Node's module resolution so a
+ *     rogue package shadows a legitimate `require`.
+ *   • GIT_SSH_COMMAND — run an arbitrary command on any git operation the harness
+ *     performs (e.g. a git: source clone).
+ */
+const CREDENTIAL_ROUTING_DENY = [
+    'ANTHROPIC_BASE_URL', 'ANTHROPIC_API_URL', 'ANTHROPIC_BEDROCK_BASE_URL', 'ANTHROPIC_VERTEX_BASE_URL',
+    'HTTP_PROXY', 'HTTPS_PROXY', 'ALL_PROXY', 'NO_PROXY',
+    'http_proxy', 'https_proxy', 'all_proxy', 'no_proxy',
+    'NODE_OPTIONS', 'NODE_EXTRA_CA_CERTS', 'NODE_PATH',
+    'LD_PRELOAD', 'LD_LIBRARY_PATH', 'DYLD_INSERT_LIBRARIES', 'DYLD_LIBRARY_PATH',
+    'GIT_SSH_COMMAND',
+];
 export function buildForwardedEnv(source, opts) {
     const out = {};
     const allow = new Set([
@@ -32,4 +79,103 @@ export function buildForwardedEnv(source, opts) {
     delete out['ANTHROPIC_ADMIN_API_KEY'];
     return out;
 }
+/**
+ * The full set of env var names protected from declared override. A declared
+ * passEnv/injectEnv entry naming one of these is ignored (the protected value
+ * wins) and a warning is surfaced — a test must never clobber PATH, the auth
+ * credentials, or the admin key. `modelVars` (run-specific model env var names)
+ * join the set so a declared var can't shadow them either.
+ */
+export function protectedEnvNames(modelVars = []) {
+    return new Set([
+        ...PROCESS_ESSENTIALS,
+        ...AUTH_CONFIG_ALLOWLIST,
+        ...INFERENCE_CREDENTIALS,
+        'ANTHROPIC_ADMIN_API_KEY',
+        ...CREDENTIAL_ROUTING_DENY,
+        ...modelVars,
+    ]);
+}
+/**
+ * Returns true if `name` is in `protectedSet`.
+ *
+ * On Windows (win32) env names are case-insensitive — `path` and `PATH` are the
+ * same variable — so we upper-case both sides before comparing. On POSIX, names
+ * are genuinely case-distinct, so the comparison stays exact.
+ *
+ * The `platform` parameter defaults to `process.platform` but is exposed so
+ * unit tests can exercise the win32 branch on any host OS.
+ */
+export function isProtectedName(name, protectedSet, platform = process.platform) {
+    if (platform === 'win32') {
+        const upper = name.toUpperCase();
+        for (const p of protectedSet) {
+            if (p.toUpperCase() === upper)
+                return true;
+        }
+        return false;
+    }
+    return protectedSet.has(name);
+}
+/**
+ * Union declared test env (Features A + B) onto an already-built forwarded env.
+ * Protected keys always win: a declared name colliding with a process-essential,
+ * auth, model, or admin var is ignored and a warning emitted. Feature-B injection
+ * (explicit value) takes precedence over Feature-A pass-through for the same key.
+ * The input `base` object is never mutated.
+ */
+export function applyDeclaredEnv(base, input) {
+    const out = { ...base };
+    const protectedNames = protectedEnvNames(input.modelVars);
+    const warnings = [];
+    const injected = [];
+    const passedThrough = [];
+    const injectKeys = new Set(Object.keys(input.injectEnv ?? {}));
+    // Feature A: pass-through by name.
+    for (const name of input.passEnv ?? []) {
+        if (isProtectedName(name, protectedNames)) {
+            warnings.push(`passEnv "${name}" ignored: it collides with a protected variable.`);
+            continue;
+        }
+        if (injectKeys.has(name))
+            continue; // Feature B wins for the same key.
+        const value = input.source[name];
+        if (value === undefined)
+            continue; // absent host var → simply not forwarded.
+        out[name] = value;
+        passedThrough.push(name);
+    }
+    // Feature B: explicit value injection.
+    for (const [name, value] of Object.entries(input.injectEnv ?? {})) {
+        if (isProtectedName(name, protectedNames)) {
+            warnings.push(`env "${name}" ignored: it collides with a protected variable.`);
+            continue;
+        }
+        out[name] = value;
+        injected.push(name);
+    }
+    return { env: out, warnings, injected, passedThrough };
+}
+/** Names whose VALUES are secrets and must be redacted in the transparency line. */
+const SECRET_NAMES = new Set(INFERENCE_CREDENTIALS);
+/**
+ * Render the single-line stderr transparency summary of the forwarded env. Key
+ * names are always shown. Auth/secret values are redacted; Feature-A pass-through
+ * values are redacted (host-sourced, may be a secret); Feature-B injected values
+ * are shown (they come from committed config).
+ */
+export function formatForwardedEnvLine(env, classified) {
+    const injected = new Set(classified.injected);
+    const passedThrough = new Set(classified.passedThrough);
+    const parts = Object.keys(env).map((name) => {
+        if (injected.has(name))
+            return `${name}=${env[name] ?? ''}`;
+        if (passedThrough.has(name))
+            return `${name}(passed-through, redacted)`;
+        if (SECRET_NAMES.has(name))
+            return `${name}(redacted)`;
+        return name;
+    });
+    return `forwarded env: ${parts.join(', ')}`;
+}
 //# sourceMappingURL=env-scrub.js.map

package/dist/skill-test/env-scrub.js.map

@@ -1 +1 @@
-{"version":3,"file":"env-scrub.js","sourceRoot":"","sources":["../../src/skill-test/env-scrub.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AASH,4EAA4E;AAC5E,MAAM,kBAAkB,GAAG,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,aAAa,EAAE,YAAY,EAAE,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ,CAAU,CAAC;AAErI,gDAAgD;AAChD,MAAM,qBAAqB,GAAG,CAAC,mBAAmB,CAAU,CAAC;AAE7D,8EAA8E;AAC9E,MAAM,qBAAqB,GAAG,CAAC,mBAAmB,EAAE,sBAAsB,CAAU,CAAC;AAErF,MAAM,UAAU,iBAAiB,CAC/B,MAAyB,EACzB,IAAuB;IAEvB,MAAM,GAAG,GAAsB,EAAE,CAAC;IAElC,MAAM,KAAK,GAAG,IAAI,GAAG,CAAS;QAC5B,GAAG,kBAAkB;QACrB,GAAG,qBAAqB;QACxB,GAAG,CAAC,IAAI,CAAC,SAAS,IAAI,EAAE,CAAC;KAC1B,CAAC,CAAC;IACH,IAAI,CAAC,IAAI,CAAC,iBAAiB,EAAE,CAAC;QAC5B,KAAK,MAAM,IAAI,IAAI,qBAAqB;YAAE,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IAC5D,CAAC;IAED,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,KAAK,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;QAC3B,IAAI,KAAK,KAAK,SAAS;YAAE,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC;IAC7C,CAAC;IAED,6EAA6E;IAC7E,8EAA8E;IAC9E,6EAA6E;IAC7E,OAAO,GAAG,CAAC,yBAAyB,CAAC,CAAC;IAEtC,OAAO,GAAG,CAAC;AACb,CAAC"}
+{"version":3,"file":"env-scrub.js","sourceRoot":"","sources":["../../src/skill-test/env-scrub.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AASH;;;;;;;;;;;;;;;;GAgBG;AACH,MAAM,kBAAkB,GAAG;IACzB,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,SAAS,EAAE,aAAa,EAAE,YAAY,EAAE,MAAM,EAAE,KAAK,EAAE,QAAQ,EAAE,MAAM,EAAE,QAAQ;IACjH,SAAS,EAAE,cAAc,EAAE,aAAa,EAAE,QAAQ,EAAE,SAAS,EAAE,SAAS;CAChE,CAAC;AAEX,gDAAgD;AAChD,MAAM,qBAAqB,GAAG,CAAC,mBAAmB,CAAU,CAAC;AAE7D,8EAA8E;AAC9E,MAAM,qBAAqB,GAAG,CAAC,mBAAmB,EAAE,sBAAsB,CAAU,CAAC;AAErF;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,uBAAuB,GAAG;IAC9B,oBAAoB,EAAE,mBAAmB,EAAE,4BAA4B,EAAE,2BAA2B;IACpG,YAAY,EAAE,aAAa,EAAE,WAAW,EAAE,UAAU;IACpD,YAAY,EAAE,aAAa,EAAE,WAAW,EAAE,UAAU;IACpD,cAAc,EAAE,qBAAqB,EAAE,WAAW;IAClD,YAAY,EAAE,iBAAiB,EAAE,uBAAuB,EAAE,mBAAmB;IAC7E,iBAAiB;CACT,CAAC;AAEX,MAAM,UAAU,iBAAiB,CAC/B,MAAyB,EACzB,IAAuB;IAEvB,MAAM,GAAG,GAAsB,EAAE,CAAC;IAElC,MAAM,KAAK,GAAG,IAAI,GAAG,CAAS;QAC5B,GAAG,kBAAkB;QACrB,GAAG,qBAAqB;QACxB,GAAG,CAAC,IAAI,CAAC,SAAS,IAAI,EAAE,CAAC;KAC1B,CAAC,CAAC;IACH,IAAI,CAAC,IAAI,CAAC,iBAAiB,EAAE,CAAC;QAC5B,KAAK,MAAM,IAAI,IAAI,qBAAqB;YAAE,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;IAC5D,CAAC;IAED,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,KAAK,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;QAC3B,IAAI,KAAK,KAAK,SAAS;YAAE,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC;IAC7C,CAAC;IAED,6EAA6E;IAC7E,8EAA8E;IAC9E,6EAA6E;IAC7E,OAAO,GAAG,CAAC,yBAAyB,CAAC,CAAC;IAEtC,OAAO,GAAG,CAAC;AACb,CAAC;AAED;;;;;;GAMG;AACH,MAAM,UAAU,iBAAiB,CAAC,YAA+B,EAAE;IACjE,OAAO,IAAI,GAAG,CAAS;QACrB,GAAG,kBAAkB;QACrB,GAAG,qBAAqB;QACxB,GAAG,qBAAqB;QACxB,yBAAyB;QACzB,GAAG,uBAAuB;QAC1B,GAAG,SAAS;KACb,CAAC,CAAC;AACL,CAAC;AAED;;;;;;;;;GASG;AACH,MAAM,UAAU,eAAe,CAC7B,IAAY,EACZ,YAAyB,EACzB,WAAmB,OAAO,CAAC,QAAQ;IAEnC,IAAI,QAAQ,KAAK,OAAO,EAAE,CAAC;QACzB,MAAM,KAAK,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;QACjC,KAAK,MAAM,CAAC,IAAI,YAAY,EAAE,CAAC;YAC7B,IAAI,CAAC,CAAC,WAAW,EAAE,KAAK,KAAK;gBAAE,OAAO,IAAI,CAAC;QAC7C,CAAC;QACD,OAAO,KAAK,CAAC;IACf,CAAC;IACD,OAAO,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC;AAChC,CAAC;AAyBD;;;;;;GAMG;AACH,MAAM,UAAU,gBAAgB,CAAC,IAAuB,EAAE,KAAuB;IAC/E,MAAM,GAAG,GAAsB,EAAE,GAAG,IAAI,EAAE,CAAC;IAC3C,MAAM,cAAc,GAAG,iBAAiB,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;IAC1D,MAAM,QAAQ,GAAa,EAAE,CAAC;IAC9B,MAAM,QAAQ,GAAa,EAAE,CAAC;IAC9B,MAAM,aAAa,GAAa,EAAE,CAAC;IAEnC,MAAM,UAAU,GAAG,IAAI,GAAG,CAAC,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,IAAI,EAAE,CAAC,CAAC,CAAC;IAE/D,mCAAmC;IACnC,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,OAAO,IAAI,EAAE,EAAE,CAAC;QACvC,IAAI,eAAe,CAAC,IAAI,EAAE,cAAc,CAAC,EAAE,CAAC;YAC1C,QAAQ,CAAC,IAAI,CAAC,YAAY,IAAI,mDAAmD,CAAC,CAAC;YACnF,SAAS;QACX,CAAC;QACD,IAAI,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,SAAS,CAAC,mCAAmC;QACvE,MAAM,KAAK,GAAG,KAAK,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;QACjC,IAAI,KAAK,KAAK,SAAS;YAAE,SAAS,CAAC,0CAA0C;QAC7E,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC;QAClB,aAAa,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IAC3B,CAAC;IAED,uCAAuC;IACvC,KAAK,MAAM,CAAC,IAAI,EAAE,KAAK,CAAC,IAAI,MAAM,CAAC,OAAO,CAAC,KAAK,CAAC,SAAS,IAAI,EAAE,CAAC,EAAE,CAAC;QAClE,IAAI,eAAe,CAAC,IAAI,EAAE,cAAc,CAAC,EAAE,CAAC;YAC1C,QAAQ,CAAC,IAAI,CAAC,QAAQ,IAAI,mDAAmD,CAAC,CAAC;YAC/E,SAAS;QACX,CAAC;QACD,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,CAAC;QAClB,QAAQ,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;IACtB,CAAC;IAED,OAAO,EAAE,GAAG,EAAE,GAAG,EAAE,QAAQ,EAAE,QAAQ,EAAE,aAAa,EAAE,CAAC;AACzD,CAAC;AAED,oFAAoF;AACpF,MAAM,YAAY,GAAG,IAAI,GAAG,CAAS,qBAAqB,CAAC,CAAC;AAE5D;;;;;GAKG;AACH,MAAM,UAAU,sBAAsB,CACpC,GAAsB,EACtB,UAA6E;IAE7E,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,UAAU,CAAC,QAAQ,CAAC,CAAC;IAC9C,MAAM,aAAa,GAAG,IAAI,GAAG,CAAC,UAAU,CAAC,aAAa,CAAC,CAAC;IACxD,MAAM,KAAK,GAAG,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;QAC1C,IAAI,QAAQ,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,OAAO,GAAG,IAAI,IAAI,GAAG,CAAC,IAAI,CAAC,IAAI,EAAE,EAAE,CAAC;QAC5D,IAAI,aAAa,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,OAAO,GAAG,IAAI,4BAA4B,CAAC;QACxE,IAAI,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,OAAO,GAAG,IAAI,YAAY,CAAC;QACvD,OAAO,IAAI,CAAC;IACd,CAAC,CAAC,CAAC;IACH,OAAO,kBAAkB,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;AAC9C,CAAC"}

package/dist/skill-test/index.d.ts

@@ -1,4 +1,4 @@
-export { buildForwardedEnv, type ForwardEnvOptions } from './env-scrub.js';
+export { applyDeclaredEnv, buildForwardedEnv, formatForwardedEnvLine, protectedEnvNames, type DeclaredEnvInput, type DeclaredEnvResult, type ForwardEnvOptions, } from './env-scrub.js';
 export { AuthPreflightError, probeAuthStatus, resolveAuth, type AuthMechanism, type AuthMode, type AuthStatusProbe, type AuthStatusResult, type ResolveAuthOptions, type ResolvedAuth, } from './auth-resolver.js';
 export { assembleClaudeArgs, spawnHeadlessClaude, type ClaudeSpawnArgs, type SpawnHeadlessOptions, type SpawnResult, } from './spawn-claude.js';
 //# sourceMappingURL=index.d.ts.map

package/dist/skill-test/index.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/skill-test/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAAE,KAAK,iBAAiB,EAAE,MAAM,gBAAgB,CAAC;AAC3E,OAAO,EACL,kBAAkB,EAClB,eAAe,EACf,WAAW,EACX,KAAK,aAAa,EAClB,KAAK,QAAQ,EACb,KAAK,eAAe,EACpB,KAAK,gBAAgB,EACrB,KAAK,kBAAkB,EACvB,KAAK,YAAY,GAClB,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EACL,kBAAkB,EAClB,mBAAmB,EACnB,KAAK,eAAe,EACpB,KAAK,oBAAoB,EACzB,KAAK,WAAW,GACjB,MAAM,mBAAmB,CAAC"}
+{"version":3,"file":"index.d.ts","sourceRoot":"","sources":["../../src/skill-test/index.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,gBAAgB,EAChB,iBAAiB,EACjB,sBAAsB,EACtB,iBAAiB,EACjB,KAAK,gBAAgB,EACrB,KAAK,iBAAiB,EACtB,KAAK,iBAAiB,GACvB,MAAM,gBAAgB,CAAC;AACxB,OAAO,EACL,kBAAkB,EAClB,eAAe,EACf,WAAW,EACX,KAAK,aAAa,EAClB,KAAK,QAAQ,EACb,KAAK,eAAe,EACpB,KAAK,gBAAgB,EACrB,KAAK,kBAAkB,EACvB,KAAK,YAAY,GAClB,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EACL,kBAAkB,EAClB,mBAAmB,EACnB,KAAK,eAAe,EACpB,KAAK,oBAAoB,EACzB,KAAK,WAAW,GACjB,MAAM,mBAAmB,CAAC"}

package/dist/skill-test/index.js

@@ -1,4 +1,4 @@
-export { buildForwardedEnv } from './env-scrub.js';
+export { applyDeclaredEnv, buildForwardedEnv, formatForwardedEnvLine, protectedEnvNames, } from './env-scrub.js';
 export { AuthPreflightError, probeAuthStatus, resolveAuth, } from './auth-resolver.js';
 export { assembleClaudeArgs, spawnHeadlessClaude, } from './spawn-claude.js';
 //# sourceMappingURL=index.js.map

package/dist/skill-test/index.js.map

@@ -1 +1 @@
-{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/skill-test/index.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,iBAAiB,EAA0B,MAAM,gBAAgB,CAAC;AAC3E,OAAO,EACL,kBAAkB,EAClB,eAAe,EACf,WAAW,GAOZ,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EACL,kBAAkB,EAClB,mBAAmB,GAIpB,MAAM,mBAAmB,CAAC"}
+{"version":3,"file":"index.js","sourceRoot":"","sources":["../../src/skill-test/index.ts"],"names":[],"mappings":"AAAA,OAAO,EACL,gBAAgB,EAChB,iBAAiB,EACjB,sBAAsB,EACtB,iBAAiB,GAIlB,MAAM,gBAAgB,CAAC;AACxB,OAAO,EACL,kBAAkB,EAClB,eAAe,EACf,WAAW,GAOZ,MAAM,oBAAoB,CAAC;AAC5B,OAAO,EACL,kBAAkB,EAClB,mBAAmB,GAIpB,MAAM,mBAAmB,CAAC"}

package/dist/skill-test/spawn-claude.d.ts

@@ -18,14 +18,32 @@ export interface SpawnResult {
     timedOut: boolean;
     stalled: boolean;
 }
+/**
+ * SIGKILL the child AND every process in its group. The child is spawned
+ * `detached` (POSIX) so it leads a new process group; killing the negated pid
+ * reaps backgrounded grandchildren (e.g. a skill that runs `nohup … &`) and
+ * orphaned MCP subprocesses that a direct `child.kill()` would leave running.
+ *
+ * - Guards an undefined pid (child never spawned — nothing to kill).
+ * - Swallows the throw (ESRCH) when the group is already gone.
+ * - Windows has no POSIX process groups: fall back to `taskkill /T /F`, which
+ *   terminates the whole process tree.
+ */
+export declare function killProcessTree(child: {
+    pid?: number | undefined;
+}): void;
 export interface SpawnHeadlessOptions extends ClaudeSpawnArgs {
     /**
-     * Path to the experimenter-prompt file. Its contents are streamed to the
-     * child's stdin (claude 2.x reads the `-p` prompt from stdin; there is no
-     * `--prompt-file` flag). Kept off argv so the prompt is never inlined
-     * (Windows cmd-quoting safety).
+     * The experimenter prompt, held IN MEMORY and streamed to the child's stdin
+     * (claude 2.x reads the `-p` prompt from stdin; there is no `--prompt-file`
+     * flag). Kept off argv so the prompt is never inlined (Windows cmd-quoting
+     * safety) AND, deliberately, off disk: the harness stamps a per-run integrity
+     * nonce into this prompt, and writing it to a file inside (or beside) the
+     * skill-writable sandbox would let untrusted skill code read the nonce back and
+     * forge a passing grading.json. Passing it in memory keeps the nonce out of any
+     * file the skill can read.
      */
-    promptFile: string;
+    prompt: string;
     cwd: string;
     env: NodeJS.ProcessEnv;
     timeoutMs: number;
@@ -33,12 +51,19 @@ export interface SpawnHeadlessOptions extends ClaudeSpawnArgs {
     stallMs?: number;
     onStdout?: (chunk: string) => void;
     onStderr?: (chunk: string) => void;
+    /**
+     * INTERNAL test seam. When set, this executable is spawned instead of resolving
+     * `claude` on PATH — letting tests exercise the wall-timeout / stall watchdog
+     * against a tiny fake child without a real `claude` install. Production callers
+     * never set this; `claude` is resolved via `which`.
+     */
+    binPath?: string;
 }
 /**
- * Spawn the headless `claude`. The prompt file is streamed to the child's stdin
- * so the session reads its prompt and then sees EOF (it cannot block on input,
- * §16). Wall-clock kill on timeout. If `stallMs` is set, a resettable stall
- * watchdog kills the child when no output is received for that duration
+ * Spawn the headless `claude`. The in-memory prompt is streamed to the child's
+ * stdin so the session reads its prompt and then sees EOF (it cannot block on
+ * input, §16). Wall-clock kill on timeout. If `stallMs` is set, a resettable
+ * stall watchdog kills the child when no output is received for that duration
  * (stalled: true in result).
  */
 export declare function spawnHeadlessClaude(opts: SpawnHeadlessOptions): Promise<SpawnResult>;

package/dist/skill-test/spawn-claude.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"spawn-claude.d.ts","sourceRoot":"","sources":["../../src/skill-test/spawn-claude.ts"],"names":[],"mappings":"AAKA,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED;;;;;;GAMG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,eAAe,GAAG,MAAM,EAAE,CAmBlE;AAED,MAAM,WAAW,WAAW;IAC1B,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,OAAO,CAAC;IAClB,OAAO,EAAE,OAAO,CAAC;CAClB;AAED,MAAM,WAAW,oBAAqB,SAAQ,eAAe;IAC3D;;;;;OAKG;IACH,UAAU,EAAE,MAAM,CAAC;IACnB,GAAG,EAAE,MAAM,CAAC;IACZ,GAAG,EAAE,MAAM,CAAC,UAAU,CAAC;IACvB,SAAS,EAAE,MAAM,CAAC;IAClB,yFAAyF;IACzF,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;IACnC,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;CACpC;AAED;;;;;;GAMG;AACH,wBAAsB,mBAAmB,CAAC,IAAI,EAAE,oBAAoB,GAAG,OAAO,CAAC,WAAW,CAAC,CAsD1F"}
+{"version":3,"file":"spawn-claude.d.ts","sourceRoot":"","sources":["../../src/skill-test/spawn-claude.ts"],"names":[],"mappings":"AAKA,MAAM,WAAW,eAAe;IAC9B,UAAU,EAAE,MAAM,EAAE,CAAC;IACrB,UAAU,EAAE,MAAM,CAAC;IACnB,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,YAAY,CAAC,EAAE,MAAM,CAAC;CACvB;AAED;;;;;;GAMG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,eAAe,GAAG,MAAM,EAAE,CAmBlE;AAED,MAAM,WAAW,WAAW;IAC1B,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,OAAO,CAAC;IAClB,OAAO,EAAE,OAAO,CAAC;CAClB;AAED;;;;;;;;;;GAUG;AACH,wBAAgB,eAAe,CAAC,KAAK,EAAE;IAAE,GAAG,CAAC,EAAE,MAAM,GAAG,SAAS,CAAA;CAAE,GAAG,IAAI,CAczE;AAED,MAAM,WAAW,oBAAqB,SAAQ,eAAe;IAC3D;;;;;;;;;OASG;IACH,MAAM,EAAE,MAAM,CAAC;IACf,GAAG,EAAE,MAAM,CAAC;IACZ,GAAG,EAAE,MAAM,CAAC,UAAU,CAAC;IACvB,SAAS,EAAE,MAAM,CAAC;IAClB,yFAAyF;IACzF,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;IACnC,QAAQ,CAAC,EAAE,CAAC,KAAK,EAAE,MAAM,KAAK,IAAI,CAAC;IACnC;;;;;OAKG;IACH,OAAO,CAAC,EAAE,MAAM,CAAC;CAClB;AAED;;;;;;GAMG;AACH,wBAAsB,mBAAmB,CAAC,IAAI,EAAE,oBAAoB,GAAG,OAAO,CAAC,WAAW,CAAC,CAwE1F"}

package/dist/skill-test/spawn-claude.js

@@ -1,5 +1,5 @@
-import { spawn } from 'node:child_process';
-import { createReadStream } from 'node:fs';
+import { spawn, spawnSync } from 'node:child_process';
+import { Readable } from 'node:stream';
 import which from 'which';
 /**
  * Pure assembly of the headless `claude -p` argv (spec §9). The prompt is never
@@ -32,22 +32,67 @@ export function assembleClaudeArgs(opts) {
     return args;
 }
 /**
- * Spawn the headless `claude`. The prompt file is streamed to the child's stdin
- * so the session reads its prompt and then sees EOF (it cannot block on input,
- * §16). Wall-clock kill on timeout. If `stallMs` is set, a resettable stall
- * watchdog kills the child when no output is received for that duration
+ * SIGKILL the child AND every process in its group. The child is spawned
+ * `detached` (POSIX) so it leads a new process group; killing the negated pid
+ * reaps backgrounded grandchildren (e.g. a skill that runs `nohup … &`) and
+ * orphaned MCP subprocesses that a direct `child.kill()` would leave running.
+ *
+ * - Guards an undefined pid (child never spawned — nothing to kill).
+ * - Swallows the throw (ESRCH) when the group is already gone.
+ * - Windows has no POSIX process groups: fall back to `taskkill /T /F`, which
+ *   terminates the whole process tree.
+ */
+export function killProcessTree(child) {
+    const { pid } = child;
+    if (pid === undefined)
+        return;
+    try {
+        if (process.platform === 'win32') {
+            // eslint-disable-next-line sonarjs/no-os-command-from-path -- taskkill is a fixed Windows system command (no PATH-injection surface); it is the only POSIX-process-group-free way to terminate the child's tree.
+            spawnSync('taskkill', ['/pid', String(pid), '/T', '/F']);
+        }
+        else {
+            // Negated pid → deliver the signal to the whole process group.
+            process.kill(-pid, 'SIGKILL');
+        }
+    }
+    catch {
+        // ESRCH (no such process/group): the child and its group are already dead.
+    }
+}
+/**
+ * Spawn the headless `claude`. The in-memory prompt is streamed to the child's
+ * stdin so the session reads its prompt and then sees EOF (it cannot block on
+ * input, §16). Wall-clock kill on timeout. If `stallMs` is set, a resettable
+ * stall watchdog kills the child when no output is received for that duration
  * (stalled: true in result).
  */
 export async function spawnHeadlessClaude(opts) {
-    const bin = which.sync('claude');
+    // opts.binPath is an internal test seam (see SpawnHeadlessOptions); production
+    // callers leave it unset and `claude` is resolved on PATH.
+    const bin = opts.binPath ?? which.sync('claude');
     const args = assembleClaudeArgs(opts);
     return await new Promise((resolve, reject) => {
-        const child = spawn(bin, args, { cwd: opts.cwd, env: opts.env, stdio: ['pipe', 'pipe', 'pipe'] });
+        const child = spawn(bin, args, {
+            cwd: opts.cwd,
+            env: opts.env,
+            stdio: ['pipe', 'pipe', 'pipe'],
+            // POSIX: lead a new process group so the kill paths can SIGKILL the whole
+            // group (backgrounded grandchildren + MCP subprocesses), not just the
+            // direct child. Windows has no process groups (taskkill handles the tree).
+            detached: process.platform !== 'win32',
+        });
         let timedOut = false;
         let stalled = false;
+        // Feed the prompt to the child via stdin (claude 2.x has no --prompt-file).
+        // Built from the in-memory string (never a file — the prompt carries a secret
+        // per-run nonce that must not land on the skill-readable filesystem). Created
+        // here (before the timers) so every kill path can destroy it if the child is
+        // SIGKILL'd before stdin is fully consumed.
+        const promptStream = Readable.from([opts.prompt], { objectMode: false });
         const wallTimer = setTimeout(() => {
             timedOut = true;
-            child.kill('SIGKILL');
+            killProcessTree(child);
         }, opts.timeoutMs);
         // Stall watchdog: reset on every stdout/stderr chunk; fire if silent for stallMs.
         let stallTimer;
@@ -58,7 +103,7 @@ export async function spawnHeadlessClaude(opts) {
                     clearTimeout(stallTimer);
                 stallTimer = setTimeout(() => {
                     stalled = true;
-                    child.kill('SIGKILL');
+                    killProcessTree(child);
                 }, stallMs);
             };
             resetStallTimer();
@@ -69,13 +114,14 @@ export async function spawnHeadlessClaude(opts) {
             clearTimeout(wallTimer);
             if (stallTimer !== undefined)
                 clearTimeout(stallTimer);
+            // Destroy the prompt stream on every terminal path (timeout/stall reach
+            // here via 'close'; error paths call it directly) so an unconsumed in-memory
+            // stream is torn down. On a clean run the stream has already ended — no-op.
+            promptStream.destroy();
         };
-        // Feed the prompt to the child via stdin (claude 2.x has no --prompt-file).
-        // eslint-disable-next-line security/detect-non-literal-fs-filename -- our own derived prompt path
-        const promptStream = createReadStream(opts.promptFile);
         promptStream.on('error', err => {
             clearAllTimers();
-            child.kill('SIGKILL');
+            killProcessTree(child);
             reject(err);
         });
         // Swallow EPIPE: if the child exits before consuming all stdin, the write

package/dist/skill-test/spawn-claude.js.map

@@ -1 +1 @@
-{"version":3,"file":"spawn-claude.js","sourceRoot":"","sources":["../../src/skill-test/spawn-claude.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,MAAM,oBAAoB,CAAC;AAC3C,OAAO,EAAE,gBAAgB,EAAE,MAAM,SAAS,CAAC;AAE3C,OAAO,KAAK,MAAM,OAAO,CAAC;AAU1B;;;;;;GAMG;AACH,MAAM,UAAU,kBAAkB,CAAC,IAAqB;IACtD,MAAM,IAAI,GAAa;QACrB,IAAI;QACJ,0EAA0E;QAC1E,sEAAsE;QACtE,4DAA4D;QAC5D,iBAAiB,EAAE,aAAa;QAChC,WAAW;QACX,mBAAmB,EAAE,EAAE;QACvB,mBAAmB,EAAE,mBAAmB;QACxC,WAAW,EAAE,IAAI,CAAC,UAAU;KAC7B,CAAC;IACF,KAAK,MAAM,GAAG,IAAI,IAAI,CAAC,UAAU,EAAE,CAAC;QAClC,IAAI,CAAC,IAAI,CAAC,cAAc,EAAE,GAAG,CAAC,CAAC;IACjC,CAAC;IACD,IAAI,IAAI,CAAC,KAAK,KAAK,SAAS;QAAE,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC;IAC/D,IAAI,IAAI,CAAC,QAAQ,KAAK,SAAS;QAAE,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,MAAM,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC;IACjF,IAAI,IAAI,CAAC,YAAY,KAAK,SAAS;QAAE,IAAI,CAAC,IAAI,CAAC,kBAAkB,EAAE,MAAM,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC;IAC9F,OAAO,IAAI,CAAC;AACd,CAAC;AAyBD;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,mBAAmB,CAAC,IAA0B;IAClE,MAAM,GAAG,GAAG,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IACjC,MAAM,IAAI,GAAG,kBAAkB,CAAC,IAAI,CAAC,CAAC;IAEtC,OAAO,MAAM,IAAI,OAAO,CAAc,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QACxD,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,EAAE,IAAI,EAAE,EAAE,GAAG,EAAE,IAAI,CAAC,GAAG,EAAE,GAAG,EAAE,IAAI,CAAC,GAAG,EAAE,KAAK,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,CAAC,CAAC;QAClG,IAAI,QAAQ,GAAG,KAAK,CAAC;QACrB,IAAI,OAAO,GAAG,KAAK,CAAC;QAEpB,MAAM,SAAS,GAAG,UAAU,CAAC,GAAG,EAAE;YAChC,QAAQ,GAAG,IAAI,CAAC;YAChB,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACxB,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,CAAC;QAEnB,kFAAkF;QAClF,IAAI,UAAqD,CAAC;QAC1D,IAAI,IAAI,CAAC,OAAO,KAAK,SAAS,EAAE,CAAC;YAC/B,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC;YAC7B,MAAM,eAAe,GAAG,GAAS,EAAE;gBACjC,IAAI,UAAU,KAAK,SAAS;oBAAE,YAAY,CAAC,UAAU,CAAC,CAAC;gBACvD,UAAU,GAAG,UAAU,CAAC,GAAG,EAAE;oBAC3B,OAAO,GAAG,IAAI,CAAC;oBACf,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;gBACxB,CAAC,EAAE,OAAO,CAAC,CAAC;YACd,CAAC,CAAC;YACF,eAAe,EAAE,CAAC;YAElB,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,EAAE,GAAG,eAAe,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;YACtD,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,EAAE,GAAG,eAAe,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;QACxD,CAAC;QAED,MAAM,cAAc,GAAG,GAAS,EAAE;YAChC,YAAY,CAAC,SAAS,CAAC,CAAC;YACxB,IAAI,UAAU,KAAK,SAAS;gBAAE,YAAY,CAAC,UAAU,CAAC,CAAC;QACzD,CAAC,CAAC;QAEF,4EAA4E;QAC5E,kGAAkG;QAClG,MAAM,YAAY,GAAG,gBAAgB,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QACvD,YAAY,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,CAAC,EAAE;YAC7B,cAAc,EAAE,CAAC;YACjB,KAAK,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;YACtB,MAAM,CAAC,GAAG,CAAC,CAAC;QACd,CAAC,CAAC,CAAC;QACH,0EAA0E;QAC1E,4EAA4E;QAC5E,6BAA6B;QAC7B,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,EAAE,GAA0C,CAAC,CAAC,CAAC;QAC1E,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAC/B,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,CAAS,EAAE,EAAE,GAAG,IAAI,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3E,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,CAAS,EAAE,EAAE,GAAG,IAAI,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3E,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,CAAC,EAAE,GAAG,cAAc,EAAE,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC7D,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,IAAI,CAAC,EAAE,GAAG,cAAc,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,IAAI,IAAI,CAAC,CAAC,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IACvG,CAAC,CAAC,CAAC;AACL,CAAC"}
+{"version":3,"file":"spawn-claude.js","sourceRoot":"","sources":["../../src/skill-test/spawn-claude.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,KAAK,EAAE,SAAS,EAAE,MAAM,oBAAoB,CAAC;AACtD,OAAO,EAAE,QAAQ,EAAE,MAAM,aAAa,CAAC;AAEvC,OAAO,KAAK,MAAM,OAAO,CAAC;AAU1B;;;;;;GAMG;AACH,MAAM,UAAU,kBAAkB,CAAC,IAAqB;IACtD,MAAM,IAAI,GAAa;QACrB,IAAI;QACJ,0EAA0E;QAC1E,sEAAsE;QACtE,4DAA4D;QAC5D,iBAAiB,EAAE,aAAa;QAChC,WAAW;QACX,mBAAmB,EAAE,EAAE;QACvB,mBAAmB,EAAE,mBAAmB;QACxC,WAAW,EAAE,IAAI,CAAC,UAAU;KAC7B,CAAC;IACF,KAAK,MAAM,GAAG,IAAI,IAAI,CAAC,UAAU,EAAE,CAAC;QAClC,IAAI,CAAC,IAAI,CAAC,cAAc,EAAE,GAAG,CAAC,CAAC;IACjC,CAAC;IACD,IAAI,IAAI,CAAC,KAAK,KAAK,SAAS;QAAE,IAAI,CAAC,IAAI,CAAC,SAAS,EAAE,IAAI,CAAC,KAAK,CAAC,CAAC;IAC/D,IAAI,IAAI,CAAC,QAAQ,KAAK,SAAS;QAAE,IAAI,CAAC,IAAI,CAAC,aAAa,EAAE,MAAM,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC,CAAC;IACjF,IAAI,IAAI,CAAC,YAAY,KAAK,SAAS;QAAE,IAAI,CAAC,IAAI,CAAC,kBAAkB,EAAE,MAAM,CAAC,IAAI,CAAC,YAAY,CAAC,CAAC,CAAC;IAC9F,OAAO,IAAI,CAAC;AACd,CAAC;AAQD;;;;;;;;;;GAUG;AACH,MAAM,UAAU,eAAe,CAAC,KAAmC;IACjE,MAAM,EAAE,GAAG,EAAE,GAAG,KAAK,CAAC;IACtB,IAAI,GAAG,KAAK,SAAS;QAAE,OAAO;IAC9B,IAAI,CAAC;QACH,IAAI,OAAO,CAAC,QAAQ,KAAK,OAAO,EAAE,CAAC;YACjC,iNAAiN;YACjN,SAAS,CAAC,UAAU,EAAE,CAAC,MAAM,EAAE,MAAM,CAAC,GAAG,CAAC,EAAE,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC;QAC3D,CAAC;aAAM,CAAC;YACN,+DAA+D;YAC/D,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,EAAE,SAAS,CAAC,CAAC;QAChC,CAAC;IACH,CAAC;IAAC,MAAM,CAAC;QACP,2EAA2E;IAC7E,CAAC;AACH,CAAC;AA8BD;;;;;;GAMG;AACH,MAAM,CAAC,KAAK,UAAU,mBAAmB,CAAC,IAA0B;IAClE,+EAA+E;IAC/E,2DAA2D;IAC3D,MAAM,GAAG,GAAG,IAAI,CAAC,OAAO,IAAI,KAAK,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IACjD,MAAM,IAAI,GAAG,kBAAkB,CAAC,IAAI,CAAC,CAAC;IAEtC,OAAO,MAAM,IAAI,OAAO,CAAc,CAAC,OAAO,EAAE,MAAM,EAAE,EAAE;QACxD,MAAM,KAAK,GAAG,KAAK,CAAC,GAAG,EAAE,IAAI,EAAE;YAC7B,GAAG,EAAE,IAAI,CAAC,GAAG;YACb,GAAG,EAAE,IAAI,CAAC,GAAG;YACb,KAAK,EAAE,CAAC,MAAM,EAAE,MAAM,EAAE,MAAM,CAAC;YAC/B,0EAA0E;YAC1E,sEAAsE;YACtE,2EAA2E;YAC3E,QAAQ,EAAE,OAAO,CAAC,QAAQ,KAAK,OAAO;SACvC,CAAC,CAAC;QACH,IAAI,QAAQ,GAAG,KAAK,CAAC;QACrB,IAAI,OAAO,GAAG,KAAK,CAAC;QAEpB,4EAA4E;QAC5E,8EAA8E;QAC9E,8EAA8E;QAC9E,6EAA6E;QAC7E,4CAA4C;QAC5C,MAAM,YAAY,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,UAAU,EAAE,KAAK,EAAE,CAAC,CAAC;QAEzE,MAAM,SAAS,GAAG,UAAU,CAAC,GAAG,EAAE;YAChC,QAAQ,GAAG,IAAI,CAAC;YAChB,eAAe,CAAC,KAAK,CAAC,CAAC;QACzB,CAAC,EAAE,IAAI,CAAC,SAAS,CAAC,CAAC;QAEnB,kFAAkF;QAClF,IAAI,UAAqD,CAAC;QAC1D,IAAI,IAAI,CAAC,OAAO,KAAK,SAAS,EAAE,CAAC;YAC/B,MAAM,OAAO,GAAG,IAAI,CAAC,OAAO,CAAC;YAC7B,MAAM,eAAe,GAAG,GAAS,EAAE;gBACjC,IAAI,UAAU,KAAK,SAAS;oBAAE,YAAY,CAAC,UAAU,CAAC,CAAC;gBACvD,UAAU,GAAG,UAAU,CAAC,GAAG,EAAE;oBAC3B,OAAO,GAAG,IAAI,CAAC;oBACf,eAAe,CAAC,KAAK,CAAC,CAAC;gBACzB,CAAC,EAAE,OAAO,CAAC,CAAC;YACd,CAAC,CAAC;YACF,eAAe,EAAE,CAAC;YAElB,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,EAAE,GAAG,eAAe,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;YACtD,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,GAAG,EAAE,GAAG,eAAe,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;QACxD,CAAC;QAED,MAAM,cAAc,GAAG,GAAS,EAAE;YAChC,YAAY,CAAC,SAAS,CAAC,CAAC;YACxB,IAAI,UAAU,KAAK,SAAS;gBAAE,YAAY,CAAC,UAAU,CAAC,CAAC;YACvD,wEAAwE;YACxE,6EAA6E;YAC7E,4EAA4E;YAC5E,YAAY,CAAC,OAAO,EAAE,CAAC;QACzB,CAAC,CAAC;QAEF,YAAY,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,CAAC,EAAE;YAC7B,cAAc,EAAE,CAAC;YACjB,eAAe,CAAC,KAAK,CAAC,CAAC;YACvB,MAAM,CAAC,GAAG,CAAC,CAAC;QACd,CAAC,CAAC,CAAC;QACH,0EAA0E;QAC1E,4EAA4E;QAC5E,6BAA6B;QAC7B,KAAK,CAAC,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,EAAE,GAA0C,CAAC,CAAC,CAAC;QAC1E,YAAY,CAAC,IAAI,CAAC,KAAK,CAAC,KAAK,CAAC,CAAC;QAC/B,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,CAAS,EAAE,EAAE,GAAG,IAAI,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3E,KAAK,CAAC,MAAM,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,CAAS,EAAE,EAAE,GAAG,IAAI,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC3E,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,GAAG,CAAC,EAAE,GAAG,cAAc,EAAE,CAAC,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;QAC7D,KAAK,CAAC,EAAE,CAAC,OAAO,EAAE,IAAI,CAAC,EAAE,GAAG,cAAc,EAAE,CAAC,CAAC,OAAO,CAAC,EAAE,MAAM,EAAE,IAAI,IAAI,CAAC,CAAC,EAAE,QAAQ,EAAE,OAAO,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IACvG,CAAC,CAAC,CAAC;AACL,CAAC"}

package/dist/yaml/surgical-yaml.d.ts

@@ -0,0 +1,71 @@
+/**
+ * Byte-surgical YAML value updater.
+ *
+ * The motivating problem: editing one value in an adopter's config file with the
+ * naive `doc.setIn(...); doc.toString()` round-trip reflows the ENTIRE document —
+ * collapsing flow sequences, re-wrapping long scalars, shifting comment alignment,
+ * normalising quote styles. That destroys hand-authored formatting the user never
+ * asked us to touch.
+ *
+ * This module instead changes the **minimum bytes possible**:
+ *
+ * - **Replace** an existing scalar: locate its source byte range via the parsed
+ *   node and splice the new serialized token in place. Every other byte is
+ *   identical.
+ * - **Insert** a new key: render ONLY the new nested fragment, indent it to the
+ *   target map's child column, and splice it in after the map's last item. The
+ *   rest of the document is never re-rendered.
+ *
+ * `verifyConfinedYamlEdit` is the double-checker: it re-parses before/after and
+ * asserts the edit landed, no other leaf value drifted, and no comment was lost.
+ * `updateYamlIn` calls it internally as a defensive post-condition.
+ *
+ * No filesystem access — these are pure string transforms.
+ */
+/** A path into a YAML document: map keys (strings) and/or sequence indices. */
+export type YamlPath = (string | number)[];
+/** A scalar value this module knows how to serialize as a single YAML token. */
+export type YamlScalarValue = string | number | boolean | null;
+/**
+ * Upsert a single scalar at `path` in `text`, changing the minimum bytes possible.
+ *
+ * - If `path` already resolves to a scalar, its source token is replaced in place
+ *   (byte-identical output except the one value, including surrounding comments,
+ *   alignment whitespace, flow-collection padding, and sibling formatting).
+ * - If `path` does not yet exist, a new nested fragment is rendered and spliced
+ *   into the deepest existing ancestor map at the correct child indentation —
+ *   without re-rendering (and thus reflowing) the rest of the document.
+ *
+ * The input's EOL style (`\n` vs `\r\n`) is detected and preserved.
+ *
+ * @param text - The full YAML source document.
+ * @param path - Key/index path to the scalar to set.
+ * @param value - The scalar value to write (string, number, boolean, or null).
+ * @returns The updated document text.
+ * @throws If `text` is not valid YAML, if `path` is empty, or if `path` resolves
+ *   to a collection that would be clobbered by a scalar (or an intermediate
+ *   ancestor is a scalar that cannot hold a child).
+ *
+ * @example
+ * updateYamlIn('model: haiku   # note\n', ['model'], 'opus')
+ * // => 'model: opus   # note\n'   (alignment + comment preserved)
+ */
+export declare function updateYamlIn(text: string, path: YamlPath, value: YamlScalarValue): string;
+/**
+ * Re-parse `before` and `after` and assert the edit was correct and confined:
+ *
+ * 1. **Correctness** — every entry in `changedPaths` resolves in `after`.
+ * 2. **Confinement** — every other leaf path present in `before` has a deep-equal
+ *    value in `after` (no collateral value edits).
+ * 3. **Comment preservation** — the multiset of all comment strings is unchanged.
+ *
+ * Safe to call as an internal assertion from {@link updateYamlIn}.
+ *
+ * @param before - The original document text.
+ * @param after - The edited document text.
+ * @param changedPaths - The paths the caller intended to change.
+ * @throws A descriptive `Error` naming the offending path or lost comment on any
+ *   violation.
+ */
+export declare function verifyConfinedYamlEdit(before: string, after: string, changedPaths: YamlPath[]): void;
+//# sourceMappingURL=surgical-yaml.d.ts.map

package/dist/yaml/surgical-yaml.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"surgical-yaml.d.ts","sourceRoot":"","sources":["../../src/yaml/surgical-yaml.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AAcH,+EAA+E;AAC/E,MAAM,MAAM,QAAQ,GAAG,CAAC,MAAM,GAAG,MAAM,CAAC,EAAE,CAAC;AAE3C,gFAAgF;AAChF,MAAM,MAAM,eAAe,GAAG,MAAM,GAAG,MAAM,GAAG,OAAO,GAAG,IAAI,CAAC;AAK/D;;;;;;;;;;;;;;;;;;;;;;;GAuBG;AACH,wBAAgB,YAAY,CAAC,IAAI,EAAE,MAAM,EAAE,IAAI,EAAE,QAAQ,EAAE,KAAK,EAAE,eAAe,GAAG,MAAM,CA2BzF;AAED;;;;;;;;;;;;;;;GAeG;AACH,wBAAgB,sBAAsB,CACpC,MAAM,EAAE,MAAM,EACd,KAAK,EAAE,MAAM,EACb,YAAY,EAAE,QAAQ,EAAE,GACvB,IAAI,CAYN"}