@gobing-ai/spur 0.3.80 → 0.3.82

package/plugins/sp/scripts/wrapup-steps.ts

@@ -0,0 +1,466 @@
+#!/usr/bin/env bun
+/**
+ * wrapup-steps — deterministic wrap-up capture, metrics and feature sync behind the
+ * wrapup-pipeline wrappers (task 0824, feature I21, governance §1.2 composition budgets).
+ *
+ * Reproduces the former wrapup-pipeline `task-resolve:onEnter:0`, `metrics-record:onEnter:0`
+ * and `feature-transition:onEnter:0` shell programs one-for-one so the workflow stays inside
+ * the shell-program caps while writing the same `.spur/run` artifacts:
+ *   - `<runId>-wrapup-tasks.json`       normalized, deduplicated WBS capture (resolve)
+ *   - `<runId>-wrapup-resolve.status`   `PASS`/`FAIL` (resolve)
+ *   - `<runId>-route-reason.txt`        route reason (written by the workflow route writer)
+ *   - `.spur/memory/wrapup-metrics.jsonl` one row per task (metrics)
+ *   - `<runId>-wrapup-metrics.status`   `PASS`/`FAIL` (metrics)
+ *   - `<runId>-wrapup-sync.status`      `PASS`/`FAIL` (feature-transition)
+ *
+ * Environment comes from the workflow vars: `__runId`, `tasks`, `feature`, `featureGateCmd`
+ * and `spurBin` (split on whitespace into a command plus prefix args, so
+ * `bun apps/cli/src/index.ts` works).
+ *
+ * Truthfulness contract (0770 + 0783): wrap-up never mutates task status; a lookup failure is
+ * recorded as FAIL, never silently omitted as success; metrics rows are serialized as JSON
+ * (never interpolated printf); and the required sync succeeds only for a valid matching
+ * unblocked proposal whose target status is freshly observed — a gate PASS can never convert
+ * a failed sync into success. The process always exits 0 after resolve/metrics/
+ * feature-transition; the verdict lives in the status file (an empty `feature` is a
+ * mis-invocation and exits 1).
+ *
+ * Node-builtin imports only; pure helpers are exported for unit testing (ADR-065).
+ */
+
+import { spawnSync } from 'node:child_process';
+import { appendFileSync, existsSync, mkdirSync, readFileSync, writeFileSync } from 'node:fs';
+import { join } from 'node:path';
+
+export interface WrapupStepsEnv {
+    __runId?: string;
+    tasks?: string;
+    feature?: string;
+    featureGateCmd?: string;
+    spurBin?: string;
+    [key: string]: string | undefined;
+}
+
+export interface WrapupStepsOptions {
+    /** Base directory for `.spur/run` / `.spur/memory`; defaults to the process cwd. */
+    cwd?: string;
+}
+
+/** jq `//` chain: first value that is neither null, undefined nor false; otherwise the fallback. */
+export function jqPick(...values: unknown[]): unknown {
+    for (const value of values) {
+        if (value !== null && value !== undefined && value !== false) return value;
+    }
+    return values[values.length - 1];
+}
+
+/** `jq -r` text rendering of a JSON value (strings raw, everything else compact JSON). */
+function jqText(value: unknown): string {
+    return typeof value === 'string' ? value : JSON.stringify(value);
+}
+
+/** Canonical four-digit WBS string: whitespace is rejected, not trimmed (0783 R1). */
+export const WBS_PATTERN = /^[0-9]{4}$/;
+
+/** `spurBin` splits on whitespace into a command plus prefix args (so `bun x.ts` works). */
+export function spurCommand(spurBin: string | undefined): { cmd: string; prefix: string[] } {
+    const parts = (spurBin ?? 'spur')
+        .trim()
+        .split(/\s+/)
+        .filter((p) => p.length > 0);
+    return { cmd: parts[0] ?? 'spur', prefix: parts.slice(1) };
+}
+
+function spur(
+    env: WrapupStepsEnv,
+    args: string[],
+    options: { cwd?: string; stderr?: 'inherit' } = {},
+): { status: number; stdout: string } {
+    const { cmd, prefix } = spurCommand(env.spurBin);
+    const result = spawnSync(cmd, [...prefix, ...args], {
+        cwd: options.cwd,
+        encoding: 'utf8',
+        ...(options.stderr === 'inherit' ? { stdio: ['ignore', 'pipe', 'inherit'] as const } : {}),
+    });
+    if (result.error !== undefined) return { status: result.status ?? 1, stdout: '' };
+    return { status: result.status ?? 1, stdout: result.stdout ?? '' };
+}
+
+/** jq status-chain semantics for `task show` output; null/missing status means the lookup failed. */
+export function taskStatusOf(taskJson: string): { resolved: unknown; present: boolean } {
+    let parsed: unknown;
+    try {
+        parsed = JSON.parse(taskJson);
+    } catch {
+        return { resolved: null, present: false };
+    }
+    if (parsed === null || typeof parsed !== 'object') return { resolved: null, present: false };
+    const frontmatter = (parsed as Record<string, unknown>).frontmatter;
+    const fmStatus =
+        frontmatter !== null && typeof frontmatter === 'object'
+            ? (frontmatter as Record<string, unknown>).status
+            : undefined;
+    const status = (parsed as Record<string, unknown>).status;
+    // jq: `.frontmatter.status // .status` is null only when frontmatter.status is null/absent
+    // AND .status is null; a trailing false is kept (false != null in jq).
+    if (fmStatus === null || fmStatus === undefined) {
+        return { resolved: status ?? null, present: status !== undefined && status !== null };
+    }
+    return { resolved: fmStatus, present: true };
+}
+
+/** One truthy status display: `unresolved` when the lookup left nothing to show. */
+function statusDisplay(taskJson: string): string {
+    const { resolved } = taskStatusOf(taskJson);
+    if (resolved === null || resolved === undefined || resolved === false) return 'unresolved';
+    return jqText(resolved);
+}
+
+export interface ResolveResult {
+    status: 'PASS' | 'FAIL';
+    statusFile: string;
+    tasksFile: string;
+    /** Exit code the workflow wrapper observes (only an empty __runId is a hard failure). */
+    exitCode: number;
+}
+
+/** `resolve` — parse and validate vars.tasks exactly once, then resolve every member. */
+export function resolveTasks(env: WrapupStepsEnv, options: WrapupStepsOptions = {}): ResolveResult {
+    const cwd = options.cwd;
+    const runId = env.__runId ?? '';
+    if (runId.length === 0) {
+        process.stderr.write('task-resolve: __runId is empty — refusing the legacy fixed-path fallback\n');
+        return { status: 'FAIL', statusFile: '', tasksFile: '', exitCode: 1 };
+    }
+    mkdirSync(cwd ? join(cwd, '.spur', 'run') : join('.spur', 'run'), { recursive: true });
+    const relTasksFile = join('.spur', 'run', `${runId}-wrapup-tasks.json`);
+    const relReasonFile = join('.spur', 'run', `${runId}-route-reason.txt`);
+    const relStatusFile = join('.spur', 'run', `${runId}-wrapup-resolve.status`);
+    const abs = (p: string): string => (cwd ? join(cwd, p) : p);
+
+    const writeFail = (reason: string): ResolveResult => {
+        writeFileSync(abs(relReasonFile), reason);
+        writeFileSync(abs(relStatusFile), 'FAIL\n');
+        return { status: 'FAIL', statusFile: relStatusFile, tasksFile: relTasksFile, exitCode: 0 };
+    };
+
+    let parsedTasks: unknown;
+    try {
+        parsedTasks = JSON.parse(env.tasks ?? '');
+    } catch {
+        parsedTasks = undefined;
+    }
+    const validArray =
+        Array.isArray(parsedTasks) && parsedTasks.every((w) => typeof w === 'string' && WBS_PATTERN.test(w));
+    if (!validArray) {
+        process.stderr.write(
+            'task-resolve: tasks must be a JSON array of canonical four-digit WBS strings (whitespace is rejected, not trimmed)\n',
+        );
+        return writeFail('failed:tasks is not a JSON array of canonical four-digit WBS strings');
+    }
+
+    // Dedupe in first-seen order (never sorted).
+    const deduped: string[] = [];
+    for (const wbs of parsedTasks as string[]) {
+        if (!deduped.includes(wbs)) deduped.push(wbs);
+    }
+    writeFileSync(abs(relTasksFile), `${JSON.stringify(deduped)}\n`);
+
+    let unresolved = false;
+    for (const wbs of deduped) {
+        const shown = spur(env, ['task', 'show', wbs, '--json'], { cwd });
+        const status = shown.status === 0 ? statusDisplay(shown.stdout) : 'unresolved';
+        if (status !== 'done' && status !== 'cancelled') {
+            process.stderr.write(
+                `task-resolve: task ${wbs} did not resolve to a completed status (status=${status})\n`,
+            );
+            unresolved = true;
+        }
+    }
+    if (unresolved) {
+        return writeFail(`failed:unresolved or non-completed task (see ${relTasksFile})`);
+    }
+    writeFileSync(abs(relStatusFile), 'PASS\n');
+    return { status: 'PASS', statusFile: relStatusFile, tasksFile: relTasksFile, exitCode: 0 };
+}
+
+export interface MetricsResult {
+    status: 'PASS' | 'FAIL';
+    statusFile: string;
+}
+
+/** `metrics` — append one JSONL row per captured task; a missing row is never silently absorbed. */
+export function runMetrics(env: WrapupStepsEnv, options: WrapupStepsOptions = {}): MetricsResult {
+    const cwd = options.cwd;
+    const runId = env.__runId ?? '';
+    mkdirSync(cwd ? join(cwd, '.spur', 'run') : join('.spur', 'run'), { recursive: true });
+    mkdirSync(cwd ? join(cwd, '.spur', 'memory') : join('.spur', 'memory'), { recursive: true });
+    const relStatusFile = join('.spur', 'run', `${runId}-wrapup-metrics.status`);
+    const relTasksFile = join('.spur', 'run', `${runId}-wrapup-tasks.json`);
+    const relMetricsFile = join('.spur', 'memory', 'wrapup-metrics.jsonl');
+    const abs = (p: string): string => (cwd ? join(cwd, p) : p);
+
+    const fail = (): MetricsResult => {
+        writeFileSync(abs(relStatusFile), 'FAIL\n');
+        return { status: 'FAIL', statusFile: relStatusFile };
+    };
+
+    let captured: unknown;
+    try {
+        captured = JSON.parse(readFileSync(abs(relTasksFile), 'utf8'));
+    } catch {
+        captured = undefined;
+    }
+    const validCapture = Array.isArray(captured) && captured.every((w) => typeof w === 'string' && WBS_PATTERN.test(w));
+    if (!validCapture) {
+        process.stderr.write(
+            'metrics-record: run-scoped task capture missing, corrupted or non-canonical — refusing to record metrics\n',
+        );
+        return fail();
+    }
+
+    let metricsRc = 0;
+    for (const wbs of captured as string[]) {
+        const shown = spur(env, ['task', 'show', wbs, '--json'], { cwd });
+        const lookup = shown.status === 0 ? taskStatusOf(shown.stdout) : { resolved: null, present: false };
+        if (!lookup.present) {
+            process.stderr.write(
+                `metrics-record: task ${wbs} lookup failed or was malformed — recording FAIL instead of silently omitting its metrics row\n`,
+            );
+            metricsRc = 1;
+            continue;
+        }
+        const parsed = JSON.parse(shown.stdout) as Record<string, unknown>;
+        const frontmatter =
+            parsed.frontmatter !== null && typeof parsed.frontmatter === 'object'
+                ? (parsed.frontmatter as Record<string, unknown>)
+                : {};
+        const featureId = String(jqPick(frontmatter.feature_id, parsed.feature_id, ''));
+        const status = String(jqPick(frontmatter.status, parsed.status, 'unknown'));
+
+        // jq `//` semantics: null and false count as missing; an empty string result stays UNKNOWN.
+        let verdict = 'UNKNOWN';
+        const verdictPath = join('.spur', 'run', `${wbs}-verdict.json`);
+        if (existsSync(abs(verdictPath))) {
+            try {
+                const raw = jqPick(JSON.parse(readFileSync(abs(verdictPath), 'utf8')).verdict, 'UNKNOWN');
+                const text = raw === 'UNKNOWN' ? 'UNKNOWN' : jqText(raw);
+                if (text.length > 0) verdict = text;
+            } catch {
+                // unreadable verdict file keeps UNKNOWN telemetry
+            }
+        }
+
+        const timestamp = new Date().toISOString().replace(/\.\d{3}Z$/, 'Z');
+        const row = { wbs, feature_id: featureId, status, verdict, timestamp };
+        try {
+            appendFileSync(abs(relMetricsFile), `${JSON.stringify(row)}\n`);
+        } catch {
+            process.stderr.write(
+                `metrics-record: metrics append failed for task ${wbs} — recording FAIL instead of claiming the row landed\n`,
+            );
+            metricsRc = 1;
+        }
+    }
+    const status: 'PASS' | 'FAIL' = metricsRc === 0 ? 'PASS' : 'FAIL';
+    writeFileSync(abs(relStatusFile), `${status}\n`);
+    return { status, statusFile: relStatusFile };
+}
+
+export interface FeatureTransitionResult {
+    status: 'PASS' | 'FAIL';
+    statusFile: string;
+    /** Only an empty vars.feature is a hard failure (mis-invocation, not a blocked sync). */
+    exitCode: number;
+}
+
+/** Classify one sync result per 0783 R4; returns the blocking reason or '' for a verified sync. */
+export function classifySync(
+    syncOutput: string,
+    syncRc: number,
+    feature: string,
+    observed: string,
+): { reason: string; applied: string; syncOk: boolean } {
+    if (syncRc !== 0) {
+        return { reason: `sync exited nonzero (rc=${syncRc})`, applied: 'unreadable', syncOk: false };
+    }
+    let parsed: unknown;
+    try {
+        parsed = JSON.parse(syncOutput);
+    } catch {
+        parsed = undefined;
+    }
+    const obj = parsed !== null && typeof parsed === 'object' ? (parsed as Record<string, unknown>) : undefined;
+    const proposal =
+        obj?.proposal !== null && typeof obj?.proposal === 'object'
+            ? (obj.proposal as Record<string, unknown>)
+            : undefined;
+    const shapeOk =
+        obj !== undefined &&
+        proposal !== undefined &&
+        typeof proposal.featureId === 'string' &&
+        typeof proposal.from === 'string' &&
+        typeof proposal.to === 'string' &&
+        typeof obj.applied === 'boolean';
+    if (!shapeOk) {
+        return { reason: 'malformed or unreadable sync result', applied: 'unreadable', syncOk: false };
+    }
+    const applied = obj.applied === true ? 'true' : 'false';
+    const pFrom = String(jqPick(proposal.from, ''));
+    const pTo = String(jqPick(proposal.to, ''));
+    if (String(jqPick(proposal.featureId, '')) !== feature) {
+        return { reason: `sync proposal does not match feature ${feature}`, applied, syncOk: false };
+    }
+    if (proposal.gateBlocked === true) {
+        return {
+            reason: 'sync proposal is gate-blocked — a blocked sync is not a no-change success',
+            applied,
+            syncOk: false,
+        };
+    }
+    if (proposal.requiresConfirm === true) {
+        return { reason: 'sync proposal requires operator confirmation', applied, syncOk: false };
+    }
+    if (applied === 'true' && observed !== pTo) {
+        return {
+            reason: `applied sync did not land on the proposal target (observed=${observed}, to=${pTo})`,
+            applied,
+            syncOk: false,
+        };
+    }
+    if (applied === 'false' && (pFrom !== pTo || observed !== pTo)) {
+        return {
+            reason: `sync applied nothing without a from==to observed no-op (from=${pFrom}, to=${pTo}, observed=${observed})`,
+            applied,
+            syncOk: false,
+        };
+    }
+    return { reason: '', applied, syncOk: true };
+}
+
+/** `feature-transition` — required bounded sync, observation and the affected-feature gate. */
+export function runFeatureTransition(env: WrapupStepsEnv, options: WrapupStepsOptions = {}): FeatureTransitionResult {
+    const cwd = options.cwd;
+    const runId = env.__runId ?? '';
+    const feature = env.feature ?? '';
+    mkdirSync(cwd ? join(cwd, '.spur', 'run') : join('.spur', 'run'), { recursive: true });
+    const relStatusFile = join('.spur', 'run', `${runId}-wrapup-sync.status`);
+    const abs = (p: string): string => (cwd ? join(cwd, p) : p);
+    if (feature.length === 0) {
+        process.stderr.write(
+            'feature-transition: vars.feature is empty — refusing no-op feature sync (mis-invocation, not a blocked sync)\n',
+        );
+        return { status: 'FAIL', statusFile: relStatusFile, exitCode: 1 };
+    }
+
+    // Three branches, first that exists relative to the cwd; current arguments including --spur-bin.
+    let syncOutput = '';
+    let syncRc = 1;
+    const boundedTs = join('plugins', 'sp', 'scripts', 'feature-sync-bounded.ts');
+    const boundedArgs = [feature, '--spur-bin', env.spurBin ?? 'spur', '--json'];
+    if (existsSync(cwd ? join(cwd, boundedTs) : boundedTs)) {
+        const result = spawnSync('bun', [boundedTs, ...boundedArgs], { cwd, encoding: 'utf8' });
+        syncOutput = result.stdout ?? '';
+        syncRc = result.status ?? 1;
+        if (result.stderr !== null && result.stderr.length > 0) process.stderr.write(result.stderr);
+    } else {
+        const probe = spawnSync('superskill', ['script', 'path', 'sp', 'feature-sync-bounded.mjs'], {
+            cwd,
+            encoding: 'utf8',
+        });
+        const twin = probe.status === 0 ? (probe.stdout ?? '').trim() : '';
+        if (twin.length > 0 && existsSync(twin)) {
+            const result = spawnSync('node', [twin, ...boundedArgs], { cwd, encoding: 'utf8' });
+            syncOutput = result.stdout ?? '';
+            syncRc = result.status ?? 1;
+            if (result.stderr !== null && result.stderr.length > 0) process.stderr.write(result.stderr);
+        } else {
+            // Last branch: stderr streams through (visible), stdout is the JSON payload.
+            const result = spur(env, ['feature', 'sync', feature, '--json'], { cwd, stderr: 'inherit' });
+            syncOutput = result.stdout;
+            syncRc = result.status;
+        }
+    }
+    process.stdout.write(`${syncOutput}\n`);
+
+    const shown = spur(env, ['feature', 'show', feature, '--json'], { cwd });
+    let observed = '';
+    if (shown.status === 0) {
+        try {
+            const parsed = JSON.parse(shown.stdout) as Record<string, unknown>;
+            const frontmatter =
+                parsed.frontmatter !== null && typeof parsed.frontmatter === 'object'
+                    ? (parsed.frontmatter as Record<string, unknown>)
+                    : {};
+            const picked = jqPick(parsed.status, frontmatter.status, '');
+            observed = picked === '' ? '' : jqText(picked);
+        } catch {
+            observed = '';
+        }
+    }
+    if (observed.length === 0) observed = 'unreadable';
+
+    const classified = classifySync(syncOutput, syncRc, feature, observed);
+    const { reason, applied } = classified;
+    const syncOk = classified.syncOk;
+
+    let gate = 'skipped';
+    if (applied === 'true' || syncRc !== 0) {
+        process.stdout.write(
+            `feature-transition: sync applied or failed after a possible partial transition for ${feature} — running feature gate: ${env.featureGateCmd ?? ''}\n`,
+        );
+        const gateResult = spawnSync('sh', ['-c', env.featureGateCmd ?? ''], { cwd, stdio: 'inherit' });
+        if ((gateResult.status ?? 1) === 0) {
+            gate = 'PASS';
+            process.stdout.write(`feature-transition: feature gate PASS for feature ${feature}\n`);
+        } else {
+            gate = 'FAIL';
+            process.stderr.write(
+                `feature-transition: feature gate FAIL for feature ${feature} — inspect findings before reporting the transition complete\n`,
+            );
+        }
+    } else {
+        process.stdout.write(
+            `feature-transition: sync did not apply a transition (rc=${syncRc}, applied=${applied}) — feature gate skipped\n`,
+        );
+    }
+
+    let syncStatus: 'PASS' | 'FAIL';
+    if (!syncOk || gate === 'FAIL') {
+        syncStatus = 'FAIL';
+        process.stderr.write(
+            `feature-transition: required synchronization failed for ${feature} — ${reason}; gate=${gate}\n`,
+        );
+    } else if (applied === 'false') {
+        syncStatus = 'PASS';
+        process.stdout.write(
+            `feature-transition: feature sync verified for ${feature} (from==to observed at ${observed}, gate=${gate}) — explicit no-change\n`,
+        );
+    } else {
+        syncStatus = 'PASS';
+        process.stdout.write(
+            `feature-transition: feature sync verified for ${feature} (applied, observed=${observed}, gate=${gate})\n`,
+        );
+    }
+    writeFileSync(abs(relStatusFile), `${syncStatus}\n`);
+    return { status: syncStatus, statusFile: relStatusFile, exitCode: 0 };
+}
+
+export const WRAPUP_STEPS_USAGE =
+    'usage: wrapup-steps.ts <resolve|metrics|feature-transition>  (env: __runId, tasks, feature, featureGateCmd, spurBin)';
+
+export function main(argv: string[], env: WrapupStepsEnv = process.env, options: WrapupStepsOptions = {}): number {
+    const sub = argv[0];
+    if (sub === 'resolve') return resolveTasks(env, options).exitCode;
+    if (sub === 'metrics') {
+        runMetrics(env, options);
+        return 0;
+    }
+    if (sub === 'feature-transition') return runFeatureTransition(env, options).exitCode;
+    process.stderr.write(`${WRAPUP_STEPS_USAGE}\n`);
+    return 2;
+}
+
+if (import.meta.main) {
+    process.exit(main(process.argv.slice(2)));
+}

package/plugins/sp/skills/code-review/references/review-lenses.md

@@ -18,6 +18,9 @@ Five dimensions for structured code review. Each lens has diagnostic questions;
 - Are edge cases handled (empty, null, boundary, error)?
 - Are invariants preserved across the changed functions?
 - Is there a test that fails if the behavior is wrong?
+- A changed route response shape or client parser is covered both ways: a server-side key-shape
+  assertion and a test that runs a real payload through the real parser (a dropped key like `teamId`
+  must fail something). Missing either → P2.
 
 **Severity guide:** Logic errors that produce wrong output → P1. Missing edge case that could cause wrong output → P2. Unclear control flow that might hide bugs → P3.
 

package/plugins/sp/skills/parallel-execution/references/dispatch-surface.md

@@ -128,7 +128,7 @@ rule at the source, so no per-path shim is needed:
 | `spur agent run` (CLI) | `AgentService.run` → resolution → child process | Declared wins; absent inherits via `SPUR_ROLE`; envelope carries `roleOrigin` |
 | Workflow `agent.run` step | `AgentRunActionRunner` → `AgentService.runTraced` | Step `role:` is **mandatory** (0538 R2, `agent-run.ts` fails a role-less step before dispatch) — always a declaration (`roleOrigin: 'declared'`); inheritance applies at the next fan-out boundary the step's subagent itself dispatches |
 | `spur agent loop` | `AgentService.run` per drained iteration | Same resolution path as `spur agent run`; inherits its own `SPUR_ROLE` |
-| `spur team` supervisor → member | spawns `spur agent loop` | Member inherits the supervisor's `SPUR_ROLE` (recursive by construction) |
+| `spur serve` team supervisor → member | spawns `spur agent loop` | Member inherits the supervisor's `SPUR_ROLE` (recursive by construction) |
 | Native subagent fan-out (this skill's default) | in-session `Task()`/`Skill()` | In-session subagents share the host session; when they themselves dispatch, the host's role is already in the session env — the rule holds at the next `spur agent run` boundary |
 | `plugins/sp/evals/run-eval.ts` | `spawnSync('spur agent run', …)` per scenario | Out of scope: a top-level eval harness, not a fan-out — no dispatcher role exists to inherit; each scenario is an independent top-level run (documented, no shim) |
 

package/plugins/sp/skills/spec-decomposition/references/decomposition.md

@@ -525,6 +525,35 @@ The payload is a top-level JSON **array** (no `tasks` wrapper):
 ]
 ```
 
+## Idea-pipeline emission
+
+When the idea-pipeline workflow dispatches you for a feature, read the brainstorm artifact, the
+feature AC, and the design doc, then emit two run-scoped artifacts.
+
+**Sizing first, before any JSON.** Apply the `Default to NOT decomposing` rubric to the whole
+unit of work — if it scores 0–2 the correct output is a ONE-entry batch, not many.
+
+**Scenario count is not task count.** Merge scenarios that one task delivers (same file surface,
+same subsystem, or unreadable apart in review), and list every scenario a task covers in its
+background. Merging never costs AC coverage — one task may carry several scenarios. Do not emit
+one entry per scenario or per requirement by reflex.
+
+**The batch.** Produce a task-batch JSON array at the workflow-provided batch path
+(`.spur/run/<runId>-idea-task-batch.json`), validated against `task-batch.schema.json`.
+Schema-permitted fields per entry: `name`, `background`, `requirements`, `design`, `plan`,
+`acceptance_criteria`, `feature_id`, `parent_wbs`, `priority`, `tags`, `template` — schema
+validation rejects anything else. `design`, `plan`, and `acceptance_criteria` are supported batch
+fields and normal default planning fills them from your analysis; the per-task refine step after
+batch-create still deepens them when a task needs more detail. Validate locally against the
+schema before emitting.
+
+**The order sidecar.** Also emit the private task-order sidecar at
+`.spur/run/<runId>-idea-task-order.json`: a JSON array (one entry per batch item) of
+`{ name: <exact batch item name>, depends_on_names: [<batch item names>] }` declaring
+ordering/dependencies between the batch items; state `depends_on_names: []` per item when no
+ordering exists. Every `name` and every dependency must match exactly one batch item `name` —
+it is private workflow data, not part of task-batch.schema.json.
+
 ## Common schema violations
 
 | Violation | Fix |

package/plugins/sp/skills/spur-cli/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: spur-cli
-description: "Reference and operate the complete `spur` CLI surface: task, feature, rule, workflow, builder, agent, message, team, self, history, and projects. Use for verb or flag lookup, machine-readable output and exit contracts, or CLI-gated corpus writes. Triggers: \"spur task\", \"spur history\", \"spur projects\", \"create a task\", \"task check\", and any Spur CLI convention. Not for planning or execution lifecycle orchestration (`sp:spur-dev`)."
+description: "Reference and operate the complete `spur` CLI surface: task, feature, rule, workflow, builder, agent, message, self, history, and projects. Use for verb or flag lookup, machine-readable output and exit contracts, or CLI-gated corpus writes. Triggers: \"spur task\", \"spur history\", \"spur projects\", \"create a task\", \"task check\", and any Spur CLI convention. Not for planning or execution lifecycle orchestration (`sp:spur-dev`)."
 license: Apache-2.0
 metadata:
   author: spur
@@ -17,7 +17,6 @@ metadata:
     - builder
     - agent
     - message
-    - team
     - self
     - history
     - projects
@@ -45,9 +44,8 @@ Pick the noun, read its reference. Each Tier A and Tier B reference owns that no
 | **Tier A** | **rule** | Constraint quality gate: run presets, author rules, fine-tune, validate rule files/presets, extend engine | [references/rules.md](references/rules.md) |
 | **Tier A** | **workflow** | Dual-mode workflow runtime: author state-machine / transition-flow workflows, validate, run, read traces | [references/workflows.md](references/workflows.md) |
 | **Tier A** | **builder** | Release plumbing: bump a package (or the `workspace:`-pinned set) with `bump-ver`, delete release tags with `drop-tags`, commit + tag + optional push | [references/builder.md](references/builder.md) |
-| **Tier B** | **agent** | Coding-agent execution surface: run prompts via detected/named agents, manage team agent specs, persistent self-draining loop, readiness check | [references/agent.md](references/agent.md) |
+| **Tier B** | **agent** | Coding-agent execution surface: run prompts via detected/named agents, list agent specs, start/stop supervised processes, readiness check | [references/agent.md](references/agent.md) |
 | **Tier B** | **message** | Durable inter-agent messaging: send, inbox, reply, watch | [references/message.md](references/message.md) |
-| **Tier B** | **team** | Team coordination and supervision: assign, status, up/down rosters, start/stop supervised processes | [references/team.md](references/team.md) |
 | **Tier B** | **self** | Self-management verbs: scaffold (`init`), schema migrations (`migrate`), local web server (`serve`), status overview (`status`); `self init` runs post-scaffold validation probes & layout classification | [references/self.md](references/self.md) |
 | **Tier B** | **history** | Import agent histories, aggregate forensic artifacts, render reports, and run the checkpoint-resumed daily pipeline | [references/history.md](references/history.md) |
 | **Tier B** | **projects** | Manage the local multi-project registry and start/stop project servers | [references/projects.md](references/projects.md) |
@@ -138,13 +136,11 @@ and spreading it; full contract in `docs/04_DESIGN.md` §1.0.1.
 
 ## See also
 
-- **[references/agent.md](references/agent.md)** - coding-agent execution surface (`run`, `loop`,
-  spec management). Cross-references the
+- **[references/agent.md](references/agent.md)** - coding-agent execution surface (`run`, `wait`,
+  `list --specs`, `start`/`stop`). Cross-references the
   [dispatch-surface rule](../parallel-execution/references/dispatch-surface.md).
 - **[references/message.md](references/message.md)** - durable inter-agent messaging (`send`,
   `inbox`, `reply`, `watch`).
-- **[references/team.md](references/team.md)** - team coordination and supervision (`assign`,
-  `status`, `up`/`down`, `start`/`stop`).
 - **[references/self.md](references/self.md)** - `spur self init|migrate|serve|status` CLI verbs
   (the four legacy top-level nouns remain hidden aliases). `self init` runs post-scaffold init
   validation (Phase 1.5/1.6 probes).