test fix

lib/screens/settings_screen.dart

@@ -18,7 +18,6 @@ class _SettingsScreenState extends State<SettingsScreen> {
   final _negativePromptController = TextEditingController();
   final _hfApiKeyController = TextEditingController();
   final _gameMasterPromptController = TextEditingController();
-  final _llmApiKeyController = TextEditingController();
   final _modelNameController = TextEditingController();
   final _settingsService = SettingsService();
   final _availabilityService = ModelAvailabilityService();
@@ -42,7 +41,6 @@ class _SettingsScreenState extends State<SettingsScreen> {
     _negativePromptController.text = _settingsService.negativeVideoPrompt;
     _hfApiKeyController.text = _settingsService.huggingfaceApiKey;
     _gameMasterPromptController.text = _settingsService.gameMasterPrompt;
-    _llmApiKeyController.text = _settingsService.llmApiKey;
     _showSceneDebugInfo = _settingsService.showSceneDebugInfo;
     _enableSimulation = _settingsService.enableSimulation;
     _simLoopDelayInSec = _settingsService.simLoopDelayInSec;
@@ -78,7 +76,6 @@ class _SettingsScreenState extends State<SettingsScreen> {
     _negativePromptController.dispose();
     _hfApiKeyController.dispose();
     _gameMasterPromptController.dispose();
-    _llmApiKeyController.dispose();
     _modelNameController.dispose();
     super.dispose();
   }
@@ -147,7 +144,6 @@ class _SettingsScreenState extends State<SettingsScreen> {
             // Switch to first compatible provider
             setState(() {
               _selectedLlmProvider = compatibleProviders.first;
-              _currentProvider = LLMProvider.getById(_selectedLlmProvider);
             });
             await _settingsService.setLlmProvider(_selectedLlmProvider);
           } else {
@@ -365,8 +361,7 @@ class _SettingsScreenState extends State<SettingsScreen> {
                           // Reset to first available provider if we had built-in selected
                           if (_selectedLlmProvider == 'built-in') {
                             _selectedLlmProvider = _availableProviders.isNotEmpty ? _availableProviders.first.id : 'hf-inference';
-                            _currentProvider = LLMProvider.getById(_selectedLlmProvider);
-                          }
+                                        }
                         });
                         await _settingsService.setLlmModel(newModel.modelId);
                         if (_selectedLlmProvider != 'built-in') {

lib/services/settings_service.dart

@@ -13,7 +13,6 @@ class SettingsService {
   static const String _gameMasterPromptKey = 'game_master_prompt';
   static const String _llmProviderKey = 'llm_provider';
   static const String _llmModelKey = 'llm_model';
-  static const String _llmApiKeyKey = 'llm_api_key';
   static final SettingsService _instance = SettingsService._internal();
   
   factory SettingsService() => _instance;
@@ -91,13 +90,6 @@ class SettingsService {
     _settingsController.add(null);
   }
 
-  String get llmApiKey => _prefs.getString(_llmApiKeyKey) ?? '';
-  
-  Future<void> setLlmApiKey(String apiKey) async {
-    await _prefs.setString(_llmApiKeyKey, apiKey);
-    _settingsController.add(null);
-  }
-
   void dispose() {
     _settingsController.close();
   }

lib/services/websocket_api_service.dart

@@ -876,7 +876,6 @@ class WebSocketApiService {
     final settings = SettingsService();
     final llmProvider = settings.llmProvider;
     final llmModel = settings.llmModel;
-    final llmApiKey = settings.llmApiKey;
     final hfApiKey = settings.huggingfaceApiKey;
 
     while (_activeSearches[query] == true && 
@@ -893,8 +892,7 @@ class WebSocketApiService {
               'llm_config': {
                 'provider': llmProvider,
                 'model': llmModel,
-                'api_key': llmApiKey,
-                'hf_token': hfApiKey,
+                  'hf_token': hfApiKey,
                 'game_master_prompt': settings.gameMasterPrompt,
               },
             },
@@ -1056,8 +1054,6 @@ class WebSocketApiService {
           debugPrint('WebSocketApiService: Processing join chat response');
           _pendingRequests[requestId]!.complete(data);
         } else if (action == 'search' && data['success'] == true && data['result'] != null) {
-          final result = VideoResult.fromJson(data['result'] as Map<String, dynamic>);
-          
           // Complete the pending request but don't add to search results here
           // The search results will be handled by the startContinuousSearch method
           _pendingRequests[requestId]!.complete(data);
@@ -1274,7 +1270,6 @@ class WebSocketApiService {
       final settings = SettingsService();
       final llmProvider = settings.llmProvider;
       final llmModel = settings.llmModel;
-      final llmApiKey = settings.llmApiKey;
       final hfApiKey = settings.huggingfaceApiKey;
       
       final response = await _sendRequest(
@@ -1285,7 +1280,7 @@ class WebSocketApiService {
             'llm_config': {
               'provider': llmProvider,
               'model': llmModel,
-              'api_key': llmApiKey,
+              'hf_token': hfApiKey,
               'game_master_prompt': settings.gameMasterPrompt,
             },
           },
@@ -1356,7 +1351,6 @@ class WebSocketApiService {
     final settings = SettingsService();
     final llmProvider = settings.llmProvider;
     final llmModel = settings.llmModel;
-    final llmApiKey = settings.llmApiKey;
     final hfApiKey = settings.huggingfaceApiKey;
     
     final response = await _sendRequest(
@@ -1368,7 +1362,7 @@ class WebSocketApiService {
           'llm_config': {
             'provider': llmProvider,
             'model': llmModel,
-            'api_key': llmApiKey,
+            'hf_token': hfApiKey,
             'game_master_prompt': settings.gameMasterPrompt,
           },
         },
@@ -1438,7 +1432,6 @@ class WebSocketApiService {
       final settings = SettingsService();
       final llmProvider = settings.llmProvider;
       final llmModel = settings.llmModel;
-      final llmApiKey = settings.llmApiKey;
       final hfApiKey = settings.huggingfaceApiKey;
       
       final response = await _sendRequest(
@@ -1455,7 +1448,7 @@ class WebSocketApiService {
             'llm_config': {
               'provider': llmProvider,
               'model': llmModel,
-              'api_key': llmApiKey,
+              'hf_token': hfApiKey,
               'game_master_prompt': settings.gameMasterPrompt,
             },
           },

server/llm_utils.py

@@ -159,29 +159,18 @@ def get_inference_client(llm_config: Optional[dict] = None) -> InferenceClient:
             raise ValueError("Built-in provider is not available. Server HF_TOKEN is not configured.")
 
     model = llm_config.get('model', '')
-    user_provider_api_key = llm_config.get('api_key', '')  # Provider-specific API key
     user_hf_token = llm_config.get('hf_token', '')  # User's HF token
     
     try:
-        # Case 1: Use a provider with a provider-specific API key if available
-        # This mode is currently hidden in the Flutter UI (we don't ask for provider-specific keys yet)
-        # but it is implemented here so that we don't forget it later
-        if user_provider_api_key:
-            return InferenceClient(
-                provider=provider,
-                model=model,
-                api_key=user_provider_api_key
-            )
-
-        # Case 2: Use a provider with user's HF token if available
-        elif user_hf_token:
+        # Use provider with user's HF token if available
+        if user_hf_token:
             return InferenceClient(
                 provider=provider,
                 model=model,
                 token=user_hf_token
             )
         else:
-            raise ValueError(f"No API key provided for provider '{provider}'. Please provide either a valid {provider} API key or your Hugging Face API key.")
+            raise ValueError(f"No Hugging Face API key provided for provider '{provider}'. Please provide your Hugging Face API key.")
 
     except ValueError:
         # Re-raise ValueError for missing API keys