Fix critical issues from code review

- Fix SQL initialization: Use executescript() instead of execute() for multiple statements in message_store.py
- Fix PROJECT_ROOT path: Reduce from 6 parents to 3 parents in server/api/mcp/config.py
- Add backward compatibility shim: Support legacy dipeo_server imports in server/__init__.py
- Remove hardcoded credentials: Use NGROK_AUTH environment variable in Makefile

Co-authored-by: Seunghyun Ji <sorryhyun@users.noreply.github.com>

Author: github-actions[bot]

Makefile

@@ -162,7 +162,11 @@ dev-server:
 	DIPEO_BASE_DIR="$(shell pwd)" python server/main.py 2>&1 | sed 's/^/[server] /' & \
 	SERVER_PID=$$!; \
 	sleep 3; \
-	ngrok http 8000 --basic-auth "sorryhyun:sorrysorry" 2>&1 | sed 's/^/[ngrok] /' & \
+	if [ -n "$$NGROK_AUTH" ]; then \
+		ngrok http 8000 --basic-auth "$$NGROK_AUTH" 2>&1 | sed 's/^/[ngrok] /' & \
+	else \
+		ngrok http 8000 2>&1 | sed 's/^/[ngrok] /' & \
+	fi; \
 	NGROK_PID=$$!; \
 	wait
 

dipeo/infrastructure/storage/message_store.py

@@ -14,7 +14,7 @@ def __init__(self, db_path: Path):
 
     async def initialize(self):
         async with aiosqlite.connect(self.db_path) as db:
-            await db.execute("""
+            await db.executescript("""
                 CREATE TABLE IF NOT EXISTS messages (
                     id TEXT PRIMARY KEY,
                     execution_id TEXT NOT NULL,

server/__init__.py

@@ -11,4 +11,31 @@
 management, and integration with external services.
 """
 
+import sys
+from types import ModuleType
+
 __version__ = "1.0.0"
+
+# Backward compatibility: Make this module accessible as 'dipeo_server'
+# to support legacy imports (e.g., 'from dipeo_server import ...')
+# This allows a gradual migration from the old module structure
+_this_module = sys.modules[__name__]
+sys.modules["dipeo_server"] = _this_module
+
+# Also create a dipeo_server namespace that mirrors this module's structure
+# This handles imports like 'from dipeo_server.api import ...'
+class _CompatModule(ModuleType):
+    """Compatibility module that redirects to the new server module."""
+
+    def __getattr__(self, name):
+        # Import the actual module from server.*
+        try:
+            actual_module = __import__(f"server.{name}", fromlist=[name])
+            return actual_module
+        except ImportError:
+            raise AttributeError(f"module 'dipeo_server' has no attribute '{name}'")
+
+# Set up the compatibility module
+_compat_module = _CompatModule("dipeo_server")
+_compat_module.__path__ = _this_module.__path__ if hasattr(_this_module, '__path__') else []
+sys.modules["dipeo_server"] = _compat_module

server/api/mcp/config.py

@@ -7,7 +7,7 @@
 
 DEFAULT_MCP_TIMEOUT = int(os.environ.get("MCP_DEFAULT_TIMEOUT", "300"))
 
-PROJECT_ROOT = Path(__file__).parent.parent.parent.parent.parent.parent
+PROJECT_ROOT = Path(__file__).parent.parent.parent
 
 mcp_server = FastMCP(
     name="dipeo-mcp-server",