perf(mcp): move table creation to background task

Moving create_tables to the background task prevents blocking the main
thread during startup, allowing the MCP server to respond to the
initial handshake immediately. This significantly improves perceived
performance and connectivity with clients that have short timeouts.

Signed-off-by: Roman Gushchin <[email protected]>

Author: rgushchin

src/bin/semcode-mcp.rs

@@ -5178,16 +5178,18 @@ async fn main() -> Result<()> {
     // Create MCP server
     let server = Arc::new(McpServer::new(&database_path, &args.git_repo, args.model_path).await?);
 
-    // Ensure tables exist
-    server.db.create_tables().await?;
-
     // Spawn background task to index current commit if needed
     eprintln!("[Background] Spawning background indexing task");
     let db_for_indexing = server.db.clone();
     let git_repo_for_indexing = args.git_repo.clone();
     let indexing_state_for_bg = server.indexing_state.clone();
     let notification_tx_for_bg = server.notification_tx.clone();
     let _indexing_handle = tokio::spawn(async move {
+        // Ensure tables exist before indexing
+        if let Err(e) = db_for_indexing.create_tables().await {
+            eprintln!("[Background] Error creating/verifying tables: {}", e);
+        }
+
         index_current_commit_background(
             db_for_indexing,
             git_repo_for_indexing,