@contentgrowth/llm-service 0.6.91 → 0.6.92

package/package.json

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

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

@@ -135,43 +135,29 @@ export class GeminiProvider extends BaseLLMProvider {
             tools: tools ? [{ functionDeclarations: tools.map(t => t.function) }] : undefined,
         });
 
-        // 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
+        // New SDK returns candidates directly on result
+        // Extract function calls from parts
         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 (result.candidates?.[0]?.content?.parts) {
+            const functionCallParts = result.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
+        // Extract text content from parts
         let textContent = '';
         try {
-            if (typeof response.text === 'function') {
-                textContent = response.text();
-            } else if (typeof response.text === 'string') {
-                textContent = response.text;
-            } else if (response.candidates?.[0]?.content?.parts) {
+            if (result.candidates?.[0]?.content?.parts) {
                 // Concatenate text from parts
-                textContent = response.candidates[0].content.parts
+                textContent = result.candidates[0].content.parts
                     .filter(p => p.text)
                     .map(p => p.text)
                     .join('');
+            } else if (result.text) {
+                textContent = typeof result.text === 'function' ? result.text() : result.text;
             }
         } catch (e) {
-            // response.text() throws if there is no text content (e.g. only tool calls)
             // This is expected behavior for tool-only responses
         }
 
@@ -372,12 +358,13 @@ export class GeminiProvider extends BaseLLMProvider {
             generationConfig
         });
 
-        const response = result.response;
-        const imagePart = response.candidates?.[0]?.content?.parts?.find(
+        // New SDK returns candidates directly on result, not result.response
+        const imagePart = result.candidates?.[0]?.content?.parts?.find(
             part => part.inlineData && part.inlineData.mimeType?.startsWith('image/')
         );
 
         if (!imagePart || !imagePart.inlineData) {
+            console.error('[GeminiProvider] No image in response. Result:', JSON.stringify(result, null, 2));
             throw new Error('No image data in response');
         }