magector 1.2.6 → 1.2.8

package/package.json

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

package/src/init.js

@@ -95,6 +95,35 @@ function writeMcpConfig(projectPath, ides, dbPath) {
 /**
  * Write IDE rules files.
  */
+/**
+ * Replace the Magector section in an existing file, or append if not present.
+ * Magector sections are delimited by marker comments.
+ */
+function upsertMagectorSection(filePath, content, markerStart, markerEnd) {
+  if (!existsSync(filePath)) {
+    writeFileSync(filePath, markerStart + '\n' + content + markerEnd + '\n');
+    return 'created';
+  }
+  const existing = readFileSync(filePath, 'utf-8');
+  const startIdx = existing.indexOf(markerStart);
+  const endIdx = existing.indexOf(markerEnd);
+  if (startIdx !== -1 && endIdx !== -1) {
+    const updated = existing.slice(0, startIdx) + markerStart + '\n' + content + existing.slice(endIdx);
+    writeFileSync(filePath, updated);
+    return 'updated';
+  }
+  if (existing.includes('Magector')) {
+    // Legacy format without markers — append fresh section
+    appendFileSync(filePath, '\n\n' + markerStart + '\n' + content + markerEnd + '\n');
+    return 'updated';
+  }
+  appendFileSync(filePath, '\n\n' + markerStart + '\n' + content + markerEnd + '\n');
+  return 'appended';
+}
+
+const MARKER_START = '<!-- magector:start -->';
+const MARKER_END = '<!-- magector:end -->';
+
 function writeRules(projectPath, ides) {
   const written = [];
 
@@ -103,34 +132,14 @@ function writeRules(projectPath, ides) {
 
   if (writeCursor) {
     const rulesPath = path.join(projectPath, '.cursorrules');
-    if (!existsSync(rulesPath)) {
-      writeFileSync(rulesPath, CURSORRULES);
-      written.push('.cursorrules (created)');
-    } else {
-      const existing = readFileSync(rulesPath, 'utf-8');
-      if (!existing.includes('Magector')) {
-        appendFileSync(rulesPath, '\n\n' + CURSORRULES);
-        written.push('.cursorrules (appended)');
-      } else {
-        written.push('.cursorrules (already configured)');
-      }
-    }
+    const result = upsertMagectorSection(rulesPath, CURSORRULES, MARKER_START, MARKER_END);
+    written.push(`.cursorrules (${result})`);
   }
 
   if (writeClaude) {
     const claudePath = path.join(projectPath, 'CLAUDE.md');
-    if (!existsSync(claudePath)) {
-      writeFileSync(claudePath, CLAUDE_MD);
-      written.push('CLAUDE.md (created)');
-    } else {
-      const existing = readFileSync(claudePath, 'utf-8');
-      if (!existing.includes('Magector')) {
-        appendFileSync(claudePath, '\n\n' + CLAUDE_MD);
-        written.push('CLAUDE.md (appended)');
-      } else {
-        written.push('CLAUDE.md (already configured)');
-      }
-    }
+    const result = upsertMagectorSection(claudePath, CLAUDE_MD, MARKER_START, MARKER_END);
+    written.push(`CLAUDE.md (${result})`);
   }
 
   return written;

package/src/mcp-server.js

@@ -40,6 +40,13 @@ const config = {
 
 // ─── Rust Core Integration ──────────────────────────────────────
 
+// Env vars to suppress ONNX Runtime native logs that would pollute stdout/JSON-RPC
+const rustEnv = {
+  ...process.env,
+  ORT_LOG_LEVEL: 'error',
+  RUST_LOG: 'error',
+};
+
 function rustSearch(query, limit = 10) {
   const result = execFileSync(config.rustBinary, [
     'search', query,
@@ -47,7 +54,7 @@ function rustSearch(query, limit = 10) {
     '-c', config.modelCache,
     '-l', String(limit),
     '-f', 'json'
-  ], { encoding: 'utf-8', timeout: 30000, stdio: ['pipe', 'pipe', 'pipe'] });
+  ], { encoding: 'utf-8', timeout: 30000, stdio: ['pipe', 'pipe', 'pipe'], env: rustEnv });
   return JSON.parse(result);
 }
 
@@ -57,7 +64,7 @@ function rustIndex(magentoRoot) {
     '-m', magentoRoot,
     '-d', config.dbPath,
     '-c', config.modelCache
-  ], { encoding: 'utf-8', timeout: 600000, stdio: ['pipe', 'pipe', 'pipe'] });
+  ], { encoding: 'utf-8', timeout: 600000, stdio: ['pipe', 'pipe', 'pipe'], env: rustEnv });
   return result;
 }
 
@@ -65,7 +72,7 @@ function rustStats() {
   const result = execFileSync(config.rustBinary, [
     'stats',
     '-d', config.dbPath
-  ], { encoding: 'utf-8', timeout: 10000, stdio: ['pipe', 'pipe', 'pipe'] });
+  ], { encoding: 'utf-8', timeout: 10000, stdio: ['pipe', 'pipe', 'pipe'], env: rustEnv });
   // Parse text output: "Total vectors: N" and "Embedding dim: N"
   const vectors = result.match(/Total vectors:\s*(\d+)/)?.[1] || '0';
   const dim = result.match(/Embedding dim:\s*(\d+)/)?.[1] || '384';