@mmmbuto/zai-codex-bridge 0.1.7 → 0.1.8

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mmmbuto/zai-codex-bridge",
-  "version": "0.1.7",
+  "version": "0.1.8",
   "description": "Local proxy that translates OpenAI Responses API format to Z.AI Chat Completions format for Codex",
   "main": "src/server.js",
   "bin": {

package/src/server.js

@@ -80,10 +80,24 @@ function translateResponsesToChat(request) {
         content: request.input
       });
     } else if (Array.isArray(request.input)) {
-      // Array of message objects
+      // Array of ResponseItem objects - filter only Message items with role
       for (const item of request.input) {
+        // Only process items with a 'role' field (Message items)
+        // Skip Reasoning, FunctionCall, LocalShellCall, etc.
+        if (!item.role) continue;
+
+        // Map non-standard roles to Z.AI-compatible roles
+        // Z.AI accepts: system, user, assistant
+        let role = item.role;
+        if (role === 'developer') {
+          role = 'user'; // Map developer to user
+        } else if (role !== 'system' && role !== 'user' && role !== 'assistant') {
+          // Skip any other non-standard roles
+          continue;
+        }
+
         const msg = {
-          role: item.role,
+          role: role,
           content: flattenContent(item.content)
         };
 
@@ -127,7 +141,21 @@ function translateResponsesToChat(request) {
   }
 
   if (request.tools && Array.isArray(request.tools)) {
-    chatRequest.tools = request.tools;
+    // Filter out tools with null or empty function
+    chatRequest.tools = request.tools.filter(tool => {
+      if (tool.type === 'function') {
+        // Check if function has required fields
+        return tool.function && typeof tool.function === 'object' &&
+               tool.function.name && tool.function.name.length > 0 &&
+               tool.function.parameters !== undefined && tool.function.parameters !== null;
+      }
+      // Keep non-function tools (if any)
+      return true;
+    });
+    // Only add tools array if there are valid tools
+    if (chatRequest.tools.length === 0) {
+      delete chatRequest.tools;
+    }
   }
 
   if (request.tool_choice) {
@@ -210,7 +238,9 @@ async function makeUpstreamRequest(path, body, headers) {
     base: ZAI_BASE_URL,
     hasAuth: !!upstreamHeaders.Authorization,
     bodyKeys: Object.keys(body),
-    bodyPreview: JSON.stringify(body).substring(0, 200)
+    bodyPreview: JSON.stringify(body).substring(0, 800),
+    messagesCount: body.messages?.length || 0,
+    allRoles: body.messages?.map(m => m.role) || []
   });
 
   const response = await fetch(url, {
@@ -229,6 +259,7 @@ async function streamChatToResponses(stream, res) {
   const decoder = new TextDecoder();
   let buffer = '';
   let chunkCount = 0;
+  let deltaCount = 0;
 
   log('debug', 'Starting to process stream');
 
@@ -252,9 +283,10 @@ async function streamChatToResponses(stream, res) {
 
       // Check for stream end
       if (data === '[DONE]') {
-        log('debug', 'Stream end received');
+        log('info', `Stream end received - wrote ${deltaCount} deltas total`);
         res.write(`event: completed\n`);
         res.write(`data: ${JSON.stringify({ status: 'completed' })}\n\n`);
+        log('info', 'Sent completed event');
         return;
       }
 
@@ -268,6 +300,7 @@ async function streamChatToResponses(stream, res) {
         const content = delta?.content || delta?.reasoning_content || '';
 
         if (content) {
+          deltaCount++;
           log('debug', 'Writing delta:', content.substring(0, 30));
           res.write(`event: output.text.delta\n`);
           res.write(`data: ${JSON.stringify({ value: content })}\n\n`);
@@ -283,7 +316,7 @@ async function streamChatToResponses(stream, res) {
     }
   }
 
-  log('debug', 'Stream ended naturally');
+  log('info', `Stream ended naturally - wrote ${deltaCount} deltas`);
 }
 
 /**