@victor-software-house/pi-openai-proxy 4.9.0 → 4.9.2

package/README.md

@@ -113,17 +113,32 @@ curl http://localhost:4141/v1/chat/completions \
 | `stop` | Via passthrough |
 | `user` | Via passthrough |
 | `stream_options.include_usage` | Final usage chunk in SSE stream |
-| `tools` / `tool_choice` | JSON Schema -> TypeBox conversion (supported subset) |
+| `tools` / `tool_choice` | JSON Schema -> TypeBox conversion (supported subset); `tool_choice` translated per-provider |
 | `tool_calls` in messages | Assistant tool call + tool result roundtrip |
+| `parallel_tool_calls` | Forwarded to OpenAI/Codex; translated to `disable_parallel_tool_use` for Anthropic |
 | `reasoning_effort` | Maps to pi's `ThinkingLevel` (`none`, `minimal`, `low`, `medium`, `high`, `xhigh`) |
-| `response_format` | `text`, `json_object`, and `json_schema` via passthrough |
-| `top_p` | Via passthrough |
-| `frequency_penalty` | Via passthrough |
-| `presence_penalty` | Via passthrough |
-| `seed` | Via passthrough |
+| `response_format` | `text`, `json_object`, and `json_schema` (OpenAI-compatible APIs only) |
+| `top_p` | Supported by OpenAI, Anthropic (native), and Google (translated to `topP`) |
+| `frequency_penalty` | OpenAI and Google only (translated to `frequencyPenalty`) |
+| `presence_penalty` | OpenAI and Google only (translated to `presencePenalty`) |
+| `seed` | OpenAI and Google only (translated to nested `generationConfig.seed`) |
+| `stop` | Translated per-provider (`stop_sequences` for Anthropic, `stopSequences` for Google) |
+| `metadata` | OpenAI passthrough; arbitrary keys silently skipped for other providers |
+| `prediction` | OpenAI passthrough only (speculative decoding) |
 
 **Not supported:** `n > 1`, `logprobs`, `logit_bias`, remote image URLs (disabled by default).
 
+### Provider-aware field translation
+
+Fields are translated to each provider's native format, not blindly forwarded:
+
+- **OpenAI / Mistral**: All fields passed directly (same wire format)
+- **Codex (Responses API)**: Only `tool_choice` and `parallel_tool_calls` (other fields rejected by API)
+- **Anthropic**: `tool_choice` → `{ type }` format, `stop` → `stop_sequences`, `user` → `metadata.user_id`, `parallel_tool_calls: false` → `disable_parallel_tool_use`
+- **Google**: Nested into `generationConfig` with camelCase names (`topP`, `stopSequences`, `seed`, etc.)
+
+Fields that have no equivalent in a target provider are silently skipped — the request succeeds and the provider's default applies.
+
 ## Model Naming and Exposure
 
 ### Public model IDs

package/dist/index.mjs

@@ -324,6 +324,8 @@ const toolArgsSchema = z.record(z.string().trim(), z.unknown());
 function convertMessages(messages) {
 	const systemParts = [];
 	const piMessages = [];
+	const toolCallNames = /* @__PURE__ */ new Map();
+	for (const msg of messages) if (msg !== void 0 && msg.role === "assistant" && "tool_calls" in msg) for (const tc of msg.tool_calls ?? []) toolCallNames.set(tc.id, tc.function.name);
 	for (let i = 0; i < messages.length; i++) {
 		const msg = messages[i];
 		if (msg === void 0) continue;
@@ -337,7 +339,8 @@ function convertMessages(messages) {
 			const assistantMsg = convertAssistantMessage(msg.content ?? null, msg.tool_calls);
 			piMessages.push(assistantMsg);
 		} else if (msg.role === "tool") {
-			const toolMsg = convertToolMessage(msg.content, msg.tool_call_id);
+			const toolName = toolCallNames.get(msg.tool_call_id) ?? "";
+			const toolMsg = convertToolMessage(msg.content, msg.tool_call_id, toolName);
 			piMessages.push(toolMsg);
 		}
 	}
@@ -442,11 +445,11 @@ function convertAssistantMessage(textContent, toolCalls) {
 		timestamp: Date.now()
 	};
 }
-function convertToolMessage(content, toolCallId) {
+function convertToolMessage(content, toolCallId, toolName) {
 	return {
 		role: "toolResult",
 		toolCallId,
-		toolName: "",
+		toolName,
 		content: [{
 			type: "text",
 			text: content
@@ -1679,6 +1682,7 @@ function createRoutes(config, configReader = fileConfigReader) {
 		const conversion = convertMessages(request.messages);
 		if (!conversion.ok) return c.json(invalidRequest(conversion.message, conversion.param), 400);
 		const context = conversion.context;
+		if (model.api === "openai-codex-responses" && context.systemPrompt === void 0) context.systemPrompt = "";
 		if (request.tools !== void 0 && request.tools.length > 0) {
 			const toolConversion = convertTools(request.tools);
 			if (!toolConversion.ok) return c.json(unsupportedParameter(toolConversion.param, toolConversion.message), 422);

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@victor-software-house/pi-openai-proxy",
-	"version": "4.9.0",
+	"version": "4.9.2",
 	"description": "OpenAI-compatible HTTP proxy for pi's multi-provider model registry",
 	"license": "MIT",
 	"author": "Victor Software House",