@oh-my-pi/pi-agent-core 15.8.2 → 15.9.0

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 ## [Unreleased]
 
+## [15.8.3] - 2026-06-03
+### Added
+
+- Added `getReadToolPath(context)` to `@oh-my-pi/pi-agent-core/compaction/tool-protection` to extract a paired `read` tool call's `path` for embedders building read-targeted protection matchers
+- Added `getReadToolPath(context)` to `@oh-my-pi/pi-agent-core/compaction/tool-protection`: the shared primitive that extracts a paired `read` tool call's `path` argument, so embedders can build their own read-targeted compaction protection matchers (e.g. plan-file reads) the same way `isSkillReadToolResult` does.
+
 ## [15.8.2] - 2026-06-03
 
 ### Added
@@ -561,4 +567,4 @@ Initial release under @oh-my-pi scope. See previous releases at [badlogic/pi-mon
 
 - `Agent` constructor now has all options optional (empty options use defaults).
 
-- `queueMessage()` is now synchronous (no longer returns a Promise).
+- `queueMessage()` is now synchronous (no longer returns a Promise).

package/dist/types/compaction/tool-protection.d.ts

@@ -7,5 +7,11 @@ export interface ProtectedToolContext {
 }
 export type ProtectedToolMatcher = string | ((context: ProtectedToolContext) => boolean);
 export declare function collectToolCallsById(entries: readonly SessionEntry[]): Map<string, AgentToolCall>;
-export declare function isSkillReadToolResult({ toolResult, toolCall }: ProtectedToolContext): boolean;
+/**
+ * Extract the `path` argument from a paired `read` tool call, when the result
+ * is a `read` result carrying a string path. Returns `undefined` otherwise.
+ * Shared primitive for read-targeted protection matchers (skills, plans, …).
+ */
+export declare function getReadToolPath({ toolResult, toolCall }: ProtectedToolContext): string | undefined;
+export declare function isSkillReadToolResult(context: ProtectedToolContext): boolean;
 export declare function isProtectedToolResult(toolResult: ToolResultMessage, toolCall: AgentToolCall | undefined, matchers: readonly ProtectedToolMatcher[]): boolean;

package/dist/types/types.d.ts

@@ -366,7 +366,7 @@ export interface AgentTool<TParameters extends TSchema = TSchema, TDetails = any
      * Controls how the INTENT_FIELD (`_i`) is handled for this tool.
      * - `"require"` (default): `_i` is injected and required in the parameter schema.
      * - `"optional"`: `_i` is injected as an optional/nullable field.
-     * - `"omit"`: `_i` is NOT injected. Use for tools where intent is obvious (yield, resolve, todo_write, …).
+     * - `"omit"`: `_i` is NOT injected. Use for tools where intent is obvious (yield, resolve, todo, …).
      * - function: `_i` is NOT injected; intent is derived dynamically from (potentially partial / streaming) args.
      */
     intent?: "omit" | "optional" | "require" | ((args: Partial<Static<TParameters>>) => string | undefined);

package/package.json

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

package/src/compaction/tool-protection.ts

@@ -24,10 +24,19 @@ export function collectToolCallsById(entries: readonly SessionEntry[]): Map<stri
 	return toolCalls;
 }
 
-export function isSkillReadToolResult({ toolResult, toolCall }: ProtectedToolContext): boolean {
-	if (toolResult.toolName !== "read" || toolCall?.name !== "read") return false;
+/**
+ * Extract the `path` argument from a paired `read` tool call, when the result
+ * is a `read` result carrying a string path. Returns `undefined` otherwise.
+ * Shared primitive for read-targeted protection matchers (skills, plans, …).
+ */
+export function getReadToolPath({ toolResult, toolCall }: ProtectedToolContext): string | undefined {
+	if (toolResult.toolName !== "read" || toolCall?.name !== "read") return undefined;
 	const path = (toolCall.arguments as Record<string, unknown>).path;
-	return typeof path === "string" && path.startsWith(SKILL_INTERNAL_URL_PREFIX);
+	return typeof path === "string" ? path : undefined;
+}
+
+export function isSkillReadToolResult(context: ProtectedToolContext): boolean {
+	return getReadToolPath(context)?.startsWith(SKILL_INTERNAL_URL_PREFIX) ?? false;
 }
 
 export function isProtectedToolResult(

package/src/types.ts

@@ -436,7 +436,7 @@ export interface AgentTool<TParameters extends TSchema = TSchema, TDetails = any
 	 * Controls how the INTENT_FIELD (`_i`) is handled for this tool.
 	 * - `"require"` (default): `_i` is injected and required in the parameter schema.
 	 * - `"optional"`: `_i` is injected as an optional/nullable field.
-	 * - `"omit"`: `_i` is NOT injected. Use for tools where intent is obvious (yield, resolve, todo_write, …).
+	 * - `"omit"`: `_i` is NOT injected. Use for tools where intent is obvious (yield, resolve, todo, …).
 	 * - function: `_i` is NOT injected; intent is derived dynamically from (potentially partial / streaming) args.
 	 */
 	intent?: "omit" | "optional" | "require" | ((args: Partial<Static<TParameters>>) => string | undefined);