fix: init config first (#33)

Author: qin-ctx

opencontext/cli.py

@@ -209,7 +209,8 @@ def handle_start(args: argparse.Namespace) -> int:
     logger.info("Starting all modules")
     lab_instance.start_capture()
 
-    web_config = lab_instance.config.get("web", {})
+    from opencontext.config.global_config import get_config
+    web_config = get_config("web")
     if web_config.get("enabled", True):
         # Command line arguments override config file
         host = args.host if args.host else web_config.get("host", "localhost")
@@ -232,13 +233,10 @@ def _setup_logging(config_path: Optional[str]) -> None:
     Args:
         config_path: Optional path to configuration file
     """
-    config_manager = ConfigManager()
-    if config_path:
-        config_manager.load_config(config_path)
-    else:
-        config_manager.update_config(config_manager.get_default_config())
-    
-    setup_logging(config_manager.get_config().get('logging', {}))
+    from opencontext.config.global_config import GlobalConfig
+    GlobalConfig.get_instance().initialize(config_path)
+
+    setup_logging(GlobalConfig.get_instance().get_config('logging'))
 
 def main() -> int:
     """Main entry point.

opencontext/config/config_manager.py

@@ -280,7 +280,7 @@ def load_user_settings(self) -> bool:
             if not user_settings:
                 return False
             self._config = self.deep_merge(self._config, user_settings)
-            logger.info(f"User settings loaded successfully: {user_settings}")
+            # logger.info(f"User settings loaded successfully: {user_settings}")
             return True
         except Exception as e:
             logger.error(f"Failed to load user settings: {e}")

opencontext/server/component_initializer.py

@@ -49,15 +49,10 @@
 class ComponentInitializer:
     """Handles initialization of various OpenContext components."""
 
-    def __init__(self, config_manager: ConfigManager = None):
-        # Backward compatibility: if config_manager is passed, use it; otherwise use global config
-        if config_manager:
-            self.config_manager = config_manager
-            GlobalConfig.get_instance().set_config_manager(config_manager)
-        else:
-            # Use global config
-            global_config = GlobalConfig.get_instance()
-            self.config_manager = global_config.get_config_manager()
+    def __init__(self):
+        # Use global config
+        global_config = GlobalConfig.get_instance()
+        self.config_manager = global_config.get_config_manager()
 
         self.config = self.config_manager.get_config() if self.config_manager else GlobalConfig.get_instance().get_config()
         self.global_config = GlobalConfig.get_instance()

opencontext/server/opencontext.py

@@ -33,8 +33,7 @@ class OpenContext:
 
     def __init__(
         self,
-        config_path: Optional[str] = None,
-        config_dict: Optional[Dict[str, Any]] = None,
+        config_path: Optional[str] = None
     ):
         """
         Initialize OpenContext.
@@ -43,11 +42,6 @@ def __init__(
             config_path: Configuration file path
             config_dict: Configuration dictionary (unused, kept for compatibility)
         """
-        # Initialize configuration manager
-        self.config_manager = ConfigManager()
-        self.config_manager.load_config(config_path)
-        self.config = self.config_manager.get_config()
-
         # Initialize core components
         self.capture_manager = ContextCaptureManager()
         self.processor_manager = ContextProcessorManager()
@@ -57,7 +51,7 @@ def __init__(
         self.completion_service = None  # Smart completion service
 
         # Helper classes
-        self.component_initializer = ComponentInitializer(self.config_manager)
+        self.component_initializer = ComponentInitializer()
         self.context_operations: Optional[ContextOperations] = None
 
         # Web server state