@gobing-ai/spur 0.3.85 → 0.3.86

package/plugins/sp/README.md

@@ -620,14 +620,11 @@ pipeline owns one lifecycle phase:
 
 | Workflow                    | Phase                             | Entry command                     |
 | --------------------------- | --------------------------------- | --------------------------------- |
-| `basic.yaml`                | Generic implement/check/fix       | direct `spur workflow run`        |
 | `feature-lifecycle.yaml`    | Feature status FSM                | `spur feature update`             |
 | `task-lifecycle.yaml`       | Task status FSM                   | `spur task update`                |
 | `task-pipeline.yaml`        | Single-task execution             | `/sp:dev-run`                     |
 | `idea-pipeline.yaml`        | Idea/planning → feature + tasks   | `/sp:dev-idea`, `/sp:dev-plan`    |
-| `feature-dev.yaml`          | Feature umbrella execution        | `/sp:dev-runall --feature`        |
 | `wrapup-pipeline.yaml`      | Post-execution wrap-up            | `/sp:dev-wrap`, `/sp:dev-wrapall` |
-| `docs-pipeline.yaml`        | Docs-only task execution          | `/sp:dev-run --mode implement`    |
 | `wayfinder-resolution.yaml` | Wayfinder ticket resolution loop  | `spur workflow run` (free-form)   |
 
 ### Lifecycle operations

package/plugins/sp/hooks/agent-hint.ts

@@ -1,3 +1,4 @@
+import { getEnvVars } from '@gobing-ai/ts-utils';
 /**
  * Shared host-agent identification helpers — the **only** host-agent detection path
  * in the sp plugin hooks.
@@ -22,11 +23,11 @@
  * 3. `TERM_PROGRAM` — terminal identifier (e.g. `claude`, `ghostty`)
  * 4. `SPUR_DEFAULT_AGENT` — configured default agent
  *
- * @param env - environment record (defaults to `process.env`)
+ * @param env - environment record (defaults to `getEnvVars()`)
  * @param fallback - value returned when no candidate matches (defaults to `undefined`)
  */
 export function resolveAgentHint(
-    env: NodeJS.ProcessEnv = process.env,
+    env: NodeJS.ProcessEnv = getEnvVars(),
     fallback: string | undefined = undefined,
 ): string | undefined {
     const candidates = [env.SPUR_AGENT, env.CLAUDE_CODE_ENTRYPOINT, env.TERM_PROGRAM, env.SPUR_DEFAULT_AGENT];
@@ -45,11 +46,11 @@ export function resolveAgentHint(
  * 3. `OPENAI_MODEL` — OpenAI API model
  * 4. `CLAUDE_MODEL` — Claude Code model setting
  *
- * @param env - environment record (defaults to `process.env`)
+ * @param env - environment record (defaults to `getEnvVars()`)
  * @param fallback - value returned when no candidate matches (defaults to `undefined`)
  */
 export function resolveModelHint(
-    env: NodeJS.ProcessEnv = process.env,
+    env: NodeJS.ProcessEnv = getEnvVars(),
     fallback: string | undefined = undefined,
 ): string | undefined {
     const candidates = [env.SPUR_MODEL, env.ANTHROPIC_MODEL, env.OPENAI_MODEL, env.CLAUDE_MODEL];

package/plugins/sp/hooks/careful-guard.ts

@@ -1,4 +1,5 @@
 #!/usr/bin/env bun
+
 /**
  * careful-guard — PreToolUse guard for destructive shell commands (task 0215, R3).
  *
@@ -21,6 +22,7 @@
  * adapter. This file owns the Claude Code payload/decision shape only.
  */
 
+import { getEnvVar } from '@gobing-ai/ts-utils';
 import { classifyCommand } from './destructive-policy';
 
 interface ToolPayload {
@@ -43,7 +45,7 @@ function preToolUseDecision(decision: Decision, reason?: string): never {
 }
 
 async function main(): Promise<void> {
-    if (process.env.SPUR_CAREFUL === 'off') preToolUseDecision('allow');
+    if (getEnvVar('SPUR_CAREFUL') === 'off') preToolUseDecision('allow');
 
     const stdinText = await Bun.stdin.text();
     let payload: ToolPayload;

package/plugins/sp/hooks/context-post-tool.ts

@@ -23,6 +23,7 @@
 import { execSync } from 'node:child_process';
 import { appendFileSync, existsSync, readFileSync, statSync, writeFileSync } from 'node:fs';
 import { join, resolve, sep } from 'node:path';
+import { getEnvVar } from '@gobing-ai/ts-utils';
 
 /** Tools recorded by this hook (must match hooks.json PostToolUse matcher). */
 export const ALLOWED_TOOLS = new Set(['Bash', 'Grep', 'Glob', 'Read', 'Write', 'Edit']);
@@ -411,7 +412,7 @@ export function readContextFreshness(contextDir: string): string | null {
 // Entrypoint — thin wrapper; logic lives in {@link recordToolUseEvent} for unit coverage.
 if (import.meta.main) {
     void (async () => {
-        const dir = join(process.env.CLAUDE_PROJECT_DIR ?? process.cwd(), '.spur', 'context');
+        const dir = join(getEnvVar('CLAUDE_PROJECT_DIR') ?? process.cwd(), '.spur', 'context');
         try {
             const stdinText = await Bun.stdin.text();
             const payload = JSON.parse(stdinText) as ToolPayload;

package/plugins/sp/hooks/context-session-start.ts

@@ -15,6 +15,7 @@
 import { execSync } from 'node:child_process';
 import { appendFileSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
 import { join } from 'node:path';
+import { getEnvVar, getEnvVars } from '@gobing-ai/ts-utils';
 import { resolveAgentHint, resolveModelHint } from './agent-hint';
 import { checkContextFreshness } from './context-post-tool';
 
@@ -82,7 +83,7 @@ interface SessionFileBody {
  * a stale file is only reachable from inside a live agent run, where reusing it is the correct
  * answer anyway.
  */
-export function resolveActiveSession(dir: string, now: Date, env: NodeJS.ProcessEnv = process.env): string | null {
+export function resolveActiveSession(dir: string, now: Date, env: NodeJS.ProcessEnv = getEnvVars()): string | null {
     let raw: string;
     try {
         raw = readFileSync(join(dir, '.session.json'), 'utf-8');
@@ -123,7 +124,7 @@ export function resolveActiveSession(dir: string, now: Date, env: NodeJS.Process
  */
 export function recordSessionStart(
     dir: string,
-    env: NodeJS.ProcessEnv = process.env,
+    env: NodeJS.ProcessEnv = getEnvVars(),
     now: () => Date = () => new Date(),
 ): string | null {
     try {
@@ -195,7 +196,7 @@ export function recordSessionStart(
 // Entrypoint — kept minimal so unit coverage focuses on pure helpers above.
 if (import.meta.main) {
     try {
-        recordSessionStart(join(process.env.CLAUDE_PROJECT_DIR ?? process.cwd(), '.spur', 'context'));
+        recordSessionStart(join(getEnvVar('CLAUDE_PROJECT_DIR') ?? process.cwd(), '.spur', 'context'));
     } catch {
         /* fail-open */
     }

package/plugins/sp/hooks/context-session-stop.ts

@@ -17,6 +17,7 @@
 
 import { appendFileSync, existsSync, readFileSync, rmSync } from 'node:fs';
 import { join } from 'node:path';
+import { getEnvVar } from '@gobing-ai/ts-utils';
 
 interface LedgerEvent {
     session: string;
@@ -53,7 +54,7 @@ function exitOk(): never {
 }
 
 async function main(): Promise<void> {
-    const dir = join(process.env.CLAUDE_PROJECT_DIR ?? process.cwd(), '.spur', 'context');
+    const dir = join(getEnvVar('CLAUDE_PROJECT_DIR') ?? process.cwd(), '.spur', 'context');
 
     const sessionFile = join(dir, '.session.json');
     if (!existsSync(sessionFile)) exitOk();

package/plugins/sp/hooks/pi/guard-extension.ts

@@ -23,6 +23,7 @@ import { appendFileSync, existsSync, mkdirSync, readFileSync, realpathSync, rmSy
 import { homedir } from 'node:os';
 import { isAbsolute, join, resolve } from 'node:path';
 import type { ExtensionAPI } from '@earendil-works/pi-coding-agent';
+import { getEnvVar, getEnvVars } from '@gobing-ai/ts-utils';
 import { resolveAgentHint as resolveAgentHintShared, resolveModelHint as resolveModelHintShared } from '../agent-hint';
 import { classifyCommand } from '../destructive-policy';
 import { couldBeTaskFile } from '../task-file-policy';
@@ -81,7 +82,7 @@ function resolveSpurTaskOwnership(filePath: string): TaskOwnership {
         spawnSync(cmd, args, { cwd: process.cwd(), encoding: 'utf-8', timeout: 8000 });
 
     // 1. SPUR_BIN env override (may include args) or `spur` on PATH
-    const envBin = process.env.SPUR_BIN || 'spur';
+    const envBin = getEnvVar('SPUR_BIN') || 'spur';
     const envParts = envBin.split(' ');
     let res = run(envParts[0] ?? 'spur', [...envParts.slice(1), 'task', 'resolve', filePath, '--strict', '--json']);
     // Only 0 (owned) / 1 (unowned) are valid spur exit codes; 127 (interpreter
@@ -203,8 +204,8 @@ function initSession(): void {
         const sessionId = generateSessionId();
         const session = {
             session_id: sessionId,
-            agent: resolveAgentHintShared(process.env, 'pi'),
-            model: resolveModelHintShared(process.env),
+            agent: resolveAgentHintShared(getEnvVars(), 'pi'),
+            model: resolveModelHintShared(getEnvVars()),
             started_at: new Date().toISOString(),
         };
         writeFileSync(sessionFilePath(), `${JSON.stringify(session, null, 2)}\n`);

package/plugins/sp/hooks/task-write-guard.ts

@@ -20,6 +20,7 @@
  */
 
 import { spawnSync } from 'node:child_process';
+import { getEnvVar } from '@gobing-ai/ts-utils';
 import { couldBeTaskFile } from './task-file-policy';
 
 interface ToolPayload {
@@ -40,7 +41,7 @@ function preToolUseDecision(decision: 'allow' | 'deny', reason?: string): never
 
 /** Resolve whether a path is owned by a Spur task via `spur task resolve --strict --json`. */
 function resolveSpurTaskOwnership(filePath: string, cwd: string): TaskOwnership {
-    const spurBin = process.env.SPUR_BIN || 'spur';
+    const spurBin = getEnvVar('SPUR_BIN') || 'spur';
     const parts = spurBin.split(' ');
     const cmd = parts[0] ?? 'spur';
     const args = [...parts.slice(1), 'task', 'resolve', filePath, '--strict', '--json'];
@@ -54,7 +55,7 @@ function resolveSpurTaskOwnership(filePath: string, cwd: string): TaskOwnership
 }
 
 async function main(): Promise<void> {
-    if (process.env.SPUR_WRITE_GUARD === 'off') preToolUseDecision('allow');
+    if (getEnvVar('SPUR_WRITE_GUARD') === 'off') preToolUseDecision('allow');
 
     const stdinText = await Bun.stdin.text();
     let payload: ToolPayload;
@@ -73,7 +74,7 @@ async function main(): Promise<void> {
     // cannot name a task file — that is every ordinary source edit.
     if (!couldBeTaskFile(filePath)) preToolUseDecision('allow');
 
-    const ownership = resolveSpurTaskOwnership(filePath, process.env.CLAUDE_PROJECT_DIR ?? process.cwd());
+    const ownership = resolveSpurTaskOwnership(filePath, getEnvVar('CLAUDE_PROJECT_DIR') ?? process.cwd());
     if (ownership === 'owned') {
         preToolUseDecision(
             'deny',