@oh-my-pi/pi-agent-core 16.5.2 → 17.0.1

package/CHANGELOG.md

@@ -2,6 +2,21 @@
 
 ## [Unreleased]
 
+## [17.0.0] - 2026-07-15
+
+### Breaking Changes
+
+- Replaced the irc, job, and launch tools with a unified hub tool.
+- Removed the tool discovery system (including the search-tool-bm25 tool) and its associated configuration settings (tools.discoveryMode, tools.essentialOverride, mcp.discoveryMode, and mcp.discoveryDefaultServers).
+- Removed the resolve tool; plan approval and preview actions now use writes to the xd://propose virtual device path.
+
+### Added
+
+- Introduced the xd:// virtual device protocol for mounting tools as URLs readable/writable via read/write tools, configurable via the new tools.xdev setting (defaults to true).
+- Added the hub tool, consolidating agent peer messaging, background job control, and supervised long-running processes.
+- Added the edit.enforceSeenLines configuration setting (defaults to false) to optionally reject edits on lines that have not been fully displayed.
+- Added the ToolLoadMode type and an optional satisfies predicate to SoftToolRequirement to support compliance checks against specific invocation shapes (such as writing to a virtual device path).
+
 ## [16.5.2] - 2026-07-14
 
 ### Fixed

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

@@ -84,11 +84,6 @@ export interface TtsrInjectionEntry extends SessionEntryBase {
     /** Names of rules that were injected */
     injectedRules: string[];
 }
-export interface MCPToolSelectionEntry extends SessionEntryBase {
-    type: "mcp_tool_selection";
-    /** MCP tool names selected for visibility in discovery mode. */
-    selectedToolNames: string[];
-}
 export interface SessionInitEntry extends SessionEntryBase {
     type: "session_init";
     /** Full system prompt sent to the model */
@@ -109,7 +104,7 @@ export interface ModeChangeEntry extends SessionEntryBase {
 }
 export interface CustomCompactionSessionEntries {
 }
-export type SessionEntry = SessionMessageEntry | ThinkingLevelChangeEntry | ModelChangeEntry | ServiceTierChangeEntry | CompactionEntry | BranchSummaryEntry | CustomEntry | CustomMessageEntry | LabelEntry | TitleChangeEntry | TtsrInjectionEntry | MCPToolSelectionEntry | SessionInitEntry | ModeChangeEntry | CustomCompactionSessionEntries[keyof CustomCompactionSessionEntries];
+export type SessionEntry = SessionMessageEntry | ThinkingLevelChangeEntry | ModelChangeEntry | ServiceTierChangeEntry | CompactionEntry | BranchSummaryEntry | CustomEntry | CustomMessageEntry | LabelEntry | TitleChangeEntry | TtsrInjectionEntry | SessionInitEntry | ModeChangeEntry | CustomCompactionSessionEntries[keyof CustomCompactionSessionEntries];
 export interface ReadonlySessionManager {
     getBranch(leafId?: string | null): SessionEntry[];
     getEntry(id: string): SessionEntry | undefined;

package/dist/types/proxy.d.ts

@@ -2,7 +2,7 @@
  * Proxy stream function for apps that route LLM calls through a server.
  * The server manages auth and proxies requests to LLM providers.
  */
-import { type AssistantMessage, type AssistantMessageEvent, type Context, EventStream, type FetchImpl, type Model, type SimpleStreamOptions, type StopReason } from "@oh-my-pi/pi-ai";
+import { type AssistantMessage, type AssistantMessageEvent, type Context, EventStream, type FetchImpl, type ImageContent, type Model, type SimpleStreamOptions, type StopReason } from "@oh-my-pi/pi-ai";
 export declare class ProxyMessageEventStream extends EventStream<AssistantMessageEvent, AssistantMessage> {
     constructor();
 }
@@ -33,6 +33,10 @@ export type ProxyAssistantMessageEvent = {
     type: "thinking_end";
     contentIndex: number;
     contentSignature?: string;
+} | {
+    type: "image_end";
+    contentIndex: number;
+    content: ImageContent;
 } | {
     type: "toolcall_start";
     contentIndex: number;

package/dist/types/types.d.ts

@@ -43,6 +43,16 @@ export interface SoftToolRequirement {
     id: string;
     /** Tool that must be called before the loop runs other tools or yields. */
     toolName: string;
+    /**
+     * Per-call compliance check: a turn satisfies the requirement only when every
+     * tool call passes. Defaults to `name === toolName`. Lets a host demand a
+     * specific invocation shape (e.g. `write` targeting a virtual device path)
+     * instead of any call to `toolName`. Escalation still forces `toolName`.
+     */
+    satisfies?(toolCall: {
+        name: string;
+        arguments?: Record<string, unknown>;
+    }): boolean;
     /** Host-owned reminder messages, injected once per `id` activation. */
     reminder: AgentMessage[];
 }
@@ -508,6 +518,16 @@ export interface RenderResultOptions {
 }
 /** Capability tier a tool exercises. Determines which approval modes auto-approve it. */
 export type ToolTier = "read" | "write" | "exec";
+/**
+ * How an enabled tool is presented to the model. `"essential"` tools are exposed
+ * as normal top-level tools. `"discoverable"` tools are removed from the top-level
+ * schema and either mounted under `xd://` device URLs (when that transport is
+ * active) or surfaced through BM25 tool search — keeping their schemas off every
+ * request. Selection (settings, `hidden`, `defaultInactive`, explicit `--tools`,
+ * provider availability) decides whether a tool is enabled; `loadMode` only
+ * decides how an enabled tool is presented.
+ */
+export type ToolLoadMode = "essential" | "discoverable";
 /**
  * Per-tool approval declaration.
  * - bare tier ("read" / "write" / "exec") — static classification.
@@ -536,8 +556,8 @@ export interface AgentTool<TParameters extends TSchema = TSchema, TDetails = any
     hidden?: boolean;
     /** If true, tool can stage a pending action that requires explicit resolution via the resolve tool. */
     deferrable?: boolean;
-    /** Built-in tool loading behavior. "essential" loads initially; "discoverable" can be activated by tool search. */
-    loadMode?: "essential" | "discoverable";
+    /** How an enabled tool is presented. See {@link ToolLoadMode}. Omitted is treated as `"essential"` for built-ins; custom-tool adapters normalize omission to `"discoverable"`. */
+    loadMode?: ToolLoadMode;
     /** Short one-line summary used for tool discovery indexes. */
     summary?: string;
     /**

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "16.5.2",
+	"version": "17.0.1",
 	"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.5.2",
-		"@oh-my-pi/pi-catalog": "16.5.2",
-		"@oh-my-pi/pi-natives": "16.5.2",
-		"@oh-my-pi/pi-utils": "16.5.2",
-		"@oh-my-pi/pi-wire": "16.5.2",
-		"@oh-my-pi/snapcompact": "16.5.2",
+		"@oh-my-pi/pi-ai": "17.0.1",
+		"@oh-my-pi/pi-catalog": "17.0.1",
+		"@oh-my-pi/pi-natives": "17.0.1",
+		"@oh-my-pi/pi-utils": "17.0.1",
+		"@oh-my-pi/pi-wire": "17.0.1",
+		"@oh-my-pi/snapcompact": "17.0.1",
 		"@opentelemetry/api": "^1.9.1"
 	},
 	"devDependencies": {

package/src/agent-loop.ts

@@ -67,6 +67,7 @@ import type {
 	AgentToolResult,
 	AgentTurnEndContext,
 	AsideMessage,
+	SoftToolRequirement,
 	SteeringInterruptSource,
 	SteeringQueueState,
 	StreamFn,
@@ -182,6 +183,7 @@ type AssistantToolCallBlock = Extract<AssistantContentBlock, { type: "toolCall"
 function snapshotAssistantContentBlock(block: AssistantContentBlock): AssistantContentBlock {
 	switch (block.type) {
 		case "text":
+		case "image":
 			return { ...block };
 		case "thinking":
 			return { ...block };
@@ -223,6 +225,7 @@ function snapshotAssistantMessageEvent(
 		case "text_start":
 		case "text_delta":
 		case "text_end":
+		case "image_end":
 		case "thinking_start":
 		case "thinking_delta":
 		case "thinking_end":
@@ -803,6 +806,7 @@ async function runLoopBody(
 		// getToolChoice is never advanced twice; the flag resets at the message boundary.
 		let hostToolChoice: ToolChoice | undefined;
 		let softRequiredTool: string | undefined;
+		let softSatisfies: SoftToolRequirement["satisfies"];
 		let directiveResolvedForTurn = false;
 
 		// Outer loop: continues when queued follow-up messages arrive after agent would stop
@@ -861,6 +865,7 @@ async function runLoopBody(
 					const softReq = isSoftToolRequirement(directive) ? directive : undefined;
 					hostToolChoice = directive === undefined || isSoftToolRequirement(directive) ? undefined : directive;
 					softRequiredTool = softReq?.toolName;
+					softSatisfies = softReq?.satisfies;
 					if (softReq !== undefined) {
 						if (softReq.id !== softRequirementId) {
 							softRequirementId = softReq.id;
@@ -1024,7 +1029,7 @@ async function runLoopBody(
 				const calledOnlyRequiredTool =
 					softRequiredTool !== undefined &&
 					toolCalls.length > 0 &&
-					toolCalls.every(toolCall => toolCall.name === softRequiredTool);
+					toolCalls.every(toolCall => softSatisfies?.(toolCall) ?? toolCall.name === softRequiredTool);
 				const softGateActive =
 					softRequiredTool !== undefined && !hardToolChoiceBlocks(config.toolChoice, softRequiredTool);
 				const softNonCompliant = softGateActive && !calledOnlyRequiredTool;
@@ -1522,6 +1527,7 @@ async function streamAssistantResponse(
 						case "text_start":
 						case "text_delta":
 						case "text_end":
+						case "image_end":
 						case "thinking_start":
 						case "thinking_delta":
 						case "thinking_end":

package/src/compaction/branch-summarization.ts

@@ -200,7 +200,6 @@ function getMessageFromEntry(entry: SessionEntry): AgentMessage | undefined {
 		case "label":
 		case "service_tier_change":
 		case "ttsr_injection":
-		case "mcp_tool_selection":
 		case "session_init":
 		case "mode_change":
 			return undefined;

package/src/compaction/entries.ts

@@ -97,12 +97,6 @@ export interface TtsrInjectionEntry extends SessionEntryBase {
 	injectedRules: string[];
 }
 
-export interface MCPToolSelectionEntry extends SessionEntryBase {
-	type: "mcp_tool_selection";
-	/** MCP tool names selected for visibility in discovery mode. */
-	selectedToolNames: string[];
-}
-
 export interface SessionInitEntry extends SessionEntryBase {
 	type: "session_init";
 	/** Full system prompt sent to the model */
@@ -137,7 +131,6 @@ export type SessionEntry =
 	| LabelEntry
 	| TitleChangeEntry
 	| TtsrInjectionEntry
-	| MCPToolSelectionEntry
 	| SessionInitEntry
 	| ModeChangeEntry
 	| CustomCompactionSessionEntries[keyof CustomCompactionSessionEntries];

package/src/proxy.ts

@@ -8,6 +8,7 @@ import {
 	type Context,
 	EventStream,
 	type FetchImpl,
+	type ImageContent,
 	type Model,
 	type SimpleStreamOptions,
 	type StopReason,
@@ -47,6 +48,7 @@ export type ProxyAssistantMessageEvent =
 	| { type: "thinking_start"; contentIndex: number }
 	| { type: "thinking_delta"; contentIndex: number; delta: string }
 	| { type: "thinking_end"; contentIndex: number; contentSignature?: string }
+	| { type: "image_end"; contentIndex: number; content: ImageContent }
 	| { type: "toolcall_start"; contentIndex: number; id: string; toolName: string }
 	| { type: "toolcall_delta"; contentIndex: number; delta: string }
 	| { type: "toolcall_end"; contentIndex: number }
@@ -315,6 +317,15 @@ function processProxyEvent(
 			throw new Error("Received thinking_end for non-thinking content");
 		}
 
+		case "image_end":
+			partial.content[proxyEvent.contentIndex] = proxyEvent.content;
+			return {
+				type: "image_end",
+				contentIndex: proxyEvent.contentIndex,
+				content: proxyEvent.content,
+				partial,
+			};
+
 		case "toolcall_start":
 			partial.content[proxyEvent.contentIndex] = {
 				type: "toolCall",

package/src/telemetry.ts

@@ -957,6 +957,9 @@ function assistantContentToOtelParts(content: AssistantMessage["content"]): Otel
 			case "text":
 				parts.push({ type: "text", content: part.text });
 				break;
+			case "image":
+				parts.push({ type: "blob", modality: "image", mime_type: part.mimeType, content: part.data });
+				break;
 			case "thinking":
 				parts.push({ type: "reasoning", content: part.thinking });
 				break;

package/src/types.ts

@@ -68,6 +68,13 @@ export interface SoftToolRequirement {
 	id: string;
 	/** Tool that must be called before the loop runs other tools or yields. */
 	toolName: string;
+	/**
+	 * Per-call compliance check: a turn satisfies the requirement only when every
+	 * tool call passes. Defaults to `name === toolName`. Lets a host demand a
+	 * specific invocation shape (e.g. `write` targeting a virtual device path)
+	 * instead of any call to `toolName`. Escalation still forces `toolName`.
+	 */
+	satisfies?(toolCall: { name: string; arguments?: Record<string, unknown> }): boolean;
 	/** Host-owned reminder messages, injected once per `id` activation. */
 	reminder: AgentMessage[];
 }
@@ -587,6 +594,17 @@ export interface RenderResultOptions {
 /** Capability tier a tool exercises. Determines which approval modes auto-approve it. */
 export type ToolTier = "read" | "write" | "exec";
 
+/**
+ * How an enabled tool is presented to the model. `"essential"` tools are exposed
+ * as normal top-level tools. `"discoverable"` tools are removed from the top-level
+ * schema and either mounted under `xd://` device URLs (when that transport is
+ * active) or surfaced through BM25 tool search — keeping their schemas off every
+ * request. Selection (settings, `hidden`, `defaultInactive`, explicit `--tools`,
+ * provider availability) decides whether a tool is enabled; `loadMode` only
+ * decides how an enabled tool is presented.
+ */
+export type ToolLoadMode = "essential" | "discoverable";
+
 /**
  * Per-tool approval declaration.
  * - bare tier ("read" / "write" / "exec") — static classification.
@@ -625,8 +643,8 @@ export interface AgentTool<TParameters extends TSchema = TSchema, TDetails = any
 	hidden?: boolean;
 	/** If true, tool can stage a pending action that requires explicit resolution via the resolve tool. */
 	deferrable?: boolean;
-	/** Built-in tool loading behavior. "essential" loads initially; "discoverable" can be activated by tool search. */
-	loadMode?: "essential" | "discoverable";
+	/** How an enabled tool is presented. See {@link ToolLoadMode}. Omitted is treated as `"essential"` for built-ins; custom-tool adapters normalize omission to `"discoverable"`. */
+	loadMode?: ToolLoadMode;
 	/** Short one-line summary used for tool discovery indexes. */
 	summary?: string;
 	/**