@oh-my-pi/pi-agent-core 14.5.0 → 14.5.2

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "14.5.0",
+	"version": "14.5.2",
 	"description": "General-purpose agent with transport abstraction, state management, and attachment support",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -35,13 +35,13 @@
 		"fmt": "biome format --write ."
 	},
 	"dependencies": {
-		"@oh-my-pi/pi-ai": "14.5.0",
-		"@oh-my-pi/pi-natives": "14.5.0",
-		"@oh-my-pi/pi-utils": "14.5.0"
+		"@oh-my-pi/pi-ai": "14.5.2",
+		"@oh-my-pi/pi-natives": "14.5.2",
+		"@oh-my-pi/pi-utils": "14.5.2"
 	},
 	"devDependencies": {
 		"@sinclair/typebox": "^0.34.49",
-		"@types/bun": "^1.3.13"
+		"@types/bun": "^1.3"
 	},
 	"engines": {
 		"bun": ">=1.3.7"

package/src/agent-loop.ts

@@ -126,7 +126,7 @@ function normalizeMessagesForProvider(
 
 export const INTENT_FIELD = "_i";
 
-function injectIntentIntoSchema(schema: unknown): unknown {
+function injectIntentIntoSchema(schema: unknown, mode: "require" | "optional" = "require"): unknown {
 	if (!schema || typeof schema !== "object" || Array.isArray(schema)) return schema;
 	const schemaRecord = schema as Record<string, unknown>;
 	const propertiesValue = schemaRecord.properties;
@@ -141,7 +141,7 @@ function injectIntentIntoSchema(schema: unknown): unknown {
 	if (INTENT_FIELD in properties) {
 		const { [INTENT_FIELD]: intentProp, ...rest } = properties;
 		const needsReorder = Object.keys(properties)[0] !== INTENT_FIELD;
-		const needsRequired = !required.includes(INTENT_FIELD);
+		const needsRequired = mode === "require" && !required.includes(INTENT_FIELD);
 		if (!needsReorder && !needsRequired) return schema;
 		return {
 			...schemaRecord,
@@ -157,16 +157,27 @@ function injectIntentIntoSchema(schema: unknown): unknown {
 			},
 			...properties,
 		},
-		required: [...required, INTENT_FIELD],
+		...(mode === "require" ? { required: [...required, INTENT_FIELD] } : {}),
 	};
 }
 
-function normalizeTools(tools: Context["tools"], injectIntent: boolean): Context["tools"] {
-	return tools?.map(tool => ({
-		...tool,
-		description: tool.description || "",
-		...(injectIntent && { parameters: injectIntentIntoSchema(tool.parameters) as typeof tool.parameters }),
-	}));
+function normalizeTools(tools: AgentContext["tools"], injectIntent: boolean): Context["tools"] {
+	injectIntent = injectIntent && Bun.env.PI_NO_INTENT !== "1";
+	return tools?.map(t => {
+		const intentMode = resolveIntentMode(t.intent);
+		const parameters =
+			injectIntent && intentMode !== "omit"
+				? (injectIntentIntoSchema(t.parameters, intentMode) as typeof t.parameters)
+				: t.parameters;
+		const description = t.description ?? "";
+		return { ...t, parameters, description };
+	});
+}
+
+function resolveIntentMode(intent: AgentTool["intent"]): "require" | "optional" | "omit" {
+	if (typeof intent === "function") return "omit";
+	if (intent === "optional" || intent === "omit") return intent;
+	return "require";
 }
 
 function extractIntent(args: Record<string, unknown>): { intent?: string; strippedArgs: Record<string, unknown> } {
@@ -542,6 +553,15 @@ async function executeToolCalls(
 			argsForExecution = strippedArgs;
 			if (intent) {
 				toolCall.intent = intent;
+			} else if (typeof tool?.intent === "function") {
+				try {
+					const derived = tool.intent(strippedArgs as never)?.trim();
+					if (derived) {
+						toolCall.intent = derived;
+					}
+				} catch {
+					// intent function must never break tool execution
+				}
 			}
 		}
 		record.args = argsForExecution;

package/src/types.ts

@@ -250,6 +250,16 @@ export interface AgentTool<TParameters extends TSchema = TSchema, TDetails = any
 	concurrency?: "shared" | "exclusive";
 	/** If true, argument validation errors are non-fatal: raw args are passed to execute() instead of returning an error to the LLM. */
 	lenientArgValidation?: boolean;
+	/**
+	 * 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, …).
+	 * - function: `_i` is NOT injected; intent is derived dynamically from (potentially partial / streaming) args.
+	 */
+	intent?: "omit" | "optional" | "require" | ((args: Partial<Static<TParameters>>) => string | undefined);
+
+	/** The main execution callback for this tool. */
 	execute: AgentToolExecFn<TParameters, TDetails, TTheme>;
 
 	/** Optional custom rendering for tool call display (returns UI component) */