@oh-my-pi/pi-coding-agent 6.8.2 → 6.8.3

package/CHANGELOG.md

@@ -2,6 +2,23 @@
 
 ## [Unreleased]
 
+## [6.8.3] - 2026-01-21
+
+### Changed
+
+- Updated keybinding system to normalize key IDs to lowercase
+- Changed label edit shortcut from 'l' to 'Shift+L' in tree selector
+- Changed output file extension from `.out.md` to `.md` for artifacts
+
+### Removed
+
+- Removed bundled worktree command from custom commands loader
+
+### Fixed
+
+- Fixed keybinding case sensitivity issues by normalizing all key IDs
+- Fixed task artifact path handling and simplified file structure
+
 ## [6.8.2] - 2026-01-21
 
 ### Fixed

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "6.8.2",
+	"version": "6.8.3",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"type": "module",
 	"ompConfig": {
@@ -40,11 +40,11 @@
 		"prepublishOnly": "bun run generate-template && bun run clean && bun run build"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-agent-core": "6.8.2",
-		"@oh-my-pi/pi-ai": "6.8.2",
-		"@oh-my-pi/pi-git-tool": "6.8.2",
-		"@oh-my-pi/pi-tui": "6.8.2",
-		"@oh-my-pi/pi-utils": "6.8.2",
+		"@oh-my-pi/pi-agent-core": "6.8.3",
+		"@oh-my-pi/pi-ai": "6.8.3",
+		"@oh-my-pi/pi-git-tool": "6.8.3",
+		"@oh-my-pi/pi-tui": "6.8.3",
+		"@oh-my-pi/pi-utils": "6.8.3",
 		"@openai/agents": "^0.3.7",
 		"@sinclair/typebox": "^0.34.46",
 		"ajv": "^8.17.1",

package/src/core/agent-session.ts

@@ -63,7 +63,6 @@ import { unmountAll } from "./ssh/sshfs-mount";
 import type { BashOperations } from "./tools/bash";
 import { normalizeDiff, normalizeToLF, ParseError, previewPatch, stripBom } from "./tools/patch";
 import { resolveToCwd } from "./tools/path-utils";
-import { getArtifactsDir } from "./tools/task/artifacts";
 import type { TodoItem } from "./tools/todo-write";
 import type { TtsrManager } from "./ttsr";
 
@@ -1991,10 +1990,7 @@ export class AgentSession {
 		const sessionFile = this.sessionManager.getSessionFile();
 		if (!sessionFile) return;
 
-		const artifactsDir = getArtifactsDir(sessionFile);
-		if (!artifactsDir) return;
-
-		const todoPath = `${artifactsDir}/todos.json`;
+		const todoPath = `${sessionFile.slice(0, -6)}/todos.json`;
 		const file = Bun.file(todoPath);
 		if (!(await file.exists())) {
 			this._todoReminderCount = 0;

package/src/core/custom-commands/loader.ts

@@ -13,7 +13,6 @@ import { getAgentDir, getConfigDirs } from "../../config";
 import * as piCodingAgent from "../../index";
 import { execCommand } from "../exec";
 import { ReviewCommand } from "./bundled/review";
-import { WorktreeCommand } from "./bundled/wt";
 import type {
 	CustomCommand,
 	CustomCommandAPI,
@@ -153,13 +152,6 @@ function loadBundledCommands(sharedApi: CustomCommandAPI): LoadedCustomCommand[]
 		source: "bundled",
 	});
 
-	bundled.push({
-		path: "bundled:wt",
-		resolvedPath: "bundled:wt",
-		command: new WorktreeCommand(sharedApi),
-		source: "bundled",
-	});
-
 	return bundled;
 }
 

package/src/core/keybindings.ts

@@ -124,6 +124,8 @@ const KEY_LABELS: Record<string, string> = {
 	right: "Right",
 };
 
+const normalizeKeyId = (key: KeyId): KeyId => key.toLowerCase() as KeyId;
+
 function formatKeyPart(part: string): string {
 	const lower = part.toLowerCase();
 	const modifier = MODIFIER_LABELS[lower];
@@ -199,14 +201,20 @@ export class KeybindingsManager {
 		// Set defaults for app actions
 		for (const [action, keys] of Object.entries(DEFAULT_APP_KEYBINDINGS)) {
 			const keyArray = Array.isArray(keys) ? keys : [keys];
-			this.appActionToKeys.set(action as AppAction, [...keyArray]);
+			this.appActionToKeys.set(
+				action as AppAction,
+				keyArray.map((key) => normalizeKeyId(key as KeyId)),
+			);
 		}
 
 		// Override with user config (app actions only)
 		for (const [action, keys] of Object.entries(this.config)) {
 			if (keys === undefined || !isAppAction(action)) continue;
 			const keyArray = Array.isArray(keys) ? keys : [keys];
-			this.appActionToKeys.set(action, keyArray);
+			this.appActionToKeys.set(
+				action,
+				keyArray.map((key) => normalizeKeyId(key as KeyId)),
+			);
 		}
 	}
 

package/src/core/tools/output.ts

@@ -25,7 +25,6 @@ import {
 	TRUNCATE_LENGTHS,
 	truncate,
 } from "./render-utils";
-import { getArtifactsDir } from "./task/artifacts";
 
 const outputSchema = Type.Object({
 	ids: Type.Array(Type.String(), {
@@ -164,7 +163,7 @@ function applyQuery(data: unknown, query: string): unknown {
 function listAvailableOutputs(artifactsDir: string): string[] {
 	try {
 		const files = fs.readdirSync(artifactsDir);
-		return files.filter((f) => f.endsWith(".out.md")).map((f) => f.replace(".out.md", ""));
+		return files.filter((f) => f.endsWith(".md")).map((f) => f.replace(".md", ""));
 	} catch {
 		return [];
 	}
@@ -274,8 +273,8 @@ export class OutputTool implements AgentTool<typeof outputSchema, OutputToolDeta
 			};
 		}
 
-		const artifactsDir = getArtifactsDir(sessionFile);
-		if (!artifactsDir || !fs.existsSync(artifactsDir)) {
+		const artifactsDir = sessionFile.slice(0, -6); // strip .jsonl extension
+		if (!fs.existsSync(artifactsDir)) {
 			return {
 				content: [{ type: "text", text: "No artifacts directory found" }],
 				details: { outputs: [], notFound: params.ids },
@@ -296,14 +295,14 @@ export class OutputTool implements AgentTool<typeof outputSchema, OutputToolDeta
 		const queryResults: Array<{ id: string; value: unknown }> = [];
 
 		for (const id of params.ids) {
-			const outputPath = path.join(artifactsDir, `${id}.out.md`);
-
-			if (!fs.existsSync(outputPath)) {
+			const outputPath = path.join(artifactsDir, `${id}.md`);
+			const file = Bun.file(outputPath);
+			if (!(await file.exists())) {
 				notFound.push(id);
 				continue;
 			}
 
-			const rawContent = fs.readFileSync(outputPath, "utf-8");
+			const rawContent = await file.text();
 			const rawLines = rawContent.split("\n");
 			const totalLines = rawLines.length;
 			const totalChars = rawContent.length;

package/src/core/tools/task/executor.ts

@@ -4,6 +4,7 @@
  * Runs each subagent in a Bun Worker and forwards AgentEvents for progress tracking.
  */
 
+import path from "node:path";
 import type { AgentEvent, ThinkingLevel } from "@oh-my-pi/pi-agent-core";
 import type { AuthStorage } from "../../auth-storage";
 import type { EventBus } from "../../event-bus";
@@ -16,7 +17,6 @@ import type { ToolSession } from "..";
 import { LspTool } from "../lsp/index";
 import type { LspParams } from "../lsp/types";
 import { PythonTool } from "../python";
-import { ensureArtifactsDir, getArtifactPaths } from "./artifacts";
 import { subprocessToolRegistry } from "./subprocess-tool-registry";
 import {
 	type AgentDefinition,
@@ -256,20 +256,9 @@ export async function runSubprocess(options: ExecutorOptions): Promise<SingleRes
 	const fullTask = context ? `${context}\n\n${task}` : task;
 
 	// Set up artifact paths and write input file upfront if artifacts dir provided
-	let artifactPaths: { inputPath: string; outputPath: string; jsonlPath: string } | undefined;
 	let subtaskSessionFile: string | undefined;
-
 	if (options.artifactsDir) {
-		ensureArtifactsDir(options.artifactsDir);
-		artifactPaths = getArtifactPaths(options.artifactsDir, taskId);
-		subtaskSessionFile = artifactPaths.jsonlPath;
-
-		// Write input file immediately (real-time visibility)
-		try {
-			await Bun.write(artifactPaths.inputPath, fullTask);
-		} catch {
-			// Non-fatal, continue without input artifact
-		}
+		subtaskSessionFile = path.join(options.artifactsDir, `${taskId}.jsonl`);
 	}
 
 	// Add tools if specified
@@ -1042,9 +1031,11 @@ export async function runSubprocess(options: ExecutorOptions): Promise<SingleRes
 	// Write output artifact (input and jsonl already written in real-time)
 	// Compute output metadata for Output tool integration
 	let outputMeta: { lineCount: number; charCount: number } | undefined;
-	if (artifactPaths) {
+	let outputPath: string | undefined;
+	if (options.artifactsDir) {
+		outputPath = path.join(options.artifactsDir, `${taskId}.md`);
 		try {
-			await Bun.write(artifactPaths.outputPath, rawOutput);
+			await Bun.write(outputPath, rawOutput);
 			outputMeta = {
 				lineCount: rawOutput.split("\n").length,
 				charCount: rawOutput.length,
@@ -1076,7 +1067,7 @@ export async function runSubprocess(options: ExecutorOptions): Promise<SingleRes
 		error: exitCode !== 0 && stderr ? stderr : undefined,
 		aborted: wasAborted,
 		usage: hasUsage ? accumulatedUsage : undefined,
-		artifactPaths,
+		outputPath,
 		extractedToolData: progress.extractedToolData,
 		outputMeta,
 	};

package/src/core/tools/task/index.ts

@@ -13,13 +13,17 @@
  *   - Session artifacts for debugging
  */
 
+import { mkdir, rm } from "node:fs/promises";
+import { tmpdir } from "node:os";
+import path from "node:path";
 import type { AgentTool, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 import type { Usage } from "@oh-my-pi/pi-ai";
+import { nanoid } from "nanoid";
 import type { Theme } from "../../../modes/interactive/theme/theme";
 import taskDescriptionTemplate from "../../../prompts/tools/task.md" with { type: "text" };
 import { renderPromptTemplate } from "../../prompt-templates";
+import type { ToolSession } from "..";
 import { formatDuration } from "../render-utils";
-import { cleanupTempDir, createTempArtifactsDir, getArtifactsDir } from "./artifacts";
 import { discoverAgents, getAgent } from "./discovery";
 import { runSubprocess } from "./executor";
 import { mapWithConcurrencyLimit } from "./parallel";
@@ -36,7 +40,6 @@ import {
 
 // Import review tools for side effects (registers subagent tool handlers)
 import "../review";
-import type { ToolSession } from "..";
 
 /** Format byte count for display */
 function formatBytes(bytes: number): string {
@@ -276,9 +279,10 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 
 		// Derive artifacts directory
 		const sessionFile = this.session.getSessionFile();
-		const artifactsDir = sessionFile ? getArtifactsDir(sessionFile) : null;
-		const tempArtifactsDir = artifactsDir ? null : createTempArtifactsDir();
+		const artifactsDir = sessionFile ? sessionFile.slice(0, -6) : null;
+		const tempArtifactsDir = artifactsDir ? null : path.join(tmpdir(), `omp-task-${nanoid()}`);
 		const effectiveArtifactsDir = artifactsDir || tempArtifactsDir!;
+		await mkdir(effectiveArtifactsDir, { recursive: true });
 
 		// Initialize progress tracking
 		const progressMap = new Map<number, AgentProgress>();
@@ -435,8 +439,8 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 			// Collect output paths (artifacts already written by executor in real-time)
 			const outputPaths: string[] = [];
 			for (const result of results) {
-				if (result.artifactPaths) {
-					outputPaths.push(result.artifactPaths.outputPath);
+				if (result.outputPath) {
+					outputPaths.push(result.outputPath);
 				}
 			}
 
@@ -468,7 +472,7 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 
 			// Cleanup temp directory if used
 			if (tempArtifactsDir) {
-				await cleanupTempDir(tempArtifactsDir);
+				await rm(tempArtifactsDir, { recursive: true, force: true });
 			}
 
 			return {
@@ -482,11 +486,6 @@ export class TaskTool implements AgentTool<typeof taskSchema, TaskToolDetails, T
 				},
 			};
 		} catch (err) {
-			// Cleanup temp directory on error
-			if (tempArtifactsDir) {
-				await cleanupTempDir(tempArtifactsDir);
-			}
-
 			return {
 				content: [{ type: "text", text: `Task execution failed: ${err}` }],
 				details: {

package/src/core/tools/task/types.ts

@@ -153,7 +153,8 @@ export interface SingleResult {
 	aborted?: boolean;
 	/** Aggregated usage from the subprocess, accumulated incrementally from message_end events. */
 	usage?: Usage;
-	artifactPaths?: { inputPath: string; outputPath: string; jsonlPath?: string };
+	/** Output path for the task result */
+	outputPath?: string;
 	/** Data extracted by registered subprocess tool handlers (keyed by tool name) */
 	extractedToolData?: Record<string, unknown[]>;
 	/** Output metadata for Output tool integration */

package/src/core/tools/todo-write.ts

@@ -1,5 +1,4 @@
 import { randomUUID } from "node:crypto";
-import { mkdirSync } from "node:fs";
 import path from "node:path";
 import type { AgentTool, AgentToolContext, AgentToolResult, AgentToolUpdateCallback } from "@oh-my-pi/pi-agent-core";
 import { StringEnum } from "@oh-my-pi/pi-ai";
@@ -13,7 +12,6 @@ import todoWriteDescription from "../../prompts/tools/todo-write.md" with { type
 import type { RenderResultOptions } from "../custom-tools/types";
 import { renderPromptTemplate } from "../prompt-templates";
 import type { ToolSession } from "../sdk";
-import { ensureArtifactsDir, getArtifactsDir } from "./task/artifacts";
 
 const todoWriteSchema = Type.Object({
 	todos: Type.Array(
@@ -135,10 +133,6 @@ async function loadTodoFile(filePath: string): Promise<TodoFile | null> {
 	}
 }
 
-async function saveTodoFile(filePath: string, data: TodoFile): Promise<void> {
-	await Bun.write(filePath, JSON.stringify(data, null, 2));
-}
-
 function formatTodoSummary(todos: TodoItem[]): string {
 	if (todos.length === 0) return "Todo list cleared.";
 	const completed = todos.filter((t) => t.status === "completed").length;
@@ -200,24 +194,14 @@ export class TodoWriteTool implements AgentTool<typeof todoWriteSchema, TodoWrit
 			};
 		}
 
-		const artifactsDir = getArtifactsDir(sessionFile);
-		if (!artifactsDir) {
-			return {
-				content: [{ type: "text", text: formatTodoSummary(todos) }],
-				details: { todos, updatedAt, storage: "memory" },
-			};
-		}
-
-		ensureArtifactsDir(artifactsDir);
-		const todoPath = path.join(artifactsDir, TODO_FILE_NAME);
+		const todoPath = path.join(sessionFile.slice(0, -6), TODO_FILE_NAME);
 		const existing = await loadTodoFile(todoPath);
 		const storedTodos = existing?.todos ?? [];
 		const merged = todos.length > 0 ? todos : [];
 		const fileData: TodoFile = { updatedAt, todos: merged };
 
 		try {
-			mkdirSync(artifactsDir, { recursive: true });
-			await saveTodoFile(todoPath, fileData);
+			await Bun.write(todoPath, JSON.stringify(fileData, null, 2));
 		} catch (error) {
 			logger.error("Failed to write todo file", { path: todoPath, error: String(error) });
 			return {

package/src/modes/interactive/components/todo-display.ts

@@ -1,7 +1,6 @@
 import * as path from "node:path";
 import { Text } from "@oh-my-pi/pi-tui";
 import { logger } from "@oh-my-pi/pi-utils";
-import { getArtifactsDir } from "../../../core/tools/task/artifacts";
 import { theme } from "../theme/theme";
 import type { TodoItem } from "../types";
 
@@ -40,13 +39,7 @@ export class TodoDisplayComponent {
 			return;
 		}
 
-		const artifactsDir = getArtifactsDir(this.sessionFile);
-		if (!artifactsDir) {
-			this.todos = [];
-			this.visible = false;
-			return;
-		}
-
+		const artifactsDir = this.sessionFile.slice(0, -6); // strip .jsonl extension
 		const todoPath = path.join(artifactsDir, TODO_FILE_NAME);
 		const data = await loadTodoFile(todoPath);
 		this.todos = data?.todos ?? [];

package/src/modes/interactive/components/tree-selector.ts

@@ -707,7 +707,7 @@ class TreeList implements Component {
 				this.searchQuery = this.searchQuery.slice(0, -1);
 				this.applyFilter();
 			}
-		} else if (keyData === "l" && !this.searchQuery) {
+		} else if (matchesKey(keyData, "shift+l") && !this.searchQuery) {
 			const selected = this.filteredNodes[this.selectedIndex];
 			if (selected && this.onLabelEdit) {
 				this.onLabelEdit(selected.node.entry.id, selected.node.label);
@@ -821,7 +821,7 @@ export class TreeSelectorComponent extends Container {
 			new TruncatedText(
 				theme.fg(
 					"muted",
-					"  Up/Down: move. Left/Right: page. l: label. Ctrl+O/Shift+Ctrl+O: filter. Alt+D/T/U/L/A: filter. Type to search",
+					"  Up/Down: move. Left/Right: page. Shift+L: label. Ctrl+O/Shift+Ctrl+O: filter. Alt+D/T/U/L/A: filter. Type to search",
 				),
 				0,
 				0,

package/src/modes/interactive/interactive-mode.ts

@@ -28,7 +28,6 @@ import { getRecentSessions } from "../../core/session-manager";
 import type { SettingsManager } from "../../core/settings-manager";
 import { loadSlashCommands } from "../../core/slash-commands";
 import { setTerminalTitle } from "../../core/title-generator";
-import { getArtifactsDir } from "../../core/tools/task/artifacts";
 import { VoiceSupervisor } from "../../core/voice-supervisor";
 import type { AssistantMessageComponent } from "./components/assistant-message";
 import type { BashExecutionComponent } from "./components/bash-execution";
@@ -384,15 +383,6 @@ export class InteractiveMode implements InteractiveModeContext {
 
 		// Initial top border update
 		this.updateEditorTopBorder();
-
-		if (!startupQuiet) {
-			const templateNames = this.session.promptTemplates.map((template) => template.name).sort();
-			if (templateNames.length > 0) {
-				const preview = templateNames.slice(0, 3).join(", ");
-				const suffix = templateNames.length > 3 ? ` +${templateNames.length - 3} more` : "";
-				this.showStatus(`Loaded prompt templates: ${preview}${suffix}`);
-			}
-		}
 	}
 
 	async getUserInput(): Promise<{ text: string; images?: ImageContent[] }> {
@@ -481,11 +471,7 @@ export class InteractiveMode implements InteractiveModeContext {
 			this.renderTodoList();
 			return;
 		}
-		const artifactsDir = getArtifactsDir(sessionFile);
-		if (!artifactsDir) {
-			this.renderTodoList();
-			return;
-		}
+		const artifactsDir = sessionFile.slice(0, -6);
 		const todoPath = path.join(artifactsDir, TODO_FILE_NAME);
 		const file = Bun.file(todoPath);
 		if (!(await file.exists())) {

package/src/utils/clipboard.ts

@@ -32,33 +32,40 @@ function selectPreferredImageMimeType(mimeTypes: string[]): string | null {
 }
 
 export async function copyToClipboard(text: string): Promise<void> {
-	let promise: Promise<void>;
+	const p = platform();
+	const timeout = 5000;
+
 	try {
-		switch (platform()) {
-			case "darwin":
-				promise = $`pbcopy ${text}`.quiet().then(() => void 0);
-				break;
-			case "win32":
-				promise = $`clip ${text}`.quiet().then(() => void 0);
-				break;
-			case "linux":
-				if (isWaylandSession()) {
-					$`wl-copy ${text}`.quiet(); // fire and forget
+		if (p === "darwin") {
+			await Bun.spawn(["pbcopy"], { stdin: Buffer.from(text), timeout }).exited;
+		} else if (p === "win32") {
+			await Bun.spawn(["clip"], { stdin: Buffer.from(text), timeout }).exited;
+		} else {
+			const wayland = isWaylandSession();
+			if (wayland) {
+				const wlCopyPath = Bun.which("wl-copy");
+				if (wlCopyPath) {
+					// Fire-and-forget: wl-copy may not exit promptly, so we unref to avoid blocking
+					void Bun.spawn([wlCopyPath], { stdin: Buffer.from(text), timeout }).unref();
 					return;
-				} else {
-					promise = $`xclip -selection clipboard -t text/plain -i ${text}`.quiet().then(() => void 0);
 				}
-				break;
-			default:
-				throw new Error(`Unsupported platform: ${platform()}`);
+			}
+
+			// Linux - try xclip first, fall back to xsel
+			try {
+				await Bun.spawn(["xclip", "-selection", "clipboard"], { stdin: Buffer.from(text), timeout }).exited;
+			} catch {
+				await Bun.spawn(["xsel", "--clipboard", "--input"], { stdin: Buffer.from(text), timeout }).exited;
+			}
 		}
 	} catch (error) {
-		if (error instanceof Error) {
-			throw new Error(`Failed to copy to clipboard: ${error.message}`, { cause: error });
+		const msg = error instanceof Error ? error.message : String(error);
+		if (p === "linux") {
+			const tools = isWaylandSession() ? "wl-copy, xclip, or xsel" : "xclip or xsel";
+			throw new Error(`Failed to copy to clipboard. Install ${tools}: ${msg}`);
 		}
-		throw new Error(`Failed to copy to clipboard: ${String(error)}`, { cause: error });
+		throw new Error(`Failed to copy to clipboard: ${msg}`);
 	}
-	await Promise.race([promise, Bun.sleep(3000)]);
 }
 
 export interface ClipboardImage {

package/src/core/tools/task/artifacts.ts

@@ -1,112 +0,0 @@
-/**
- * Session artifacts for subagent outputs.
- *
- * When a session exists, writes agent outputs to a sibling directory.
- * Otherwise uses temp files that are cleaned up after execution.
- */
-
-import * as fs from "node:fs";
-import * as os from "node:os";
-import * as path from "node:path";
-import { nanoid } from "nanoid";
-
-/**
- * Derive artifacts directory from session file path.
- *
- * /path/to/sessions/project/2026-01-01T14-28-11-636Z_uuid.jsonl
- *   → /path/to/sessions/project/2026-01-01T14-28-11-636Z_uuid/
- */
-export function getArtifactsDir(sessionFile: string | null): string | null {
-	if (!sessionFile) return null;
-	// Strip .jsonl extension to get directory path
-	if (sessionFile.endsWith(".jsonl")) {
-		return sessionFile.slice(0, -6);
-	}
-	return sessionFile;
-}
-
-/**
- * Ensure artifacts directory exists.
- */
-export function ensureArtifactsDir(dir: string): void {
-	if (!fs.existsSync(dir)) {
-		fs.mkdirSync(dir, { recursive: true });
-	}
-}
-
-/**
- * Generate artifact file paths for an agent run.
- */
-export function getArtifactPaths(
-	dir: string,
-	taskId: string,
-): { inputPath: string; outputPath: string; jsonlPath: string } {
-	return {
-		inputPath: path.join(dir, `${taskId}.in.md`),
-		outputPath: path.join(dir, `${taskId}.out.md`),
-		jsonlPath: path.join(dir, `${taskId}.jsonl`),
-	};
-}
-
-/**
- * Write artifacts for an agent run.
- */
-export async function writeArtifacts(
-	dir: string,
-	taskId: string,
-	input: string,
-	output: string,
-	jsonlEvents?: string[],
-): Promise<{ inputPath: string; outputPath: string; jsonlPath?: string }> {
-	ensureArtifactsDir(dir);
-
-	const paths = getArtifactPaths(dir, taskId);
-
-	// Write input
-	await Bun.write(paths.inputPath, input);
-
-	// Write output
-	await Bun.write(paths.outputPath, output);
-
-	// Write JSONL if events provided
-	if (jsonlEvents && jsonlEvents.length > 0) {
-		await Bun.write(paths.jsonlPath, jsonlEvents.join("\n"));
-		return paths;
-	}
-
-	return { inputPath: paths.inputPath, outputPath: paths.outputPath };
-}
-
-/**
- * Create a temporary artifacts directory.
- */
-export function createTempArtifactsDir(runId?: string): string {
-	const id = runId || nanoid();
-	const dir = path.join(os.tmpdir(), `omp-task-${id}`);
-	ensureArtifactsDir(dir);
-	return dir;
-}
-
-/**
- * Clean up temporary artifacts.
- */
-export async function cleanupTempArtifacts(paths: string[]): Promise<void> {
-	for (const p of paths) {
-		try {
-			await fs.promises.unlink(p);
-		} catch {
-			// Ignore cleanup errors
-		}
-	}
-}
-
-/**
- * Clean up a temporary directory and its contents.
- */
-export async function cleanupTempDir(dir: string): Promise<void> {
-	try {
-		await fs.promises.rm(dir, { recursive: true, force: true });
-	} catch {
-		// Ignore cleanup errors
-	}
-}