@victor-software-house/pi-openai-proxy 4.8.0 → 4.9.1

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

@@ -1247,15 +1247,23 @@ const REASONING_EFFORT_MAP = {
 	xhigh: "xhigh"
 };
 /**
-* APIs that use the OpenAI chat completions wire format and accept standard
-* passthrough fields (stop, seed, top_p, tool_choice, etc.) in the payload.
+* APIs that accept the full set of OpenAI passthrough fields (stop, seed, top_p,
+* tool_choice, frequency_penalty, etc.) as top-level payload properties.
 */
-const OPENAI_COMPATIBLE_APIS = new Set([
+const OPENAI_FULL_PASSTHROUGH_APIS = new Set([
 	"openai-completions",
 	"openai-responses",
 	"azure-openai-responses",
 	"mistral-conversations"
 ]);
+/**
+* The Codex Responses API accepts only tool_choice and parallel_tool_calls.
+* Other standard fields (top_p, seed, stop, user, frequency_penalty, etc.)
+* are rejected with "Unsupported parameter". The SDK hardcodes
+* tool_choice: "auto" and parallel_tool_calls: true; onPayload overrides
+* those when the client sends explicit values.
+*/
+const CODEX_APIS = new Set(["openai-codex-responses"]);
 const ANTHROPIC_APIS = new Set(["anthropic-messages"]);
 const GOOGLE_APIS = new Set([
 	"google-generative-ai",
@@ -1462,13 +1470,36 @@ function patchGooglePayload(payload, request) {
 	return patched;
 }
 /**
+* Collect the restricted set of fields the Codex Responses API accepts.
+*
+* Only tool_choice and parallel_tool_calls are supported. The SDK hardcodes
+* tool_choice: "auto" and parallel_tool_calls: true; these overrides let clients
+* control tool behavior explicitly.
+*
+* @internal Exported for unit testing only.
+*/
+function collectCodexPayloadFields(request) {
+	const fields = {};
+	let hasFields = false;
+	if (request.tool_choice !== void 0) {
+		fields["tool_choice"] = request.tool_choice;
+		hasFields = true;
+	}
+	if (request.parallel_tool_calls !== void 0) {
+		fields["parallel_tool_calls"] = request.parallel_tool_calls;
+		hasFields = true;
+	}
+	return hasFields ? fields : void 0;
+}
+/**
 * Collect API-specific payload fields from an OpenAI request.
 *
 * Dispatches to the appropriate translator based on the target API:
-* - OpenAI-compatible: flat field injection (same names)
+* - OpenAI full passthrough: all standard fields (same names)
+* - Codex: restricted to tool_choice + parallel_tool_calls only
 * - Anthropic: translated field names and formats
 * - Google: nested generationConfig patching (handled separately in onPayload)
-* - Others (Bedrock, Codex): no passthrough
+* - Others (Bedrock): no passthrough
 *
 * For Google APIs, returns undefined (patching is done directly in onPayload
 * via patchGooglePayload because the payload structure is nested).
@@ -1476,7 +1507,8 @@ function patchGooglePayload(payload, request) {
 * @internal Exported for unit testing only.
 */
 function collectPayloadFields(request, api) {
-	if (OPENAI_COMPATIBLE_APIS.has(api)) return collectOpenAIPayloadFields(request);
+	if (OPENAI_FULL_PASSTHROUGH_APIS.has(api)) return collectOpenAIPayloadFields(request);
+	if (CODEX_APIS.has(api)) return collectCodexPayloadFields(request);
 	if (ANTHROPIC_APIS.has(api)) return collectAnthropicPayloadFields(request);
 }
 /**
@@ -1647,6 +1679,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.8.0",
+	"version": "4.9.1",
 	"description": "OpenAI-compatible HTTP proxy for pi's multi-provider model registry",
 	"license": "MIT",
 	"author": "Victor Software House",