pi-subagents 0.8.3 → 0.8.5

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 ## [Unreleased]
 
+## [0.8.5] - 2026-02-16
+
+### Fixed
+- Async subagent execution no longer fails with "jiti not found" on machines without a global `jiti` install. The jiti resolution now tries three strategies: vanilla `jiti`, the `@mariozechner/jiti` fork, and finally resolves `@mariozechner/jiti` from pi's own installation via `process.argv[1]`. Since pi always ships the fork as a dependency, async mode now works out of the box.
+- Improved the "jiti not found" error message to explain what's needed and how to fix it.
+
+## [0.8.4] - 2026-02-13
+
+### Fixed
+- JSONL artifact files no longer written by default — they duplicated pi's own session files and were the sole cause of `subagent-artifacts` directories growing to 10+ GB. Changed `includeJsonl` default from `true` to `false`. `_output.md` and `_meta.json` still capture the useful data.
+- Artifact cleanup now covers session-based directories, not just the temp dir. Previously `cleanupOldArtifacts` only ran on `os.tmpdir()/pi-subagent-artifacts` at startup, while sync runs (the common path) wrote to `<session-dir>/subagent-artifacts/` which was never cleaned. Now scans all `~/.pi/agent/sessions/*/subagent-artifacts/` dirs on startup and cleans the current session's artifacts dir on session lifecycle events.
+- JSONL writer now enforces a 50 MB size cap (`maxBytes` on `JsonlWriterDeps`) as defense-in-depth for users who opt into JSONL. Silently stops writing at the cap without pausing the source stream, so the progress tracker keeps working.
+
 ## [0.8.3] - 2026-02-11
 
 ### Added

package/artifacts.ts

@@ -69,3 +69,24 @@ export function cleanupOldArtifacts(dir: string, maxAgeDays: number): void {
 
 	fs.writeFileSync(markerPath, String(now));
 }
+
+export function cleanupAllArtifactDirs(maxAgeDays: number): void {
+	cleanupOldArtifacts(TEMP_ARTIFACTS_DIR, maxAgeDays);
+
+	const sessionsBase = path.join(os.homedir(), ".pi", "agent", "sessions");
+	if (!fs.existsSync(sessionsBase)) return;
+
+	let dirs: string[];
+	try {
+		dirs = fs.readdirSync(sessionsBase);
+	} catch {
+		return;
+	}
+
+	for (const dir of dirs) {
+		const artifactsDir = path.join(sessionsBase, dir, "subagent-artifacts");
+		try {
+			cleanupOldArtifacts(artifactsDir, maxAgeDays);
+		} catch {}
+	}
+}

package/async-execution.ts

@@ -24,11 +24,22 @@ import {
 
 const require = createRequire(import.meta.url);
 const jitiCliPath: string | undefined = (() => {
-	try {
-		return path.join(path.dirname(require.resolve("jiti/package.json")), "lib/jiti-cli.mjs");
-	} catch {
-		return undefined;
+	const candidates: Array<() => string> = [
+		() => path.join(path.dirname(require.resolve("jiti/package.json")), "lib/jiti-cli.mjs"),
+		() => path.join(path.dirname(require.resolve("@mariozechner/jiti/package.json")), "lib/jiti-cli.mjs"),
+		() => {
+			const piEntry = fs.realpathSync(process.argv[1]);
+			const piRequire = createRequire(piEntry);
+			return path.join(path.dirname(piRequire.resolve("@mariozechner/jiti/package.json")), "lib/jiti-cli.mjs");
+		},
+	];
+	for (const candidate of candidates) {
+		try {
+			const p = candidate();
+			if (fs.existsSync(p)) return p;
+		} catch {}
 	}
+	return undefined;
 })();
 
 export interface AsyncExecutionContext {

package/index.ts

@@ -22,7 +22,7 @@ import { type AgentConfig, type AgentScope, discoverAgents, discoverAgentsAll }
 import { resolveExecutionAgentScope } from "./agent-scope.js";
 import { cleanupOldChainDirs, getStepAgents, isParallelStep, resolveStepBehavior, type ChainStep, type SequentialStep } from "./settings.js";
 import { ChainClarifyComponent, type ChainClarifyResult, type ModelInfo } from "./chain-clarify.js";
-import { cleanupOldArtifacts, getArtifactsDir } from "./artifacts.js";
+import { cleanupAllArtifactDirs, cleanupOldArtifacts, getArtifactsDir } from "./artifacts.js";
 import {
 	type AgentProgress,
 	type ArtifactConfig,
@@ -78,7 +78,7 @@ export default function registerSubagentExtension(pi: ExtensionAPI): void {
 	const asyncByDefault = config.asyncByDefault === true;
 
 	const tempArtifactsDir = getArtifactsDir(null);
-	cleanupOldArtifacts(tempArtifactsDir, DEFAULT_ARTIFACT_CONFIG.cleanupDays);
+	cleanupAllArtifactDirs(DEFAULT_ARTIFACT_CONFIG.cleanupDays);
 	let baseCwd = process.cwd();
 	let currentSessionId: string | null = null;
 	const asyncJobs = new Map<string, AsyncJobState>();
@@ -335,7 +335,7 @@ MANAGEMENT (use action field — omit agent/task/chain/tasks):
 			if (effectiveAsync) {
 				if (!isAsyncAvailable()) {
 					return {
-						content: [{ type: "text", text: "jiti not found" }],
+						content: [{ type: "text", text: "Async mode requires jiti for TypeScript execution but it could not be found. Install globally: npm install -g jiti" }],
 						isError: true,
 						details: { mode: "single" as const, results: [] },
 					};
@@ -1143,9 +1143,19 @@ MANAGEMENT (use action field — omit agent/task/chain/tasks):
 		}
 	});
 
+	const cleanupSessionArtifacts = (ctx: ExtensionContext) => {
+		try {
+			const sessionFile = ctx.sessionManager.getSessionFile();
+			if (sessionFile) {
+				cleanupOldArtifacts(getArtifactsDir(sessionFile), DEFAULT_ARTIFACT_CONFIG.cleanupDays);
+			}
+		} catch {}
+	};
+
 	pi.on("session_start", (_event, ctx) => {
 		baseCwd = ctx.cwd;
 		currentSessionId = ctx.sessionManager.getSessionFile() ?? `session-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
+		cleanupSessionArtifacts(ctx);
 		for (const timer of cleanupTimers.values()) clearTimeout(timer);
 		cleanupTimers.clear();
 		asyncJobs.clear();
@@ -1158,6 +1168,7 @@ MANAGEMENT (use action field — omit agent/task/chain/tasks):
 	pi.on("session_switch", (_event, ctx) => {
 		baseCwd = ctx.cwd;
 		currentSessionId = ctx.sessionManager.getSessionFile() ?? `session-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
+		cleanupSessionArtifacts(ctx);
 		for (const timer of cleanupTimers.values()) clearTimeout(timer);
 		cleanupTimers.clear();
 		asyncJobs.clear();
@@ -1170,6 +1181,7 @@ MANAGEMENT (use action field — omit agent/task/chain/tasks):
 	pi.on("session_branch", (_event, ctx) => {
 		baseCwd = ctx.cwd;
 		currentSessionId = ctx.sessionManager.getSessionFile() ?? `session-${Date.now()}-${Math.random().toString(36).slice(2, 8)}`;
+		cleanupSessionArtifacts(ctx);
 		for (const timer of cleanupTimers.values()) clearTimeout(timer);
 		cleanupTimers.clear();
 		asyncJobs.clear();

package/jsonl-writer.ts

@@ -11,8 +11,11 @@ export interface JsonlWriteStream {
 	end(callback?: () => void): void;
 }
 
+const DEFAULT_MAX_JSONL_BYTES = 50 * 1024 * 1024;
+
 export interface JsonlWriterDeps {
 	createWriteStream?: (filePath: string) => JsonlWriteStream;
+	maxBytes?: number;
 }
 
 export interface JsonlWriter {
@@ -45,12 +48,18 @@ export function createJsonlWriter(
 
 	let backpressured = false;
 	let closed = false;
+	let bytesWritten = 0;
+	const maxBytes = deps.maxBytes ?? DEFAULT_MAX_JSONL_BYTES;
 
 	return {
 		writeLine(line: string) {
 			if (!stream || closed || !line.trim()) return;
+			const chunk = `${line}\n`;
+			const chunkBytes = Buffer.byteLength(chunk, "utf-8");
+			if (bytesWritten + chunkBytes > maxBytes) return;
 			try {
-				const ok = stream.write(`${line}\n`);
+				const ok = stream.write(chunk);
+				bytesWritten += chunkBytes;
 				if (!ok && !backpressured) {
 					backpressured = true;
 					source.pause();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-subagents",
-  "version": "0.8.3",
+  "version": "0.8.5",
   "description": "Pi extension for delegating tasks to subagents with chains, parallel execution, and TUI clarification",
   "author": "Nico Bailon",
   "license": "MIT",

package/types.ts

@@ -232,7 +232,7 @@ export const DEFAULT_ARTIFACT_CONFIG: ArtifactConfig = {
 	enabled: true,
 	includeInput: true,
 	includeOutput: true,
-	includeJsonl: true,
+	includeJsonl: false,
 	includeMetadata: true,
 	cleanupDays: 7,
 };