@oh-my-pi/pi-agent-core 15.3.0 → 15.3.1

package/dist/types/agent-loop.d.ts

@@ -2,7 +2,7 @@
  * Agent loop that works with AgentMessage throughout.
  * Transforms to Message[] only at the LLM call boundary.
  */
-import { EventStream } from "@oh-my-pi/pi-ai";
+import { type Context, EventStream } from "@oh-my-pi/pi-ai";
 import { type AgentRunCoverage, type AgentRunSummary } from "./run-collector";
 import type { AgentContext, AgentEvent, AgentLoopConfig, AgentMessage, StreamFn } from "./types";
 /**
@@ -52,3 +52,4 @@ export declare function agentLoopContinueDetailed(context: AgentContext, config:
     readonly detailed: () => Promise<AgentLoopDetailedResult>;
 };
 export declare const INTENT_FIELD = "_i";
+export declare function normalizeTools(tools: AgentContext["tools"], injectIntent: boolean): Context["tools"];

package/dist/types/append-only-context.d.ts

@@ -21,6 +21,11 @@ export interface StablePrefixSnapshot {
     tools: Tool[];
     fingerprint: string;
 }
+/** Options threaded through `build()` so the snapshot reflects loop-time settings. */
+export interface BuildOptions {
+    /** Inject the `_i` intent field into tool schemas (must match agent-loop's normalizeTools). */
+    intentTracing: boolean;
+}
 /**
  * A frozen prefix (system prompt + tools) that produces stable byte
  * sequences across `build()` calls.
@@ -38,7 +43,7 @@ export declare class StablePrefix {
      * Build or rebuild from live context.
      * Returns `true` if the prefix actually changed (cache miss imminent).
      */
-    build(context: AgentContext): boolean;
+    build(context: AgentContext, options: BuildOptions): boolean;
     /** Force rebuild on the next `build()` call. */
     invalidate(): void;
     /**
@@ -89,7 +94,7 @@ export declare class AppendOnlyContextManager {
     #private;
     readonly prefix: StablePrefix;
     readonly log: AppendOnlyLog;
-    build(context: AgentContext): Context;
+    build(context: AgentContext, options: BuildOptions): Context;
     /**
      * Sync normalized (provider-level) messages into the append-only log.
      *
@@ -104,5 +109,5 @@ export declare class AppendOnlyContextManager {
     appendMessage(message: any): void;
     replaceTailMessage(message: any): void;
     invalidate(): void;
-    reset(context: AgentContext): void;
+    reset(context: AgentContext, options: BuildOptions): void;
 }

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-agent-core",
-	"version": "15.3.0",
+	"version": "15.3.1",
 	"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.3.0",
-		"@oh-my-pi/pi-natives": "15.3.0",
-		"@oh-my-pi/pi-utils": "15.3.0",
+		"@oh-my-pi/pi-ai": "15.3.1",
+		"@oh-my-pi/pi-natives": "15.3.1",
+		"@oh-my-pi/pi-utils": "15.3.1",
 		"@opentelemetry/api": "^1.9.0"
 	},
 	"devDependencies": {

package/src/agent-loop.ts

@@ -363,7 +363,7 @@ function injectIntentIntoSchema(schema: unknown, mode: "require" | "optional" =
 	};
 }
 
-function normalizeTools(tools: AgentContext["tools"], injectIntent: boolean): Context["tools"] {
+export 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);
@@ -652,7 +652,7 @@ async function streamAssistantResponse(
 	let llmContext: Context;
 	if (config.appendOnlyContext) {
 		config.appendOnlyContext.syncMessages(normalizedMessages);
-		llmContext = config.appendOnlyContext.build(context);
+		llmContext = config.appendOnlyContext.build(context, { intentTracing: !!config.intentTracing });
 	} else {
 		llmContext = {
 			systemPrompt: context.systemPrompt,

package/src/append-only-context.ts

@@ -15,7 +15,8 @@
  */
 
 import type { Context, Message, Tool } from "@oh-my-pi/pi-ai";
-import type { AgentContext, AgentTool } from "./types";
+import { normalizeTools } from "./agent-loop";
+import type { AgentContext } from "./types";
 
 // ---------------------------------------------------------------------------
 // StablePrefix (formerly ImmutablePrefix)
@@ -28,6 +29,12 @@ export interface StablePrefixSnapshot {
 	fingerprint: string;
 }
 
+/** Options threaded through `build()` so the snapshot reflects loop-time settings. */
+export interface BuildOptions {
+	/** Inject the `_i` intent field into tool schemas (must match agent-loop's normalizeTools). */
+	intentTracing: boolean;
+}
+
 /**
  * A frozen prefix (system prompt + tools) that produces stable byte
  * sequences across `build()` calls.
@@ -54,8 +61,8 @@ export class StablePrefix {
 	 * Build or rebuild from live context.
 	 * Returns `true` if the prefix actually changed (cache miss imminent).
 	 */
-	build(context: AgentContext): boolean {
-		const snapshot = takeSnapshot(context);
+	build(context: AgentContext, options: BuildOptions): boolean {
+		const snapshot = takeSnapshot(context, options);
 		if (this.#snapshot && this.#snapshot.fingerprint === snapshot.fingerprint) {
 			return false;
 		}
@@ -154,8 +161,8 @@ export class AppendOnlyContextManager {
 	/** Rolling digest of synced message content — detects in-place rewrites. */
 	#syncedDigest = 0;
 
-	build(context: AgentContext): Context {
-		this.prefix.build(context);
+	build(context: AgentContext, options: BuildOptions): Context {
+		this.prefix.build(context, options);
 		const { systemPrompt, tools } = this.prefix.toContext();
 		return { systemPrompt, messages: this.log.toMessages(), tools };
 	}
@@ -219,25 +226,36 @@ export class AppendOnlyContextManager {
 		this.prefix.invalidate();
 	}
 
-	reset(context: AgentContext): void {
+	reset(context: AgentContext, options: BuildOptions): void {
 		this.prefix.invalidate();
 		this.log.clear();
 		this.#lastSyncCount = 0;
 		this.#syncedDigest = 0;
-		this.prefix.build(context);
+		this.prefix.build(context, options);
 	}
 
-	/** Fast rolling digest of message content. */
-	#computeDigest(messages: any[]): number {
+	/**
+	 * Deterministic digest over every field the provider may serialize — role,
+	 * content, tool calls (both `toolCalls` and OpenAI-wire `tool_calls`),
+	 * `tool_call_id`, `name`, `id`. Hashed with the same FNV-style rolling
+	 * accumulator so in-place rewrites of *any* of these fields are visible.
+	 */
+	#computeDigest(messages: readonly unknown[]): number {
 		let hash = 0;
 		for (let i = 0; i < messages.length; i++) {
 			const msg = messages[i];
-			if (msg && typeof msg === "object") {
-				const payload =
-					String(msg.role) + (typeof msg.content === "string" ? msg.content : JSON.stringify(msg.content ?? ""));
-				for (let j = 0; j < payload.length; j++) {
-					hash = ((hash << 5) - hash + payload.charCodeAt(j)) | 0;
-				}
+			if (!msg || typeof msg !== "object") continue;
+			const m = msg as Record<string, unknown>;
+			const payload = JSON.stringify({
+				r: m.role ?? null,
+				c: m.content ?? null,
+				tc: m.toolCalls ?? m.tool_calls ?? null,
+				tcid: m.tool_call_id ?? null,
+				n: m.name ?? null,
+				id: m.id ?? null,
+			});
+			for (let j = 0; j < payload.length; j++) {
+				hash = ((hash << 5) - hash + payload.charCodeAt(j)) | 0;
 			}
 		}
 		return hash >>> 0;
@@ -248,30 +266,17 @@ export class AppendOnlyContextManager {
 // Snapshot helpers
 // ---------------------------------------------------------------------------
 
-/**
- * Produce a stable serialization of tools that matches what
- * `normalizeTools(tools, false)` outputs (no intent injection).
- *
- * The spread `{ ...agentTool }` preserves all own enumerable properties
- * that survive JSON.stringify — functions are dropped, but strings,
- * booleans, objects are included.
- */
-function normalizeTool(t: AgentTool): Tool {
-	const description = t.description ?? "";
-	return { ...t, parameters: t.parameters, description };
-}
-
-function takeSnapshot(context: AgentContext): StablePrefixSnapshot {
+function takeSnapshot(context: AgentContext, options: BuildOptions): StablePrefixSnapshot {
 	const systemPrompt = [...context.systemPrompt];
-	const tools = (context.tools ?? []).map(normalizeTool);
+	const tools = normalizeTools(context.tools, options.intentTracing) ?? [];
 	return {
 		systemPrompt,
 		tools,
-		fingerprint: computeFingerprint(systemPrompt, tools),
+		fingerprint: computeFingerprint(systemPrompt, tools, options),
 	};
 }
 
-function computeFingerprint(systemPrompt: string[], tools: Tool[]): string {
+function computeFingerprint(systemPrompt: string[], tools: Tool[], options: BuildOptions): string {
 	const payload = JSON.stringify({
 		s: systemPrompt,
 		t: tools.map(t => ({
@@ -282,6 +287,7 @@ function computeFingerprint(systemPrompt: string[], tools: Tool[]): string {
 			cf: t.customFormat,
 			cw: t.customWireName,
 		})),
+		i: options.intentTracing,
 	});
 	let hash = 0;
 	for (let i = 0; i < payload.length; i++) {