@entro314labs/ai-changelog-generator 3.1.1 → 3.2.0

package/src/infrastructure/providers/utils/provider-utils.js

@@ -12,40 +12,40 @@
 export function selectModelByComplexity(commitDetails, modelConfig) {
   const {
     complexModel = 'default-complex',
-    standardModel = 'default-standard', 
+    standardModel = 'default-standard',
     mediumModel = 'default-medium',
-    smallModel = 'default-small'
-  } = modelConfig;
+    smallModel = 'default-small',
+  } = modelConfig
 
   // Breaking or complex changes need the most capable model
   if (commitDetails.breaking || commitDetails.complex || commitDetails.files > 20) {
-    return { 
-      model: complexModel, 
-      reason: 'Complex or breaking change requiring advanced reasoning' 
-    };
+    return {
+      model: complexModel,
+      reason: 'Complex or breaking change requiring advanced reasoning',
+    }
   }
-  
+
   // Large commits need standard model
   if (commitDetails.lines > 1000 || commitDetails.files > 10) {
-    return { 
-      model: standardModel, 
-      reason: 'Large change requiring standard capabilities' 
-    };
+    return {
+      model: standardModel,
+      reason: 'Large change requiring standard capabilities',
+    }
   }
-  
+
   // Medium commits
   if (commitDetails.lines > 200 || commitDetails.files > 5) {
-    return { 
-      model: mediumModel, 
-      reason: 'Medium-sized change' 
-    };
+    return {
+      model: mediumModel,
+      reason: 'Medium-sized change',
+    }
   }
-  
+
   // Small commits can use the most efficient model
-  return { 
-    model: smallModel, 
-    reason: 'Small change, optimized for efficiency' 
-  };
+  return {
+    model: smallModel,
+    reason: 'Small change, optimized for efficiency',
+  }
 }
 
 /**
@@ -56,24 +56,22 @@ export function selectModelByComplexity(commitDetails, modelConfig) {
  */
 export async function standardConnectionTest(generateCompletion, defaultModel) {
   try {
-    const response = await generateCompletion([
-      { role: 'user', content: 'Test connection' }
-    ], { 
-      max_tokens: 5, 
+    const response = await generateCompletion([{ role: 'user', content: 'Test connection' }], {
+      max_tokens: 5,
       model: defaultModel,
-      temperature: 0.1 
-    });
-    
-    return { 
-      success: true, 
-      response: response.content || response.text || 'Connection successful', 
-      model: response.model || defaultModel
-    };
+      temperature: 0.1,
+    })
+
+    return {
+      success: true,
+      response: response.content || response.text || 'Connection successful',
+      model: response.model || defaultModel,
+    }
   } catch (error) {
-    return { 
-      success: false, 
-      error: error.message 
-    };
+    return {
+      success: false,
+      error: error.message,
+    }
   }
 }
 
@@ -92,8 +90,8 @@ export function createProviderErrorResponse(providerName, operation, reason, alt
     operation,
     error: reason,
     alternatives: alternatives.length > 0 ? alternatives : [`Check ${providerName} configuration`],
-    timestamp: new Date().toISOString()
-  };
+    timestamp: new Date().toISOString(),
+  }
 }
 
 /**
@@ -107,8 +105,8 @@ export function createProviderSuccessResponse(providerName, data = {}) {
     available: true,
     provider: providerName,
     ...data,
-    timestamp: new Date().toISOString()
-  };
+    timestamp: new Date().toISOString(),
+  }
 }
 
 /**
@@ -126,14 +124,14 @@ export function buildCapabilities(baseCapabilities = {}, modelSpecificCapabiliti
     large_context: false,
     streaming: false,
     temperature_control: true,
-    max_tokens_control: true
-  };
+    max_tokens_control: true,
+  }
 
   return {
     ...defaultCapabilities,
     ...baseCapabilities,
-    ...modelSpecificCapabilities
-  };
+    ...modelSpecificCapabilities,
+  }
 }
 
 /**
@@ -144,28 +142,28 @@ export function buildCapabilities(baseCapabilities = {}, modelSpecificCapabiliti
  * @returns {Object} Provider-specific configuration
  */
 export function extractProviderConfig(config, providerPrefix, defaults = {}) {
-  const providerConfig = {};
-  
+  const providerConfig = {}
+
   // Extract all config keys that start with the provider prefix
-  Object.keys(config).forEach(key => {
-    if (key.startsWith(providerPrefix + '_') || key === providerPrefix) {
-      providerConfig[key] = config[key];
+  Object.keys(config).forEach((key) => {
+    if (key.startsWith(`${providerPrefix}_`) || key === providerPrefix) {
+      providerConfig[key] = config[key]
     }
-  });
-  
+  })
+
   // Add fallback AI_MODEL configs
-  if (config.AI_MODEL && !providerConfig[providerPrefix + '_MODEL']) {
-    providerConfig[providerPrefix + '_MODEL'] = config.AI_MODEL;
+  if (config.AI_MODEL && !providerConfig[`${providerPrefix}_MODEL`]) {
+    providerConfig[`${providerPrefix}_MODEL`] = config.AI_MODEL
   }
-  
+
   // Apply defaults for missing values
-  Object.keys(defaults).forEach(key => {
+  Object.keys(defaults).forEach((key) => {
     if (providerConfig[key] === undefined) {
-      providerConfig[key] = defaults[key];
+      providerConfig[key] = defaults[key]
     }
-  });
-  
-  return providerConfig;
+  })
+
+  return providerConfig
 }
 
 /**
@@ -175,12 +173,14 @@ export function extractProviderConfig(config, providerPrefix, defaults = {}) {
  * @returns {number} Parsed numeric value
  */
 export function parseNumericConfig(value, defaultValue) {
-  if (typeof value === 'number') return value;
+  if (typeof value === 'number') {
+    return value
+  }
   if (typeof value === 'string') {
-    const parsed = parseInt(value, 10);
-    return isNaN(parsed) ? defaultValue : parsed;
+    const parsed = Number.parseInt(value, 10)
+    return Number.isNaN(parsed) ? defaultValue : parsed
   }
-  return defaultValue;
+  return defaultValue
 }
 
 /**
@@ -192,17 +192,21 @@ export function parseNumericConfig(value, defaultValue) {
  */
 export async function validateModelWithFallbacks(testModelFn, modelName, fallbackModels = []) {
   try {
-    const result = await testModelFn(modelName);
+    const result = await testModelFn(modelName)
     if (result.success || result.available) {
       return createProviderSuccessResponse('validation', {
         model: modelName,
-        capabilities: result.capabilities || {}
-      });
-    } else {
-      return createProviderErrorResponse('validation', 'model_test', result.error || 'Model not available', fallbackModels);
+        capabilities: result.capabilities || {},
+      })
     }
+    return createProviderErrorResponse(
+      'validation',
+      'model_test',
+      result.error || 'Model not available',
+      fallbackModels
+    )
   } catch (error) {
-    return createProviderErrorResponse('validation', 'model_test', error.message, fallbackModels);
+    return createProviderErrorResponse('validation', 'model_test', error.message, fallbackModels)
   }
 }
 
@@ -214,27 +218,27 @@ export async function validateModelWithFallbacks(testModelFn, modelName, fallbac
  */
 export function formatMessagesForProvider(messages, format) {
   switch (format) {
-    case 'anthropic':
-      const systemMessage = messages.find(m => m.role === 'system');
-      const userMessages = messages.filter(m => m.role !== 'system');
+    case 'anthropic': {
+      const systemMessage = messages.find((m) => m.role === 'system')
+      const userMessages = messages.filter((m) => m.role !== 'system')
       return {
         system: systemMessage ? systemMessage.content : undefined,
-        messages: userMessages
-      };
-      
-    case 'google':
-      const systemInstruction = messages.find(m => m.role === 'system')?.content;
+        messages: userMessages,
+      }
+    }
+
+    case 'google': {
+      const systemInstruction = messages.find((m) => m.role === 'system')?.content
       const history = messages
-        .filter(m => m.role !== 'system')
-        .map(m => ({
+        .filter((m) => m.role !== 'system')
+        .map((m) => ({
           role: m.role === 'assistant' ? 'model' : 'user',
-          parts: [{ text: m.content }]
-        }));
-      return { systemInstruction, history };
-      
-    case 'openai':
+          parts: [{ text: m.content }],
+        }))
+      return { systemInstruction, history }
+    }
     default:
-      return { messages };
+      return { messages }
   }
 }
 
@@ -245,24 +249,24 @@ export function formatMessagesForProvider(messages, format) {
  * @returns {Object} Client initialization options
  */
 export function buildClientOptions(config, defaults = {}) {
-  const options = { ...defaults };
-  
+  const options = { ...defaults }
+
   // Common timeout handling
   if (config.timeout || config.TIMEOUT) {
-    options.timeout = parseNumericConfig(config.timeout || config.TIMEOUT, 60000);
+    options.timeout = parseNumericConfig(config.timeout || config.TIMEOUT, 60000)
   }
-  
+
   // Common retry handling
   if (config.maxRetries || config.MAX_RETRIES) {
-    options.maxRetries = parseNumericConfig(config.maxRetries || config.MAX_RETRIES, 2);
+    options.maxRetries = parseNumericConfig(config.maxRetries || config.MAX_RETRIES, 2)
   }
-  
+
   // Common base URL handling
   if (config.baseURL || config.BASE_URL || config.API_URL) {
-    options.baseURL = config.baseURL || config.BASE_URL || config.API_URL;
+    options.baseURL = config.baseURL || config.BASE_URL || config.API_URL
   }
-  
-  return options;
+
+  return options
 }
 
 /**
@@ -277,10 +281,11 @@ export function buildRequestParams(messages, options, defaults = {}) {
     messages,
     model: options.model || defaults.model,
     max_tokens: options.max_tokens || defaults.max_tokens || 1000,
-    temperature: options.temperature !== undefined ? options.temperature : (defaults.temperature || 0.3),
+    temperature:
+      options.temperature !== undefined ? options.temperature : defaults.temperature || 0.3,
     stream: !!options.stream,
     ...(options.tools && { tools: options.tools }),
     ...(options.tool_choice && { tool_choice: options.tool_choice }),
-    ...defaults.extraParams
-  };
-}
+    ...defaults.extraParams,
+  }
+}