@oh-my-pi/pi-coding-agent 13.1.0 → 13.1.2

package/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 ## [Unreleased]
 
+## [13.1.2] - 2026-02-23
+### Breaking Changes
+
+- Removed `timeout` parameter from await tool—tool now waits indefinitely until jobs complete or the call is aborted
+- Renamed `job_ids` parameter to `jobs` in await tool schema
+- Removed `timedOut` field from await tool result details
+
+## [13.1.1] - 2026-02-23
+
+### Fixed
+
+- Fixed bash internal URL expansion to resolve `local://` targets to concrete filesystem paths, including newly created destination files for commands like `mv src.json local://dest.json`
+- Fixed bash local URL resolution to create missing parent directories under the session local root before command execution, preventing `mv` destination failures for new paths
 ## [13.1.0] - 2026-02-23
 ### Breaking Changes
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-coding-agent",
-	"version": "13.1.0",
+	"version": "13.1.2",
 	"description": "Coding agent CLI with read, bash, edit, write tools and session management",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -41,12 +41,12 @@
 	},
 	"dependencies": {
 		"@mozilla/readability": "^0.6",
-		"@oh-my-pi/omp-stats": "13.1.0",
-		"@oh-my-pi/pi-agent-core": "13.1.0",
-		"@oh-my-pi/pi-ai": "13.1.0",
-		"@oh-my-pi/pi-natives": "13.1.0",
-		"@oh-my-pi/pi-tui": "13.1.0",
-		"@oh-my-pi/pi-utils": "13.1.0",
+		"@oh-my-pi/omp-stats": "13.1.2",
+		"@oh-my-pi/pi-agent-core": "13.1.2",
+		"@oh-my-pi/pi-ai": "13.1.2",
+		"@oh-my-pi/pi-natives": "13.1.2",
+		"@oh-my-pi/pi-tui": "13.1.2",
+		"@oh-my-pi/pi-utils": "13.1.2",
 		"@sinclair/typebox": "^0.34",
 		"@xterm/headless": "^6.0",
 		"ajv": "^8.18",

package/src/prompts/system/custom-system-prompt.md

@@ -16,6 +16,16 @@
 </file>
 {{/list}}
 </instructions>
+{{/if}}
+{{#if git.isRepo}}
+## Version Control
+Snapshot; does not update during conversation.
+Current branch: {{git.currentBranch}}
+Main branch: {{git.mainBranch}}
+{{git.status}}
+### History
+{{git.commits}}
+{{/if}}
 </project>
 {{/ifAny}}
 {{#if skills.length}}

package/src/tools/await-tool.ts

@@ -5,16 +5,11 @@ import awaitDescription from "../prompts/tools/await.md" with { type: "text" };
 import type { ToolSession } from "./index";
 
 const awaitSchema = Type.Object({
-	job_ids: Type.Optional(
+	jobs: Type.Optional(
 		Type.Array(Type.String(), {
 			description: "Specific job IDs to wait for. If omitted, waits for any running job.",
 		}),
 	),
-	timeout: Type.Optional(
-		Type.Number({
-			description: "Maximum seconds to wait before returning (default: 300)",
-		}),
-	),
 });
 
 type AwaitParams = Static<typeof awaitSchema>;
@@ -31,7 +26,6 @@ interface AwaitResult {
 
 export interface AwaitToolDetails {
 	jobs: AwaitResult[];
-	timedOut: boolean;
 }
 
 export class AwaitTool implements AgentTool<typeof awaitSchema, AwaitToolDetails> {
@@ -61,12 +55,11 @@ export class AwaitTool implements AgentTool<typeof awaitSchema, AwaitToolDetails
 		if (!manager) {
 			return {
 				content: [{ type: "text", text: "Async execution is disabled; no background jobs to poll." }],
-				details: { jobs: [], timedOut: false },
+				details: { jobs: [] },
 			};
 		}
 
-		const timeoutMs = (params.timeout ?? 300) * 1000;
-		const requestedIds = params.job_ids;
+		const requestedIds = params.jobs;
 
 		// Resolve which jobs to watch
 		const jobsToWatch = requestedIds?.length
@@ -79,19 +72,18 @@ export class AwaitTool implements AgentTool<typeof awaitSchema, AwaitToolDetails
 				: "No running background jobs to wait for.";
 			return {
 				content: [{ type: "text", text: message }],
-				details: { jobs: [], timedOut: false },
+				details: { jobs: [] },
 			};
 		}
 
 		// If all watched jobs are already done, return immediately
 		const runningJobs = jobsToWatch.filter(j => j.status === "running");
 		if (runningJobs.length === 0) {
-			return this.#buildResult(jobsToWatch, false);
+			return this.#buildResult(jobsToWatch);
 		}
 
-		// Block until at least one running job finishes or timeout
+		// Block until at least one running job finishes or the call is aborted
 		const racePromises: Promise<unknown>[] = runningJobs.map(j => j.promise);
-		racePromises.push(Bun.sleep(timeoutMs));
 
 		if (signal) {
 			const { promise: abortPromise, resolve: abortResolve } = Promise.withResolvers<void>();
@@ -108,14 +100,10 @@ export class AwaitTool implements AgentTool<typeof awaitSchema, AwaitToolDetails
 		}
 
 		if (signal?.aborted) {
-			return this.#buildResult(jobsToWatch, false);
+			return this.#buildResult(jobsToWatch);
 		}
 
-		// Check if we timed out (all watched jobs still running)
-		const stillRunning = jobsToWatch.filter(j => j.status === "running");
-		const timedOut = stillRunning.length === runningJobs.length;
-
-		return this.#buildResult(jobsToWatch, timedOut);
+		return this.#buildResult(jobsToWatch);
 	}
 
 	#buildResult(
@@ -128,7 +116,6 @@ export class AwaitTool implements AgentTool<typeof awaitSchema, AwaitToolDetails
 			resultText?: string;
 			errorText?: string;
 		}[],
-		timedOut: boolean,
 	): AgentToolResult<AwaitToolDetails> {
 		const now = Date.now();
 		const jobResults: AwaitResult[] = jobs.map(j => ({
@@ -145,10 +132,6 @@ export class AwaitTool implements AgentTool<typeof awaitSchema, AwaitToolDetails
 		const running = jobResults.filter(j => j.status === "running");
 
 		const lines: string[] = [];
-		if (timedOut) {
-			lines.push("Timed out waiting for jobs to complete.\n");
-		}
-
 		if (completed.length > 0) {
 			lines.push(`## Completed (${completed.length})\n`);
 			for (const j of completed) {
@@ -173,7 +156,7 @@ export class AwaitTool implements AgentTool<typeof awaitSchema, AwaitToolDetails
 
 		return {
 			content: [{ type: "text", text: lines.join("\n") }],
-			details: { jobs: jobResults, timedOut },
+			details: { jobs: jobResults },
 		};
 	}
 }

package/src/tools/bash-skill-urls.ts

@@ -1,5 +1,7 @@
+import * as fs from "node:fs/promises";
 import * as path from "node:path";
 import type { Skill } from "../extensibility/skills";
+import { type LocalProtocolOptions, resolveLocalUrlToPath } from "../internal-urls";
 import { validateRelativePath } from "../internal-urls/skill-protocol";
 import type { InternalResource } from "../internal-urls/types";
 import { ToolError } from "./tool-errors";
@@ -9,9 +11,9 @@ const SKILL_URL_PATTERN = /'skill:\/\/[^'\s")`\\]+'|"skill:\/\/[^"\s')`\\]+"|ski
 
 /** Regex to find supported internal URL tokens in command text. */
 const INTERNAL_URL_PATTERN =
-	/'(?:skill|agent|artifact|plan|memory|rule):\/\/[^'\s")`\\]+'|"(?:skill|agent|artifact|plan|memory|rule):\/\/[^"\s')`\\]+"|(?:skill|agent|artifact|plan|memory|rule):\/\/[^\s'")`\\]+/g;
+	/'(?:skill|agent|artifact|plan|memory|rule|local):\/\/[^'\s")`\\]+'|"(?:skill|agent|artifact|plan|memory|rule|local):\/\/[^"\s')`\\]+"|(?:skill|agent|artifact|plan|memory|rule|local):\/\/[^\s'")`\\]+/g;
 
-const SUPPORTED_INTERNAL_SCHEMES = ["skill", "agent", "artifact", "plan", "memory", "rule"] as const;
+const SUPPORTED_INTERNAL_SCHEMES = ["skill", "agent", "artifact", "plan", "memory", "rule", "local"] as const;
 
 type SupportedInternalScheme = (typeof SUPPORTED_INTERNAL_SCHEMES)[number];
 
@@ -24,6 +26,8 @@ export interface InternalUrlExpansionOptions {
 	skills: readonly Skill[];
 	noEscape?: boolean;
 	internalRouter?: InternalUrlResolver;
+	localOptions?: LocalProtocolOptions;
+	ensureLocalParentDirs?: boolean;
 }
 
 /**
@@ -102,6 +106,8 @@ async function resolveInternalUrlToPath(
 	url: string,
 	skills: readonly Skill[],
 	internalRouter?: InternalUrlResolver,
+	localOptions?: LocalProtocolOptions,
+	ensureLocalParentDirs?: boolean,
 ): Promise<string> {
 	const scheme = extractScheme(url);
 	if (!scheme) {
@@ -112,6 +118,19 @@ async function resolveInternalUrlToPath(
 		return resolveSkillUrlToPath(url, skills);
 	}
 
+	if (scheme === "local") {
+		if (!localOptions) {
+			throw new ToolError(
+				"Cannot resolve local:// URL in bash command: local protocol options are unavailable for this session.",
+			);
+		}
+		const resolvedLocalPath = resolveLocalUrlToPath(url, localOptions);
+		if (ensureLocalParentDirs) {
+			await fs.mkdir(path.dirname(resolvedLocalPath), { recursive: true });
+		}
+		return resolvedLocalPath;
+	}
+
 	if (!internalRouter || !internalRouter.canHandle(url)) {
 		throw new ToolError(
 			`Cannot resolve ${scheme}:// URL in bash command: ${url}\n` +
@@ -169,7 +188,13 @@ export async function expandInternalUrls(command: string, options: InternalUrlEx
 		if (index === undefined) continue;
 
 		const url = unquoteToken(token);
-		const resolvedPath = await resolveInternalUrlToPath(url, options.skills, options.internalRouter);
+		const resolvedPath = await resolveInternalUrlToPath(
+			url,
+			options.skills,
+			options.internalRouter,
+			options.localOptions,
+			options.ensureLocalParentDirs,
+		);
 		const replacement = options.noEscape ? resolvedPath : shellEscape(resolvedPath);
 		expanded = `${expanded.slice(0, index)}${replacement}${expanded.slice(index + token.length)}`;
 	}

package/src/tools/bash.ts

@@ -173,8 +173,12 @@ export class BashTool implements AgentTool<BashToolSchema, BashToolDetails> {
 		const internalUrlOptions: InternalUrlExpansionOptions = {
 			skills: this.session.skills ?? [],
 			internalRouter: this.session.internalRouter,
+			localOptions: {
+				getArtifactsDir: this.session.getArtifactsDir,
+				getSessionId: this.session.getSessionId,
+			},
 		};
-		command = await expandInternalUrls(command, internalUrlOptions);
+		command = await expandInternalUrls(command, { ...internalUrlOptions, ensureLocalParentDirs: true });
 
 		// Resolve protocol URLs (skill://, agent://, etc.) in extracted cwd.
 		if (cwd?.includes("://")) {