@contentgrowth/llm-service 0.7.9 → 0.8.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@contentgrowth/llm-service",
-  "version": "0.7.9",
+  "version": "0.8.0",
   "description": "Unified LLM Service for Content Growth",
   "main": "src/index.js",
   "type": "module",

package/src/llm/providers/gemini-provider.js

@@ -113,29 +113,18 @@ export class GeminiProvider extends BaseLLMProvider {
                     break;
                 case 'assistant':
                     role = 'model';
-                    parts = [];
-
-                    // Always include text if present (Thought)
-                    if (msg.content) {
-                        parts.push({ text: msg.content });
-                    }
-
-                    // Append tool calls if present (Call)
                     if (msg.tool_calls) {
-                        const callParts = msg.tool_calls.map(tc => ({
-                            functionCall: { name: tc.function.name, args: tc.function.arguments }
-                        }));
-                        parts.push(...callParts);
-                    }
-
-                    // Safety fallback: if nothing was added, ensure valid structure
-                    // CRITICAL: Newer Gemini models (and SDK 1.34+) REQUIRE a non-empty thought_signature (text)
-                    // before a function call. If we have tool calls but no content, we must fabricate one.
-                    const hasText = parts.some(p => p.text);
-                    if (!hasText && msg.tool_calls) {
-                        parts.unshift({ text: "Processing tool execution..." });
-                    } else if (parts.length === 0) {
-                        parts.push({ text: "" }); // Fallback for truly empty message (shouldn't happen)
+                        parts = msg.tool_calls.map(tc => {
+                            const part = {
+                                functionCall: { name: tc.function.name, args: tc.function.arguments || tc.function.args }
+                            };
+                            if (tc.thought_signature) {
+                                part.thought_signature = tc.thought_signature;
+                            }
+                            return part;
+                        });
+                    } else {
+                        parts = [{ text: msg.content || '' }];
                     }
                     break;
                 case 'tool':
@@ -228,6 +217,10 @@ export class GeminiProvider extends BaseLLMProvider {
             }
             if (part.functionCall) {
                 if (!toolCalls) toolCalls = [];
+                // Preserve thought_signature if present (Gemini 3 requirement)
+                if (part.thought_signature) {
+                    part.functionCall.thought_signature = part.thought_signature;
+                }
                 toolCalls.push(part.functionCall);
             }
         }
@@ -252,7 +245,11 @@ export class GeminiProvider extends BaseLLMProvider {
 
         return {
             content: textContent,
-            tool_calls: toolCalls ? (Array.isArray(toolCalls) ? toolCalls : [toolCalls]).map(fc => ({ type: 'function', function: fc })) : null,
+            tool_calls: toolCalls ? (Array.isArray(toolCalls) ? toolCalls : [toolCalls]).map(fc => ({
+                type: 'function',
+                function: fc,
+                thought_signature: fc.thought_signature
+            })) : null,
             finishReason: normalizedFinishReason, // Standardized: 'completed', 'truncated', etc.
             _rawFinishReason: candidate.finishReason, // Keep original for debugging
             _responseFormat: options.responseFormat,