fix: hoted workflow

Author: emir-karabeg

sim/lib/utils.ts

@@ -197,34 +197,3 @@ export function formatDuration(durationMs: number): string {
 export function generateApiKey(): string {
   return `sim_${nanoid(32)}`
 }
-
-/**
- * Rotates through available API keys for a provider
- * @param provider - The provider to get a key for (e.g., 'openai')
- * @returns The selected API key
- * @throws Error if no API keys are configured for rotation
- */
-export function getRotatingApiKey(provider: string): string {
-  if (provider !== 'openai') {
-    throw new Error(`No rotation implemented for provider: ${provider}`)
-  }
-
-  // Get all OpenAI keys from environment
-  const keys = []
-  
-  // Add keys if they exist in environment variables
-  if (process.env.OPENAI_API_KEY_1) keys.push(process.env.OPENAI_API_KEY_1)
-  if (process.env.OPENAI_API_KEY_2) keys.push(process.env.OPENAI_API_KEY_2)
-  if (process.env.OPENAI_API_KEY_3) keys.push(process.env.OPENAI_API_KEY_3)
-  
-  if (keys.length === 0) {
-    throw new Error('No API keys configured for rotation. Please configure OPENAI_API_KEY_1, OPENAI_API_KEY_2, or OPENAI_API_KEY_3.')
-  }
-  
-  // Simple round-robin rotation based on current minute
-  // This distributes load across keys and is stateless
-  const currentMinute = new Date().getMinutes()
-  const keyIndex = currentMinute % keys.length
-  
-  return keys[keyIndex]
-}

sim/providers/openai/index.ts

@@ -1,53 +1,10 @@
 import OpenAI from 'openai'
 import { createLogger } from '@/lib/logs/console-logger'
-import { getRotatingApiKey } from '@/lib/utils'
 import { executeTool } from '@/tools'
 import { ProviderConfig, ProviderRequest, ProviderResponse, TimeSegment } from '../types'
 
 const logger = createLogger('OpenAI Provider')
 
-/**
- * Helper function to handle API key rotation for GPT-4o
- * @param apiKey - The original API key from the request
- * @param model - The model being used
- * @returns The API key to use (original or rotating)
- */
-async function getApiKey(apiKey: string | undefined, model: string): Promise<string> {
-  // Check if we should use a rotating key
-  const isHostedVersion = process.env.NEXT_PUBLIC_APP_URL === 'https://www.simstudio.ai'
-  const isGPT4o = model === 'gpt-4o'
-  
-  // On hosted version, always use rotating key for GPT-4o models
-  // This handles both null/undefined API keys and user-provided API keys
-  if (isHostedVersion && isGPT4o) {
-    try {
-      // Use the rotating key mechanism
-      return getRotatingApiKey('openai')
-    } catch (error) {
-      logger.warn('Failed to get rotating API key', {
-        error: error instanceof Error ? error.message : String(error)
-      })
-      
-      // If we couldn't get a rotating key and have a user key, use it as fallback
-      // This should only happen if rotation system is misconfigured
-      if (apiKey) {
-        logger.info('Falling back to user-provided API key')
-        return apiKey
-      }
-      
-      // No rotating key and no user key - throw specific error
-      throw new Error('No API key available for OpenAI. Please configure API key rotation or provide a valid API key.')
-    }
-  }
-  
-  // For non-hosted versions or non-GPT4o models, require the provided key
-  if (!apiKey) {
-    throw new Error('API key is required for OpenAI')
-  }
-  
-  return apiKey
-}
-
 /**
  * OpenAI provider configuration
  */
@@ -69,11 +26,18 @@ export const openaiProvider: ProviderConfig = {
       hasResponseFormat: !!request.responseFormat,
     })
 
-    // Get the appropriate API key based on our rotation logic
-    const apiKey = await getApiKey(request.apiKey, request.model || 'gpt-4o')
+    if (!request.apiKey) {
+      logger.error('OpenAI API key missing in request', {
+        hasModel: !!request.model,
+        hasSystemPrompt: !!request.systemPrompt,
+        hasMessages: !!request.messages,
+        hasTools: !!request.tools,
+      })
+      throw new Error('API key is required for OpenAI')
+    }
 
     const openai = new OpenAI({
-      apiKey: apiKey,
+      apiKey: request.apiKey,
       dangerouslyAllowBrowser: true,
     })