@contentgrowth/llm-service 0.7.1 → 0.7.2

package/package.json

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

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

@@ -180,7 +180,7 @@ export class GeminiProvider extends BaseLLMProvider {
             }
         }
 
-        console.log('[GeminiProvider] generateContent request:', JSON.stringify(requestOptions, null, 2));
+        // console.log('[GeminiProvider] generateContent request:', JSON.stringify(requestOptions, null, 2));
 
         let response;
         try {
@@ -228,7 +228,7 @@ export class GeminiProvider extends BaseLLMProvider {
             );
         }
 
-        console.log('Gemini returns:', textContent);
+        // console.log('Gemini returns:', textContent);
         // Return with parsed JSON if applicable
         return {
             content: textContent,
@@ -265,7 +265,7 @@ export class GeminiProvider extends BaseLLMProvider {
                     // Use responseSchema for strict structured output
                     // Must convert to Gemini Schema format (Uppercase types)
                     config.responseSchema = this._convertToGeminiSchema(schema);
-                    console.log('[GeminiProvider] Using Strict JSON mode with schema (responseSchema)');
+                    // console.log('[GeminiProvider] Using Strict JSON mode with schema (responseSchema)');
                 } else {
                     console.warn('[GeminiProvider] Using legacy JSON mode without schema - may produce markdown wrappers');
                 }
@@ -336,9 +336,7 @@ export class GeminiProvider extends BaseLLMProvider {
         // - Brace extraction as fallback
         const parsed = extractJsonFromResponse(content);
 
-        if (parsed) {
-            console.log('[GeminiProvider] Successfully parsed JSON from response');
-        } else {
+        if (!parsed) {
             console.error('[GeminiProvider] Failed to extract valid JSON from response');
             console.error('[GeminiProvider] Content preview:', content.substring(0, 200));
         }
@@ -354,7 +352,7 @@ export class GeminiProvider extends BaseLLMProvider {
                 const tool_call_id = `gemini-tool-call-${index}`;
                 toolCall.id = tool_call_id;
 
-                console.log(`[Tool Call] ${toolName} with arguments:`, toolCall.function.args);
+                // console.log(`[Tool Call] ${toolName} with arguments:`, toolCall.function.args);
 
                 if (!tool) {
                     console.error(`[Tool Error] Tool '${toolName}' not found`);
@@ -362,7 +360,7 @@ export class GeminiProvider extends BaseLLMProvider {
                 }
                 try {
                     const output = await tool(toolCall.function.args, { env, tenantId });
-                    console.log(`[Tool Result] ${toolName} returned:`, output.substring(0, 200) + (output.length > 200 ? '...' : ''));
+                    // console.log(`[Tool Result] ${toolName} returned:`, output.substring(0, 200) + (output.length > 200 ? '...' : ''));
                     return { tool_call_id, output };
                 } catch (error) {
                     console.error(`[Tool Error] ${toolName} failed:`, error.message);
@@ -411,7 +409,7 @@ export class GeminiProvider extends BaseLLMProvider {
             requestOptions.config.systemInstruction = { parts: [{ text: systemPrompt }] };
         }
 
-        console.log('[GeminiProvider] imageGeneration request:', JSON.stringify(requestOptions, null, 2));
+        // console.log('[GeminiProvider] imageGeneration request:', JSON.stringify(requestOptions, null, 2));
 
         const response = await this.client.models.generateContent(requestOptions);
 
@@ -449,9 +447,12 @@ export class GeminiProvider extends BaseLLMProvider {
 
     async startVideoGeneration(prompt, images, modelName, systemPrompt, options = {}) {
         // Use unified client for video generation
-        const operation = await this.client.models.generateVideos({
+        // Prepend system prompt to user prompt if provided, as video models often expect instructions in the prompt
+        const effectivePrompt = systemPrompt ? `${systemPrompt}\n\n${prompt}` : prompt;
+
+        const requestConfig = {
             model: modelName,
-            prompt: prompt,
+            prompt: effectivePrompt,
             config: {
                 durationSeconds: options.durationSeconds || 6,
                 aspectRatio: options.aspectRatio || '16:9',
@@ -459,15 +460,35 @@ export class GeminiProvider extends BaseLLMProvider {
                 // Pass reference images if provided
                 ...(images && images.length > 0 ? { referenceImages: images } : {}),
             }
-        });
+        };
 
-        // Store operation for later polling
-        this._pendingOperations.set(operation.name, operation);
+        // Create a loggable copy of the config
+        const logConfig = JSON.parse(JSON.stringify(requestConfig));
+        if (logConfig.config && logConfig.config.referenceImages) {
+            logConfig.config.referenceImages = logConfig.config.referenceImages.map(img => ({
+                ...img,
+                data: `... (${img.data ? img.data.length : 0} bytes)` // Summarize data
+            }));
+        }
+
+        console.log('[GeminiProvider] startVideoGeneration request:', JSON.stringify(logConfig, null, 2));
+
+        try {
+            const operation = await this.client.models.generateVideos(requestConfig);
 
-        return { operationName: operation.name };
+            // Store operation for later polling
+            this._pendingOperations.set(operation.name, operation);
+
+            return { operationName: operation.name };
+        } catch (error) {
+            console.error('[GeminiProvider] startVideoGeneration failed:', error);
+            throw error;
+        }
     }
 
     async getVideoGenerationStatus(operationName) {
+        console.log(`[GeminiProvider] Checking status for operation: ${operationName}`);
+
         // Get the operation from cache or fetch it
         let operation = this._pendingOperations.get(operationName);
 
@@ -488,6 +509,8 @@ export class GeminiProvider extends BaseLLMProvider {
             state: operation.metadata?.state || (operation.done ? 'COMPLETED' : 'PROCESSING'),
         };
 
+        console.log(`[GeminiProvider] Operation status: ${result.state}, Progress: ${result.progress}%`);
+
         if (operation.done) {
             // Clean up from cache
             this._pendingOperations.delete(operationName);