@oh-my-pi/pi-ai 5.6.70 → 5.7.67

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@oh-my-pi/pi-ai",
-	"version": "5.6.70",
+	"version": "5.7.67",
 	"description": "Unified LLM API with automatic model discovery and provider configuration",
 	"type": "module",
 	"main": "./src/index.ts",

package/src/cli.ts

@@ -98,6 +98,9 @@ async function login(provider: OAuthProvider): Promise<void> {
 					onProgress: (msg) => console.log(msg),
 				});
 				break;
+
+			default:
+				throw new Error(`Unknown provider: ${provider}`);
 		}
 
 		const auth = loadAuth();

package/src/providers/transform-messages.ts

@@ -127,16 +127,27 @@ export function transformMessages<TApi extends Api>(messages: Message[], model:
 
 			// For errored/aborted messages with tool calls, insert synthetic results immediately
 			// to maintain tool_use/tool_result pairing required by the API
+			// BUT only if there aren't already tool results for these calls later in the array
 			if (isErroredAssistant && toolCalls.length > 0) {
+				// Look ahead to find existing tool results
+				const existingResultIds = new Set<string>();
+				for (let j = i + 1; j < transformed.length; j++) {
+					if (transformed[j].role === "toolResult") {
+						existingResultIds.add((transformed[j] as ToolResultMessage).toolCallId);
+					}
+				}
+				// Only add synthetic results for tool calls that don't have results
 				for (const tc of toolCalls) {
-					result.push({
-						role: "toolResult",
-						toolCallId: tc.id,
-						toolName: tc.name,
-						content: [{ type: "text", text: "Tool execution was aborted" }],
-						isError: true,
-						timestamp: Date.now(),
-					} as ToolResultMessage);
+					if (!existingResultIds.has(tc.id)) {
+						result.push({
+							role: "toolResult",
+							toolCallId: tc.id,
+							toolName: tc.name,
+							content: [{ type: "text", text: "Tool execution was aborted" }],
+							isError: true,
+							timestamp: Date.now(),
+						} as ToolResultMessage);
+					}
 				}
 			} else if (!isErroredAssistant && toolCalls.length > 0) {
 				// Track tool calls to check for orphaned calls later