@gobing-ai/spur 0.2.10 → 0.2.11

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@gobing-ai/spur",
-    "version": "0.2.10",
+    "version": "0.2.11",
     "description": "Spur CLI — local-first harness for mainstream coding agents: constraint checking, workflow orchestration, agent health, and history analytics. Bun-native; exposes the `spur` command.",
     "keywords": [
         "spur",

package/spur-cli/config/workflows/basic.yaml

@@ -1,8 +1,8 @@
-$schema: "@gobing-ai/spur/schemas/state-machine-workflow.schema.json"
 # Canonical basic workflow: implement -> check -> fix until the check passes or the
 # iteration bound is exhausted. Authored for the @gobing-ai/ts-dual-workflow-engine
 # state-machine schema (initialState / states[].id / onEnter / top-level transitions).
 # See Architecture Section 5 and Design Section 3.3.
+"$schema": "@gobing-ai/spur/schemas/state-machine-workflow.schema.json"
 name: basic
 kind: state-machine
 description: The canonical implement-check-fix-until-pass loop

package/spur-cli/config/workflows/task-pipeline.yaml

@@ -97,10 +97,9 @@ states:
   - id: approve
     description: >
       Human-in-the-loop approval gate. Under `profile=auto` this state is never entered
-      (review routes around it straight to verify). NOTE: to make this state PAUSE for `spur workflow continue`
-      (E3), add `pause: true` here — deferred until the globally-installed `@gobing-ai/spur`
-      schema (currently a stale 0.2.5 that lacks the `pause` field) is refreshed, so
-      full `spur workflow validate` stays green. The workspace schema already supports it.
+      (review routes around it straight to verify). In interactive mode this state pauses
+      the run for `spur workflow continue` (E3), making approval an explicit operator action.
+    pause: true
     onEnter:
       - kind: hitl.confirm
         options:

package/spur.js

@@ -32493,7 +32493,40 @@ var init_template_renderer = () => {};
 // ../../packages/config/src/loader.ts
 import { existsSync as existsSync3 } from "fs";
 import { homedir } from "os";
-import { join as join2 } from "path";
+import { dirname as dirname3, join as join2 } from "path";
+import { fileURLToPath as fileURLToPath3 } from "url";
+function embeddedSchemasId(embeddedSchemas) {
+  if (embeddedSchemas === undefined)
+    return "disk";
+  let id = embeddedSchemasIds.get(embeddedSchemas);
+  if (id === undefined) {
+    id = nextEmbeddedSchemasId;
+    nextEmbeddedSchemasId += 1;
+    embeddedSchemasIds.set(embeddedSchemas, id);
+  }
+  return `embedded:${id}`;
+}
+function cacheKey(configPath, opts, validateJsonSchema2) {
+  return [
+    configPath,
+    validateJsonSchema2 ? "schema" : "zod",
+    opts?.schemaManifestSpecifier ?? "@gobing-ai/spur/package.json",
+    embeddedSchemasId(opts?.embeddedSchemas)
+  ].join("\x00");
+}
+function resolveSchemaSpecifier(specifier, manifestSpecifier) {
+  if (specifier === manifestSpecifier) {
+    const workspaceManifest = join2(LOADER_DIR, "..", "..", "..", "apps", "cli", "package.json");
+    if (existsSync3(workspaceManifest))
+      return workspaceManifest;
+  }
+  try {
+    const resolved = import.meta.resolve(specifier);
+    return resolved.startsWith("file:") ? fileURLToPath3(resolved) : resolved;
+  } catch {
+    return specifier;
+  }
+}
 function resolveConfigFile(cwd) {
   const projectConfig = join2(cwd ?? process.cwd(), SPUR_CONFIG_DIR, SPUR_CONFIG_FILE);
   if (existsSync3(projectConfig))
@@ -32518,11 +32551,21 @@ function makeEmbeddedReader(embeddedSchemas) {
   };
 }
 async function loadSpurConfig(cwd = process.cwd(), opts) {
-  const configPath = join2(cwd, SPUR_CONFIG_DIR, SPUR_CONFIG_FILE);
-  if (!existsSync3(configPath)) {
+  const configPath = resolveConfigFile(cwd);
+  if (configPath === undefined) {
     return spurConfigSchema.parse({});
   }
   const validateJsonSchema2 = opts?.validateJsonSchema ?? true;
+  const key = cacheKey(configPath, opts, validateJsonSchema2);
+  const cached2 = spurConfigCache.get(key);
+  if (cached2 !== undefined)
+    return cached2;
+  const promise2 = loadSpurConfigFile(configPath, opts, validateJsonSchema2);
+  spurConfigCache.set(key, promise2);
+  promise2.catch(() => spurConfigCache.delete(key));
+  return promise2;
+}
+async function loadSpurConfigFile(configPath, opts, validateJsonSchema2) {
   let raw;
   if (validateJsonSchema2) {
     const embeddedSchemas = opts?.embeddedSchemas;
@@ -32531,7 +32574,7 @@ async function loadSpurConfig(cwd = process.cwd(), opts) {
       if (embeddedSchemas !== undefined && specifier === manifestSpecifier) {
         return `${EMBEDDED_PREFIX}/package.json`;
       }
-      return specifier;
+      return resolveSchemaSpecifier(specifier, manifestSpecifier);
     };
     const fileSystem = embeddedSchemas ? { readFile: makeEmbeddedReader(embeddedSchemas) } : undefined;
     raw = await loadStructuredConfig(configPath, {
@@ -32561,7 +32604,7 @@ async function loadStructuredSpurConfig(configPath, opts) {
     if (embeddedSchemas !== undefined && specifier === manifestSpecifier) {
       return `${EMBEDDED_PREFIX}/package.json`;
     }
-    return specifier;
+    return resolveSchemaSpecifier(specifier, manifestSpecifier);
   };
   const fileSystem = embeddedSchemas ? { readFile: makeEmbeddedReader(embeddedSchemas) } : undefined;
   return await loadStructuredConfig(configPath, {
@@ -32584,6 +32627,14 @@ function defaultPlanningFolders() {
   };
 }
 async function resolvePlanningFolders(fs) {
+  const cached2 = planningFoldersCache.get(fs);
+  if (cached2 !== undefined)
+    return cached2;
+  const promise2 = resolvePlanningFoldersUncached(fs);
+  planningFoldersCache.set(fs, promise2);
+  return promise2;
+}
+async function resolvePlanningFoldersUncached(fs) {
   const configPath = fs.resolve(join2(SPUR_CONFIG_DIR, SPUR_CONFIG_FILE));
   if (!await fs.exists(configPath))
     return defaultPlanningFolders();
@@ -32614,7 +32665,7 @@ async function resolvePlanningFolders(fs) {
     return defaultPlanningFolders();
   }
 }
-var SPUR_CONFIG_DIR = ".spur", SPUR_CONFIG_FILE = "config.yaml", GLOBAL_CONFIG_FILE, EMBEDDED_PREFIX = "\x00embedded-spur";
+var SPUR_CONFIG_DIR = ".spur", SPUR_CONFIG_FILE = "config.yaml", GLOBAL_CONFIG_FILE, LOADER_DIR, nextEmbeddedSchemasId = 1, embeddedSchemasIds, spurConfigCache, planningFoldersCache, EMBEDDED_PREFIX = "\x00embedded-spur";
 var init_loader = __esm(() => {
   init_dist3();
   init_dist2();
@@ -32622,6 +32673,10 @@ var init_loader = __esm(() => {
   init_bundled_config();
   init_template_renderer();
   GLOBAL_CONFIG_FILE = join2(homedir(), ".config", "spur", "config.yaml");
+  LOADER_DIR = dirname3(fileURLToPath3(import.meta.url));
+  embeddedSchemasIds = new WeakMap;
+  spurConfigCache = new Map;
+  planningFoldersCache = new WeakMap;
 });
 
 // ../../node_modules/.bun/@gobing-ai+ts-runtime@0.4.2/node_modules/@gobing-ai/ts-runtime/dist/bun-sqlite.js
@@ -55614,7 +55669,7 @@ var init_db2 = __esm(() => {
 
 // ../../packages/domain/src/planning/locks.ts
 import { closeSync, fsyncSync, openSync, renameSync as renameSync2 } from "fs";
-import { dirname as dirname5, join as join5 } from "path";
+import { dirname as dirname6, join as join5 } from "path";
 function lockPathForEntity(folder, entityId) {
   return join5(folder, `.${entityId}.lock`);
 }
@@ -55701,7 +55756,7 @@ function fsyncPath(path8, swallowErrors = false) {
   }
 }
 function atomicWrite(dest, content, entityId, fs3, projectName = "spur") {
-  const dir = dirname5(dest);
+  const dir = dirname6(dest);
   const rand = Math.random().toString(36).slice(2, 10);
   const tempPath = join5(dir, `.${projectName}.${entityId}.${rand}.tmp`);
   fs3.writeFile(tempPath, content);
@@ -55710,7 +55765,7 @@ function atomicWrite(dest, content, entityId, fs3, projectName = "spur") {
   fsyncPath(dir, true);
 }
 async function atomicWriteAsync(dest, content, entityId, fs3, projectName = "spur") {
-  const dir = dirname5(dest);
+  const dir = dirname6(dest);
   const rand = Math.random().toString(36).slice(2, 10);
   const tempPath = join5(dir, `.${projectName}.${entityId}.${rand}.tmp`);
   await fs3.writeFile(tempPath, content);
@@ -55774,6 +55829,19 @@ function findHeadings(body, hashes) {
   }
   return hits;
 }
+function normalizeYamlScalars(data) {
+  const out = {};
+  for (const [key, value] of Object.entries(data)) {
+    if (value instanceof Date) {
+      out[key] = value.toISOString();
+    } else if (value !== null && typeof value === "object" && !Array.isArray(value)) {
+      out[key] = normalizeYamlScalars(value);
+    } else {
+      out[key] = value;
+    }
+  }
+  return out;
+}
 
 class MarkdownDocument {
   _domain;
@@ -55804,7 +55872,7 @@ class MarkdownDocument {
       try {
         const parsed = $parse(raw);
         if (parsed !== null && typeof parsed === "object" && !Array.isArray(parsed)) {
-          data = parsed;
+          data = normalizeYamlScalars(parsed);
         }
       } catch {}
       frontmatter = { raw, data };
@@ -55971,8 +56039,19 @@ ${body.trimEnd()}
 ${content}
 ${END_MARKER}${after}`;
   }
+  yamlSafeValue(value) {
+    if (value.startsWith('"') && value.endsWith('"'))
+      return value;
+    if (/[:{}[\],&*?|>!%@`#"'\\\n\r]/.test(value))
+      return `"${value}"`;
+    if (/^(null|true|false|yes|no|on|off)$/i.test(value))
+      return `"${value}"`;
+    if (/^\d/.test(value) && !/^\d{4}-\d{2}-\d{2}/.test(value))
+      return `"${value}"`;
+    return value;
+  }
   setFrontmatterField(key, value) {
-    const fieldLine = `${key}: ${value}`;
+    const fieldLine = `${key}: ${this.yamlSafeValue(value)}`;
     if (this._frontmatter === null) {
       const raw2 = fieldLine;
       this._frontmatter = { raw: raw2, data: { [key]: value } };
@@ -58476,7 +58555,7 @@ var init_rule_service = __esm(() => {
 });
 
 // ../../packages/app/src/services/task-check.ts
-import { dirname as dirname6, join as join8 } from "path";
+import { dirname as dirname7, join as join8 } from "path";
 function isPlaceholderBody(body) {
   const stripped = body.replace(/<!--[\s\S]*?-->/g, "").replace(/^\s*>\s*TBD\s*$/gim, "").trim();
   return stripped.length === 0;
@@ -58526,6 +58605,29 @@ function isProseOnlyReview(body) {
   return !stripped.split(`
 `).some((l) => l.includes("|"));
 }
+function hasAdjacentFileLineColumns(body) {
+  const lines = body.split(`
+`);
+  for (const line of lines) {
+    if (!line.includes("|"))
+      continue;
+    const cells = line.split("|").map((c3) => c3.trim());
+    if (cells.length < 3)
+      continue;
+    for (let i2 = 0;i2 < cells.length - 1; i2++) {
+      const a2 = cells[i2];
+      const b = cells[i2 + 1];
+      if (a2 === undefined || b === undefined)
+        continue;
+      const fileText = a2.replace(/`/g, "");
+      const fileCell = /\.(tsx?|jsx?|mjs|cjs|md|ya?ml|json|css|html|sql|sh|toml)$/i.test(fileText);
+      const lineCell = /^\d+(-\d+)?$/.test(b);
+      if (fileCell && lineCell)
+        return true;
+    }
+  }
+  return false;
+}
 var TaskCheckService;
 var init_task_check = __esm(() => {
   init_src2();
@@ -58554,8 +58656,8 @@ var init_task_check = __esm(() => {
       const entry = this.resolveMatrixEntry(variant, status);
       this.runL2(doc2, entry, findings);
       this.runL3(doc2, entry, findings);
-      const tasksDir = dirname6(filePath);
-      const featuresDir = join8(dirname6(tasksDir), "features");
+      const tasksDir = dirname7(filePath);
+      const featuresDir = join8(dirname7(tasksDir), "features");
       await this.runL4(doc2, fm, findings, featuresDir, tasksDir);
       await this.runL4Rollup(doc2, wbs, status, findings, tasksDir);
       return { wbs, ...this.summarizeWithStatus(status, findings, strict) };
@@ -58586,7 +58688,8 @@ var init_task_check = __esm(() => {
       const solBody = doc2.getSection("Solution");
       if (solBody !== null && !isPlaceholderBody(solBody)) {
         const hasFileLine = /`[^`]+?:\d+(-\d+)?`/.test(solBody) || /[^\s`]\.\w+:\d+/.test(solBody);
-        if (!hasFileLine) {
+        const hasTableFileLine = hasFileLine || hasAdjacentFileLineColumns(solBody);
+        if (!hasTableFileLine) {
           findings.push({
             layer: "L3",
             severity: "error",
@@ -58997,7 +59100,7 @@ function gitDiffU0(cwd) {
 var init_task_record = () => {};
 
 // ../../packages/app/src/services/task-service.ts
-import { dirname as dirname7, isAbsolute, relative as relative2, resolve as resolve2 } from "path";
+import { dirname as dirname8, isAbsolute, relative as relative2, resolve as resolve2 } from "path";
 function patchFrontmatterField(rendered, key, value) {
   const existingRe = new RegExp(`^(?=\\s*${escapeRegex2(key)}:)`, "m");
   if (existingRe.test(rendered)) {
@@ -59590,7 +59693,7 @@ ${block}` : block);
   resolveListDir(folder) {
     if (folder === undefined)
       return this.ctx.tasksDir;
-    const root = resolve2(dirname7(this.ctx.tasksDir));
+    const root = resolve2(dirname8(this.ctx.tasksDir));
     const candidate = resolve2(root, folder);
     const rel = relative2(root, candidate);
     if (rel.startsWith("..") || isAbsolute(rel)) {
@@ -59864,7 +59967,7 @@ var init_team_service = __esm(() => {
 
 // ../../packages/app/src/workflow/actions/agent-run.ts
 import { mkdir, writeFile } from "fs/promises";
-import { dirname as dirname8, isAbsolute as isAbsolute2, join as join10 } from "path";
+import { dirname as dirname9, isAbsolute as isAbsolute2, join as join10 } from "path";
 
 class AgentRunActionRunner {
   kind = KIND;
@@ -59916,7 +60019,7 @@ class AgentRunActionRunner {
       const ok2 = exitCode2 === 0;
       if (answerFile !== undefined) {
         const target = isAbsolute2(answerFile) ? answerFile : join10(cwd, answerFile);
-        await mkdir(dirname8(target), { recursive: true });
+        await mkdir(dirname9(target), { recursive: true });
         await writeFile(target, answer, "utf8");
       }
       return {
@@ -61232,7 +61335,7 @@ init_loader();
 init_dist4();
 init_dist3();
 import { appendFileSync as appendFileSync3, mkdirSync as mkdirSync2, readFileSync as readFileSync4 } from "fs";
-import { dirname as dirname3 } from "path";
+import { dirname as dirname4 } from "path";
 
 // ../../node_modules/.bun/@gobing-ai+ts-infra@0.4.2+be3e9d72efacb189/node_modules/@gobing-ai/ts-infra/dist/application/index.js
 init_default_observers();
@@ -64634,7 +64737,7 @@ function validateAppConfig(validator, raw, section, filePath) {
 function createFileSink(filePath) {
   return (line) => {
     try {
-      mkdirSync2(dirname3(filePath), { recursive: true });
+      mkdirSync2(dirname4(filePath), { recursive: true });
       appendFileSync3(filePath, line);
     } catch {}
   };
@@ -66052,8 +66155,8 @@ var figlet = (() => {
 })();
 
 // ../../node_modules/.bun/figlet@1.11.0/node_modules/figlet/dist/node-figlet.mjs
-import { fileURLToPath as fileURLToPath3 } from "url";
-var __filename2 = fileURLToPath3(import.meta.url);
+import { fileURLToPath as fileURLToPath4 } from "url";
+var __filename2 = fileURLToPath4(import.meta.url);
 var __dirname2 = path7.dirname(__filename2);
 var fontPath = path7.join(__dirname2, "/../fonts/");
 var nodeFiglet = figlet;
@@ -68864,7 +68967,7 @@ import { join as join13, resolve as resolve4 } from "path";
 var CLI_CONFIG = {
   binaryName: "spur",
   binaryLabel: "spur",
-  binaryVersion: "0.2.10",
+  binaryVersion: "0.2.11",
   configDir: ".spur",
   configFile: ".spur/config.yaml",
   databaseFile: ".spur/spur.db"
@@ -69394,7 +69497,7 @@ init_src();
 
 // ../server/src/serve.ts
 init_src3();
-import { dirname as dirname9, isAbsolute as isAbsolute3, join as join16 } from "path";
+import { dirname as dirname10, isAbsolute as isAbsolute3, join as join16 } from "path";
 init_dist3();
 
 // ../server/src/bootstrap.ts
@@ -72635,15 +72738,15 @@ var getPattern = (label, next) => {
   }
   const match = label.match(/^\:([^\{\}]+)(?:\{(.+)\})?$/);
   if (match) {
-    const cacheKey = `${label}#${next}`;
-    if (!patternCache[cacheKey]) {
+    const cacheKey2 = `${label}#${next}`;
+    if (!patternCache[cacheKey2]) {
       if (match[2]) {
-        patternCache[cacheKey] = next && next[0] !== ":" && next[0] !== "*" ? [cacheKey, match[1], new RegExp(`^${match[2]}(?=/${next})`)] : [label, match[1], new RegExp(`^${match[2]}$`)];
+        patternCache[cacheKey2] = next && next[0] !== ":" && next[0] !== "*" ? [cacheKey2, match[1], new RegExp(`^${match[2]}(?=/${next})`)] : [label, match[1], new RegExp(`^${match[2]}$`)];
       } else {
-        patternCache[cacheKey] = [label, match[1], true];
+        patternCache[cacheKey2] = [label, match[1], true];
       }
     }
-    return patternCache[cacheKey];
+    return patternCache[cacheKey2];
   }
   return null;
 };
@@ -77367,7 +77470,7 @@ var defaultDeps = {
 async function resolveWebDistPath(configuredPath) {
   const candidates = configuredPath && configuredPath.trim() !== "" ? [isAbsolute3(configuredPath) ? configuredPath : join16(process.cwd(), configuredPath)] : [
     join16(process.cwd(), "dist/web"),
-    join16(dirname9(process.execPath), "../web"),
+    join16(dirname10(process.execPath), "../web"),
     join16(import.meta.dir, "../../../dist/web")
   ];
   for (const candidate of candidates) {
@@ -78135,6 +78238,15 @@ ${result.content}`);
         } else {
           context3.output.write(`${result.ref.id}: ${result.fromStatus} \u2192 ${result.toStatus}`);
         }
+        if (status === "done" && !options.json) {
+          const task2 = await svc.show(wbs);
+          const featureId2 = task2.frontmatter.feature_id;
+          if (!featureId2 || featureId2.length === 0) {
+            context3.output.write(`\u24D8  Task ${wbs} is now done but has no feature_id.
+` + `   Link it with: spur task update ${wbs} --feature <id>
+` + "   (Advisory only \u2014 the task is already done.)");
+          }
+        }
       } else {
         context3.output.error("Either <status>, --section/--from-file, or --feature/--priority is required");
         context3.setExitCode(2);
@@ -78436,7 +78548,17 @@ async function runDoneGateCheck(context3, wbs, folderOverride) {
   const result = await svc.check(`${tasksDir}/${fileName}`, wbs, { strict: false });
   return result.pass;
 }
+var sectionMatrixCache = new Map;
 async function loadSectionMatrix(projectRoot) {
+  const cached2 = sectionMatrixCache.get(projectRoot);
+  if (cached2 !== undefined)
+    return cached2;
+  const promise2 = loadSectionMatrixUncached(projectRoot);
+  sectionMatrixCache.set(projectRoot, promise2);
+  promise2.catch(() => sectionMatrixCache.delete(projectRoot));
+  return promise2;
+}
+async function loadSectionMatrixUncached(projectRoot) {
   const localPath = join18(projectRoot, ".spur", "tasks", "section-matrix.yaml");
   if (existsSync5(localPath)) {
     const data = await loadStructuredSpurConfig(localPath, {
@@ -78826,7 +78948,7 @@ init_src3();
 init_src();
 init_src2();
 init_dist3();
-import { dirname as dirname10, join as join19, resolve as resolve6 } from "path";
+import { dirname as dirname11, join as join19, resolve as resolve6 } from "path";
 import { isatty as isatty3 } from "tty";
 
 // ../../node_modules/.bun/@clack+core@1.4.1/node_modules/@clack/core/dist/index.mjs
@@ -80238,7 +80360,7 @@ async function createMigratedDbAdapter(cwd = process.cwd(), env = process.env, d
   const configuredUrl = env.DATABASE_URL === undefined ? join19(cwd, CLI_CONFIG.databaseFile) : config3.database.url;
   const url2 = dbUrl ?? configuredUrl;
   if (url2 !== ":memory:") {
-    await createNodeFileSystem().ensureDir(dirname10(url2));
+    await createNodeFileSystem().ensureDir(dirname11(url2));
   }
   return createMigratedDb({ url: url2 });
 }