pi-subagents 0.8.2 → 0.8.4

package/subagent-runner.ts

@@ -5,6 +5,8 @@ import * as os from "node:os";
 import * as path from "node:path";
 import { pathToFileURL } from "node:url";
 import { appendJsonl, getArtifactPaths } from "./artifacts.js";
+import { getPiSpawnCommand } from "./pi-spawn.js";
+import { persistSingleOutput } from "./single-output.js";
 import {
 	type ArtifactConfig,
 	type ArtifactPaths,
@@ -20,9 +22,11 @@ interface SubagentStep {
 	cwd?: string;
 	model?: string;
 	tools?: string[];
+	extensions?: string[];
 	mcpDirectTools?: string[];
 	systemPrompt?: string | null;
 	skills?: string[];
+	outputPath?: string;
 }
 
 interface SubagentRunConfig {
@@ -104,7 +108,8 @@ function runPiStreaming(
 	return new Promise((resolve) => {
 		const outputStream = fs.createWriteStream(outputFile, { flags: "w" });
 		const spawnEnv = { ...process.env, ...(env ?? {}), ...getSubagentDepthEnv() };
-		const child = spawn("pi", args, { cwd, stdio: ["ignore", "pipe", "pipe"], env: spawnEnv });
+		const spawnSpec = getPiSpawnCommand(args);
+		const child = spawn(spawnSpec.command, spawnSpec.args, { cwd, stdio: ["ignore", "pipe", "pipe"], env: spawnEnv });
 		let stdout = "";
 
 		child.stdout.on("data", (chunk: Buffer) => {
@@ -337,19 +342,27 @@ async function runSubagent(config: SubagentRunConfig): Promise<void> {
 			} catch {}
 			args.push("--session-dir", config.sessionDir);
 		}
-		if (step.model) args.push("--model", step.model);
+		// Use --models (not --model) because pi CLI silently ignores --model
+		// without a companion --provider flag. --models resolves the provider
+		// automatically via resolveModelScope. See: #8
+		if (step.model) args.push("--models", step.model);
+		const toolExtensionPaths: string[] = [];
 		if (step.tools?.length) {
 			const builtinTools: string[] = [];
-			const extensionPaths: string[] = [];
 			for (const tool of step.tools) {
 				if (tool.includes("/") || tool.endsWith(".ts") || tool.endsWith(".js")) {
-					extensionPaths.push(tool);
+					toolExtensionPaths.push(tool);
 				} else {
 					builtinTools.push(tool);
 				}
 			}
 			if (builtinTools.length > 0) args.push("--tools", builtinTools.join(","));
-			for (const extPath of extensionPaths) args.push("--extension", extPath);
+		}
+		if (step.extensions !== undefined) {
+			args.push("--no-extensions");
+			for (const extPath of step.extensions) args.push("--extension", extPath);
+		} else {
+			for (const extPath of toolExtensionPaths) args.push("--extension", extPath);
 		}
 
 		let tmpDir: string | null = null;
@@ -392,6 +405,19 @@ async function runSubagent(config: SubagentRunConfig): Promise<void> {
 
 		const output = (result.stdout || "").trim();
 		previousOutput = output;
+		let outputForSummary = output;
+		if (step.outputPath && result.exitCode === 0) {
+			const persisted = persistSingleOutput(step.outputPath, output);
+			if (persisted.savedPath) {
+				outputForSummary = output
+					? `${output}\n\n📄 Output saved to: ${persisted.savedPath}`
+					: `📄 Output saved to: ${persisted.savedPath}`;
+			} else if (persisted.error) {
+				outputForSummary = output
+					? `${output}\n\n⚠️ Failed to save output to: ${step.outputPath}\n${persisted.error}`
+					: `⚠️ Failed to save output to: ${step.outputPath}\n${persisted.error}`;
+			}
+		}
 
 		const cumulativeTokens = config.sessionDir ? parseSessionTokens(config.sessionDir) : null;
 		const stepTokens: TokenUsage | null = cumulativeTokens
@@ -407,7 +433,7 @@ async function runSubagent(config: SubagentRunConfig): Promise<void> {
 
 		const stepResult: StepResult = {
 			agent: step.agent,
-			output,
+			output: outputForSummary,
 			success: result.exitCode === 0,
 			artifactPaths,
 		};

package/types.ts

@@ -2,6 +2,8 @@
  * Type definitions for the subagent extension
  */
 
+import * as os from "node:os";
+import * as path from "node:path";
 import type { Message } from "@mariozechner/pi-ai";
 
 // ============================================================================
@@ -230,15 +232,15 @@ export const DEFAULT_ARTIFACT_CONFIG: ArtifactConfig = {
 	enabled: true,
 	includeInput: true,
 	includeOutput: true,
-	includeJsonl: true,
+	includeJsonl: false,
 	includeMetadata: true,
 	cleanupDays: 7,
 };
 
 export const MAX_PARALLEL = 8;
 export const MAX_CONCURRENCY = 4;
-export const RESULTS_DIR = "/tmp/pi-async-subagent-results";
-export const ASYNC_DIR = "/tmp/pi-async-subagent-runs";
+export const RESULTS_DIR = path.join(os.tmpdir(), "pi-async-subagent-results");
+export const ASYNC_DIR = path.join(os.tmpdir(), "pi-async-subagent-runs");
 export const WIDGET_KEY = "subagent-async";
 export const POLL_INTERVAL_MS = 250;
 export const MAX_WIDGET_JOBS = 4;