@aexol/spectral 0.7.6 → 0.7.8

package/dist/agent/index.js

@@ -26,7 +26,6 @@
  */
 import { agentLoop } from "../pi/agent-core/index.js";
 import { StringEnum } from "../pi/ai/index.js";
-import { Text } from "../pi/tui/index.js";
 import { Type } from "typebox";
 import { discoverAgents } from "./agents.js";
 // ---------------------------------------------------------------------------
@@ -235,17 +234,19 @@ async function runSingleAgent(defaultCwd, agents, agentName, task, cwd, step, si
         agent: agentName,
         agentSource: agent.source,
         task,
-        exitCode: 0,
+        exitCode: -1, // -1 = still running; 0 = success; 1 = error
         messages: [],
         stderr: "",
         usage: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0, cost: 0, contextTokens: 0, turns: 0 },
         model: model.id,
         step,
     };
+    // Accumulated streaming text from text_delta events (reset per assistant turn).
+    let streamingText = "";
     const emitUpdate = () => {
         if (onUpdate) {
             onUpdate({
-                content: [{ type: "text", text: getFinalOutput(currentResult.messages) || "(running...)" }],
+                content: [{ type: "text", text: getFinalOutput(currentResult.messages) || streamingText || "(running...)" }],
                 details: makeDetails([currentResult]),
             });
         }
@@ -271,10 +272,22 @@ async function runSingleAgent(defaultCwd, agents, agentName, task, cwd, step, si
                 hadError = true;
                 break;
             }
+            if (event.type === "message_update" && event.message?.role === "assistant") {
+                // Forward streaming text deltas so the UI can show the subagent's
+                // response as it's generated, not just the final result.
+                const inner = event.assistantMessageEvent;
+                if (inner?.type === "text_delta" && inner.delta) {
+                    streamingText += inner.delta;
+                    emitUpdate();
+                }
+                continue;
+            }
             if (event.type === "message_end" && event.message) {
                 const msg = event.message;
                 collectedMessages.push(msg);
                 if (msg.role === "assistant") {
+                    // Reset streaming accumulator — final text is now in messages
+                    streamingText = "";
                     currentResult.usage.turns++;
                     const usage = msg.usage;
                     if (usage) {
@@ -600,142 +613,5 @@ export default function subagentExtension(pi) {
                 details: makeDetails("single")([]),
             };
         },
-        // ── Rendering (headless-compatible: plain text strings, no TUI imports) ──
-        renderCall(args, _theme, _context) {
-            const scope = args.agentScope ?? "user";
-            if (args.chain && args.chain.length > 0) {
-                let text = `subagent chain (${args.chain.length} steps) [${scope}]`;
-                for (let i = 0; i < Math.min(args.chain.length, 3); i++) {
-                    const step = args.chain[i];
-                    const cleanTask = step.task.replace(/\{previous\}/g, "").trim();
-                    const preview = cleanTask.length > 40 ? `${cleanTask.slice(0, 40)}...` : cleanTask;
-                    text += `\n  ${i + 1}. ${step.agent} ${preview}`;
-                }
-                if (args.chain.length > 3)
-                    text += `\n  ... +${args.chain.length - 3} more`;
-                return new Text(text);
-            }
-            if (args.tasks && args.tasks.length > 0) {
-                let text = `subagent parallel (${args.tasks.length} tasks) [${scope}]`;
-                for (const t of args.tasks.slice(0, 3)) {
-                    const preview = t.task.length > 40 ? `${t.task.slice(0, 40)}...` : t.task;
-                    text += `\n  ${t.agent} ${preview}`;
-                }
-                if (args.tasks.length > 3)
-                    text += `\n  ... +${args.tasks.length - 3} more`;
-                return new Text(text);
-            }
-            const agentName = args.agent || "...";
-            const preview = args.task
-                ? args.task.length > 60 ? `${args.task.slice(0, 60)}...` : args.task
-                : "...";
-            return new Text(`subagent ${agentName} [${scope}]\n  ${preview}`);
-        },
-        renderResult(result, _opts, _theme, _context) {
-            const details = result.details;
-            if (!details || details.results.length === 0) {
-                const text = result.content[0];
-                return new Text(text?.type === "text" ? text.text : "(no output)");
-            }
-            const lines = [];
-            if (details.mode === "single" && details.results.length === 1) {
-                const r = details.results[0];
-                const isError = r.exitCode !== 0 || r.stopReason === "error" || r.stopReason === "aborted";
-                const icon = isError ? "✗" : "✓";
-                const finalOutput = getFinalOutput(r.messages);
-                lines.push(`${icon} ${r.agent} (${r.agentSource})${isError && r.stopReason ? ` [${r.stopReason}]` : ""}`);
-                if (isError && r.errorMessage)
-                    lines.push(`Error: ${r.errorMessage}`);
-                lines.push("");
-                lines.push("─── Task ───");
-                lines.push(r.task);
-                lines.push("");
-                lines.push("─── Output ───");
-                if (!finalOutput) {
-                    lines.push("(no output)");
-                }
-                else {
-                    lines.push(finalOutput.trim());
-                }
-                const usageStr = formatUsageStats(r.usage, r.model);
-                if (usageStr) {
-                    lines.push("");
-                    lines.push(usageStr);
-                }
-                return new Text(lines.join("\n"));
-            }
-            const aggregateUsage = (results) => {
-                const total = { input: 0, output: 0, cacheRead: 0, cacheWrite: 0, cost: 0, turns: 0 };
-                for (const r of results) {
-                    total.input += r.usage.input;
-                    total.output += r.usage.output;
-                    total.cacheRead += r.usage.cacheRead;
-                    total.cacheWrite += r.usage.cacheWrite;
-                    total.cost += r.usage.cost;
-                    total.turns += r.usage.turns;
-                }
-                return total;
-            };
-            if (details.mode === "chain") {
-                const successCount = details.results.filter((r) => r.exitCode === 0).length;
-                const icon = successCount === details.results.length ? "✓" : "✗";
-                lines.push(`${icon} chain ${successCount}/${details.results.length} steps`);
-                for (const r of details.results) {
-                    const rIcon = r.exitCode === 0 ? "✓" : "✗";
-                    const finalOutput = getFinalOutput(r.messages);
-                    lines.push("");
-                    lines.push(`─── Step ${r.step}: ${r.agent} ${rIcon}`);
-                    lines.push(`Task: ${r.task}`);
-                    if (finalOutput)
-                        lines.push(finalOutput.trim());
-                    else
-                        lines.push("(no output)");
-                    const stepUsage = formatUsageStats(r.usage, r.model);
-                    if (stepUsage)
-                        lines.push(stepUsage);
-                }
-                const usageStr = formatUsageStats(aggregateUsage(details.results));
-                if (usageStr) {
-                    lines.push("");
-                    lines.push(`Total: ${usageStr}`);
-                }
-                return new Text(lines.join("\n"));
-            }
-            if (details.mode === "parallel") {
-                const running = details.results.filter((r) => r.exitCode === -1).length;
-                const successCount = details.results.filter((r) => r.exitCode === 0).length;
-                const failCount = details.results.filter((r) => r.exitCode > 0).length;
-                const isRunning = running > 0;
-                const icon = isRunning ? "⏳" : failCount > 0 ? "◐" : "✓";
-                const status = isRunning
-                    ? `${successCount + failCount}/${details.results.length} done, ${running} running`
-                    : `${successCount}/${details.results.length} tasks`;
-                lines.push(`${icon} parallel ${status}`);
-                for (const r of details.results) {
-                    const rIcon = r.exitCode === -1 ? "⏳" : r.exitCode === 0 ? "✓" : "✗";
-                    const finalOutput = getFinalOutput(r.messages);
-                    lines.push("");
-                    lines.push(`─── ${r.agent} ${rIcon}`);
-                    lines.push(`Task: ${r.task}`);
-                    if (finalOutput)
-                        lines.push(finalOutput.trim());
-                    else
-                        lines.push(r.exitCode === -1 ? "(running...)" : "(no output)");
-                    const taskUsage = formatUsageStats(r.usage, r.model);
-                    if (taskUsage)
-                        lines.push(taskUsage);
-                }
-                if (!isRunning) {
-                    const usageStr = formatUsageStats(aggregateUsage(details.results));
-                    if (usageStr) {
-                        lines.push("");
-                        lines.push(`Total: ${usageStr}`);
-                    }
-                }
-                return new Text(lines.join("\n"));
-            }
-            const text = result.content[0];
-            return new Text(text?.type === "text" ? text.text : "(no output)");
-        },
     });
 }

package/dist/cli.js

@@ -2,120 +2,26 @@
 /**
  * @aexol/spectral — Coding agent that never sleeps
  *
- * Thin branding wrapper around @mariozechner/pi-coding-agent (pi).
- *
- * Delegation strategy: SUBPROCESS spawn of pi's bin.
- *
- * Why subprocess and not in-process import?
- *  - pi's CLI entry has top-level side effects (TUI bootstrap, signal handlers,
- *    raw stdin mode). Running it in-process means our wrapper process owns those,
- *    and any future pre-processing we add (skills, system prompts) becomes
- *    entangled with pi's lifecycle.
- *  - A child process gives us clean exit-code propagation, clean SIGINT
- *    forwarding, and a stable boundary for future iterations to inject things
- *    like extra args, env vars, or post-run hooks without monkey-patching pi.
- *  - stdio: "inherit" gives pi direct TTY access, so its TUI renders correctly
- *    and raw stdin works as expected.
+ * Headless CLI wrapper — no TUI mode supported.
  *
  * Subcommand routing:
  *  - `spectral login`  → interactive auth flow (writes ~/.spectral/config.json)
  *  - `spectral logout` → deletes ~/.spectral/config.json
+ *  - `spectral serve`  → start the Aexol relay WebSocket server (headless)
+ *  - `spectral bind`   → link directory to an Aexol Studio project
+ *  - `spectral unbind` → remove Aexol Studio project binding
  *  - `spectral --version` / `--help` → branded short-circuits, no auth needed.
- *  - anything else → pre-flight: require ~/.spectral/config.json, then spawn pi
- *    with the bundled Aexol MCP extension auto-loaded.
+ *  - anything else → error with usage hint.
  */
-import { spawn } from "node:child_process";
-import { existsSync, readFileSync } from "node:fs";
-import { constants as osConstants } from "node:os";
+import { readFileSync } from "node:fs";
 import { dirname, resolve } from "node:path";
 import { fileURLToPath } from "node:url";
-import { requireLogin } from "./preflight.js";
 const __dirname = dirname(fileURLToPath(import.meta.url));
 // ---- Read our own version ----------------------------------------------------
 const pkgPath = resolve(__dirname, "..", "package.json");
 const pkg = JSON.parse(readFileSync(pkgPath, "utf8"));
 const VERSION = pkg.version;
 const TAGLINE = "Coding agent that never sleeps";
-// ---- Resolve pi's bin --------------------------------------------------------
-// pi exports `.` as ESM only and does not export `./package.json`, so
-// require.resolve(...) is unreliable across Node versions. Instead we walk
-// upward from this file's location looking for a `node_modules/<pi>/package.json`.
-function resolvePiBin() {
-    const piPkgRel = "node_modules/@mariozechner/pi-coding-agent/package.json";
-    let dir = __dirname;
-    let piPkgJsonPath;
-    for (let i = 0; i < 20; i++) {
-        const candidate = resolve(dir, piPkgRel);
-        try {
-            readFileSync(candidate, "utf8");
-            piPkgJsonPath = candidate;
-            break;
-        }
-        catch {
-            /* keep walking */
-        }
-        const parent = dirname(dir);
-        if (parent === dir)
-            break;
-        dir = parent;
-    }
-    if (!piPkgJsonPath) {
-        throw new Error("Unable to locate @mariozechner/pi-coding-agent in any ancestor node_modules.");
-    }
-    const piPkg = JSON.parse(readFileSync(piPkgJsonPath, "utf8"));
-    let binRel;
-    if (typeof piPkg.bin === "string") {
-        binRel = piPkg.bin;
-    }
-    else if (piPkg.bin && typeof piPkg.bin === "object") {
-        binRel = piPkg.bin.pi ?? Object.values(piPkg.bin)[0];
-    }
-    if (!binRel) {
-        throw new Error("Unable to locate pi bin in @mariozechner/pi-coding-agent package.json");
-    }
-    return resolve(dirname(piPkgJsonPath), binRel);
-}
-/**
- * Resolve an extension entry-point path, falling back from .js to .ts.
- *
- * In production __dirname is dist/ and compiled .js files exist. In dev
- * (tsx src/cli.ts) __dirname is src/ and only .ts sources are present.
- * pi's extension loader uses jiti which handles .ts transparently.
- */
-function resolveExtensionEntry(relativePath, fileName) {
-    const dir = resolve(__dirname, relativePath);
-    const jsPath = resolve(dir, fileName + ".js");
-    if (existsSync(jsPath))
-        return jsPath;
-    const tsPath = resolve(dir, fileName + ".ts");
-    if (existsSync(tsPath))
-        return tsPath;
-    return jsPath; // let the missing-file check produce a clear error
-}
-/** Absolute path to the bundled aexol-mcp extension. */
-function resolveAexolExtensionPath() {
-    return resolveExtensionEntry("extensions", "aexol-mcp");
-}
-/** Absolute path to the bundled observations-memory extension. */
-function resolveObservationalMemoryPath() {
-    return resolveExtensionEntry("memory", "index");
-}
-/** Absolute path to the bundled pi-mcp-adapter extension. */
-function resolveMcpExtensionPath() {
-    return resolveExtensionEntry("mcp", "index");
-}
-/** Absolute path to the bundled openrouter-attribution extension. */
-function resolveOpenRouterAttributionPath() {
-    return resolveExtensionEntry("extensions", "openrouter-attribution");
-}
-/** Absolute path to the bundled agent subagent extension. */
-function resolveAgentExtensionPath() {
-    return resolveExtensionEntry("agent", "index");
-}
-/** Absolute path to the bundled designer extension. */
-function resolveDesignerExtensionPath() {
-    return resolveExtensionEntry("designer", "index");
-}
 // ---- Branded helpers ---------------------------------------------------------
 function printVersion() {
     process.stdout.write(`spectral ${VERSION} — ${TAGLINE}\n`);
@@ -128,77 +34,29 @@ function printHeader() {
         "Subcommands:",
         "  spectral login         Interactive authentication (team API key or OAuth)",
         "  spectral logout        Remove stored Aexol credentials",
-        "  spectral serve    Connect this machine to the Aexol relay backend",
-        "  spectral bind     Link this directory to an Aexol Studio project",
-        "  spectral unbind   Remove the Aexol Studio project binding",
+        "  spectral serve         Connect this machine to the Aexol relay backend",
+        "  spectral bind          Link this directory to an Aexol Studio project",
+        "  spectral unbind        Remove the Aexol Studio project binding",
+        "  spectral --version     Print version and exit",
+        "  spectral --help        Print this help and exit",
         "",
-        "Powered by pi (@mariozechner/pi-coding-agent).",
-        "All other flags are forwarded to pi. Run: spectral <pi-args>",
+        "TUI/interactive mode has been removed. Use `spectral serve` and connect",
+        "via the Aexol Studio web frontend or relay WebSocket.",
         "",
     ].join("\n"));
 }
-// ---- Delegate to pi ----------------------------------------------------------
-function delegateToPi(args) {
-    const piBin = resolvePiBin();
-    // The subprocess inherits a snapshot of process.env (which already has
-    // all interactive-editor vars disabled by disableInteractiveGitEditors()),
-    // but we pass it explicitly anyway as defense-in-depth in case the user
-    // overrides any of these in their shell rc files before launching spectral.
-    const child = spawn(process.execPath, [piBin, ...args], {
-        stdio: "inherit",
-        env: {
-            ...process.env,
-            GIT_EDITOR: "true",
-            GIT_SEQUENCE_EDITOR: "true",
-            EDITOR: "true",
-            VISUAL: "true",
-            GIT_PAGER: "cat",
-        },
-    });
-    // Forward common termination signals to pi so its TUI can clean up.
-    const signals = ["SIGINT", "SIGTERM", "SIGHUP", "SIGQUIT"];
-    for (const sig of signals) {
-        process.on(sig, () => {
-            if (!child.killed) {
-                try {
-                    child.kill(sig);
-                }
-                catch {
-                    /* ignore */
-                }
-            }
-        });
-    }
-    child.on("exit", (code, signal) => {
-        if (signal) {
-            const sigNum = osConstants.signals[signal] ?? 0;
-            process.exit(128 + sigNum);
-        }
-        process.exit(code ?? 0);
-    });
-    child.on("error", (err) => {
-        process.stderr.write(`spectral: failed to launch pi: ${err.message}\n`);
-        process.exit(1);
-    });
-    // spawn returns; keep the type system happy.
-    // We never actually reach here because of the exit handlers above, but
-    // TypeScript needs a `never` terminator.
-    return undefined;
-}
 // ---- Disable interactive git editors ----------------------------------------
-// Set at process level so BOTH direct mode (delegateToPi → subprocess inherits)
-// AND serve mode (PiBridge in-process → getShellEnv() returns process.env) are
-// covered. Without this, `git rebase --continue`, `git commit` (without -m),
-// `git merge`, etc. open an editor that hangs forever — the agent's bash tool
-// runs with `stdio: ["ignore", ...]`, so there is no TTY to interact with.
+// Set at process level for serve mode (PiBridge in-process → getShellEnv()
+// returns process.env). Without this, `git rebase --continue`, `git commit`
+// (without -m), `git merge`, etc. open an editor that hangs forever — the
+// agent's bash tool runs with `stdio: ["ignore", ...]`, so there is no TTY
+// to interact with.
 function disableInteractiveGitEditors() {
     const editorVars = {
         GIT_EDITOR: "true",
         GIT_SEQUENCE_EDITOR: "true",
         EDITOR: "true",
         VISUAL: "true",
-        // cat pipes output straight through; prevents git's pager (less) from
-        // blocking on a non-existent TTY.
         GIT_PAGER: "cat",
     };
     for (const [key, value] of Object.entries(editorVars)) {
@@ -209,7 +67,6 @@ function disableInteractiveGitEditors() {
 }
 // ---- Main --------------------------------------------------------------------
 async function main() {
-    // Must be called before any pi process is spawned (direct or serve).
     disableInteractiveGitEditors();
     const args = process.argv.slice(2);
     const first = args[0];
@@ -220,12 +77,7 @@ async function main() {
     }
     if (first === "--help" || first === "-h") {
         printHeader();
-        // Spawn pi to append its own help, then bail. delegateToPi is
-        // annotated `: never` because its child.on("exit") handler calls
-        // process.exit, but the function itself returns synchronously after
-        // spawn — so we must not fall through to the pre-flight check below.
-        delegateToPi(args);
-        return;
+        process.exit(0);
     }
     // Subcommands. Dynamic import keeps the cold-start path light when users
     // are just running pi.
@@ -257,59 +109,12 @@ async function main() {
         await runUnbind();
         process.exit(0);
     }
-    // Pre-flight: every other invocation requires authenticated state. We do NOT
-    // auto-launch the login flow — that would fight pi's stdio inheritance and
-    // hide auth state changes from the user.
-    await requireLogin();
-    // Resolve and inject the Aexol MCP extension. Sanity-check existence so we
-    // fail with a clear message if someone publishes a broken bundle.
-    const aexolExtPath = resolveAexolExtensionPath();
-    if (!existsSync(aexolExtPath)) {
-        process.stderr.write(`spectral: bundled Aexol MCP extension not found at ${aexolExtPath}. This is a packaging bug.\n`);
-        process.exit(1);
-    }
-    // Bundled pi-mcp-adapter extension for standard MCP server support
-    // (stdio + SSE/HTTP transports, lazy loading, OAuth, /mcp panel).
-    const mcpExtPath = resolveMcpExtensionPath();
-    if (!existsSync(mcpExtPath)) {
-        process.stderr.write(`spectral: bundled MCP extension not found at ${mcpExtPath}. This is a packaging bug.\n`);
-        process.exit(1);
-    }
-    // Bundled observational memory extension for tiered agent memory
-    // (observer → reflector → pruner pipeline with /om-status, /om-view, recall tool).
-    const obsMemPath = resolveObservationalMemoryPath();
-    if (!existsSync(obsMemPath)) {
-        process.stderr.write(`spectral: bundled observational-memory extension not found at ${obsMemPath}. This is a packaging bug.\n`);
-        process.exit(1);
-    }
-    // OpenRouter attribution extension – adds headers so Aexol appears in OpenRouter rankings
-    const openrouterAttrPath = resolveOpenRouterAttributionPath();
-    if (!existsSync(openrouterAttrPath)) {
-        process.stderr.write(`spectral: bundled OpenRouter attribution extension not found at ${openrouterAttrPath}. This is a packaging bug.\n`);
-        process.exit(1);
-    }
-    // Bundled agent delegation extension for subagent task delegation
-    const agentExtPath = resolveAgentExtensionPath();
-    if (!existsSync(agentExtPath)) {
-        process.stderr.write(`spectral: bundled agent extension not found at ${agentExtPath}. This is a packaging bug.\n`);
-        process.exit(1);
-    }
-    // Bundled designer extension for design tasks (prototypes, slide decks, mobile apps, etc.)
-    const designerExtPath = resolveDesignerExtensionPath();
-    if (!existsSync(designerExtPath)) {
-        process.stderr.write(`spectral: bundled designer extension not found at ${designerExtPath}. This is a packaging bug.\n`);
-        process.exit(1);
-    }
-    const extFlags = [
-        "--extension", aexolExtPath,
-        "--extension", mcpExtPath,
-        "--extension", obsMemPath,
-        "--extension", openrouterAttrPath,
-        "--extension", agentExtPath,
-        "--extension", designerExtPath,
-    ];
-    const finalArgs = [...extFlags, ...args];
-    delegateToPi(finalArgs);
+    // No TUI/interactive mode is supported. All other invocations must use
+    // explicit subcommands.
+    process.stderr.write(`spectral: unknown command "${first ?? ""}".\n` +
+        `Use "spectral --help" for available subcommands.\n` +
+        `To connect this machine, run: spectral serve\n`);
+    process.exit(1);
 }
 main().catch((err) => {
     const msg = err instanceof Error ? err.message : String(err);