Update config_provider.py

config_provider.py

@@ -81,7 +81,10 @@ class ConfigProvider:
             # Ensure required fields
             if 'config' not in config_data:
                 config_data['config'] = {}
-            
+
+            # Ensure providers exist
+            cls._ensure_providers(config_data)
+
             # Parse API configs (handle JSON strings)
             if 'apis' in config_data:
                 cls._parse_api_configs(config_data['apis'])
@@ -228,8 +231,11 @@ class ConfigProvider:
                 )
                 
             except Exception as e:
-                log_error("Failed to save configuration", error=str(e), user=username)
-                raise
+                log_error(f"Failed to save config", error=str(e))
+                raise ConfigurationError(
+                    f"Failed to save configuration: {str(e)}",
+                    config_key="service_config.jsonc"
+                )
     
     # ===================== Environment Methods =====================
     
@@ -258,7 +264,72 @@ class ConfigProvider:
             
             # Save
             cls.save(config, username)
-    
+
+    @classmethod
+    def _ensure_providers(cls, config_data: Dict[str, Any]) -> None:
+        """Ensure config has required provider structure"""
+        if 'config' not in config_data:
+            config_data['config'] = {}
+        
+        config = config_data['config']
+        
+        # Ensure provider settings exist
+        if 'llm_provider' not in config:
+            config['llm_provider'] = {
+                'name': 'spark_cloud',
+                'api_key': '',
+                'endpoint': 'http://localhost:8080',
+                'settings': {}
+            }
+        
+        if 'tts_provider' not in config:
+            config['tts_provider'] = {
+                'name': 'no_tts',
+                'api_key': '',
+                'endpoint': None,
+                'settings': {}
+            }
+        
+        if 'stt_provider' not in config:
+            config['stt_provider'] = {
+                'name': 'no_stt',
+                'api_key': '',
+                'endpoint': None,
+                'settings': {}
+            }
+        
+        # Ensure providers list exists
+        if 'providers' not in config:
+            config['providers'] = [
+                {
+                    "type": "llm",
+                    "name": "spark_cloud",
+                    "display_name": "Spark LLM (Cloud)",
+                    "requires_endpoint": True,
+                    "requires_api_key": True,
+                    "requires_repo_info": False,
+                    "description": "Spark Cloud LLM Service"
+                },
+                {
+                    "type": "tts",
+                    "name": "no_tts",
+                    "display_name": "No TTS",
+                    "requires_endpoint": False,
+                    "requires_api_key": False,
+                    "requires_repo_info": False,
+                    "description": "Text-to-Speech disabled"
+                },
+                {
+                    "type": "stt",
+                    "name": "no_stt",
+                    "display_name": "No STT",
+                    "requires_endpoint": False,
+                    "requires_api_key": False,
+                    "requires_repo_info": False,
+                    "description": "Speech-to-Text disabled"
+                }
+            ]
+                
     # ===================== Project Methods =====================
     
     @classmethod
@@ -636,8 +707,21 @@ class ConfigProvider:
             # Update fields
             for key, value in update_data.items():
                 if hasattr(version, key) and key not in ['no', 'created_date', 'created_by', 'published', 'last_update_date']:
-                    setattr(version, key, value)
-            
+                    # Handle LLM config
+                    if key == 'llm' and isinstance(value, dict):
+                        setattr(version, key, LLMConfiguration(**value))
+                    # Handle intents
+                    elif key == 'intents' and isinstance(value, list):
+                        intents = []
+                        for intent_data in value:
+                            if isinstance(intent_data, dict):
+                                intents.append(IntentConfig(**intent_data))
+                            else:
+                                intents.append(intent_data)
+                        setattr(version, key, intents)
+                    else:
+                        setattr(version, key, value)
+                        
             version.last_update_date = get_current_timestamp()
             version.last_update_user = username
             
@@ -705,7 +789,6 @@ class ConfigProvider:
             )
     
     # ===================== API Methods =====================
-    
     @classmethod
     def create_api(cls, api_data: dict, username: str) -> APIConfig:
         """Create new API"""