@contentgrowth/llm-service 0.6.96 → 0.6.98

package/package.json

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

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

@@ -139,7 +139,7 @@ export class GeminiProvider extends BaseLLMProvider {
         }
 
         if (tools && tools.length > 0) {
-            requestOptions.tools = [{ functionDeclarations: tools.map(t => t.function) }];
+            requestOptions.config.tools = [{ functionDeclarations: tools.map(t => t.function) }];
         }
 
         console.log('[GeminiProvider] generateContent request:', JSON.stringify(requestOptions, null, 2));
@@ -180,9 +180,12 @@ 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] Finish Reason:', candidate.finishReason);
+            console.error('[GeminiProvider] Safety Ratings:', JSON.stringify(candidate.safetyRatings, null, 2));
+            console.error('[GeminiProvider] Full Candidate:', JSON.stringify(candidate, null, 2));
+
             throw new LLMServiceException(
-                'Model returned empty response. This usually means the prompt or schema is confusing the model.',
+                `Model returned empty response. Finish Reason: ${candidate.finishReason}.`,
                 500
             );
         }
@@ -381,10 +384,19 @@ export class GeminiProvider extends BaseLLMProvider {
         if (!imagePart || !imagePart.inlineData) {
             // Fallback: Check if it returned a URI or other format, or just text
             const textPart = response.candidates?.[0]?.content?.parts?.find(p => p.text);
+            const candidate = response.candidates?.[0];
+
+            console.error('[GeminiProvider] Image generation failed (no image data)');
+            if (candidate) {
+                console.error('[GeminiProvider] Finish Reason:', candidate.finishReason);
+                console.error('[GeminiProvider] Safety Ratings:', JSON.stringify(candidate.safetyRatings, null, 2));
+                console.error('[GeminiProvider] Full Candidate:', JSON.stringify(candidate, null, 2));
+            }
+
             if (textPart) {
                 console.warn('[GeminiProvider] Model returned text instead of image:', textPart.text);
             }
-            throw new Error('No image data in response');
+            throw new Error(`No image data in response. Finish Reason: ${candidate?.finishReason}`);
         }
 
         return {
@@ -443,6 +455,7 @@ export class GeminiProvider extends BaseLLMProvider {
             this._pendingOperations.delete(operationName);
 
             if (operation.error) {
+                console.error('[GeminiProvider] Video generation failed:', JSON.stringify(operation.error, null, 2));
                 result.error = operation.error;
             } else {
                 const videoResult = operation.response;