fix: defer server settings access to resolve circular imports

Author: lemorage

python/cocoindex/cli.py

@@ -151,17 +151,23 @@ def evaluate(flow_name: str | None, output_dir: str | None, cache: bool = True):
     options = flow.EvaluateAndDumpOptions(output_dir=output_dir, use_cache=cache)
     fl.evaluate_and_dump(options)
 
-_default_server_settings = lib.ServerSettings.from_env()
-
 COCOINDEX_HOST = 'https://cocoindex.io'
 
+_default_server_settings = None
+
+def _get_default_server_settings():
+    global _default_server_settings
+    if _default_server_settings is None:
+        _default_server_settings = lib.ServerSettings.from_env()
+    return _default_server_settings
+
 @cli.command()
 @click.option(
-    "-a", "--address", type=str, default=_default_server_settings.address,
+    "-a", "--address", type=str, default=lambda: _get_default_server_settings().address,
     help="The address to bind the server to, in the format of IP:PORT.")
 @click.option(
     "-c", "--cors-origin", type=str,
-    default=_default_server_settings.cors_origins and ','.join(_default_server_settings.cors_origins),
+    default=lambda: _get_default_server_settings().cors_origins and ','.join(_get_default_server_settings().cors_origins),
     help="The origins of the clients (e.g. CocoInsight UI) to allow CORS from. "
          "Multiple origins can be specified as a comma-separated list. "
          "e.g. `https://cocoindex.io,http://localhost:3000`")
@@ -184,9 +190,9 @@ def server(address: str, live_update: bool, quiet: bool, cors_origin: str | None
 
     It will allow tools like CocoInsight to access the server.
     """
-    cors_origins : set[str] = set()
+    cors_origins: set[str] = set()
     if cors_origin is not None:
-        cors_origins.update(s for o in cors_origin.split(',') if (s:= o.strip()) != '')
+        cors_origins.update(s for o in cors_origin.split(',') if (s := o.strip()) != '')
     if cors_cocoindex:
         cors_origins.add(COCOINDEX_HOST)
     if cors_local is not None: