mage-remote-run 0.23.0 → 0.23.1

package/bin/mage-remote-run.js

@@ -10,6 +10,7 @@ const pkg = require('../package.json');
 
 const program = new Command();
 
+
 program
   .name('mage-remote-run')
   .description('The remote swiss army knife for Magento Open Source, Mage-OS, Adobe Commerce')
@@ -48,12 +49,18 @@ import { startMcpServer } from '../lib/mcp.js';
 // But we need them registered early if we want them to show up in help even if config fails?
 // Actually registerCommands handles null profile by registering connection commands only.
 
-program.command('mcp')
+program.command('mcp [args...]')
   .description('Run as MCP server')
   .option('--transport <type>', 'Transport type (stdio, http)', 'stdio')
   .option('--host <host>', 'HTTP Host', '127.0.0.1')
   .option('--port <port>', 'HTTP Port', '18098')
-  .action(async (options) => {
+  .allowExcessArguments(true)
+  .allowUnknownOption(true)
+  .action(async (args, options) => {
+    // We ignore extra arguments but log them for debugging purposes
+    if (args && args.length > 0) {
+      // console.error(chalk.yellow(`[mage-remote-run] Warning: Received extra arguments for mcp command: ${args.join(' ')}`));
+    }
     await startMcpServer(options);
   });
 
@@ -68,7 +75,8 @@ program.hook('preAction', async (thisCommand, actionCommand) => {
     const config = await loadConfig();
     if (config.showActiveConnectionHeader !== false) {
       const opts = actionCommand.opts();
-      if (opts.format !== 'json' && opts.format !== 'xml') {
+      // Standard output corruption check: Don't print header if output is json/xml OR if running mcp command (which uses stdio)
+      if (opts.format !== 'json' && opts.format !== 'xml' && actionCommand.name() !== 'mcp') {
         console.log(chalk.cyan(`Active Connection: ${chalk.bold(profile.name)} (${profile.type})`));
         console.log(chalk.gray('━'.repeat(60)) + '\n');
       }

package/lib/mcp.js

@@ -62,8 +62,8 @@ export async function startMcpServer(options) {
 
     } else {
         // STDIO
-        console.error(`Protocol: stdio`);
-        console.error(`Registered Tools: ${toolsCount}`);
+         console.error(`Protocol: stdio`);
+         console.error(`Registered Tools: ${toolsCount}`);
         const transport = new StdioServerTransport();
         await server.connect(transport);
     }
@@ -82,7 +82,7 @@ function registerTools(server, program) {
         }
 
         // It's a leaf command, register as tool
-        // Tool name: Replace spaces/colons with underscores. 
+        // Tool name: Replace spaces/colons with underscores.
         // Example: website list -> website_list
         const toolName = cmdName.replace(/[^a-zA-Z0-9_]/g, '_');
 
@@ -105,7 +105,7 @@ function registerTools(server, program) {
 
         cmd.options.forEach(opt => {
             const name = opt.name(); // e.g. "format" for --format
-            // Check flags to guess type. 
+            // Check flags to guess type.
             // -f, --format <type> -> string
             // -v, --verbose -> boolean
 
@@ -186,9 +186,9 @@ async function executeCommand(cmdDefinition, args, parentName) {
             // Wait, registerTools calls: `executeCommand(cmd, args, parentName)`
             // If parentName is "website", and cmd is "list", we need "website list"
 
-            // NOTE: parentName in processCommand is built recursively with underscores? 
-            // In processCommand(subCmd, cmdName), cmdName is "parent_sub". 
-            // So if we have website -> list. 
+            // NOTE: parentName in processCommand is built recursively with underscores?
+            // In processCommand(subCmd, cmdName), cmdName is "parent_sub".
+            // So if we have website -> list.
             // processCommand(website) -> processCommand(list, "website")
             // parentName in executeCommand is "website".
             // cmd.name() is "list".
@@ -212,7 +212,7 @@ async function executeCommand(cmdDefinition, args, parentName) {
             // `cmdName` = `parentName_cmd.name()`.
             // So for `website list`:
             // `processCommand(website, '')` -> `cmdName="website"`.
-            //   -> `processCommand(list, "website")`. 
+            //   -> `processCommand(list, "website")`.
             //      -> register tool "website_list". `parentName` passed to execute is "website".
 
             // So `parentName` is the accumulated prefix with underscores.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mage-remote-run",
-  "version": "0.23.0",
+  "version": "0.23.1",
   "description": "The remote swiss army knife for Magento Open Source, Mage-OS, Adobe Commerce",
   "main": "index.js",
   "scripts": {