magector 2.1.3 → 2.1.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "magector",
-  "version": "2.1.3",
+  "version": "2.1.5",
   "description": "Semantic code search for Magento 2 — index, search, MCP server",
   "type": "module",
   "main": "src/mcp-server.js",
@@ -39,10 +39,10 @@
     "ruvector": "^0.1.96"
   },
   "optionalDependencies": {
-    "@magector/cli-darwin-arm64": "2.1.3",
-    "@magector/cli-linux-x64": "2.1.3",
-    "@magector/cli-linux-arm64": "2.1.3",
-    "@magector/cli-win32-x64": "2.1.3"
+    "@magector/cli-darwin-arm64": "2.1.5",
+    "@magector/cli-linux-x64": "2.1.5",
+    "@magector/cli-linux-arm64": "2.1.5",
+    "@magector/cli-win32-x64": "2.1.5"
   },
   "keywords": [
     "magento",

package/src/mcp-server.js

@@ -81,8 +81,8 @@ function logToFile(level, message) {
   }
 }
 
-// Initialize log file on startup
-try { writeFileSync(LOG_PATH, `[${new Date().toISOString()}] [INFO] Magector MCP server starting\n`); } catch {}
+// Initialize log on startup — append to preserve history across MCP restarts
+try { appendFileSync(LOG_PATH, `\n[${new Date().toISOString()}] [INFO] ════ Magector MCP server starting ════\n`); } catch {}
 
 // Log resolved configuration so the log file is self-contained for debugging
 logToFile('INFO', `Config: MAGENTO_ROOT=${config.magentoRoot}`);
@@ -264,11 +264,28 @@ function startBackgroundReindex() {
     console.error(`Reindex already running (PID ${existingPid}) — skipping`);
     reindexInProgress = true; // mark locally so tools know
     // Poll the external process and react when it finishes
+    const tempDbPath = config.dbPath + '.new';
     const pollInterval = setInterval(() => {
       if (!getRunningReindexPid()) {
         clearInterval(pollInterval);
         reindexInProgress = false;
-        logToFile('INFO', 'External reindex finished. Restarting serve process.');
+        // Swap the new DB into place if the external reindex produced one
+        if (existsSync(tempDbPath)) {
+          try {
+            if (existsSync(config.dbPath)) {
+              const backupPath = config.dbPath + '.bak';
+              if (existsSync(backupPath)) { try { unlinkSync(backupPath); } catch {} }
+              renameSync(config.dbPath, backupPath);
+              logToFile('INFO', 'Old DB moved to .bak');
+            }
+            renameSync(tempDbPath, config.dbPath);
+            logToFile('INFO', 'External reindex complete — new index swapped into place.');
+          } catch (e) {
+            logToFile('ERR', `Failed to swap index after external reindex: ${e.message}`);
+          }
+        } else {
+          logToFile('INFO', 'External reindex finished but no .new file found — skipping swap.');
+        }
         if (serveProcess) serveProcess.kill();
         searchCache.clear();
         startServeProcess();
@@ -320,12 +337,17 @@ function startBackgroundReindex() {
   // Write PID file so other MCP instances know a reindex is running
   writeReindexPidFile(reindexProcess.pid);
 
-  reindexProcess.stdout.on('data', (d) => {
-    const text = d.toString().replace(/\x1b\[[0-9;]*m/g, '').trim();
+  // Log stdout/stderr line-by-line using readline to avoid buffering issues.
+  // Without this, Rust tracing output accumulates in pipe buffers and progress
+  // entries arrive in large chunks instead of in real time.
+  const indexStdout = createInterface({ input: reindexProcess.stdout });
+  const indexStderr = createInterface({ input: reindexProcess.stderr });
+  indexStdout.on('line', (line) => {
+    const text = line.replace(/\x1b\[[0-9;]*m/g, '').trim();
     if (text) logToFile('INDEX', text);
   });
-  reindexProcess.stderr.on('data', (d) => {
-    const text = d.toString().replace(/\x1b\[[0-9;]*m/g, '').trim();
+  indexStderr.on('line', (line) => {
+    const text = line.replace(/\x1b\[[0-9;]*m/g, '').trim();
     if (text) logToFile('INDEX', text);
   });