@oh-my-pi/pi-ai 14.8.0 → 14.8.1

package/CHANGELOG.md

@@ -7,6 +7,7 @@
 ### Fixed
 - Fixed Gemini 3 Pro thinking metadata so `medium` effort is rejected with the expected error instead of being silently accepted: `ThinkingConfig` now carries an optional explicit `levels` list that survives `expandEffortRange`, letting non-contiguous supported sets (e.g. `[low, high]`) round-trip through enrichment.
 - Fixed Kimi Code OAuth expiry handling to refresh access tokens 5 minutes before server expiry, avoiding daily 401s from using tokens right up to the cutoff.
+- Fixed OpenAI Responses custom tool replay to preserve custom tool call item IDs with the `ctc_` prefix instead of rewriting them as `fc_` function-call IDs ([#977](https://github.com/can1357/oh-my-pi/issues/977)).
 
 ## [14.7.6] - 2026-05-07
 

package/package.json

@@ -1,7 +1,7 @@
 {
 	"type": "module",
 	"name": "@oh-my-pi/pi-ai",
-	"version": "14.8.0",
+	"version": "14.8.1",
 	"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.39",
 		"@bufbuild/protobuf": "^2.12.0",
 		"@google/genai": "^1.52.0",
-		"@oh-my-pi/pi-natives": "14.8.0",
-		"@oh-my-pi/pi-utils": "14.8.0",
+		"@oh-my-pi/pi-natives": "14.8.1",
+		"@oh-my-pi/pi-utils": "14.8.1",
 		"@sinclair/typebox": "^0.34.49",
 		"@smithy/node-http-handler": "^4.6.1",
 		"ajv": "^8.20.0",

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

@@ -2438,7 +2438,7 @@ function convertMessages(model: Model<"openai-codex-responses">, context: Contex
 				}
 				if (block.type === "toolCall") {
 					const toolCall = block as ToolCall;
-					const normalized = normalizeResponsesToolCallId(toolCall.id);
+					const normalized = normalizeResponsesToolCallId(toolCall.id, toolCall.customWireName ? "ctc" : "fc");
 					if (toolCall.customWireName) {
 						const rawInput = typeof toolCall.arguments?.input === "string" ? toolCall.arguments.input : "";
 						customCallIds.add(normalized.callId);

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

@@ -187,9 +187,9 @@ export function convertResponsesAssistantMessage<TApi extends Api>(
 			continue;
 		}
 
-		const normalized = normalizeResponsesToolCallId(block.id);
+		const normalized = normalizeResponsesToolCallId(block.id, block.customWireName ? "ctc" : "fc");
 		let itemId: string | undefined = normalized.itemId;
-		if (isDifferentModel && (itemId?.startsWith("fc_") || itemId?.startsWith("fcr_"))) {
+		if (isDifferentModel && (itemId?.startsWith("fc_") || itemId?.startsWith("fcr_") || itemId?.startsWith("ctc_"))) {
 			itemId = undefined;
 		}
 		knownCallIds.add(normalized.callId);

package/src/utils.ts

@@ -36,16 +36,21 @@ export function normalizeToolCallId(id: string): string {
 	return sanitized.length > 64 ? sanitized.slice(0, 64) : sanitized;
 }
 
-export function normalizeResponsesToolCallId(id: string): { callId: string; itemId: string } {
+type ResponsesToolItemIdPrefix = "fc" | "ctc";
+
+export function normalizeResponsesToolCallId(
+	id: string,
+	itemPrefix: ResponsesToolItemIdPrefix = "fc",
+): { callId: string; itemId: string } {
 	const [callId, itemId] = id.split("|");
 	if (callId && itemId) {
 		const normalizedCallId = truncateResponseItemId(callId, getIdPrefix(callId, "call"));
-		const normalizedItemId = normalizeResponsesItemId(itemId);
+		const normalizedItemId = normalizeResponsesItemId(itemId, itemPrefix);
 		return { callId: normalizedCallId, itemId: normalizedItemId };
 	}
 	const hash = Bun.hash(id).toString(36);
 	const normalizedCallId = id.startsWith("call_") ? truncateResponseItemId(id, "call") : `call_${hash}`;
-	return { callId: normalizedCallId, itemId: `fc_${hash}` };
+	return { callId: normalizedCallId, itemId: `${itemPrefix}_${hash}` };
 }
 
 function getIdPrefix(id: string, fallback: string): string {
@@ -53,10 +58,19 @@ function getIdPrefix(id: string, fallback: string): string {
 	return prefix || fallback;
 }
 
-function normalizeResponsesItemId(itemId: string): string {
-	const prefix = getIdPrefix(itemId, "fc");
-	if (prefix !== "fc" && prefix !== "fcr") {
-		return `fc_${Bun.hash(itemId).toString(36)}`;
+function getExplicitIdPrefix(id: string): string | undefined {
+	return id.match(/^([a-zA-Z][a-zA-Z0-9]*)_/)?.[1];
+}
+
+function normalizeResponsesItemId(itemId: string, fallbackPrefix: ResponsesToolItemIdPrefix): string {
+	const prefix = getExplicitIdPrefix(itemId);
+	const isAllowedPrefix = prefix
+		? fallbackPrefix === "ctc"
+			? prefix === "ctc"
+			: prefix === "fc" || prefix === "fcr"
+		: false;
+	if (!prefix || !isAllowedPrefix) {
+		return `${fallbackPrefix}_${Bun.hash(itemId).toString(36)}`;
 	}
 	return truncateResponseItemId(itemId, prefix);
 }