@victor-software-house/pi-openai-proxy 4.6.0 → 4.6.1

package/dist/index.mjs

@@ -1141,7 +1141,17 @@ const chatCompletionRequestSchema = z.object({
 }).strict();
 /**
 * Fields that are explicitly rejected with a helpful error.
-* These are not supported and won't be promoted.
+*
+* `n`, `logprobs`, `top_logprobs`, `logit_bias`: not supported by the pi SDK's
+* simple completion interface and unlikely to be promoted.
+*
+* `functions`, `function_call`: deprecated OpenAI fields, superseded by `tools`
+* and `tool_choice`.
+*
+* `parallel_tool_calls`: the pi SDK does not expose parallel tool call control.
+* The SSE streaming code handles multiple tool calls per response, so the response
+* side is capable, but the proxy cannot guarantee the flag reaches the provider.
+* Needs deeper analysis — see Phase 3D in TODO.md.
 */
 const rejectedFields = [
 	"n",
@@ -1227,6 +1237,8 @@ const SKIP_PAYLOAD_PASSTHROUGH_APIS = new Set(["openai-codex-responses"]);
 /**
 * Collect fields that need to be injected via onPayload.
 * Skips passthrough for APIs that use non-standard request formats.
+*
+* @internal Exported for unit testing only.
 */
 function collectPayloadFields(request, api) {
 	if (SKIP_PAYLOAD_PASSTHROUGH_APIS.has(api)) return;
@@ -1260,9 +1272,55 @@ function collectPayloadFields(request, api) {
 		fields["response_format"] = request.response_format;
 		hasFields = true;
 	}
+	if (request.tool_choice !== void 0) {
+		fields["tool_choice"] = request.tool_choice;
+		hasFields = true;
+	}
 	return hasFields ? fields : void 0;
 }
 /**
+* Collect tool strict flags from the original OpenAI request.
+*
+* The pi SDK's `Tool` interface has no `strict` field, so the SDK always sets
+* `strict: false` when building the upstream payload. This function extracts
+* the per-tool strict flags from the original request so they can be restored
+* via `onPayload` after the SDK builds the payload.
+*
+* Returns a map of tool index -> true for tools that requested strict mode,
+* or undefined if no tools use strict mode.
+*
+* @internal Exported for unit testing only.
+*/
+function collectToolStrictFlags(tools) {
+	if (tools === void 0 || tools.length === 0) return;
+	let flags;
+	for (let i = 0; i < tools.length; i++) if (tools[i]?.function.strict === true) {
+		flags ??= /* @__PURE__ */ new Map();
+		flags.set(i, true);
+	}
+	return flags;
+}
+/**
+* Apply strict flags to tool definitions in the upstream payload.
+*
+* The pi SDK always sets `strict: false` on tool definitions. This function
+* patches the payload's `tools` array to restore the client's requested
+* `strict: true` flags on the matching tool definitions.
+*
+* @internal Exported for unit testing only.
+*/
+function applyToolStrictFlags(payload, strictFlags) {
+	const tools = payload["tools"];
+	if (!Array.isArray(tools)) return;
+	for (const [index, _flag] of strictFlags) {
+		const tool = tools[index];
+		if (isRecord(tool)) {
+			const fn = tool["function"];
+			if (isRecord(fn)) fn["strict"] = true;
+		}
+	}
+}
+/**
 * Combine a client disconnect signal with an upstream timeout into a single signal.
 * Returns the combined signal, or undefined if neither is provided.
 */
@@ -1293,8 +1351,12 @@ async function buildStreamOptions(model, request, options) {
 		if (apiKey !== void 0) opts.apiKey = apiKey;
 	}
 	const payloadFields = collectPayloadFields(request, model.api);
-	if (payloadFields !== void 0) opts.onPayload = (payload) => {
-		if (isRecord(payload)) for (const [key, value] of Object.entries(payloadFields)) payload[key] = value;
+	const strictFlags = collectToolStrictFlags(request.tools);
+	if (payloadFields !== void 0 || strictFlags !== void 0) opts.onPayload = (payload) => {
+		if (isRecord(payload)) {
+			if (payloadFields !== void 0) for (const [key, value] of Object.entries(payloadFields)) payload[key] = value;
+			if (strictFlags !== void 0) applyToolStrictFlags(payload, strictFlags);
+		}
 		return payload;
 	};
 	return opts;

package/package.json

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