magector 1.2.5 → 1.2.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "magector",
-  "version": "1.2.5",
+  "version": "1.2.7",
   "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.5",
-    "@magector/cli-linux-x64": "1.2.5",
-    "@magector/cli-linux-arm64": "1.2.5",
-    "@magector/cli-win32-x64": "1.2.5"
+    "@magector/cli-darwin-arm64": "1.2.7",
+    "@magector/cli-linux-x64": "1.2.7",
+    "@magector/cli-linux-arm64": "1.2.7",
+    "@magector/cli-win32-x64": "1.2.7"
   },
   "keywords": [
     "magento",

package/src/cli.js

@@ -77,17 +77,19 @@ async function runIndex(targetPath) {
   console.log(`Database: ${path.resolve(config.dbPath)}\n`);
 
   try {
-    const output = execFileSync(binary, [
+    execFileSync(binary, [
       'index',
       '-m', path.resolve(root),
       '-d', path.resolve(config.dbPath),
       '-c', modelPath
-    ], { encoding: 'utf-8', timeout: 600000, stdio: ['pipe', 'pipe', 'pipe'] });
-    if (output.trim()) console.log(output.trim());
+    ], { timeout: 600000, stdio: 'inherit' });
     console.log('\nIndexing complete.');
   } catch (err) {
-    const output = err.stderr || err.stdout || err.message;
-    console.error(`Indexing error: ${output}`);
+    if (err.status) {
+      console.error('Indexing failed.');
+      process.exit(err.status);
+    }
+    console.error(`Indexing error: ${err.message}`);
     process.exit(1);
   }
 }

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;
@@ -199,18 +208,18 @@ export async function init(projectPath) {
   console.log('\nIndexing codebase...');
   const startTime = Date.now();
   try {
-    const output = execFileSync(binary, [
+    execFileSync(binary, [
       'index',
       '-m', projectPath,
       '-d', dbPath,
       '-c', modelPath
-    ], { encoding: 'utf-8', timeout: 600000, stdio: ['pipe', 'pipe', 'pipe'] });
-    if (output.trim()) {
-      console.log(output.trim().split('\n').map(l => `  ${l}`).join('\n'));
-    }
+    ], { timeout: 600000, stdio: 'inherit' });
   } catch (err) {
-    const output = err.stderr || err.stdout || err.message;
-    console.error(`Indexing error: ${output}`);
+    if (err.status) {
+      console.error('Indexing failed.');
+      process.exit(err.status);
+    }
+    console.error(`Indexing error: ${err.message}`);
     process.exit(1);
   }
   const elapsed = ((Date.now() - startTime) / 1000).toFixed(1);