@oh-my-pi/pi-ai 14.4.0 → 14.4.3

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-ai",
-	"version": "14.4.0",
+	"version": "14.4.3",
 	"description": "Unified LLM API with automatic model discovery and provider configuration",
 	"homepage": "https://github.com/can1357/oh-my-pi",
 	"author": "Can Boluk",
@@ -46,8 +46,8 @@
 		"@aws-sdk/credential-provider-node": "^3.972.36",
 		"@bufbuild/protobuf": "^2.12.0",
 		"@google/genai": "^1.50.1",
-		"@oh-my-pi/pi-natives": "14.4.0",
-		"@oh-my-pi/pi-utils": "14.4.0",
+		"@oh-my-pi/pi-natives": "14.4.3",
+		"@oh-my-pi/pi-utils": "14.4.3",
 		"@sinclair/typebox": "^0.34.49",
 		"@smithy/node-http-handler": "^4.6.1",
 		"ajv": "^8.20.0",

package/src/providers/anthropic.ts

@@ -84,7 +84,6 @@ export function buildBetaHeader(baseBetas: string[], extraBetas: string[]): stri
 const claudeCodeBetaDefaults = [
 	"claude-code-20250219",
 	"oauth-2025-04-20",
-	"interleaved-thinking-2025-05-14",
 	"context-management-2025-06-27",
 	"prompt-caching-scope-2026-01-05",
 ];
@@ -1160,6 +1159,7 @@ export function buildAnthropicClientOptions(args: AnthropicClientOptionsArgs): A
 		dynamicHeaders,
 		isOAuth,
 	} = args;
+	const needsInterleavedBeta = interleavedThinking && !supportsAdaptiveThinkingDisplay(model.id);
 	const oauthToken = isOAuth ?? isAnthropicOAuthToken(apiKey);
 	const baseUrl = resolveAnthropicBaseUrl(model, apiKey);
 	const foundryCustomHeaders = resolveAnthropicCustomHeaders(model);
@@ -1193,7 +1193,7 @@ export function buildAnthropicClientOptions(args: AnthropicClientOptionsArgs): A
 	}
 
 	const betaFeatures = [...extraBetas];
-	if (interleavedThinking) {
+	if (needsInterleavedBeta) {
 		betaFeatures.push("interleaved-thinking-2025-05-14");
 	}
 
@@ -1498,9 +1498,6 @@ function buildParams(
 		stream: true,
 	};
 
-	if (options?.temperature !== undefined) {
-		params.temperature = options.temperature;
-	}
 	if (options?.topP !== undefined) {
 		params.top_p = options.topP;
 	}
@@ -1510,13 +1507,16 @@ function buildParams(
 
 	// Opus 4.7+ rejects non-default sampling parameters with 400 error.
 	if (hasOpus47ApiRestrictions(model.id)) {
-		delete params.temperature;
-		delete (params as AnthropicSamplingParams).top_p;
-		delete (params as AnthropicSamplingParams).top_k;
+		delete params.top_p;
+		delete params.top_k;
 	}
 
 	if (context.tools) {
-		params.tools = convertTools(context.tools, isOAuthToken, disableStrictTools);
+		params.tools = convertTools(
+			context.tools,
+			isOAuthToken,
+			disableStrictTools || model.provider === "github-copilot",
+		);
 	}
 
 	if (options?.thinkingEnabled && model.reasoning && model.provider !== "github-copilot") {

package/src/providers/azure-openai-responses.ts

@@ -10,11 +10,11 @@ import {
 	type Api,
 	type AssistantMessage,
 	type Context,
-	isSpecialServiceTier,
 	type Model,
 	type ServiceTier,
 	type StreamFunction,
 	type StreamOptions,
+	shouldSendServiceTier,
 	type Tool,
 	type ToolChoice,
 } from "../types";
@@ -298,7 +298,7 @@ function buildParams(
 	if (options?.repetitionPenalty !== undefined) {
 		params.repetition_penalty = options.repetitionPenalty;
 	}
-	if (isSpecialServiceTier(options?.serviceTier)) {
+	if (shouldSendServiceTier(options?.serviceTier, model.provider)) {
 		params.service_tier = options.serviceTier;
 	}
 

package/src/providers/openai-codex-responses.ts

@@ -19,12 +19,12 @@ import {
 	type Api,
 	type AssistantMessage,
 	type Context,
-	isSpecialServiceTier,
 	type Model,
 	type ProviderSessionState,
 	type ServiceTier,
 	type StreamFunction,
 	type StreamOptions,
+	shouldSendServiceTier,
 	type TextContent,
 	type ThinkingContent,
 	type Tool,
@@ -493,7 +493,7 @@ async function buildTransformedCodexRequestBody(
 	if (options?.repetitionPenalty !== undefined) {
 		params.repetition_penalty = options.repetitionPenalty;
 	}
-	if (isSpecialServiceTier(options?.serviceTier)) {
+	if (shouldSendServiceTier(options?.serviceTier, model.provider)) {
 		params.service_tier = options.serviceTier;
 	}
 	if (context.tools && context.tools.length > 0) {

package/src/providers/openai-completions.ts

@@ -14,7 +14,6 @@ import { getEnvApiKey } from "../stream";
 import {
 	type AssistantMessage,
 	type Context,
-	isSpecialServiceTier,
 	type Message,
 	type MessageAttribution,
 	type Model,
@@ -23,6 +22,7 @@ import {
 	type StopReason,
 	type StreamFunction,
 	type StreamOptions,
+	shouldSendServiceTier,
 	type TextContent,
 	type ThinkingContent,
 	type Tool,
@@ -811,7 +811,7 @@ function buildParams(
 	if (options?.repetitionPenalty !== undefined) {
 		params.repetition_penalty = options.repetitionPenalty;
 	}
-	if (isSpecialServiceTier(options?.serviceTier)) {
+	if (shouldSendServiceTier(options?.serviceTier, model.provider)) {
 		params.service_tier = options.serviceTier;
 	}
 

package/src/providers/openai-responses.ts

@@ -11,13 +11,13 @@ import {
 	type AssistantMessage,
 	type CacheRetention,
 	type Context,
-	isSpecialServiceTier,
 	type MessageAttribution,
 	type Model,
 	type ProviderSessionState,
 	type ServiceTier,
 	type StreamFunction,
 	type StreamOptions,
+	shouldSendServiceTier,
 	type Tool,
 	type ToolChoice,
 } from "../types";
@@ -384,7 +384,7 @@ function buildParams(
 	if (options?.repetitionPenalty !== undefined) {
 		params.repetition_penalty = options.repetitionPenalty;
 	}
-	if (isSpecialServiceTier(options?.serviceTier)) {
+	if (shouldSendServiceTier(options?.serviceTier, model.provider)) {
 		params.service_tier = options.serviceTier;
 	}
 

package/src/types.ts

@@ -152,7 +152,13 @@ export type CacheRetention = "none" | "short" | "long";
 /** OpenAI service tier for processing priority. Only applies to OpenAI-compatible APIs. */
 export type ServiceTier = "auto" | "default" | "flex" | "scale" | "priority";
 
-export function isSpecialServiceTier(serviceTier?: ServiceTier | null): serviceTier is "flex" | "scale" | "priority" {
+export function shouldSendServiceTier(
+	serviceTier?: ServiceTier | null,
+	provider?: Provider,
+): serviceTier is "flex" | "scale" | "priority" {
+	if (provider !== "openai" && provider !== "openai-codex") {
+		return false;
+	}
 	return serviceTier === "flex" || serviceTier === "scale" || serviceTier === "priority";
 }
 

package/src/utils/validation.ts

@@ -585,6 +585,26 @@ function normalizeOptionalNullsForSchema(schema: unknown, value: unknown): { val
 		nextValue[key] = normalized.value;
 	}
 
+	// Strip unknown keys with null/"null" values when the schema forbids extras.
+	// LLMs sometimes hallucinate verbs alongside valid ones (e.g. `split: null`,
+	// `original: null`). Rejecting the entire tool call wastes a turn; treating
+	// these the same as null on known optional fields is a safer fallback. Keys
+	// with non-null unknown values are left intact so genuine schema mistakes
+	// still surface as validation errors.
+	if (schemaObject.additionalProperties === false) {
+		const knownKeys = new Set(Object.keys(properties));
+		for (const key of Object.keys(nextValue)) {
+			if (knownKeys.has(key)) continue;
+			const v = nextValue[key];
+			if (v !== null && v !== "null") continue;
+			if (!changed) {
+				nextValue = { ...nextValue };
+				changed = true;
+			}
+			delete nextValue[key];
+		}
+	}
+
 	return { value: changed ? nextValue : value, changed };
 }