@contentgrowth/llm-service 0.6.9 → 0.6.91

package/package.json

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

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

@@ -135,12 +135,41 @@ export class GeminiProvider extends BaseLLMProvider {
             tools: tools ? [{ functionDeclarations: tools.map(t => t.function) }] : undefined,
         });
 
-        const response = result.response;
-        const toolCalls = response.functionCalls?.() || response.functionCalls || null;
+        // New SDK returns result directly, not result.response
+        // Debug log to understand structure
+        console.log('[GeminiProvider] Result structure:', JSON.stringify(Object.keys(result), null, 2));
+
+        // Handle both old (result.response) and new (direct result) structures
+        const response = result.response || result;
+
+        // Extract function calls - try multiple possible locations
+        let toolCalls = null;
+        if (typeof response.functionCalls === 'function') {
+            toolCalls = response.functionCalls();
+        } else if (response.functionCalls) {
+            toolCalls = response.functionCalls;
+        } else if (response.candidates?.[0]?.content?.parts) {
+            // Check parts for function calls
+            const functionCallParts = response.candidates[0].content.parts.filter(p => p.functionCall);
+            if (functionCallParts.length > 0) {
+                toolCalls = functionCallParts.map(p => p.functionCall);
+            }
+        }
 
+        // Extract text content - try multiple possible locations
         let textContent = '';
         try {
-            textContent = typeof response.text === 'function' ? response.text() : (response.text || '');
+            if (typeof response.text === 'function') {
+                textContent = response.text();
+            } else if (typeof response.text === 'string') {
+                textContent = response.text;
+            } else if (response.candidates?.[0]?.content?.parts) {
+                // Concatenate text from parts
+                textContent = response.candidates[0].content.parts
+                    .filter(p => p.text)
+                    .map(p => p.text)
+                    .join('');
+            }
         } catch (e) {
             // response.text() throws if there is no text content (e.g. only tool calls)
             // This is expected behavior for tool-only responses
@@ -149,7 +178,7 @@ export class GeminiProvider extends BaseLLMProvider {
         // Validate that we have EITHER content OR tool calls
         if (!textContent && (!toolCalls || toolCalls.length === 0)) {
             console.error('[GeminiProvider] Model returned empty response (no text, no tool calls)');
-            console.error('[GeminiProvider] Contents:', JSON.stringify(contents, null, 2));
+            console.error('[GeminiProvider] Full result:', JSON.stringify(result, null, 2));
             throw new LLMServiceException(
                 'Model returned empty response. This usually means the prompt or schema is confusing the model.',
                 500