@bastani/atomic 0.8.13 → 0.8.14-0

package/dist/builtin/subagents/src/slash/slash-commands.ts

@@ -3,683 +3,526 @@ import * as fs from "node:fs";
 import * as path from "node:path";
 import type { ExtensionAPI, ExtensionContext } from "@bastani/atomic";
 import { Key, matchesKey } from "@earendil-works/pi-tui";
-import {
-  discoverAgents,
-  discoverAgentsAll,
-  type ChainConfig,
-} from "../agents/agents.ts";
+import { discoverAgents, discoverAgentsAll, type ChainConfig } from "../agents/agents.ts";
 import type { SubagentParamsLike } from "../runs/foreground/subagent-executor.ts";
 import { isParallelStep, type ChainStep } from "../shared/settings.ts";
-import type {
-  SlashSubagentResponse,
-  SlashSubagentUpdate,
-} from "./slash-bridge.ts";
+import type { SlashSubagentResponse, SlashSubagentUpdate } from "./slash-bridge.ts";
 import {
-  applySlashUpdate,
-  buildSlashInitialResult,
-  failSlashResult,
-  finalizeSlashResult,
+	applySlashUpdate,
+	buildSlashInitialResult,
+	failSlashResult,
+	finalizeSlashResult,
 } from "./slash-live-state.ts";
 import {
-  SLASH_RESULT_TYPE,
-  SLASH_SUBAGENT_CANCEL_EVENT,
-  SLASH_SUBAGENT_REQUEST_EVENT,
-  SLASH_SUBAGENT_RESPONSE_EVENT,
-  SLASH_SUBAGENT_STARTED_EVENT,
-  SLASH_SUBAGENT_UPDATE_EVENT,
-  type SingleResult,
-  type SubagentState,
+	SLASH_RESULT_TYPE,
+	SLASH_SUBAGENT_CANCEL_EVENT,
+	SLASH_SUBAGENT_REQUEST_EVENT,
+	SLASH_SUBAGENT_RESPONSE_EVENT,
+	SLASH_SUBAGENT_STARTED_EVENT,
+	SLASH_SUBAGENT_UPDATE_EVENT,
+	type SingleResult,
+	type SubagentState,
 } from "../shared/types.ts";
 
 interface InlineConfig {
-  output?: string | false;
-  outputMode?: "inline" | "file-only";
-  reads?: string[] | false;
-  model?: string;
-  skill?: string[] | false;
-  progress?: boolean;
+	output?: string | false;
+	outputMode?: "inline" | "file-only";
+	reads?: string[] | false;
+	model?: string;
+	skill?: string[] | false;
+	progress?: boolean;
 }
 
 const parseInlineConfig = (raw: string): InlineConfig => {
-  const config: InlineConfig = {};
-  for (const part of raw.split(",")) {
-    const trimmed = part.trim();
-    if (!trimmed) continue;
-    const eq = trimmed.indexOf("=");
-    if (eq === -1) {
-      if (trimmed === "progress") config.progress = true;
-      continue;
-    }
-    const key = trimmed.slice(0, eq).trim();
-    const val = trimmed.slice(eq + 1).trim();
-    switch (key) {
-      case "output":
-        config.output = val === "false" ? false : val;
-        break;
-      case "outputMode":
-        if (val === "inline" || val === "file-only") config.outputMode = val;
-        break;
-      case "reads":
-        config.reads = val === "false" ? false : val.split("+").filter(Boolean);
-        break;
-      case "model":
-        config.model = val || undefined;
-        break;
-      case "skill":
-      case "skills":
-        config.skill = val === "false" ? false : val.split("+").filter(Boolean);
-        break;
-      case "progress":
-        config.progress = val !== "false";
-        break;
-    }
-  }
-  return config;
+	const config: InlineConfig = {};
+	for (const part of raw.split(",")) {
+		const trimmed = part.trim();
+		if (!trimmed) continue;
+		const eq = trimmed.indexOf("=");
+		if (eq === -1) {
+			if (trimmed === "progress") config.progress = true;
+			continue;
+		}
+		const key = trimmed.slice(0, eq).trim();
+		const val = trimmed.slice(eq + 1).trim();
+		switch (key) {
+			case "output": config.output = val === "false" ? false : val; break;
+			case "outputMode": if (val === "inline" || val === "file-only") config.outputMode = val; break;
+			case "reads": config.reads = val === "false" ? false : val.split("+").filter(Boolean); break;
+			case "model": config.model = val || undefined; break;
+			case "skill": case "skills": config.skill = val === "false" ? false : val.split("+").filter(Boolean); break;
+			case "progress": config.progress = val !== "false"; break;
+		}
+	}
+	return config;
 };
 
-const parseAgentToken = (
-  token: string,
-): { name: string; config: InlineConfig } => {
-  const bracket = token.indexOf("[");
-  if (bracket === -1) return { name: token, config: {} };
-  const end = token.lastIndexOf("]");
-  return {
-    name: token.slice(0, bracket),
-    config: parseInlineConfig(
-      token.slice(bracket + 1, end !== -1 ? end : undefined),
-    ),
-  };
+const parseAgentToken = (token: string): { name: string; config: InlineConfig } => {
+	const bracket = token.indexOf("[");
+	if (bracket === -1) return { name: token, config: {} };
+	const end = token.lastIndexOf("]");
+	return { name: token.slice(0, bracket), config: parseInlineConfig(token.slice(bracket + 1, end !== -1 ? end : undefined)) };
 };
 
-const extractExecutionFlags = (
-  rawArgs: string,
-): { args: string; bg: boolean; fork: boolean } => {
-  let args = rawArgs.trim();
-  let bg = false;
-  let fork = false;
-
-  while (true) {
-    if (args.endsWith(" --bg") || args === "--bg") {
-      bg = true;
-      args = args === "--bg" ? "" : args.slice(0, -5).trim();
-      continue;
-    }
-    if (args.endsWith(" --fork") || args === "--fork") {
-      fork = true;
-      args = args === "--fork" ? "" : args.slice(0, -7).trim();
-      continue;
-    }
-    break;
-  }
-
-  return { args, bg, fork };
+const extractExecutionFlags = (rawArgs: string): { args: string; bg: boolean; fork: boolean } => {
+	let args = rawArgs.trim();
+	let bg = false;
+	let fork = false;
+
+	while (true) {
+		if (args.endsWith(" --bg") || args === "--bg") {
+			bg = true;
+			args = args === "--bg" ? "" : args.slice(0, -5).trim();
+			continue;
+		}
+		if (args.endsWith(" --fork") || args === "--fork") {
+			fork = true;
+			args = args === "--fork" ? "" : args.slice(0, -7).trim();
+			continue;
+		}
+		break;
+	}
+
+	return { args, bg, fork };
 };
 
-const makeAgentCompletions =
-  (state: SubagentState, multiAgent: boolean) => (prefix: string) => {
-    if (!state.baseCwd) return null;
-    const agents = discoverAgents(state.baseCwd, "both").agents;
-    if (!multiAgent) {
-      if (prefix.includes(" ")) return null;
-      return agents
-        .filter((a) => a.name.startsWith(prefix))
-        .map((a) => ({ value: a.name, label: a.name }));
-    }
-
-    const lastArrow = prefix.lastIndexOf(" -> ");
-    const segment = lastArrow !== -1 ? prefix.slice(lastArrow + 4) : prefix;
-    if (
-      segment.includes(" -- ") ||
-      segment.includes('"') ||
-      segment.includes("'")
-    )
-      return null;
-
-    const lastWord = (prefix.match(/(\S*)$/) || ["", ""])[1];
-    const beforeLastWord = prefix.slice(0, prefix.length - lastWord.length);
-
-    if (lastWord === "->") {
-      return agents.map((a) => ({
-        value: `${prefix} ${a.name}`,
-        label: a.name,
-      }));
-    }
-
-    return agents
-      .filter((a) => a.name.startsWith(lastWord))
-      .map((a) => ({ value: `${beforeLastWord}${a.name}`, label: a.name }));
-  };
+const makeAgentCompletions = (state: SubagentState, multiAgent: boolean) => (prefix: string) => {
+	if (!state.baseCwd) return null;
+	const agents = discoverAgents(state.baseCwd, "both").agents;
+	if (!multiAgent) {
+		if (prefix.includes(" ")) return null;
+		return agents.filter((a) => a.name.startsWith(prefix)).map((a) => ({ value: a.name, label: a.name }));
+	}
+
+	const lastArrow = prefix.lastIndexOf(" -> ");
+	const segment = lastArrow !== -1 ? prefix.slice(lastArrow + 4) : prefix;
+	if (segment.includes(" -- ") || segment.includes('"') || segment.includes("'")) return null;
+
+	const lastWord = (prefix.match(/(\S*)$/) || ["", ""])[1];
+	const beforeLastWord = prefix.slice(0, prefix.length - lastWord.length);
+
+	if (lastWord === "->") {
+		return agents.map((a) => ({ value: `${prefix} ${a.name}`, label: a.name }));
+	}
+
+	return agents.filter((a) => a.name.startsWith(lastWord)).map((a) => ({ value: `${beforeLastWord}${a.name}`, label: a.name }));
+};
 
 const discoverSavedChains = (cwd: string): ChainConfig[] => {
-  const chainsByName = new Map<string, ChainConfig>();
-  for (const chain of discoverAgentsAll(cwd).chains) {
-    chainsByName.set(chain.name, chain);
-  }
-  return Array.from(chainsByName.values());
+	const chainsByName = new Map<string, ChainConfig>();
+	for (const chain of discoverAgentsAll(cwd).chains) {
+		chainsByName.set(chain.name, chain);
+	}
+	return Array.from(chainsByName.values());
 };
 
 const makeChainCompletions = (state: SubagentState) => (prefix: string) => {
-  if (prefix.includes(" ") || !state.baseCwd) return null;
-  return discoverSavedChains(state.baseCwd)
-    .filter((chain) => chain.name.startsWith(prefix))
-    .map((chain) => ({ value: chain.name, label: chain.name }));
+	if (prefix.includes(" ") || !state.baseCwd) return null;
+	return discoverSavedChains(state.baseCwd)
+		.filter((chain) => chain.name.startsWith(prefix))
+		.map((chain) => ({ value: chain.name, label: chain.name }));
 };
 
-const mapSavedChainSteps = (
-  chain: ChainConfig,
-  worktree = false,
-): ChainStep[] => {
-  return (chain.steps as Array<ChainStep & { skills?: string[] | false }>).map(
-    (step) => {
-      if (isParallelStep(step))
-        return worktree ? { ...step, worktree: true } : { ...step };
-      return {
-        agent: step.agent,
-        task: step.task || undefined,
-        output: step.output,
-        outputMode: step.outputMode,
-        reads: step.reads,
-        progress: step.progress,
-        skill: step.skill ?? step.skills,
-        model: step.model,
-      };
-    },
-  );
+const mapSavedChainSteps = (chain: ChainConfig, worktree = false): ChainStep[] => {
+	return (chain.steps as Array<ChainStep & { skills?: string[] | false }>).map((step) => {
+		if (isParallelStep(step)) return worktree ? { ...step, worktree: true } : { ...step };
+		return {
+			agent: step.agent,
+			task: step.task || undefined,
+			output: step.output,
+			outputMode: step.outputMode,
+			reads: step.reads,
+			progress: step.progress,
+			skill: step.skill ?? step.skills,
+			model: step.model,
+		};
+	});
 };
 
 async function requestSlashRun(
-  pi: ExtensionAPI,
-  ctx: ExtensionContext,
-  requestId: string,
-  params: SubagentParamsLike,
+	pi: ExtensionAPI,
+	ctx: ExtensionContext,
+	requestId: string,
+	params: SubagentParamsLike,
 ): Promise<SlashSubagentResponse> {
-  return new Promise((resolve, reject) => {
-    let done = false;
-    let started = false;
-
-    const startTimeoutMs = 15_000;
-    const startTimeout = setTimeout(() => {
-      finish(() =>
-        reject(
-          new Error(
-            "Slash subagent bridge did not start within 15s. Ensure the extension is loaded correctly.",
-          ),
-        ),
-      );
-    }, startTimeoutMs);
-
-    const onStarted = (data: unknown) => {
-      if (done || !data || typeof data !== "object") return;
-      if ((data as { requestId?: unknown }).requestId !== requestId) return;
-      started = true;
-      clearTimeout(startTimeout);
-      if (ctx.hasUI) ctx.ui.setStatus("subagent-slash", "running...");
-    };
-
-    const onResponse = (data: unknown) => {
-      if (done || !data || typeof data !== "object") return;
-      const response = data as Partial<SlashSubagentResponse>;
-      if (response.requestId !== requestId) return;
-      clearTimeout(startTimeout);
-      finish(() => resolve(response as SlashSubagentResponse));
-    };
-
-    const onUpdate = (data: unknown) => {
-      if (done || !data || typeof data !== "object") return;
-      const update = data as SlashSubagentUpdate;
-      if (update.requestId !== requestId) return;
-      applySlashUpdate(requestId, update);
-      if (!ctx.hasUI) return;
-      const tool = update.currentTool ? ` ${update.currentTool}` : "";
-      const count = update.toolCount ?? 0;
-      ctx.ui.setStatus(
-        "subagent-slash",
-        `${count} tools${tool} | ctrl+o live detail`,
-      );
-    };
-
-    const onTerminalInput = ctx.hasUI
-      ? ctx.ui.onTerminalInput((input) => {
-          if (!matchesKey(input, Key.escape)) return undefined;
-          pi.events.emit(SLASH_SUBAGENT_CANCEL_EVENT, { requestId });
-          finish(() => reject(new Error("Cancelled")));
-          return { consume: true };
-        })
-      : undefined;
-
-    const unsubStarted = pi.events.on(SLASH_SUBAGENT_STARTED_EVENT, onStarted);
-    const unsubResponse = pi.events.on(
-      SLASH_SUBAGENT_RESPONSE_EVENT,
-      onResponse,
-    );
-    const unsubUpdate = pi.events.on(SLASH_SUBAGENT_UPDATE_EVENT, onUpdate);
-
-    const finish = (next: () => void) => {
-      if (done) return;
-      done = true;
-      clearTimeout(startTimeout);
-      unsubStarted();
-      unsubResponse();
-      unsubUpdate();
-      onTerminalInput?.();
-      next();
-    };
-
-    pi.events.emit(SLASH_SUBAGENT_REQUEST_EVENT, { requestId, params });
-
-    // Bridge emits STARTED synchronously during REQUEST emit.
-    // If not started, no bridge received the request.
-    if (!started && done) return;
-    if (!started) {
-      finish(() =>
-        reject(
-          new Error(
-            "No slash subagent bridge responded. Ensure the subagent extension is loaded correctly.",
-          ),
-        ),
-      );
-    }
-  });
+	return new Promise((resolve, reject) => {
+		let done = false;
+		let started = false;
+
+		const startTimeoutMs = 15_000;
+		const startTimeout = setTimeout(() => {
+			finish(() => reject(new Error(
+				"Slash subagent bridge did not start within 15s. Ensure the extension is loaded correctly.",
+			)));
+		}, startTimeoutMs);
+
+		const onStarted = (data: unknown) => {
+			if (done || !data || typeof data !== "object") return;
+			if ((data as { requestId?: unknown }).requestId !== requestId) return;
+			started = true;
+			clearTimeout(startTimeout);
+			if (ctx.hasUI) ctx.ui.setStatus("subagent-slash", "running...");
+		};
+
+		const onResponse = (data: unknown) => {
+			if (done || !data || typeof data !== "object") return;
+			const response = data as Partial<SlashSubagentResponse>;
+			if (response.requestId !== requestId) return;
+			clearTimeout(startTimeout);
+			finish(() => resolve(response as SlashSubagentResponse));
+		};
+
+		const onUpdate = (data: unknown) => {
+			if (done || !data || typeof data !== "object") return;
+			const update = data as SlashSubagentUpdate;
+			if (update.requestId !== requestId) return;
+			applySlashUpdate(requestId, update);
+			if (!ctx.hasUI) return;
+			const tool = update.currentTool ? ` ${update.currentTool}` : "";
+			const count = update.toolCount ?? 0;
+			ctx.ui.setStatus("subagent-slash", `${count} tools${tool} | ctrl+o live detail`);
+		};
+
+		const onTerminalInput = ctx.hasUI
+			? ctx.ui.onTerminalInput((input) => {
+				if (!matchesKey(input, Key.escape)) return undefined;
+				pi.events.emit(SLASH_SUBAGENT_CANCEL_EVENT, { requestId });
+				finish(() => reject(new Error("Cancelled")));
+				return { consume: true };
+			})
+			: undefined;
+
+		const unsubStarted = pi.events.on(SLASH_SUBAGENT_STARTED_EVENT, onStarted);
+		const unsubResponse = pi.events.on(SLASH_SUBAGENT_RESPONSE_EVENT, onResponse);
+		const unsubUpdate = pi.events.on(SLASH_SUBAGENT_UPDATE_EVENT, onUpdate);
+
+		const finish = (next: () => void) => {
+			if (done) return;
+			done = true;
+			clearTimeout(startTimeout);
+			unsubStarted();
+			unsubResponse();
+			unsubUpdate();
+			onTerminalInput?.();
+			next();
+		};
+
+		pi.events.emit(SLASH_SUBAGENT_REQUEST_EVENT, { requestId, params });
+
+		// Bridge emits STARTED synchronously during REQUEST emit.
+		// If not started, no bridge received the request.
+		if (!started && done) return;
+		if (!started) {
+			finish(() => reject(new Error(
+				"No slash subagent bridge responded. Ensure the subagent extension is loaded correctly.",
+			)));
+		}
+	});
 }
 
-function extractSlashMessageText(
-  content: string | Array<{ type?: string; text?: string }>,
-): string {
-  if (typeof content === "string") return content;
-  if (!Array.isArray(content)) return "";
-  return content
-    .filter(
-      (part): part is { type: "text"; text: string } =>
-        part?.type === "text" && typeof part.text === "string",
-    )
-    .map((part) => part.text)
-    .join("\n");
+function extractSlashMessageText(content: string | Array<{ type?: string; text?: string }>): string {
+	if (typeof content === "string") return content;
+	if (!Array.isArray(content)) return "";
+	return content
+		.filter((part): part is { type: "text"; text: string } => part?.type === "text" && typeof part.text === "string")
+		.map((part) => part.text)
+		.join("\n");
 }
 
 function formatExportPathList(paths: string[]): string {
-  return paths.map((file) => `- \`${file}\``).join("\n");
+	return paths.map((file) => `- \`${file}\``).join("\n");
 }
 
-function collectResultPaths(
-  results: SingleResult[],
-  getPath: (result: SingleResult) => string | undefined,
-): string[] {
-  return results
-    .map(getPath)
-    .filter(
-      (file): file is string => typeof file === "string" && file.length > 0,
-    );
+function collectResultPaths(results: SingleResult[], getPath: (result: SingleResult) => string | undefined): string[] {
+	return results
+		.map(getPath)
+		.filter((file): file is string => typeof file === "string" && file.length > 0);
 }
 
 function buildSlashExportText(response: SlashSubagentResponse): string {
-  const output =
-    extractSlashMessageText(response.result.content) ||
-    response.errorText ||
-    "(no output)";
-  const results = response.result.details?.results ?? [];
-  const sessionFiles = collectResultPaths(
-    results,
-    (result) => result.sessionFile,
-  );
-  const savedOutputs = collectResultPaths(
-    results,
-    (result) => result.savedOutputPath,
-  );
-  const artifactOutputs = collectResultPaths(
-    results,
-    (result) => result.artifactPaths?.outputPath,
-  );
-  const sections = ["## Subagent result", output];
-  if (sessionFiles.length > 0)
-    sections.push(
-      "## Child session exports",
-      formatExportPathList(sessionFiles),
-    );
-  if (savedOutputs.length > 0)
-    sections.push("## Saved outputs", formatExportPathList(savedOutputs));
-  if (artifactOutputs.length > 0)
-    sections.push("## Artifact outputs", formatExportPathList(artifactOutputs));
-  return sections.join("\n\n");
+	const output = extractSlashMessageText(response.result.content) || response.errorText || "(no output)";
+	const results = response.result.details?.results ?? [];
+	const sessionFiles = collectResultPaths(results, (result) => result.sessionFile);
+	const savedOutputs = collectResultPaths(results, (result) => result.savedOutputPath);
+	const artifactOutputs = collectResultPaths(results, (result) => result.artifactPaths?.outputPath);
+	const sections = ["## Subagent result", output];
+	if (sessionFiles.length > 0) sections.push("## Child session exports", formatExportPathList(sessionFiles));
+	if (savedOutputs.length > 0) sections.push("## Saved outputs", formatExportPathList(savedOutputs));
+	if (artifactOutputs.length > 0) sections.push("## Artifact outputs", formatExportPathList(artifactOutputs));
+	return sections.join("\n\n");
 }
 
 function persistSlashSessionSnapshot(ctx: ExtensionContext): void {
-  try {
-    if (!ctx.sessionManager) return;
-    const sessionManager = ctx.sessionManager as typeof ctx.sessionManager & {
-      _rewriteFile?: () => void;
-      flushed?: boolean;
-    };
-    const sessionFile = sessionManager.getSessionFile();
-    if (!sessionFile || typeof sessionManager._rewriteFile !== "function")
-      return;
-    fs.mkdirSync(path.dirname(sessionFile), { recursive: true });
-    sessionManager._rewriteFile();
-    sessionManager.flushed = true;
-  } catch (error) {
-    console.error(
-      "Failed to persist slash session snapshot for export:",
-      error,
-    );
-  }
+	try {
+		if (!ctx.sessionManager) return;
+		const sessionManager = ctx.sessionManager as typeof ctx.sessionManager & {
+			_rewriteFile?: () => void;
+			flushed?: boolean;
+		};
+		const sessionFile = sessionManager.getSessionFile();
+		if (!sessionFile || typeof sessionManager._rewriteFile !== "function") return;
+		fs.mkdirSync(path.dirname(sessionFile), { recursive: true });
+		sessionManager._rewriteFile();
+		sessionManager.flushed = true;
+	} catch (error) {
+		console.error("Failed to persist slash session snapshot for export:", error);
+	}
 }
 
 async function runSlashSubagent(
-  pi: ExtensionAPI,
-  ctx: ExtensionContext,
-  params: SubagentParamsLike,
+	pi: ExtensionAPI,
+	ctx: ExtensionContext,
+	params: SubagentParamsLike,
 ): Promise<void> {
-  if (ctx.hasUI) ctx.ui.setToolsExpanded(false);
-  const requestId = randomUUID();
-  const initialDetails = buildSlashInitialResult(requestId, params);
-  const initialText =
-    extractSlashMessageText(initialDetails.result.content) ||
-    "Running subagent...";
-  pi.sendMessage({
-    customType: SLASH_RESULT_TYPE,
-    content: initialText,
-    display: true,
-    details: initialDetails,
-  });
-  persistSlashSessionSnapshot(ctx);
-
-  try {
-    const response = await requestSlashRun(pi, ctx, requestId, params);
-    const finalDetails = finalizeSlashResult(response);
-    pi.sendMessage({
-      customType: SLASH_RESULT_TYPE,
-      content: buildSlashExportText(response),
-      display: true,
-      details: finalDetails,
-    });
-    persistSlashSessionSnapshot(ctx);
-    if (ctx.hasUI) {
-      ctx.ui.setStatus("subagent-slash", undefined);
-    }
-    if (response.isError && ctx.hasUI) {
-      ctx.ui.notify(response.errorText || "Subagent failed", "error");
-    }
-  } catch (error) {
-    const message = error instanceof Error ? error.message : String(error);
-    const failedDetails = failSlashResult(requestId, params, message);
-    pi.sendMessage({
-      customType: SLASH_RESULT_TYPE,
-      content: `## Subagent result\n\n${message}`,
-      display: true,
-      details: failedDetails,
-    });
-    persistSlashSessionSnapshot(ctx);
-    if (ctx.hasUI) {
-      ctx.ui.setStatus("subagent-slash", undefined);
-    }
-    if (message === "Cancelled") {
-      if (ctx.hasUI) ctx.ui.notify("Cancelled", "warning");
-      return;
-    }
-    if (ctx.hasUI) ctx.ui.notify(message, "error");
-  }
+	if (ctx.hasUI) ctx.ui.setToolsExpanded(false);
+	const requestId = randomUUID();
+	const initialDetails = buildSlashInitialResult(requestId, params);
+	const initialText = extractSlashMessageText(initialDetails.result.content) || "Running subagent...";
+	pi.sendMessage({
+		customType: SLASH_RESULT_TYPE,
+		content: initialText,
+		display: true,
+		details: initialDetails,
+	});
+	persistSlashSessionSnapshot(ctx);
+
+	try {
+		const response = await requestSlashRun(pi, ctx, requestId, params);
+		const finalDetails = finalizeSlashResult(response);
+		pi.sendMessage({
+			customType: SLASH_RESULT_TYPE,
+			content: buildSlashExportText(response),
+			display: true,
+			details: finalDetails,
+		});
+		persistSlashSessionSnapshot(ctx);
+		if (ctx.hasUI) {
+			ctx.ui.setStatus("subagent-slash", undefined);
+		}
+		if (response.isError && ctx.hasUI) {
+			ctx.ui.notify(response.errorText || "Subagent failed", "error");
+		}
+	} catch (error) {
+		const message = error instanceof Error ? error.message : String(error);
+		const failedDetails = failSlashResult(requestId, params, message);
+		pi.sendMessage({
+			customType: SLASH_RESULT_TYPE,
+			content: `## Subagent result\n\n${message}`,
+			display: true,
+			details: failedDetails,
+		});
+		persistSlashSessionSnapshot(ctx);
+		if (ctx.hasUI) {
+			ctx.ui.setStatus("subagent-slash", undefined);
+		}
+		if (message === "Cancelled") {
+			if (ctx.hasUI) ctx.ui.notify("Cancelled", "warning");
+			return;
+		}
+		if (ctx.hasUI) ctx.ui.notify(message, "error");
+	}
 }
 
-interface ParsedStep {
-  name: string;
-  config: InlineConfig;
-  task?: string;
-}
+
+interface ParsedStep { name: string; config: InlineConfig; task?: string }
 
 const parseAgentArgs = (
-  state: SubagentState,
-  args: string,
-  command: string,
-  ctx: ExtensionContext,
+	state: SubagentState,
+	args: string,
+	command: string,
+	ctx: ExtensionContext,
 ): { steps: ParsedStep[]; task: string } | null => {
-  const input = args.trim();
-  const usage = `Usage: /${command} agent1 "task1" -> agent2 "task2"`;
-  let steps: ParsedStep[];
-  let sharedTask: string;
-  let perStep = false;
-
-  if (input.includes(" -> ")) {
-    perStep = true;
-    const segments = input.split(" -> ");
-    steps = [];
-    for (const seg of segments) {
-      const trimmed = seg.trim();
-      if (!trimmed) continue;
-      let agentPart: string;
-      let task: string | undefined;
-      const qMatch = trimmed.match(
-        /^(\S+(?:\[[^\]]*\])?)\s+(?:"([^"]*)"|'([^']*)')$/,
-      );
-      if (qMatch) {
-        agentPart = qMatch[1]!;
-        task = (qMatch[2] ?? qMatch[3]) || undefined;
-      } else {
-        const dashIdx = trimmed.indexOf(" -- ");
-        if (dashIdx !== -1) {
-          agentPart = trimmed.slice(0, dashIdx).trim();
-          task = trimmed.slice(dashIdx + 4).trim() || undefined;
-        } else {
-          agentPart = trimmed;
-        }
-      }
-      const parsed = parseAgentToken(agentPart);
-      steps.push({ ...parsed, task });
-    }
-    sharedTask = steps.find((s) => s.task)?.task ?? "";
-  } else {
-    const delimiterIndex = input.indexOf(" -- ");
-    if (delimiterIndex === -1) {
-      ctx.ui.notify(usage, "error");
-      return null;
-    }
-    const agentsPart = input.slice(0, delimiterIndex).trim();
-    sharedTask = input.slice(delimiterIndex + 4).trim();
-    if (!agentsPart || !sharedTask) {
-      ctx.ui.notify(usage, "error");
-      return null;
-    }
-    steps = agentsPart
-      .split(/\s+/)
-      .filter(Boolean)
-      .map((t) => parseAgentToken(t));
-  }
-
-  if (steps.length === 0) {
-    ctx.ui.notify(usage, "error");
-    return null;
-  }
-  if (!state.baseCwd) {
-    ctx.ui.notify("Subagent session cwd is not initialized yet", "error");
-    return null;
-  }
-  const agents = discoverAgents(state.baseCwd, "both").agents;
-  for (const step of steps) {
-    if (!agents.find((a) => a.name === step.name)) {
-      ctx.ui.notify(`Unknown agent: ${step.name}`, "error");
-      return null;
-    }
-  }
-  if (command === "chain" && !steps[0]?.task && (perStep || !sharedTask)) {
-    ctx.ui.notify(
-      `First step must have a task: /chain agent "task" -> agent2`,
-      "error",
-    );
-    return null;
-  }
-  if (command === "parallel" && !steps.some((s) => s.task) && !sharedTask) {
-    ctx.ui.notify("At least one step must have a task", "error");
-    return null;
-  }
-  return { steps, task: sharedTask };
+	const input = args.trim();
+	const usage = `Usage: /${command} agent1 "task1" -> agent2 "task2"`;
+	let steps: ParsedStep[];
+	let sharedTask: string;
+	let perStep = false;
+
+	if (input.includes(" -> ")) {
+		perStep = true;
+		const segments = input.split(" -> ");
+		steps = [];
+		for (const seg of segments) {
+			const trimmed = seg.trim();
+			if (!trimmed) continue;
+			let agentPart: string;
+			let task: string | undefined;
+			const qMatch = trimmed.match(/^(\S+(?:\[[^\]]*\])?)\s+(?:"([^"]*)"|'([^']*)')$/);
+			if (qMatch) {
+				agentPart = qMatch[1]!;
+				task = (qMatch[2] ?? qMatch[3]) || undefined;
+			} else {
+				const dashIdx = trimmed.indexOf(" -- ");
+				if (dashIdx !== -1) {
+					agentPart = trimmed.slice(0, dashIdx).trim();
+					task = trimmed.slice(dashIdx + 4).trim() || undefined;
+				} else {
+					agentPart = trimmed;
+				}
+			}
+			const parsed = parseAgentToken(agentPart);
+			steps.push({ ...parsed, task });
+		}
+		sharedTask = steps.find((s) => s.task)?.task ?? "";
+	} else {
+		const delimiterIndex = input.indexOf(" -- ");
+		if (delimiterIndex === -1) {
+			ctx.ui.notify(usage, "error");
+			return null;
+		}
+		const agentsPart = input.slice(0, delimiterIndex).trim();
+		sharedTask = input.slice(delimiterIndex + 4).trim();
+		if (!agentsPart || !sharedTask) {
+			ctx.ui.notify(usage, "error");
+			return null;
+		}
+		steps = agentsPart.split(/\s+/).filter(Boolean).map((t) => parseAgentToken(t));
+	}
+
+	if (steps.length === 0) {
+		ctx.ui.notify(usage, "error");
+		return null;
+	}
+	if (!state.baseCwd) {
+		ctx.ui.notify("Subagent session cwd is not initialized yet", "error");
+		return null;
+	}
+	const agents = discoverAgents(state.baseCwd, "both").agents;
+	for (const step of steps) {
+		if (!agents.find((a) => a.name === step.name)) {
+			ctx.ui.notify(`Unknown agent: ${step.name}`, "error");
+			return null;
+		}
+	}
+	if (command === "chain" && !steps[0]?.task && (perStep || !sharedTask)) {
+		ctx.ui.notify(`First step must have a task: /chain agent "task" -> agent2`, "error");
+		return null;
+	}
+	if (command === "parallel" && !steps.some((s) => s.task) && !sharedTask) {
+		ctx.ui.notify("At least one step must have a task", "error");
+		return null;
+	}
+	return { steps, task: sharedTask };
 };
 
 export function registerSlashCommands(
-  pi: ExtensionAPI,
-  state: SubagentState,
+	pi: ExtensionAPI,
+	state: SubagentState,
 ): void {
-  pi.registerCommand("run", {
-    description:
-      "Run a subagent directly: /run agent[output=file] [task] [--bg] [--fork]",
-    getArgumentCompletions: makeAgentCompletions(state, false),
-    handler: async (args, ctx) => {
-      const { args: cleanedArgs, bg, fork } = extractExecutionFlags(args);
-      const input = cleanedArgs.trim();
-      const firstSpace = input.indexOf(" ");
-      if (!input) {
-        ctx.ui.notify("Usage: /run <agent> [task] [--bg] [--fork]", "error");
-        return;
-      }
-      const { name: agentName, config: inline } = parseAgentToken(
-        firstSpace === -1 ? input : input.slice(0, firstSpace),
-      );
-      const task = firstSpace === -1 ? "" : input.slice(firstSpace + 1).trim();
-
-      if (!state.baseCwd) {
-        ctx.ui.notify("Subagent session cwd is not initialized yet", "error");
-        return;
-      }
-      const agents = discoverAgents(state.baseCwd, "both").agents;
-      if (!agents.find((a) => a.name === agentName)) {
-        ctx.ui.notify(`Unknown agent: ${agentName}`, "error");
-        return;
-      }
-
-      let finalTask = task;
-      if (
-        inline.reads &&
-        Array.isArray(inline.reads) &&
-        inline.reads.length > 0
-      ) {
-        finalTask = `[Read from: ${inline.reads.join(", ")}]\n\n${finalTask}`;
-      }
-      const params: SubagentParamsLike = {
-        agent: agentName,
-        task: finalTask,
-        clarify: false,
-        agentScope: "both",
-      };
-      if (inline.output !== undefined) params.output = inline.output;
-      if (inline.outputMode !== undefined)
-        params.outputMode = inline.outputMode;
-      if (inline.skill !== undefined) params.skill = inline.skill;
-      if (inline.model) params.model = inline.model;
-      if (bg) params.async = true;
-      if (fork) params.context = "fork";
-      await runSlashSubagent(pi, ctx, params);
-    },
-  });
-
-  pi.registerCommand("chain", {
-    description:
-      'Run agents in sequence: /chain scout "task" -> planner [--bg] [--fork]',
-    getArgumentCompletions: makeAgentCompletions(state, true),
-    handler: async (args, ctx) => {
-      const { args: cleanedArgs, bg, fork } = extractExecutionFlags(args);
-      const parsed = parseAgentArgs(state, cleanedArgs, "chain", ctx);
-      if (!parsed) return;
-      const chain = parsed.steps.map(({ name, config, task: stepTask }, i) => ({
-        agent: name,
-        ...(stepTask
-          ? { task: stepTask }
-          : i === 0 && parsed.task
-            ? { task: parsed.task }
-            : {}),
-        ...(config.output !== undefined ? { output: config.output } : {}),
-        ...(config.outputMode !== undefined
-          ? { outputMode: config.outputMode }
-          : {}),
-        ...(config.reads !== undefined ? { reads: config.reads } : {}),
-        ...(config.model ? { model: config.model } : {}),
-        ...(config.skill !== undefined ? { skill: config.skill } : {}),
-        ...(config.progress !== undefined ? { progress: config.progress } : {}),
-      }));
-      const params: SubagentParamsLike = {
-        chain,
-        task: parsed.task,
-        clarify: false,
-        agentScope: "both",
-      };
-      if (bg) params.async = true;
-      if (fork) params.context = "fork";
-      await runSlashSubagent(pi, ctx, params);
-    },
-  });
-
-  pi.registerCommand("run-chain", {
-    description:
-      "Run a saved chain: /run-chain chainName -- task [--bg] [--fork]",
-    getArgumentCompletions: makeChainCompletions(state),
-    handler: async (args, ctx) => {
-      const { args: cleanedArgs, bg, fork } = extractExecutionFlags(args);
-      const delimiterIndex = cleanedArgs.indexOf(" -- ");
-      const usage = "Usage: /run-chain <chainName> -- <task> [--bg] [--fork]";
-      if (delimiterIndex === -1) {
-        ctx.ui.notify(usage, "error");
-        return;
-      }
-      const chainName = cleanedArgs.slice(0, delimiterIndex).trim();
-      const task = cleanedArgs.slice(delimiterIndex + 4).trim();
-      if (!chainName || !task) {
-        ctx.ui.notify(usage, "error");
-        return;
-      }
-      if (!state.baseCwd) {
-        ctx.ui.notify("Subagent session cwd is not initialized yet", "error");
-        return;
-      }
-      const chain = discoverSavedChains(state.baseCwd).find(
-        (candidate) => candidate.name === chainName,
-      );
-      if (!chain) {
-        ctx.ui.notify(`Unknown chain: ${chainName}`, "error");
-        return;
-      }
-      const params: SubagentParamsLike = {
-        chain: mapSavedChainSteps(chain),
-        task,
-        clarify: false,
-        agentScope: "both",
-      };
-      if (bg) params.async = true;
-      if (fork) params.context = "fork";
-      await runSlashSubagent(pi, ctx, params);
-    },
-  });
-
-  pi.registerCommand("parallel", {
-    description:
-      'Run agents in parallel: /parallel scout "task1" -> reviewer "task2" [--bg] [--fork]',
-    getArgumentCompletions: makeAgentCompletions(state, true),
-    handler: async (args, ctx) => {
-      const { args: cleanedArgs, bg, fork } = extractExecutionFlags(args);
-      const parsed = parseAgentArgs(state, cleanedArgs, "parallel", ctx);
-      if (!parsed) return;
-      const tasks = parsed.steps.map(({ name, config, task: stepTask }) => ({
-        agent: name,
-        task: stepTask ?? parsed.task,
-        ...(config.output !== undefined ? { output: config.output } : {}),
-        ...(config.outputMode !== undefined
-          ? { outputMode: config.outputMode }
-          : {}),
-        ...(config.reads !== undefined ? { reads: config.reads } : {}),
-        ...(config.model ? { model: config.model } : {}),
-        ...(config.skill !== undefined ? { skill: config.skill } : {}),
-        ...(config.progress !== undefined ? { progress: config.progress } : {}),
-      }));
-      const params: SubagentParamsLike = {
-        tasks,
-        clarify: false,
-        agentScope: "both",
-      };
-      if (bg) params.async = true;
-      if (fork) params.context = "fork";
-      await runSlashSubagent(pi, ctx, params);
-    },
-  });
-
-  pi.registerCommand("subagents-doctor", {
-    description: "Show subagent diagnostics",
-    handler: async (_args, ctx) => {
-      await runSlashSubagent(pi, ctx, { action: "doctor" });
-    },
-  });
+	pi.registerCommand("run", {
+		description: "Run a subagent directly: /run agent[output=file] [task] [--bg] [--fork]",
+		getArgumentCompletions: makeAgentCompletions(state, false),
+		handler: async (args, ctx) => {
+			const { args: cleanedArgs, bg, fork } = extractExecutionFlags(args);
+			const input = cleanedArgs.trim();
+			const firstSpace = input.indexOf(" ");
+			if (!input) { ctx.ui.notify("Usage: /run <agent> [task] [--bg] [--fork]", "error"); return; }
+			const { name: agentName, config: inline } = parseAgentToken(firstSpace === -1 ? input : input.slice(0, firstSpace));
+			const task = firstSpace === -1 ? "" : input.slice(firstSpace + 1).trim();
+
+			if (!state.baseCwd) { ctx.ui.notify("Subagent session cwd is not initialized yet", "error"); return; }
+			const agents = discoverAgents(state.baseCwd, "both").agents;
+			if (!agents.find((a) => a.name === agentName)) { ctx.ui.notify(`Unknown agent: ${agentName}`, "error"); return; }
+
+			let finalTask = task;
+			if (inline.reads && Array.isArray(inline.reads) && inline.reads.length > 0) {
+				finalTask = `[Read from: ${inline.reads.join(", ")}]\n\n${finalTask}`;
+			}
+			const params: SubagentParamsLike = { agent: agentName, task: finalTask, clarify: false, agentScope: "both" };
+			if (inline.output !== undefined) params.output = inline.output;
+			if (inline.outputMode !== undefined) params.outputMode = inline.outputMode;
+			if (inline.skill !== undefined) params.skill = inline.skill;
+			if (inline.model) params.model = inline.model;
+			if (bg) params.async = true;
+			if (fork) params.context = "fork";
+			await runSlashSubagent(pi, ctx, params);
+		},
+	});
+
+	pi.registerCommand("chain", {
+		description: "Run agents in sequence: /chain scout \"task\" -> planner [--bg] [--fork]",
+		getArgumentCompletions: makeAgentCompletions(state, true),
+		handler: async (args, ctx) => {
+			const { args: cleanedArgs, bg, fork } = extractExecutionFlags(args);
+			const parsed = parseAgentArgs(state, cleanedArgs, "chain", ctx);
+			if (!parsed) return;
+			const chain = parsed.steps.map(({ name, config, task: stepTask }, i) => ({
+				agent: name,
+				...(stepTask ? { task: stepTask } : i === 0 && parsed.task ? { task: parsed.task } : {}),
+				...(config.output !== undefined ? { output: config.output } : {}),
+				...(config.outputMode !== undefined ? { outputMode: config.outputMode } : {}),
+				...(config.reads !== undefined ? { reads: config.reads } : {}),
+				...(config.model ? { model: config.model } : {}),
+				...(config.skill !== undefined ? { skill: config.skill } : {}),
+				...(config.progress !== undefined ? { progress: config.progress } : {}),
+			}));
+			const params: SubagentParamsLike = { chain, task: parsed.task, clarify: false, agentScope: "both" };
+			if (bg) params.async = true;
+			if (fork) params.context = "fork";
+			await runSlashSubagent(pi, ctx, params);
+		},
+	});
+
+	pi.registerCommand("run-chain", {
+		description: "Run a saved chain: /run-chain chainName -- task [--bg] [--fork]",
+		getArgumentCompletions: makeChainCompletions(state),
+		handler: async (args, ctx) => {
+			const { args: cleanedArgs, bg, fork } = extractExecutionFlags(args);
+			const delimiterIndex = cleanedArgs.indexOf(" -- ");
+			const usage = "Usage: /run-chain <chainName> -- <task> [--bg] [--fork]";
+			if (delimiterIndex === -1) {
+				ctx.ui.notify(usage, "error");
+				return;
+			}
+			const chainName = cleanedArgs.slice(0, delimiterIndex).trim();
+			const task = cleanedArgs.slice(delimiterIndex + 4).trim();
+			if (!chainName || !task) {
+				ctx.ui.notify(usage, "error");
+				return;
+			}
+			if (!state.baseCwd) { ctx.ui.notify("Subagent session cwd is not initialized yet", "error"); return; }
+			const chain = discoverSavedChains(state.baseCwd).find((candidate) => candidate.name === chainName);
+			if (!chain) {
+				ctx.ui.notify(`Unknown chain: ${chainName}`, "error");
+				return;
+			}
+			const params: SubagentParamsLike = { chain: mapSavedChainSteps(chain), task, clarify: false, agentScope: "both" };
+			if (bg) params.async = true;
+			if (fork) params.context = "fork";
+			await runSlashSubagent(pi, ctx, params);
+		},
+	});
+
+	pi.registerCommand("parallel", {
+		description: "Run agents in parallel: /parallel scout \"task1\" -> reviewer \"task2\" [--bg] [--fork]",
+		getArgumentCompletions: makeAgentCompletions(state, true),
+		handler: async (args, ctx) => {
+			const { args: cleanedArgs, bg, fork } = extractExecutionFlags(args);
+			const parsed = parseAgentArgs(state, cleanedArgs, "parallel", ctx);
+			if (!parsed) return;
+			const tasks = parsed.steps.map(({ name, config, task: stepTask }) => ({
+				agent: name,
+				task: stepTask ?? parsed.task,
+				...(config.output !== undefined ? { output: config.output } : {}),
+				...(config.outputMode !== undefined ? { outputMode: config.outputMode } : {}),
+				...(config.reads !== undefined ? { reads: config.reads } : {}),
+				...(config.model ? { model: config.model } : {}),
+				...(config.skill !== undefined ? { skill: config.skill } : {}),
+				...(config.progress !== undefined ? { progress: config.progress } : {}),
+			}));
+			const params: SubagentParamsLike = { tasks, clarify: false, agentScope: "both" };
+			if (bg) params.async = true;
+			if (fork) params.context = "fork";
+			await runSlashSubagent(pi, ctx, params);
+		},
+	});
+
+
+	pi.registerCommand("subagents-doctor", {
+		description: "Show subagent diagnostics",
+		handler: async (_args, ctx) => {
+			await runSlashSubagent(pi, ctx, { action: "doctor" });
+		},
+	});
+
 }