@oh-my-pi/pi-agent-core 16.2.1 → 16.2.2

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [16.2.2] - 2026-06-27
+
+### Added
+
+- Added optional AgentTool.matcherPaths(args) and AgentTool.matcherEntries(args) hooks to allow tools to surface target file paths and isolate file evaluations for path-scoped stream matchers (e.g., when handling multi-file payloads or embedded paths in streamed arguments).
+
+### Removed
+
+- Removed support for Pi dialect integration.
+
 ## [16.2.0] - 2026-06-27
 
 ### Added

package/dist/types/types.d.ts

@@ -545,6 +545,28 @@ export interface AgentTool<TParameters extends TSchema = TSchema, TDetails = any
      * matching.
      */
     matcherDigest?: (args: unknown) => string | undefined;
+    /**
+     * Surface the target file paths a (potentially partial) streamed call would
+     * touch, so path-scoped stream matchers (e.g. TTSR `tool:edit(*.ts)` globs)
+     * can match without a top-level `path`/`paths` argument. Used for tools whose
+     * wire grammar embeds paths inside the streamed payload (hashline section
+     * headers, apply_patch envelope markers). Return `undefined` (or an empty
+     * array) to fall back to the caller's top-level argument scan.
+     */
+    matcherPaths?: (args: unknown) => readonly string[] | undefined;
+    /**
+     * Per-file projection of a (potentially partial) streamed call, pairing each
+     * touched file path with the digest of only the lines added to that file.
+     * Path-scoped stream matchers (TTSR) evaluate each entry in isolation, so a
+     * scoped rule like `tool:edit(*.ts)` never fires on text that actually
+     * belongs to a sibling Markdown hunk in a multi-file payload. Takes
+     * precedence over {@link matcherDigest} + {@link matcherPaths} when present;
+     * returns `undefined` (or empty) to fall back to the combined hooks.
+     */
+    matcherEntries?: (args: unknown) => readonly {
+        path: string;
+        digest: string;
+    }[] | undefined;
     /** Capability tier declaration used by approval gates. Omitted means "exec". */
     approval?: ToolApproval;
     /** Lines appended after the standard approval prompt header. */

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "16.2.1",
+	"version": "16.2.2",
 	"description": "General-purpose agent with transport abstraction, state management, and attachment support",
 	"homepage": "https://omp.sh",
 	"author": "Can Boluk",
@@ -35,12 +35,12 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "16.2.1",
-		"@oh-my-pi/pi-catalog": "16.2.1",
-		"@oh-my-pi/pi-natives": "16.2.1",
-		"@oh-my-pi/pi-utils": "16.2.1",
-		"@oh-my-pi/pi-wire": "16.2.1",
-		"@oh-my-pi/snapcompact": "16.2.1",
+		"@oh-my-pi/pi-ai": "16.2.2",
+		"@oh-my-pi/pi-catalog": "16.2.2",
+		"@oh-my-pi/pi-natives": "16.2.2",
+		"@oh-my-pi/pi-utils": "16.2.2",
+		"@oh-my-pi/pi-wire": "16.2.2",
+		"@oh-my-pi/snapcompact": "16.2.2",
 		"@opentelemetry/api": "^1.9.1"
 	},
 	"devDependencies": {

package/src/agent-loop.ts

@@ -25,6 +25,7 @@ import {
 	renderToolExamples,
 	wrapInbandToolStream,
 } from "@oh-my-pi/pi-ai/dialect";
+import * as AIError from "@oh-my-pi/pi-ai/error";
 import {
 	createHarmonyAuditEvent,
 	detectHarmonyLeakInAssistantMessage,
@@ -134,7 +135,6 @@ export function resolveOwnedDialectFromEnv(value: string | undefined): Dialect |
 		case "anthropic":
 		case "deepseek":
 		case "harmony":
-		case "pi":
 		case "qwen3":
 		case "gemini":
 		case "gemma":
@@ -1557,8 +1557,12 @@ function emitAbortedAssistantMessage(
 	requestSignal: AbortSignal | undefined,
 ): AssistantMessage {
 	const errorMessage = abortReasonText(requestSignal);
+	const errorId =
+		errorMessage === "Request was aborted"
+			? AIError.create(AIError.Flag.Abort)
+			: AIError.classify(requestSignal?.reason) || undefined;
 	const base: AssistantMessage = partialMessage
-		? { ...partialMessage, stopReason: "aborted", errorMessage }
+		? { ...partialMessage, stopReason: "aborted", errorMessage, errorId }
 		: {
 				role: "assistant",
 				content: [],
@@ -1575,6 +1579,7 @@ function emitAbortedAssistantMessage(
 				},
 				stopReason: "aborted",
 				errorMessage,
+				errorId,
 				timestamp: Date.now(),
 			};
 	// Only tool calls that reached `toolcall_end` survive abort/error replay. A

package/src/compaction/compaction.ts

@@ -14,12 +14,12 @@ import {
 	type Message,
 	type MessageAttribution,
 	type Model,
-	ProviderHttpError,
 	type SimpleStreamOptions,
 	type Tool,
 	type Usage,
 	withAuth,
 } from "@oh-my-pi/pi-ai";
+import { ProviderHttpError } from "@oh-my-pi/pi-ai/error";
 import { preferredDialect } from "@oh-my-pi/pi-catalog/identity";
 import { clampThinkingLevelForModel } from "@oh-my-pi/pi-catalog/model-thinking";
 import { logger, prompt } from "@oh-my-pi/pi-utils";

package/src/compaction/openai.ts

@@ -12,7 +12,7 @@
  *   with `{ summary, shortSummary? }`.
  */
 
-import { ProviderHttpError } from "@oh-my-pi/pi-ai/errors";
+import { ProviderHttpError } from "@oh-my-pi/pi-ai/error";
 import { parseAzureDeploymentNameMap, parseTextSignature } from "@oh-my-pi/pi-ai/providers/openai-shared";
 import { transformMessages } from "@oh-my-pi/pi-ai/providers/transform-messages";
 import type { Api, AssistantMessage, FetchImpl, Message, Model } from "@oh-my-pi/pi-ai/types";

package/src/proxy.ts

@@ -13,6 +13,12 @@ import {
 	type StopReason,
 	type ToolCall,
 } from "@oh-my-pi/pi-ai";
+import {
+	clearStreamingPartialJson,
+	kStreamingPartialJson,
+	type StreamingPartialJsonCarrier,
+	setStreamingPartialJson,
+} from "@oh-my-pi/pi-ai/utils/block-symbols";
 import { calculateCost } from "@oh-my-pi/pi-catalog/models";
 import { parseStreamingJson, readSseJson } from "@oh-my-pi/pi-utils";
 
@@ -183,8 +189,8 @@ export function streamProxy(model: Model, context: Context, options: ProxyStream
 			const errorMessage = error instanceof Error ? error.message : String(error);
 			const reason = options.signal?.aborted ? "aborted" : "error";
 			partial.stopReason = reason;
-			scrubPartialJson(partial);
 			partial.errorMessage = errorMessage;
+			scrubPartialJson(partial);
 			stream.push({
 				type: "error",
 				reason,
@@ -202,14 +208,13 @@ export function streamProxy(model: Model, context: Context, options: ProxyStream
 }
 
 /**
- * Remove the `partialJson` streaming field from any tool-call content blocks
- * that still carry it (e.g. when the stream ended without a `toolcall_end`).
+ * Clear the `partialJson` streaming symbol from any tool-call content blocks
+ * that still carry it (e.g. when the stream ended without a `toolcall_end`), so
+ * the finalized `AssistantMessage` no longer reads as still-streaming.
  */
 function scrubPartialJson(partial: AssistantMessage): void {
 	for (const block of partial.content) {
-		if (block?.type === "toolCall") {
-			delete (block as ToolCall & { partialJson?: string }).partialJson;
-		}
+		if (block?.type === "toolCall") clearStreamingPartialJson(block);
 	}
 }
 
@@ -218,10 +223,10 @@ function scrubPartialJson(partial: AssistantMessage): void {
  *
  * Streaming `partialJson` for in-progress tool calls is accumulated in a
  * side-channel map keyed by `contentIndex` and also written onto the content
- * object (as a typed intersection field) so downstream renderers can read it
- * during streaming. The field is deleted at `toolcall_end` and scrubbed from
- * any remaining blocks at `done`/`error` to guarantee it never leaks into the
- * final `AssistantMessage`.
+ * object as a symbol-keyed field so downstream renderers can read it
+ * during streaming. The field is cleared at `toolcall_end` and scrubbed from any
+ * remaining blocks at `done`/`error` so the finalized `AssistantMessage` never
+ * reads as still-streaming.
  */
 function processProxyEvent(
 	model: Model,
@@ -240,9 +245,10 @@ function processProxyEvent(
 				totalTokens: 0,
 				cost: { input: 0, output: 0, cacheRead: 0, cacheWrite: 0, total: 0 },
 			};
-			delete (partial as { stopReason?: string }).stopReason;
-			delete (partial as { errorMessage?: string }).errorMessage;
-			delete (partial as { duration?: number }).duration;
+			partial.errorMessage = undefined;
+			partial.errorId = undefined;
+			partial.duration = undefined;
+			(partial as { stopReason?: string }).stopReason = undefined;
 			return { type: "start", partial };
 
 		case "text_start":
@@ -315,8 +321,8 @@ function processProxyEvent(
 				id: proxyEvent.id,
 				name: proxyEvent.toolName,
 				arguments: {},
-				partialJson: "",
-			} as ToolCall & { partialJson: string };
+				[kStreamingPartialJson]: "",
+			} as ToolCall & StreamingPartialJsonCarrier;
 			partialJsonByIndex.set(proxyEvent.contentIndex, "");
 			return { type: "toolcall_start", contentIndex: proxyEvent.contentIndex, partial };
 		case "toolcall_delta": {
@@ -325,7 +331,7 @@ function processProxyEvent(
 				const acc = (partialJsonByIndex.get(proxyEvent.contentIndex) ?? "") + proxyEvent.delta;
 				partialJsonByIndex.set(proxyEvent.contentIndex, acc);
 				content.arguments = parseStreamingJson(acc) || {};
-				(content as ToolCall & { partialJson: string }).partialJson = acc;
+				setStreamingPartialJson(content, acc);
 				partial.content[proxyEvent.contentIndex] = { ...content }; // Trigger reactivity
 				return {
 					type: "toolcall_delta",
@@ -341,7 +347,7 @@ function processProxyEvent(
 			const content = partial.content[proxyEvent.contentIndex];
 			if (content?.type === "toolCall") {
 				partialJsonByIndex.delete(proxyEvent.contentIndex);
-				delete (content as ToolCall & { partialJson?: string }).partialJson;
+				clearStreamingPartialJson(content);
 				return {
 					type: "toolcall_end",
 					contentIndex: proxyEvent.contentIndex,

package/src/types.ts

@@ -633,6 +633,27 @@ export interface AgentTool<TParameters extends TSchema = TSchema, TDetails = any
 	 */
 	matcherDigest?: (args: unknown) => string | undefined;
 
+	/**
+	 * Surface the target file paths a (potentially partial) streamed call would
+	 * touch, so path-scoped stream matchers (e.g. TTSR `tool:edit(*.ts)` globs)
+	 * can match without a top-level `path`/`paths` argument. Used for tools whose
+	 * wire grammar embeds paths inside the streamed payload (hashline section
+	 * headers, apply_patch envelope markers). Return `undefined` (or an empty
+	 * array) to fall back to the caller's top-level argument scan.
+	 */
+	matcherPaths?: (args: unknown) => readonly string[] | undefined;
+
+	/**
+	 * Per-file projection of a (potentially partial) streamed call, pairing each
+	 * touched file path with the digest of only the lines added to that file.
+	 * Path-scoped stream matchers (TTSR) evaluate each entry in isolation, so a
+	 * scoped rule like `tool:edit(*.ts)` never fires on text that actually
+	 * belongs to a sibling Markdown hunk in a multi-file payload. Takes
+	 * precedence over {@link matcherDigest} + {@link matcherPaths} when present;
+	 * returns `undefined` (or empty) to fall back to the combined hooks.
+	 */
+	matcherEntries?: (args: unknown) => readonly { path: string; digest: string }[] | undefined;
+
 	/** Capability tier declaration used by approval gates. Omitted means "exec". */
 	approval?: ToolApproval;