added load_dotenv monkey patch to display a warning if it is called after ibind has been imported

Author: Voyz

ibind/__init__.py

@@ -9,8 +9,7 @@
 from ibind.client.ibkr_definitions import snapshot_keys_to_ids
 from ibind.support.errors import ExternalBrokerError
 from ibind.support.logs import ibind_logs_initialize
-from ibind.support.py_utils import execute_in_parallel
-
+from ibind.support.py_utils import execute_in_parallel, patch_dotenv
 
 __all__ = [
     'ibind_logs_initialize',
@@ -31,3 +30,4 @@
     'ExternalBrokerError',
 ]
 
+patch_dotenv()

ibind/support/py_utils.py

@@ -1,9 +1,11 @@
 import copy
 import inspect
 import os
+import sys
 import threading
 import time
 import traceback
+import warnings
 from concurrent.futures import ThreadPoolExecutor, as_completed
 from enum import Enum, EnumMeta
 from functools import wraps
@@ -324,4 +326,22 @@ def print_table(my_dict, column_order=None):
     column_size = [max(map(len,col)) for col in zip(*rv)]
     formatter = '   '.join(["{{:>{}}}".format(i) for i in column_size])
     for i, item in enumerate(rv):
-        print(formatter.format(*item))
+        print(formatter.format(*item))
+
+def patch_dotenv():
+    try:
+        import dotenv
+        from dotenv import load_dotenv
+
+        # Wrap the original load_dotenv function
+        def warn_if_late_load(*args, **kwargs):
+            if 'ibind.var' in sys.modules:
+                warnings.warn("⚠️ WARNING: `load_dotenv()` was called after `ibind` was imported. Environment variables were already read and changes may not take effect. Call `load_dotenv()` before importing `ibind` to ensure proper behavior.", RuntimeWarning, stacklevel=2)
+            return load_dotenv(*args, **kwargs)
+
+        # Replace the original load_dotenv with the wrapped version
+        dotenv.load_dotenv = warn_if_late_load
+
+    except ImportError:
+        pass  # dotenv is not installed, nothing to patch
+