Fix Redis connection timing by deferring tool imports

- Move tool imports into _import_tools() function called after config reload
- This ensures RedisConnectionManager singleton is never instantiated before config is set
- Much cleaner solution than resetting singleton - prevents the problem instead of fixing it
- Tools are imported after reload_redis_config() so they use the correct Redis configuration
- Maintains backward compatibility with main() function

Author: mortensi

src/common/connection.py

@@ -32,6 +32,7 @@ def get_connection(cls, decode_responses=True) -> Redis:
                         "max_connections_per_node": 10 
                     }
                 else:
+                    print("Redis config:", REDIS_CFG, file=sys.stderr)
                     redis_class: Type[Union[Redis, RedisCluster]] = redis.Redis
                     connection_params = {
                         "host": REDIS_CFG["host"],

src/main.py

@@ -2,22 +2,25 @@
 import os
 import click
 
-from src.common.connection import RedisConnectionManager
 from src.common.server import mcp
-import src.tools.server_management
-import src.tools.misc
-import src.tools.redis_query_engine
-import src.tools.hash
-import src.tools.list
-import src.tools.string
-import src.tools.json
-import src.tools.sorted_set
-import src.tools.set
-import src.tools.stream
-import src.tools.pub_sub
 from src.common.config import MCP_TRANSPORT, parse_redis_uri, set_redis_env_from_config, reload_redis_config
 
 
+def _import_tools():
+    """Import all tool modules after configuration is set up."""
+    import src.tools.server_management
+    import src.tools.misc
+    import src.tools.redis_query_engine
+    import src.tools.hash
+    import src.tools.list
+    import src.tools.string
+    import src.tools.json
+    import src.tools.sorted_set
+    import src.tools.set
+    import src.tools.stream
+    import src.tools.pub_sub
+
+
 class RedisMCPServer:
     def __init__(self):
         print("Starting the Redis MCP Server", file=sys.stderr)
@@ -87,6 +90,9 @@ def cli(url, host, port, db, username, password,
     # Reload Redis configuration to pick up the new environment variables
     reload_redis_config()
 
+    # Import tools after configuration is set up (ensures Redis connection uses new config)
+    _import_tools()
+
     # Set MCP transport settings
     os.environ['MCP_TRANSPORT'] = mcp_transport
     os.environ['MCP_HOST'] = mcp_host
@@ -99,6 +105,8 @@ def cli(url, host, port, db, username, password,
 
 def main():
     """Legacy main function for backward compatibility."""
+    # Import tools (uses default environment variables)
+    _import_tools()
     server = RedisMCPServer()
     server.run()