@wrongstack/providers 0.257.2 → 0.264.0

package/dist/index.d.ts

@@ -164,6 +164,26 @@ interface ConvertOptions {
     flattenContentToString?: boolean | undefined;
     stripCacheControl?: boolean | undefined;
     systemAsMessage?: boolean | undefined;
+    /**
+     * What to write as the assistant message's `content` field when the
+     * message has tool_calls but no prose. Two values:
+     *
+     *   - `'empty_string'` (default): writes `content: ''`. This is the
+     *     OpenAI 2024-2025 wire-format contract. Vanilla OpenAI, K2P7,
+     *     strict Mistral / OpenRouter / DeepSeek proxies all reject
+     *     requests where `content` is missing or `null` on a tool_call
+     *     assistant message.
+     *
+     *   - `'null'`: writes `content: null` explicitly. Some older or
+     *     permissive proxies (e.g. certain vLLM builds, local llama.cpp
+     *     servers) prefer this. Set this only if a specific provider
+     *     rejects the empty-string form.
+     *
+     * The default is `'empty_string'` (NOT undefined) because omitting
+     * `content` entirely (the pre-2024 behaviour) breaks too many
+     * providers to be the safe default in 2025. Callers that need the
+     * old behaviour can opt in with `emptyToolCallContent: 'null'`.
+     */
     emptyToolCallContent?: 'null' | 'empty_string' | undefined;
 }
 declare function messagesToOpenAI(system: TextBlock[] | undefined, messages: Message[], opts?: ConvertOptions): OpenAIMessage[];

package/dist/index.js

@@ -1,5 +1,6 @@
 import { expectDefined, ProviderError, StreamHangError, safeParse, sanitizeJsonString, WrongStackError, ERROR_CODES } from '@wrongstack/core';
 import { Readable } from 'stream';
+import { toErrorMessage } from '@wrongstack/core/utils';
 import { randomUUID } from 'crypto';
 
 var __defProp = Object.defineProperty;
@@ -657,9 +658,9 @@ var WireAdapter = class {
       httpRes = raw;
     } catch (err) {
       if (opts.signal.aborted) throw err;
-      throw new ProviderError(err instanceof Error ? err.message : String(err), 0, true, this.id, {
+      throw new ProviderError(toErrorMessage(err), 0, true, this.id, {
         cause: err,
-        body: { message: err instanceof Error ? err.message : String(err) }
+        body: { message: toErrorMessage(err) }
       });
     }
     if (!httpRes.ok) {
@@ -1209,6 +1210,7 @@ function toolsToOpenAI(tools) {
   }));
 }
 function messagesToOpenAI(system, messages, opts = {}) {
+  const emptyContentMode = opts.emptyToolCallContent ?? "empty_string";
   const out = [];
   if (system && system.length > 0) {
     const sysText = system.map((b) => b.text).join("\n\n");
@@ -1255,7 +1257,6 @@ function messagesToOpenAI(system, messages, opts = {}) {
         if (text) {
           message.content = text;
         } else {
-          const emptyContentMode = opts.emptyToolCallContent ?? "empty_string";
           message.content = emptyContentMode === "null" ? null : "";
         }
       } else {
@@ -1627,7 +1628,7 @@ var mistralWireFormat = defineWireFormat({
   buildBody: (req) => {
     const body = {
       model: req.model,
-      messages: messagesToOpenAI(stripCacheControl(req.system), req.messages, {}),
+      messages: messagesToOpenAI(stripCacheControl(req.system), req.messages),
       max_tokens: req.maxTokens,
       stream: true
     };
@@ -1908,7 +1909,7 @@ var openaiWireFormat = defineWireFormat({
   buildBody: (req) => {
     const body = {
       model: req.model,
-      messages: messagesToOpenAI(stripCacheControl2(req.system), req.messages, {}),
+      messages: messagesToOpenAI(stripCacheControl2(req.system), req.messages),
       // Real OpenAI requires `max_completion_tokens`; newer model families
       // (gpt-4o, o1/o3/o4) 400 on the deprecated `max_tokens`. See issue #10.
       max_completion_tokens: req.maxTokens,