magector 2.9.1 → 2.11.0

package/package.json

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

package/src/mcp-server.js

@@ -4143,13 +4143,61 @@ server.setRequestHandler(ListToolsRequestSchema, async () => ({
           },
           context: {
             type: 'number',
-            description: 'Lines of context around each match (default: 0). Like grep -C.',
-            default: 0
+            description: 'Lines of context around each match (default: 2). Like grep -C.',
+            default: 2
           }
         },
         required: ['pattern']
       }
     },
+    {
+      name: 'magento_trace_api',
+      description: 'Trace a REST or GraphQL API endpoint from URL to implementation. Parses webapi.xml to find the service interface, resolves the DI preference to the concrete class, reads the execute/method body, and checks di.xml for constructor arguments. Returns the complete chain in one call.',
+      inputSchema: {
+        type: 'object',
+        properties: {
+          url: {
+            type: 'string',
+            description: 'REST API URL pattern to trace. Example: "/V1/orders/:orderId/items", "/V1/carts/mine/payment-information"'
+          },
+          interfaceName: {
+            type: 'string',
+            description: 'Alternative: service interface class name. Example: "ChangePaymentMethodInterface"'
+          },
+          method: {
+            type: 'string',
+            description: 'HTTP method (GET, PUT, POST, DELETE). Default: any.',
+            enum: ['GET', 'PUT', 'POST', 'DELETE']
+          }
+        }
+      }
+    },
+    {
+      name: 'magento_read',
+      description: 'Read a file from the Magento codebase. Use in magento_batch to read multiple files in a single MCP call (e.g., grep finds 5 files → read all 5 in one batch). Supports line ranges and method extraction.',
+      inputSchema: {
+        type: 'object',
+        properties: {
+          path: {
+            type: 'string',
+            description: 'File path relative to MAGENTO_ROOT. Example: "vendor/acme/module-sales/Model/OrderService.php"'
+          },
+          startLine: {
+            type: 'number',
+            description: 'Start reading from this line number (1-based). Default: 1 (beginning of file).'
+          },
+          endLine: {
+            type: 'number',
+            description: 'Stop reading at this line number (inclusive). Default: end of file. Use with startLine for large files.'
+          },
+          methodName: {
+            type: 'string',
+            description: 'Extract only this method from the file (uses brace-counting). Returns the complete method body with line numbers. Much more token-efficient than reading the whole file. Example: "execute"'
+          }
+        },
+        required: ['path']
+      }
+    },
   ]
 }));
 
@@ -4168,7 +4216,7 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
     // These tools have filesystem/di.xml fallbacks — work without serve process
     'magento_find_class', 'magento_find_method', 'magento_find_plugin',
     'magento_find_observer', 'magento_find_di_wiring', 'magento_module_structure',
-    'magento_batch', 'magento_find_config', 'magento_find_callers', 'magento_grep'];
+    'magento_batch', 'magento_find_config', 'magento_find_callers', 'magento_grep', 'magento_read', 'magento_trace_api'];
   if (warmupInProgress && !indexFreeTools.includes(name)) {
     logToFile('REQ', `${name} → blocked (warmup: loading index)`);
     return {
@@ -4969,7 +5017,22 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
           structure: structureOutput.categories
         });
 
-        return { content: [{ type: 'text', text: jsonOutput }] };
+        // Include README.md if it exists in the module directory
+        let readmeText = '';
+        if (results.length > 0) {
+          // Find module root from first result path
+          const firstPath = results[0].path || '';
+          const moduleRoot = firstPath.split('/').slice(0, 3).join('/');
+          if (moduleRoot) {
+            const readmePath = path.join(config.magentoRoot, moduleRoot, 'README.md');
+            try {
+              const readme = readFileSync(readmePath, 'utf-8');
+              readmeText = '\n\n## README.md\n\n' + readme.slice(0, 2000) + (readme.length > 2000 ? '\n...(truncated)' : '');
+            } catch { /* no README */ }
+          }
+        }
+
+        return { content: [{ type: 'text', text: jsonOutput + readmeText }] };
       }
 
       case 'magento_analyze_diff': {
@@ -5983,13 +6046,39 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 }
                 break;
               }
+              case 'magento_read': {
+                const filePath = path.join(config.magentoRoot, a.path);
+                let fileContent;
+                try { fileContent = readFileSync(filePath, 'utf-8'); } catch {
+                  text = `File not found: ${a.path}`;
+                  break;
+                }
+                if (a.methodName) {
+                  const body = readFullMethodBody(filePath, a.methodName);
+                  if (!body) { text = `Method ${a.methodName} not found in ${a.path}`; break; }
+                  const fLines = fileContent.split('\n');
+                  let mLine = 0;
+                  for (let i = 0; i < fLines.length; i++) {
+                    if (fLines[i].includes(`function ${a.methodName}(`)) { mLine = i + 1; break; }
+                  }
+                  text = body.split('\n').map((line, i) => `${mLine + i}\t${line}`).join('\n');
+                  break;
+                }
+                const allLines = fileContent.split('\n');
+                const s = Math.max((a.startLine || 1) - 1, 0);
+                const e = a.endLine ? Math.min(a.endLine, allLines.length) : allLines.length;
+                const sl = allLines.slice(s, e);
+                text = sl.map((line, i) => `${s + i + 1}\t${line}`).join('\n');
+                break;
+              }
               case 'magento_grep': {
                 const searchPath = a.path || '.';
                 const include = a.include || '*.php';
                 const maxRes = Math.min(a.maxResults || 30, 100);
-                const gArgs = ['-rn'];
+                const batchCtx = a.context !== undefined ? a.context : 2;
+                const gArgs = ['-rn', '-E'];
                 if (a.ignoreCase) gArgs.push('-i');
-                if (a.context) gArgs.push('-C', String(a.context));
+                if (batchCtx > 0) gArgs.push('-C', String(batchCtx));
                 for (const pat of include.split(',').map(p => p.trim())) gArgs.push('--include=' + pat);
                 gArgs.push('--', a.pattern, searchPath);
                 let out;
@@ -6023,9 +6112,10 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         const searchPath = args.path || '.';
         const include = args.include || '*.php';
         const maxResults = Math.min(args.maxResults || 50, 200);
-        const grepArgs = ['-rn'];
+        const ctxLines = args.context !== undefined ? args.context : 2;
+        const grepArgs = ['-rn', '-E'];
         if (args.ignoreCase) grepArgs.push('-i');
-        if (args.context) grepArgs.push('-C', String(args.context));
+        if (ctxLines > 0) grepArgs.push('-C', String(ctxLines));
         // Support multiple include patterns (e.g., "*.{php,xml}")
         for (const pat of include.split(',').map(p => p.trim())) {
           grepArgs.push('--include=' + pat);
@@ -6053,6 +6143,188 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
         return { content: [{ type: 'text', text }] };
       }
 
+      case 'magento_trace_api': {
+        const root = config.magentoRoot;
+        if (!root) return { content: [{ type: 'text', text: 'MAGENTO_ROOT not set.' }], isError: true };
+        let text = '';
+
+        // 1. Find the endpoint in webapi.xml files
+        const webapiFiles = await glob('**/etc/webapi.xml', { cwd: root, absolute: true, nodir: true });
+        let matchedRoute = null;
+        const searchUrl = args.url || '';
+        const searchInterface = args.interfaceName || '';
+        const searchMethod = args.method || '';
+
+        for (const wf of webapiFiles) {
+          let wContent;
+          try { wContent = readFileSync(wf, 'utf-8'); } catch { continue; }
+          const relPath = wf.replace(root + '/', '');
+
+          const routeRegex = /<route\s+url="([^"]+)"\s+method="([^"]+)"[^>]*>([\s\S]*?)<\/route>/g;
+          let rm;
+          while ((rm = routeRegex.exec(wContent)) !== null) {
+            const routeUrl = rm[1];
+            const routeMethod = rm[2];
+            const routeBody = rm[3];
+
+            const urlMatch = searchUrl ? routeUrl.includes(searchUrl) || searchUrl.includes(routeUrl) : false;
+            const ifaceMatch = searchInterface ? routeBody.includes(searchInterface) : false;
+            const methodMatch = searchMethod ? routeMethod === searchMethod : true;
+
+            if ((urlMatch || ifaceMatch) && methodMatch) {
+              const serviceMatch = routeBody.match(/class="([^"]+)"\s+method="([^"]+)"/);
+              if (serviceMatch) {
+                matchedRoute = {
+                  url: routeUrl,
+                  httpMethod: routeMethod,
+                  serviceClass: serviceMatch[1],
+                  serviceMethod: serviceMatch[2],
+                  file: relPath
+                };
+                // Extract resource
+                const resMatch = routeBody.match(/resource\s+ref="([^"]+)"/);
+                if (resMatch) matchedRoute.acl = resMatch[1];
+                break;
+              }
+            }
+          }
+          if (matchedRoute) break;
+        }
+
+        if (!matchedRoute) {
+          return { content: [{ type: 'text', text: `No API endpoint found matching url="${searchUrl}" interface="${searchInterface}"` }] };
+        }
+
+        text += `## API Endpoint\n\n`;
+        text += `- **URL:** \`${matchedRoute.httpMethod} ${matchedRoute.url}\`\n`;
+        text += `- **Interface:** \`${matchedRoute.serviceClass}::${matchedRoute.serviceMethod}()\`\n`;
+        text += `- **ACL:** \`${matchedRoute.acl || 'none'}\`\n`;
+        text += `- **webapi.xml:** \`${matchedRoute.file}\`\n\n`;
+
+        // 2. Find DI preference (implementation)
+        const diFiles = await getDiXmlFiles(root);
+        const shortIface = matchedRoute.serviceClass.split('\\').pop();
+        let implClass = null;
+        let implFile = null;
+
+        for (const { content: diContent, relPath } of diFiles) {
+          if (!diContent.includes(shortIface)) continue;
+          const prefRegex = /<preference\s+for="([^"]+)"\s+type="([^"]+)"\s*\/?>/g;
+          let pm;
+          while ((pm = prefRegex.exec(diContent)) !== null) {
+            if (pm[1].includes(shortIface)) {
+              implClass = pm[2];
+              implFile = relPath;
+              break;
+            }
+          }
+          if (implClass) break;
+        }
+
+        if (implClass) {
+          text += `## Implementation\n\n`;
+          text += `- **Class:** \`${implClass}\`\n`;
+          text += `- **di.xml:** \`${implFile}\`\n\n`;
+
+          // 3. Read the implementation method body
+          const implPhpFile = findClassFile(root, implClass);
+          if (implPhpFile) {
+            const relImpl = implPhpFile.replace(root + '/', '');
+            const body = readFullMethodBody(implPhpFile, matchedRoute.serviceMethod);
+            if (body) {
+              const implContent = readFileSync(implPhpFile, 'utf-8');
+              const lines = implContent.split('\n');
+              let mLine = 0;
+              for (let i = 0; i < lines.length; i++) {
+                if (lines[i].includes(`function ${matchedRoute.serviceMethod}(`)) { mLine = i + 1; break; }
+              }
+              text += `## ${matchedRoute.serviceMethod}() — \`${relImpl}\` (line ${mLine})\n\n`;
+              text += '```php\n' + body + '\n```\n\n';
+
+              // 4. Check for collectTotals / key patterns
+              const hasCollectTotals = body.includes('collectTotals');
+              const hasEventDispatch = body.includes('dispatch') || body.includes('eventManager');
+              text += `## Quick checks\n\n`;
+              text += `- collectTotals() called: **${hasCollectTotals ? 'YES' : 'NO'}**\n`;
+              text += `- Event dispatched: **${hasEventDispatch ? 'YES' : 'NO'}**\n`;
+
+              // 5. Extract constructor for DI understanding
+              const ctorBody = readFullMethodBody(implPhpFile, '__construct');
+              if (ctorBody) {
+                text += `\n## Constructor\n\n`;
+                text += '```php\n' + ctorBody + '\n```\n';
+              }
+            }
+          }
+
+          // 6. Check di.xml for type arguments (e.g., allowed payment methods)
+          for (const { content: diContent, relPath } of diFiles) {
+            const implShort = implClass.split('\\').pop();
+            if (!diContent.includes(implShort)) continue;
+            const typeBlockRegex = /<type\s+name="([^"]+)"[^>]*>([\s\S]*?)<\/type>/g;
+            let tm;
+            while ((tm = typeBlockRegex.exec(diContent)) !== null) {
+              if (tm[1].includes(implShort)) {
+                text += `\n## DI Arguments — \`${relPath}\`\n\n`;
+                text += '```xml\n' + tm[0] + '\n```\n';
+              }
+            }
+          }
+        }
+
+        // 7. Check for plugins on the interface
+        for (const { content: diContent, relPath } of diFiles) {
+          if (!diContent.includes(shortIface)) continue;
+          const typeBlockRegex = /<type\s+name="([^"]+)"[^>]*>([\s\S]*?)<\/type>/g;
+          let tm;
+          while ((tm = typeBlockRegex.exec(diContent)) !== null) {
+            if (tm[1].includes(shortIface) && tm[2].includes('<plugin')) {
+              text += `\n## Plugins on \`${tm[1]}\` — \`${relPath}\`\n\n`;
+              text += '```xml\n' + tm[0] + '\n```\n';
+            }
+          }
+        }
+
+        return { content: [{ type: 'text', text }] };
+      }
+
+      case 'magento_read': {
+        const root = config.magentoRoot;
+        if (!root) return { content: [{ type: 'text', text: 'MAGENTO_ROOT not set.' }], isError: true };
+        const filePath = path.join(root, args.path);
+        let content;
+        try { content = readFileSync(filePath, 'utf-8'); } catch (err) {
+          return { content: [{ type: 'text', text: `File not found: ${args.path}` }], isError: true };
+        }
+
+        // Method extraction mode: return only the specified method
+        if (args.methodName) {
+          const body = readFullMethodBody(filePath, args.methodName);
+          if (!body) {
+            return { content: [{ type: 'text', text: `## ${args.path}\n\nMethod \`${args.methodName}\` not found in file.` }] };
+          }
+          // Find line number of the method
+          const lines = content.split('\n');
+          let methodLine = 0;
+          for (let i = 0; i < lines.length; i++) {
+            if (lines[i].includes(`function ${args.methodName}(`)) { methodLine = i + 1; break; }
+          }
+          const bodyLines = body.split('\n');
+          const numbered = bodyLines.map((line, i) => `${methodLine + i}\t${line}`).join('\n');
+          return { content: [{ type: 'text', text: `## ${args.path}::${args.methodName}() (line ${methodLine})\n\n${numbered}` }] };
+        }
+
+        const allLines = content.split('\n');
+        const start = Math.max((args.startLine || 1) - 1, 0);
+        const end = args.endLine ? Math.min(args.endLine, allLines.length) : allLines.length;
+        const sliced = allLines.slice(start, end);
+        const numbered = sliced.map((line, i) => `${start + i + 1}\t${line}`).join('\n');
+        let text = `## ${args.path}`;
+        if (args.startLine || args.endLine) text += ` (lines ${start + 1}-${end})`;
+        text += `\n\n${numbered}`;
+        return { content: [{ type: 'text', text }] };
+      }
+
       default:
         return {
           content: [{