magector 2.6.3 → 2.6.5

package/package.json

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

package/src/mcp-server.js

@@ -2283,6 +2283,7 @@ async function analyzeImpact(className) {
 
   // Filesystem fallback: if vector search found too few files, find the class file via glob
   if (relatedPaths.length < 5 && root) {
+    logToFile('INFO', `impact_analysis: vector search returned ${relatedPaths.length} files for "${className}" — using filesystem fallback`);
     try {
       const classFiles = await glob(`**/${shortName}.php`, { cwd: root, absolute: false, nodir: true });
       const existingPaths = new Set(relatedPaths.map(r => r.path));
@@ -4097,6 +4098,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         if (results.length === 0 && config.magentoRoot) {
           const shortName = args.className.split('\\').pop();
           const globPattern = `**/${shortName}.php`;
+          logToFile('INFO', `find_class: vector search returned 0 results for "${args.className}" — using filesystem fallback (glob ${globPattern})`);
           try {
             const files = await glob(globPattern, { cwd: config.magentoRoot, absolute: false, nodir: true, ignore: ['**/test/**', '**/tests/**', '**/Test/**'] });
             // Filter by namespace if provided
@@ -4128,6 +4130,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 results.push({ path: filePath, className, score: 0.5 });
               }
             }
+            if (results.length > 0) logToFile('INFO', `find_class: filesystem fallback found ${results.length} files for "${args.className}"`);
           } catch {}
         }
 
@@ -4156,16 +4159,28 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
           );
           results = results.concat(pathFallback);
         }
-        // Filesystem fallback: grep for the method definition in PHP files
+        // Filesystem fallback: use grep to find method definition across all PHP files
         if (results.length === 0 && config.magentoRoot) {
+          logToFile('INFO', `find_method: vector search returned 0 results for "${args.methodName}" — using filesystem fallback (grep -rl)`);
           try {
             const methodSig = `function ${args.methodName}(`;
-            // Use className to narrow the glob if provided
+            // Use className to narrow search if provided, otherwise grep all PHP
             const classShort = args.className ? args.className.split('\\').pop() : null;
-            const globPattern = classShort ? `**/${classShort}.php` : '**/*.php';
-            const files = await glob(globPattern, { cwd: config.magentoRoot, absolute: false, nodir: true, ignore: ['**/test/**', '**/tests/**', '**/Test/**'] });
-            for (const f of files.slice(0, classShort ? 50 : 500)) {
-              const absPath = path.join(config.magentoRoot, f);
+            let files = [];
+            if (classShort) {
+              files = await glob(`**/${classShort}.php`, { cwd: config.magentoRoot, absolute: false, nodir: true });
+            } else {
+              // Use grep -rl for fast search across all PHP files (much faster than reading each file)
+              try {
+                const grepResult = execFileSync('grep', ['-rl', '--include=*.php', methodSig, '.'],
+                  { cwd: config.magentoRoot, encoding: 'utf-8', timeout: 15000, stdio: ['pipe', 'pipe', 'pipe'] });
+                files = grepResult.trim().split('\n').filter(Boolean).map(f => f.replace(/^\.\//, ''));
+              } catch {
+                // grep returns exit code 1 when no matches — fall through with empty files
+              }
+            }
+            for (const f of files.slice(0, 20)) {
+              const absPath = f.startsWith('/') ? f : path.join(config.magentoRoot, f);
               let content;
               try { content = readFileSync(absPath, 'utf-8'); } catch { continue; }
               if (!content.includes(methodSig)) continue;
@@ -4189,7 +4204,10 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
               });
               if (results.length >= 5) break;
             }
-          } catch {}
+            if (results.length > 0) logToFile('INFO', `find_method: filesystem fallback found ${results.length} files containing "${args.methodName}"`);
+          } catch (err) {
+            logToFile('WARN', `find_method: filesystem fallback error: ${err.message}`);
+          }
         }
         // Boost exact method matches to top
         results = results.map(r => {
@@ -4734,6 +4752,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
 
         // Filesystem fallback: if vector search found nothing, glob the module directory
         if (results.length === 0 && config.magentoRoot && vendorPath) {
+          logToFile('INFO', `module_structure: vector search returned 0 results for "${args.moduleName}" — using filesystem fallback (glob ${vendorPath})`);
           try {
             const vendorGlob = `**/${vendorPath}**/*.{php,xml,phtml}`;
             const files = await glob(vendorGlob, { cwd: config.magentoRoot, absolute: false, nodir: true });