@oh-my-pi/pi-agent-core 18.14.1 → 18.14.2

package/CHANGELOG.md

@@ -2,6 +2,16 @@
 
 ## [Unreleased]
 
+## [18.14.2] - 2026-10-31
+
+### Added
+
+- The soft steering signal a tool sees as `toolCall.steeringSignal` now aborts with the source of the queued message (`"user"`, `"irc"`, …) as its `reason`, so a tool can yield only to the sources it should ([#17537](https://github.com/can1357/oh-my-pi/issues/17537))
+
+### Fixed
+
+- Reject incomplete local compaction, turn-prefix, handoff, and branch summaries ending with `stopReason: "length"` instead of using truncated text as complete context. Omit incomplete display-only short summaries without discarding a completed full summary ([#17499](https://github.com/can1357/oh-my-pi/pull/17499) by [@Xytronix](https://github.com/Xytronix)).
+
 ## [18.14.1] - 2026-10-31
 
 ### Breaking Changes

package/dist/types/compaction/utils.d.ts

@@ -10,6 +10,8 @@ export interface FileOperations {
     edited: Set<string>;
 }
 export declare function createFileOps(): FileOperations;
+/** Error text for a summarization call that hit its output limit (`stopReason: "length"`): partial text is never used. */
+export declare function incompleteOutputError(prefix: string): string;
 /**
  * Split a read-tool path into its base path and trailing selector, mirroring the
  * read tool's own splitter. Single source of the grammar in this package: the

package/dist/types/types.d.ts

@@ -588,7 +588,8 @@ export interface ToolCallContext {
     providerMetadata?: ToolCallProviderMetadata;
     /**
      * Cooperative steering signal: aborted when a queued user/steering message
-     * (or an interrupting peer IRC) is detected while this tool batch runs.
+     * (or an interrupting peer IRC) is detected while this tool batch runs. Its
+     * `reason` names the source: a {@link SteeringInterruptSource} or `"irc"`.
      * Unlike the hard abort signal it NEVER kills the tool — long-running
      * tools MAY observe it (via `ctx.toolCall.steeringSignal`) to finish early
      * or background themselves so the message injects promptly; ignoring it is

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "18.14.1",
+	"version": "18.14.2",
 	"description": "General-purpose agent with transport abstraction, state management, and attachment support",
 	"homepage": "https://omp.sh",
 	"author": {
@@ -38,16 +38,16 @@
 		"fmt": "oxfmt --no-error-on-unmatched-pattern 'src/**/*.{ts,tsx}' '{test,bench,examples,scripts}/**/*.ts' '*.ts'"
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "18.14.1",
-		"@oh-my-pi/pi-catalog": "18.14.1",
-		"@oh-my-pi/pi-natives": "18.14.1",
-		"@oh-my-pi/pi-utils": "18.14.1",
-		"@oh-my-pi/pi-wire": "18.14.1",
-		"@oh-my-pi/snapcompact": "18.14.1",
+		"@oh-my-pi/pi-ai": "18.14.2",
+		"@oh-my-pi/pi-catalog": "18.14.2",
+		"@oh-my-pi/pi-natives": "18.14.2",
+		"@oh-my-pi/pi-utils": "18.14.2",
+		"@oh-my-pi/pi-wire": "18.14.2",
+		"@oh-my-pi/snapcompact": "18.14.2",
 		"@opentelemetry/api": "^1.9.1"
 	},
 	"devDependencies": {
-		"@oh-my-pi/omptype": "18.14.1",
+		"@oh-my-pi/omptype": "18.14.2",
 		"@opentelemetry/context-async-hooks": "^2.9.0",
 		"@opentelemetry/sdk-trace-base": "^2.9.0",
 		"@types/bun": "^1.3.14"

package/src/agent-loop.ts

@@ -3402,7 +3402,7 @@ async function executeToolCalls(
 		// also cascades: the freshly detached job's own completion re-triggers
 		// this check for the next command, so millisecond-long commands chain
 		// into separate background deliveries (#12869).
-		if (source === "irc" && softInterrupts) steeringSoftController.abort();
+		if (source === "irc" && softInterrupts) steeringSoftController.abort("irc");
 	};
 
 	const checkSteering = async (): Promise<void> => {
@@ -3441,7 +3441,7 @@ async function executeToolCalls(
 				interruptState.triggered = true;
 				interruptState.source = steeringSource ?? "unknown";
 				steeringAbortController.abort(TOOL_INTERRUPT_ABORT_REASON);
-				if (softInterrupts) steeringSoftController.abort();
+				if (softInterrupts) steeringSoftController.abort(interruptState.source);
 			}
 			return;
 		}

package/src/compaction/branch-summarization.ts

@@ -28,6 +28,7 @@ import {
 	createFileOps,
 	extractFileOpsFromMessage,
 	type FileOperations,
+	incompleteOutputError,
 	SUMMARIZATION_SYSTEM_PROMPT,
 	serializeConversationForSummary,
 	stripReadSelector,
@@ -386,6 +387,9 @@ export async function generateBranchSummary(
 	if (response.stopReason === "error") {
 		return { error: response.errorMessage || "Summarization failed" };
 	}
+	if (response.stopReason === "length") {
+		return { error: incompleteOutputError("Summarization failed") };
+	}
 
 	let summary = response.content
 		.filter((c): c is { type: "text"; text: string } => c.type === "text")

package/src/compaction/compaction.ts

@@ -91,6 +91,7 @@ import {
 	escapeSummaryBoundaryTags,
 	extractFileOpsFromMessage,
 	type FileOperations,
+	incompleteOutputError,
 	SUMMARIZATION_SYSTEM_PROMPT,
 	serializeConversationForSummary,
 	stripReadSelector,
@@ -1075,6 +1076,9 @@ async function summarizeConversationWindow(
 	if (response.stopReason === "error") {
 		throw createSummarizationError("Summarization failed", response);
 	}
+	if (response.stopReason === "length") {
+		throw new Error(incompleteOutputError("Summarization failed"));
+	}
 
 	const textContent = response.content
 		.filter((c): c is { type: "text"; text: string } => c.type === "text")
@@ -1182,6 +1186,9 @@ export async function generateHandoffFromContext(
 	if (response.stopReason === "error") {
 		throw createSummarizationError("Handoff generation failed", response);
 	}
+	if (response.stopReason === "length") {
+		throw new Error(incompleteOutputError("Handoff generation failed"));
+	}
 
 	return response.content
 		.filter((c): c is { type: "text"; text: string } => c.type === "text")
@@ -1305,8 +1312,10 @@ async function generateShortSummary(
 				case "error":
 					outcome = { kind: "failed", error: response.errorMessage || "Unknown error" };
 					break;
-				case "stop":
 				case "length":
+					outcome = { kind: "failed", error: incompleteOutputError("Short summary failed") };
+					break;
+				case "stop":
 				case "toolUse":
 					outcome = {
 						kind: "text",
@@ -2435,6 +2444,9 @@ async function generateTurnPrefixSummary(
 	if (response.stopReason === "error") {
 		throw createSummarizationError("Turn prefix summarization failed", response);
 	}
+	if (response.stopReason === "length") {
+		throw new Error(incompleteOutputError("Turn prefix summarization failed"));
+	}
 
 	return response.content
 		.filter((c): c is { type: "text"; text: string } => c.type === "text")

package/src/compaction/utils.ts

@@ -28,6 +28,11 @@ export function createFileOps(): FileOperations {
 	};
 }
 
+/** Error text for a summarization call that hit its output limit (`stopReason: "length"`): partial text is never used. */
+export function incompleteOutputError(prefix: string): string {
+	return `${prefix}: model output ended incomplete (stopReason: length)`;
+}
+
 // Read-tool selector grammar, mirrored from the conservative filesystem splitter in
 // packages/coding-agent/src/tools/path-utils.ts (splitPathAndSel). Keep in sync.
 // A trailing `:chunk` is a selector only when it is a line-range list

package/src/types.ts

@@ -672,7 +672,8 @@ export interface ToolCallContext {
 	providerMetadata?: ToolCallProviderMetadata;
 	/**
 	 * Cooperative steering signal: aborted when a queued user/steering message
-	 * (or an interrupting peer IRC) is detected while this tool batch runs.
+	 * (or an interrupting peer IRC) is detected while this tool batch runs. Its
+	 * `reason` names the source: a {@link SteeringInterruptSource} or `"irc"`.
 	 * Unlike the hard abort signal it NEVER kills the tool — long-running
 	 * tools MAY observe it (via `ctx.toolCall.steeringSignal`) to finish early
 	 * or background themselves so the message injects promptly; ignoring it is