Update config_provider.py

config_provider.py

@@ -78,18 +78,6 @@ class GlobalConfig(BaseModel):
 
     users: List["UserConfig"] = []
 
-    def is_gpt_mode(self) -> bool:
-        """Check if running in GPT mode (any variant)"""
-        return self.work_mode in ("gpt4o", "gpt4o-mini")
-    
-    def get_gpt_model(self) -> str:
-        """Get the GPT model name for OpenAI API"""
-        if self.work_mode == "gpt4o":
-            return "gpt-4o"
-        elif self.work_mode == "gpt4o-mini":
-            return "gpt-4o-mini"
-        return None
-
     def get_plain_token(self) -> Optional[str]:
         if self.cloud_token:
             # Lazy import to avoid circular dependency
@@ -143,13 +131,25 @@ class GlobalConfig(BaseModel):
         }
         
     def is_cloud_mode(self) -> bool:
-        """Check if running in cloud mode (hfcloud or cloud)"""
-        return self.work_mode in ("hfcloud", "cloud")
+        """Check if running in cloud mode (anything except on-premise)"""
+        return self.work_mode != "on-premise"
     
     def is_on_premise(self) -> bool:
         """Check if running in on-premise mode"""
         return self.work_mode == "on-premise"
 
+    def is_gpt_mode(self) -> bool:
+        """Check if running in GPT mode (any variant)"""
+        return self.work_mode in ("gpt4o", "gpt4o-mini")
+    
+    def get_gpt_model(self) -> str:
+        """Get the GPT model name for OpenAI API"""
+        if self.work_mode == "gpt4o":
+            return "gpt-4o"
+        elif self.work_mode == "gpt4o-mini":
+            return "gpt-4o-mini"
+        return None
+
 # ---------------- Global -----------------
 class UserConfig(BaseModel):
     username: str
@@ -337,20 +337,27 @@ class ServiceConfig(BaseModel):
 class ConfigProvider:
     _instance: Optional[ServiceConfig] = None
     _CONFIG_PATH = Path(__file__).parent / "service_config.jsonc"
+    _lock = threading.Lock()  # Thread-safe access için lock
 
     @classmethod
     def get(cls) -> ServiceConfig:
+        """Get cached config - thread-safe"""
         if cls._instance is None:
-            cls._instance = cls._load()
-            cls._instance.build_index()
-            cls._check_environment_setup()
+            with cls._lock:
+                # Double-checked locking pattern
+                if cls._instance is None:
+                    cls._instance = cls._load()
+                    cls._instance.build_index()
+                    cls._check_environment_setup()
         return cls._instance
 
     @classmethod
     def reload(cls) -> ServiceConfig:
-        """Force reload configuration from file"""
-        cls._instance = None
-        return cls.get()
+        """Force reload configuration from file - used after UI saves"""
+        with cls._lock:
+            log("🔄 Reloading configuration...")
+            cls._instance = None
+            return cls.get()
     
     @classmethod
     def update_config(cls, config_dict: dict):