pi-sap-aicore 0.3.1 → 0.3.2

package/CHANGELOG.md

@@ -7,6 +7,17 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [0.3.2] - 2026-07-01
+
+### Fixed
+
+- Direct AWS Bedrock and GCP Vertex AI foundation routes now advertise pi tools
+  using provider-native tool/function declaration schemas, so Claude and Gemini
+  foundation models can perform real coding-agent tool calls instead of acting as
+  text-only generators.
+- Vertex/Gemini foundation tool-call replay now preserves Gemini thought signatures
+  and normalizes provider-prefixed function names before emitting pi tool calls.
+
 ## [0.3.1] - 2026-07-01
 
 ### Added
@@ -135,7 +146,8 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
   `reasoning_effort` for OpenAI).
 - MIT license and npm packaging.
 
-[Unreleased]: https://github.com/ttiimmaahh/pi-sap-aicore/compare/v0.3.1...HEAD
+[Unreleased]: https://github.com/ttiimmaahh/pi-sap-aicore/compare/v0.3.2...HEAD
+[0.3.2]: https://github.com/ttiimmaahh/pi-sap-aicore/compare/v0.3.1...v0.3.2
 [0.3.1]: https://github.com/ttiimmaahh/pi-sap-aicore/compare/v0.3.0...v0.3.1
 [0.3.0]: https://github.com/ttiimmaahh/pi-sap-aicore/compare/v0.2.2...v0.3.0
 [0.2.2]: https://github.com/ttiimmaahh/pi-sap-aicore/compare/v0.2.1...v0.2.2

package/README.md

@@ -132,8 +132,9 @@ The extension registers **two providers**, both backed by the same service key:
 |---|---|---|
 | SAP deployment | one orchestration deployment fronts **every** model | one foundation deployment **per model** |
 | Models | Claude, GPT-5*, Gemini | GPT/OpenAI (`azure-openai`), Anthropic/Claude (`aws-bedrock`), and Gemini (`gcp-vertexai`) |
+| Tool use | yes | yes — tools are translated to OpenAI, Bedrock Converse, or Vertex/Gemini function declarations by executable |
 | Streaming | subject to orchestration's per-model allow-list — new models can 400 `Streaming is not supported` (we fall back to non-streaming) | Azure OpenAI streams natively; AWS Bedrock and Vertex AI currently use non-streaming endpoints and replay responses into pi stream events |
-| Reasoning effort | tunable (`reasoning_effort` / `thinking`) | model **default** only for Azure; Bedrock/Anthropic and Vertex/Gemini thinking controls are not fully wired yet |
+| Reasoning effort | tunable (`reasoning_effort` / `thinking`) | model **default** only for Azure; Bedrock/Anthropic and Vertex/Gemini effort controls are intentionally conservative |
 | Content filter / grounding / templating | yes | no — raw model access |
 | SDK / endpoint | `@sap-ai-sdk/orchestration` | `AzureOpenAiChatClient` for `azure-openai`; SAP `/inference/deployments/{id}/converse` for `aws-bedrock`; SAP `/inference/deployments/{id}/models/{model}:generateContent` for `gcp-vertexai` |
 
@@ -337,7 +338,8 @@ the direct Azure OpenAI SDK pinned to API version `2024-10-21`, which has no
 thinking-level cycle is a no-op there. Anthropic/Claude models use SAP's AWS
 Bedrock `/converse` endpoint; Gemini models use SAP's Vertex AI `generateContent`
 endpoint with `thinkingBudget: 0` by default so small pi output budgets produce
-visible text instead of only hidden thoughts. Use the orchestration route if you
+visible text instead of only hidden thoughts. Tool/function calling is supported
+on all three direct foundation executables. Use the orchestration route if you
 need explicit effort control.
 
 To override budgets per model, edit `thinkingLevelMap` on the relevant entry in

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-sap-aicore",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "description": "SAP AI Core (orchestration + foundation) provider for the pi coding agent",
   "license": "MIT",
   "author": "Tim Pearson (https://github.com/ttiimmaahh)",

package/src/stream-foundation-vertexai.ts

@@ -29,6 +29,7 @@ type VertexGenerateContentResponse = {
 			role?: string;
 			parts?: Array<{
 				text?: string;
+				thoughtSignature?: string;
 				functionCall?: { name?: string; args?: unknown };
 			}>;
 		};
@@ -192,14 +193,18 @@ function replayVertexGenerateContentResponse(
 
 		if (part.functionCall) {
 			const contentIndex = output.content.length;
+			const providerName = part.functionCall.name ?? "";
 			const toolCall = {
 				type: "toolCall" as const,
 				id: randomUUID(),
-				name: part.functionCall.name ?? "",
+				name: normalizeVertexFunctionName(providerName),
 				arguments:
 					part.functionCall.args && typeof part.functionCall.args === "object"
 						? (part.functionCall.args as Record<string, unknown>)
 						: {},
+				...(part.thoughtSignature
+					? { thoughtSignature: part.thoughtSignature }
+					: {}),
 			};
 			output.content.push(toolCall);
 			stream.push({ type: "toolcall_start", contentIndex, partial: output });
@@ -212,3 +217,8 @@ function replayVertexGenerateContentResponse(
 		}
 	}
 }
+
+function normalizeVertexFunctionName(name: string): string {
+	const index = name.indexOf(":");
+	return index >= 0 ? name.slice(index + 1) : name;
+}

package/src/translate-foundation-bedrock.ts

@@ -3,6 +3,7 @@ import type {
 	Context,
 	Message,
 	TextContent,
+	Tool,
 	ToolResultMessage,
 	UserMessage,
 } from "@earendil-works/pi-ai";
@@ -35,9 +36,20 @@ export type BedrockConverseMessage = {
 	content: BedrockConverseContentBlock[];
 };
 
+export type BedrockToolConfig = {
+	tools: Array<{
+		toolSpec: {
+			name: string;
+			description: string;
+			inputSchema: { json: Record<string, unknown> };
+		};
+	}>;
+};
+
 export function piContextToBedrockConverse(context: Context): {
 	system?: Array<{ text: string }>;
 	messages: BedrockConverseMessage[];
+	toolConfig?: BedrockToolConfig;
 } {
 	const messages: BedrockConverseMessage[] = [];
 
@@ -46,9 +58,11 @@ export function piContextToBedrockConverse(context: Context): {
 		if (translated) messages.push(translated);
 	}
 
+	const tools = (context.tools ?? []).map(piToolToBedrockToolSpec);
 	return {
 		...(context.systemPrompt ? { system: [{ text: context.systemPrompt }] } : {}),
 		messages: coalesceAdjacentMessages(messages),
+		...(tools.length > 0 ? { toolConfig: { tools } } : {}),
 	};
 }
 
@@ -159,6 +173,18 @@ function imageFormatFromMimeType(mimeType: string): string {
 	return "png";
 }
 
+function piToolToBedrockToolSpec(tool: Tool): BedrockToolConfig["tools"][number] {
+	return {
+		toolSpec: {
+			name: tool.name,
+			description: tool.description,
+			inputSchema: {
+				json: tool.parameters as unknown as Record<string, unknown>,
+			},
+		},
+	};
+}
+
 function coalesceAdjacentMessages(
 	messages: BedrockConverseMessage[],
 ): BedrockConverseMessage[] {

package/src/translate-foundation-vertexai.ts

@@ -3,6 +3,7 @@ import type {
 	Context,
 	Message,
 	TextContent,
+	Tool,
 	ToolResultMessage,
 	UserMessage,
 } from "@earendil-works/pi-ai";
@@ -10,7 +11,7 @@ import type {
 export type VertexPart =
 	| { text: string }
 	| { inlineData: { mimeType: string; data: string } }
-	| { functionCall: { name: string; args: unknown } }
+	| { functionCall: { name: string; args: unknown }; thoughtSignature?: string }
 	| { functionResponse: { name: string; response: Record<string, unknown> } };
 
 export type VertexContent = {
@@ -18,9 +19,18 @@ export type VertexContent = {
 	parts: VertexPart[];
 };
 
+export type VertexTool = {
+	functionDeclarations: Array<{
+		name: string;
+		description: string;
+		parameters: Record<string, unknown>;
+	}>;
+};
+
 export function piContextToVertexGenerateContent(context: Context): {
 	systemInstruction?: { parts: Array<{ text: string }> };
 	contents: VertexContent[];
+	tools?: VertexTool[];
 } {
 	const contents: VertexContent[] = [];
 
@@ -29,11 +39,17 @@ export function piContextToVertexGenerateContent(context: Context): {
 		if (translated) contents.push(translated);
 	}
 
+	const functionDeclarations = (context.tools ?? []).map(
+		piToolToVertexFunctionDeclaration,
+	);
 	return {
 		...(context.systemPrompt
 			? { systemInstruction: { parts: [{ text: context.systemPrompt }] } }
 			: {}),
 		contents: coalesceAdjacentContents(contents),
+		...(functionDeclarations.length > 0
+			? { tools: [{ functionDeclarations }] }
+			: {}),
 	};
 }
 
@@ -68,7 +84,13 @@ function piAssistantToVertexContent(msg: AssistantMessage): VertexContent {
 			parts.push({ text: block.text });
 		} else if (block.type === "toolCall") {
 			parts.push({
-				functionCall: { name: block.name, args: block.arguments },
+				functionCall: {
+					name: block.name,
+					args: block.arguments,
+				},
+				...(block.thoughtSignature
+					? { thoughtSignature: block.thoughtSignature }
+					: {}),
 			});
 		}
 	}
@@ -111,6 +133,16 @@ function toolResultImagesAsUserParts(msg: ToolResultMessage): VertexPart[] {
 		}));
 }
 
+function piToolToVertexFunctionDeclaration(
+	tool: Tool,
+): VertexTool["functionDeclarations"][number] {
+	return {
+		name: tool.name,
+		description: tool.description,
+		parameters: tool.parameters as unknown as Record<string, unknown>,
+	};
+}
+
 function coalesceAdjacentContents(contents: VertexContent[]): VertexContent[] {
 	const result: VertexContent[] = [];
 	for (const content of contents) {