@estebanforge/pi-antigravity-bridge 1.6.1 → 1.6.2

package/CHANGELOG.md

@@ -2,6 +2,12 @@
 
 All notable changes to this project will be documented in this file.
 
+## [1.6.2] - 2026-09-19
+
+### Fixed
+
+- **Compatibility with host 0.86.0's normalized provider stream input.** Stream entry points now receive a branded `TranscriptContext` (`{ messages }` only): the `context.systemPrompt` / `context.tools` fields are gone, and only `normalizeContext()` produces the new type. G10 fresh-conversation delivery read `context.systemPrompt`, so every fresh agy conversation silently lost the system prompt block (operating instructions, AGENTS.md project context, tool-priority note); the prompt now comes from the transcript via `getCurrentSystemPrompt(context.messages)`. `emitToolUse` arguments are restricted to `Record<string, JsonValue>` per the tightened `ToolCall.arguments` contract (driver args arrive JSON-decoded off the wire, so the two call-site casts are sound). Test fixtures in `provider-{sysprompt,streaming,late-result,escalation}.test.ts` and the `scripts/test-provider.ts` smoke context wrap raw objects with `normalizeContext()`. Dev pins `@earendil-works/*` raised `^0.85.0` to `^0.86.0`; `tsc` clean, 360 tests green.
+
 ## [1.6.1] - 2026-09-18
 
 ### Added

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@estebanforge/pi-antigravity-bridge",
-  "version": "1.6.1",
+  "version": "1.6.2",
   "description": "Gemini provider for Pi on the Antigravity ACP server (official Google ACP) or the stream-json agy CLI. antigravity/* models in Pi's /model picker, no-patch MCP bridge: agy runs Pi's tools. ToS safe to use.",
   "keywords": [
     "pi-package",
@@ -59,10 +59,10 @@
     }
   },
   "devDependencies": {
-    "@earendil-works/pi-ai": "^0.85.0",
-    "@earendil-works/pi-coding-agent": "^0.85.0",
-    "@earendil-works/pi-server": "^0.85.0",
-    "@earendil-works/pi-tui": "^0.85.0",
+    "@earendil-works/pi-ai": "^0.86.0",
+    "@earendil-works/pi-coding-agent": "^0.86.0",
+    "@earendil-works/pi-server": "^0.86.0",
+    "@earendil-works/pi-tui": "^0.86.0",
     "@types/node": "^22.0.0",
     "tsx": "^4.19.0",
     "typebox": "^1.1.38",

package/src/provider.ts

@@ -19,13 +19,15 @@
 
 import {
 	createAssistantMessageEventStream,
+	getCurrentSystemPrompt,
 	type AssistantMessage,
 	type AssistantMessageEventStream,
-	type Context,
+	type JsonValue,
 	type Message,
 	type Model,
 	type SimpleStreamOptions,
 	type ThinkingLevel,
+	type TranscriptContext,
 	type Usage,
 } from "@earendil-works/pi-ai";
 import fs from "node:fs";
@@ -59,7 +61,7 @@ function zeroUsage(): Usage {
 /** Extract the latest user message as a flat prompt string. agy maintains its
  *  own conversation history via --conversation, so we collapse pi's structured
  *  message to text. Returns null if the last message isn't a user message. */
-function extractUserPrompt(context: Context): string | null {
+function extractUserPrompt(context: TranscriptContext): string | null {
 	const last = context.messages[context.messages.length - 1];
 	if (!last || last.role !== "user") return null;
 	const content = last.content;
@@ -75,7 +77,7 @@ function extractUserPrompt(context: Context): string | null {
 /** Image blocks of the latest user message (pi-ai ImageContent: base64 data
  *  + mimeType). The ACP engine forwards them as typed content blocks; the
  *  stream-json CLI prompt is text-only, so its driver simply ignores these. */
-function extractImages(context: Context): Array<{ data: string; mimeType: string }> {
+function extractImages(context: TranscriptContext): Array<{ data: string; mimeType: string }> {
 	const last = context.messages[context.messages.length - 1];
 	if (!last || last.role !== "user" || typeof last.content === "string") return [];
 	return last.content
@@ -100,7 +102,8 @@ const DIGEST_PREAMBLE =
 
 // --- pi system prompt (G10) ----------------------------------------------------
 //
-// pi composes context.systemPrompt every turn: its own operating instructions
+// pi normalizes the system prompt into the transcript every turn (read via
+// getCurrentSystemPrompt): its own operating instructions
 // plus every AGENTS.md/CLAUDE.md it loaded (global agent dir first, then
 // ancestors). The provider used to drop it, so agy models never saw the user's
 // machine-level or project-level instructions. agy has no system-prompt flag
@@ -921,13 +924,14 @@ function nextRtId(kind: "nat" | "wrap"): string {
 	return `${kind}-${++RT_SEQ}`;
 }
 
-/** Emit a complete toolCall block and end the pi call with toolUse. */
+/** Emit a complete toolCall block and end the pi call with toolUse.
+ *  pi 0.86 restricts ToolCall.arguments to JSON-compatible values. */
 function emitToolUse(
 	stream: AssistantMessageEventStream,
 	blocks: BlockState,
 	id: string,
 	name: string,
-	args: Record<string, unknown>,
+	args: Record<string, JsonValue>,
 ): void {
 	const partial = blocks.partial;
 	closeThinking(stream, blocks);
@@ -1050,7 +1054,7 @@ function acpEditFileArg(args: Record<string, unknown>): string | undefined {
 				emitToolUse(stream, blocks, id, mapped.tool, {
 					reasoning: `re-exec of agy ${activity.name} for display`,
 					...mapped.args,
-				});
+				} as Record<string, JsonValue>);
 				return "parked";
 			}
 			{
@@ -1067,7 +1071,9 @@ function acpEditFileArg(args: Record<string, unknown>): string | undefined {
 		case "bridge_call": {
 			// Park the pi call: real tool name + args, toolUse stopReason. pi
 			// executes; the toolResult returns on the next stream call.
-			emitToolUse(stream, blocks, activity.callId, activity.name, activity.args);
+			// Driver args are JSON-decoded off the agy wire, so the JsonValue
+			// cast is sound.
+			emitToolUse(stream, blocks, activity.callId, activity.name, activity.args as Record<string, JsonValue>);
 			return "parked";
 		}
 	}
@@ -1077,7 +1083,7 @@ function acpEditFileArg(args: Record<string, unknown>): string | undefined {
 async function runTurnDriver(
 	stream: AssistantMessageEventStream,
 	model: Model<Api>,
-	context: Context,
+	context: TranscriptContext,
 	options: SimpleStreamOptions | undefined,
 	entries: AgyModelEntry[],
 	store: SessionStore,
@@ -1179,7 +1185,9 @@ async function runTurnDriver(
 		// Fresh conversation only: agy stores the block in its own history, so
 		// re-sending it every turn would bloat each prompt and bust the cache.
 		const sysPrompt =
-			config.systemPrompt && !existing?.conversationId ? context.systemPrompt : undefined;
+			config.systemPrompt && !existing?.conversationId
+				? getCurrentSystemPrompt(context.messages) || undefined
+				: undefined;
 		const fullPrompt =
 			late.length > 0
 				? buildLateResultPrompt(late, prompt || undefined)
@@ -1258,7 +1266,7 @@ async function runTurnDriver(
  *  fallback). */
 export function createStreamSimple(
 	deps: StreamSimpleDeps,
-): (model: Model<Api>, context: Context, options?: SimpleStreamOptions) => AssistantMessageEventStream {
+): (model: Model<Api>, context: TranscriptContext, options?: SimpleStreamOptions) => AssistantMessageEventStream {
 	const { entries, store, roundTrips } = deps;
 
 	return function streamSimple(model, context, options) {