smart-terminal-mcp 1.2.31 → 1.2.34

package/.claude/settings.local.json

@@ -3,7 +3,10 @@
     "allow": [
       "WebSearch",
       "Bash(dir:*)",
-      "Bash(npm test:*)"
+      "Bash(npm test:*)",
+      "Bash(\"C:\\\\Users\\\\Alessandro\\\\AppData\\\\Local\\\\mcp-publisher\\\\mcp-publisher.exe\" login:*)",
+      "Bash(node -e \"console.log\\(require\\('./package.json'\\).version\\)\")",
+      "Bash(npm view *)"
     ]
   }
 }

package/.mcpregistry_github_token

@@ -1 +1 @@
-ghu_mi7nqUh7yokw3ywHX7jTSENKl3oizw1yvyfE
+ghu_oOdnT8lgxyaN964Yo3rMrjHyBj8zcd2uGDnB

package/.mcpregistry_registry_token

@@ -1 +1 @@
-{"token":"eyJhbGciOiJFZERTQSIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJtY3AtcmVnaXN0cnkiLCJleHAiOjE3NzMwMDM0NDQsIm5iZiI6MTc3MzAwMzE0NCwiaWF0IjoxNzczMDAzMTQ0LCJhdXRoX21ldGhvZCI6ImdpdGh1Yi1hdCIsImF1dGhfbWV0aG9kX3N1YiI6InB1bmdnZ2kiLCJwZXJtaXNzaW9ucyI6W3siYWN0aW9uIjoicHVibGlzaCIsInJlc291cmNlIjoiaW8uZ2l0aHViLnB1bmdnZ2kvKiJ9XX0.WdnInC_87a16F4MtyjLr4XCG-DDGAGMUOxKY2Y9SWnxiy8QBY25fBY4dbR53gQwveB2-L86e6MjtQ4Yj7kXtCA","expires_at":1773003444}
+{"token":"eyJhbGciOiJFZERTQSIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJtY3AtcmVnaXN0cnkiLCJleHAiOjE3NzUyMDMwODcsIm5iZiI6MTc3NTIwMjc4NywiaWF0IjoxNzc1MjAyNzg3LCJhdXRoX21ldGhvZCI6ImdpdGh1Yi1hdCIsImF1dGhfbWV0aG9kX3N1YiI6InB1bmdnZ2kiLCJwZXJtaXNzaW9ucyI6W3siYWN0aW9uIjoicHVibGlzaCIsInJlc291cmNlIjoiaW8uZ2l0aHViLnB1bmdnZ2kvKiJ9XX0.m5_YrGmyIMZ7vVMZBU5G31PBgEfVfWI4WmGZvkNhmk4wSL_cxEF-M2ARhgoymUInngh6qgdql2FLX_SJszlqAw","expires_at":1775203087}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "smart-terminal-mcp",
-  "version": "1.2.31",
+  "version": "1.2.34",
   "description": "MCP PTY server providing AI agents with real interactive terminal access",
   "mcpName": "io.github.pungggi/smart-terminal",
   "repository": {
@@ -15,7 +15,7 @@
   "scripts": {
     "start": "node src/index.js",
     "test": "node --test",
-    "stable": "npm dist-tag add smart-terminal-mcp@1.2.29 stable",
+    "stable": "npm dist-tag add smart-terminal-mcp@1.2.30 stable",
     "release": "node scripts/publish.js"
   },
   "keywords": [

package/scripts/publish.js

@@ -48,6 +48,16 @@ function runCommand(command, errorMessage, extraEnv = {}) {
   }
 }
 
+function parseFlags(args) {
+  const flags = new Set();
+  const positional = [];
+  for (const arg of args) {
+    if (arg.startsWith('--')) flags.add(arg);
+    else positional.push(arg);
+  }
+  return { flags, positional };
+}
+
 function createServerCardReaderInit() {
   return parseExpression(
     `JSON.parse(readFileSync(new URL('${SERVER_CARD_FILE_URL}', import.meta.url), 'utf8'))`,
@@ -510,10 +520,16 @@ function updateVersionFiles(nextVersion) {
 }
 
 export async function main(args = process.argv.slice(2)) {
+  const { flags, positional } = parseFlags(args);
+  const skipNpm = flags.has('--skip-npm');
+  const skipRegistry = flags.has('--skip-registry');
+  const skipSmithery = flags.has('--skip-smithery');
+
   let fileSnapshots = null;
+  let npmPublished = false;
 
   try {
-    const versionArg = args[0] || 'patch';
+    const versionArg = positional[0] || 'patch';
     const pkg = JSON.parse(readFileSync(packageJsonPath, 'utf8'));
     const currentVersion = pkg.version;
     const nextVersion = bumpVersion(currentVersion, versionArg);
@@ -529,23 +545,55 @@ export async function main(args = process.argv.slice(2)) {
     updateServerCard(tools, nextVersion);
 
     // ── Step 3: npm ──
-    console.log('\n📦 Phase 1: Publishing to npm...');
-    runCommand('npm publish --access public', 'npm publishing failed.');
+    if (skipNpm) {
+      console.log('\n📦 Phase 1: Publishing to npm... SKIPPED (--skip-npm)');
+    } else {
+      console.log('\n📦 Phase 1: Publishing to npm...');
+      runCommand('npm publish --access public', 'npm publishing failed.');
+      npmPublished = true;
+    }
 
     // ── Step 4: Official MCP Registry ──
-    console.log('\n🌍 Phase 2: Publishing to Official MCP Registry...');
-    runCommand(`${getMcpPublisher()} publish`, 'Official MCP Registry publishing failed.');
+    if (skipRegistry) {
+      console.log('\n🌍 Phase 2: Publishing to Official MCP Registry... SKIPPED (--skip-registry)');
+    } else {
+      console.log('\n🌍 Phase 2: Publishing to Official MCP Registry...');
+      try {
+        runCommand(`${getMcpPublisher()} publish`, 'Official MCP Registry publishing failed.');
+      } catch (error) {
+        const causeStderr = error.cause?.stderr?.toString?.() ?? '';
+        const causeMessage = error.message ?? '';
+        if (causeStderr.includes('401') || causeStderr.includes('Unauthorized') || causeStderr.includes('expired')
+          || causeMessage.includes('401') || causeMessage.includes('Unauthorized') || causeMessage.includes('expired')) {
+          const pub = getMcpPublisher();
+          console.error(`\n💡 Auth token expired. Run: ${pub} login github`);
+          console.error(`   Then retry with: npm run release -- ${versionArg} --skip-npm`);
+        }
+        throw error;
+      }
+    }
 
     // ── Step 5: Smithery.ai (HTTP server + tunnel + publish) ──
-    await publishToSmithery();
+    if (skipSmithery) {
+      console.log('\n💎 Phase 3: Publishing to Smithery.ai... SKIPPED (--skip-smithery)');
+    } else {
+      await publishToSmithery();
+    }
 
-    console.log(`\n✅ SUCCESSFULLY PUBLISHED version ${nextVersion} to all registries!`);
+    const skipped = [skipNpm && 'npm', skipRegistry && 'registry', skipSmithery && 'smithery'].filter(Boolean);
+    const suffix = skipped.length ? ` (skipped: ${skipped.join(', ')})` : ' to all registries';
+    console.log(`\n✅ SUCCESSFULLY PUBLISHED version ${nextVersion}${suffix}!`);
   } catch (error) {
     console.error('\n❌ Publish failed.');
     console.error(error);
     if (fileSnapshots) {
-      // Restore only files this script manages so unrelated local edits stay intact.
-      revertVersionChanges(fileSnapshots);
+      if (npmPublished) {
+        // npm already has this version — don't revert local files or we'll get version drift.
+        console.error('\n⚠️  npm was already published. Local version files kept at the new version.');
+        console.error('   Fix the issue and re-run with --skip-npm to complete remaining phases.');
+      } else {
+        revertVersionChanges(fileSnapshots);
+      }
     }
     throw error;
   }

package/server-card.json

@@ -1,7 +1,7 @@
 {
   "serverInfo": {
     "name": "smart-terminal-mcp",
-    "version": "1.2.31"
+    "version": "1.2.34"
   },
   "tools": [
     {
@@ -77,7 +77,7 @@
     },
     {
       "name": "terminal_run",
-      "description": "Run a binary directly; no shell quoting needed.",
+      "description": "Run a binary directly. shell=true for built-ins/pipes/redirects.",
       "inputSchema": {
         "type": "object",
         "properties": {
@@ -138,6 +138,11 @@
           "successFilePattern": {
             "type": "string",
             "description": "Regex"
+          },
+          "shell": {
+            "type": "boolean",
+            "default": false,
+            "description": "Run via system shell"
           }
         },
         "required": [

package/server.json

@@ -6,12 +6,12 @@
     "url": "https://github.com/pungggi/smart-terminal-mcp",
     "source": "github"
   },
-  "version": "1.2.31",
+  "version": "1.2.34",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "smart-terminal-mcp",
-      "version": "1.2.31",
+      "version": "1.2.34",
       "transport": {
         "type": "stdio"
       }

package/smithery.json

@@ -1,7 +1,7 @@
 {
   "name": "pungggi/smart-terminal",
   "description": "MCP PTY server providing AI agents with real interactive terminal access",
-  "version": "1.2.31",
+  "version": "1.2.34",
   "repository": "https://github.com/pungggi/smart-terminal-mcp",
   "tags": [
     "terminal",

package/src/command-runner.js

@@ -26,11 +26,12 @@ export async function runCommand({
   successExitCode = 0,
   successFile,
   successFilePattern,
+  shell = false,
 }) {
   assertSuccessChecksAreValid({ successFile, successFilePattern });
   const resolvedCwd = resolvePath(cwd ?? process.cwd());
   const startedAt = Date.now();
-  const spawnPlan = buildSpawnPlan({ cmd, args, cwd: resolvedCwd });
+  const spawnPlan = buildSpawnPlan({ cmd, args, cwd: resolvedCwd, useShell: shell });
 
   return new Promise((resolve, reject) => {
     const stdoutChunks = [];
@@ -42,7 +43,7 @@ export async function runCommand({
 
     const child = spawn(spawnPlan.command, spawnPlan.args, {
       cwd: resolvedCwd,
-      shell: false,
+      shell: spawnPlan.shell ?? false,
       windowsHide: true,
       windowsVerbatimArguments: spawnPlan.windowsVerbatimArguments,
       stdio: ['ignore', 'pipe', 'pipe'],
@@ -222,7 +223,16 @@ function formatFileCheckError(error) {
   return error?.message ?? String(error);
 }
 
-function buildSpawnPlan({ cmd, args, cwd }) {
+function buildSpawnPlan({ cmd, args, cwd, useShell = false }) {
+  if (useShell) {
+    return {
+      command: cmd,
+      args,
+      shell: true,
+      windowsVerbatimArguments: false,
+    };
+  }
+
   if (process.platform !== 'win32') {
     return {
       command: cmd,
@@ -372,11 +382,11 @@ function quoteWindowsBatchArgument(value) {
 
 function formatStartError({ cmd, err }) {
   const baseMessage = `Failed to start command "${cmd}": ${err.message}`;
-  if (process.platform !== 'win32' || err?.code !== 'ENOENT' || looksLikePath(cmd)) {
+  if (err?.code !== 'ENOENT' || looksLikePath(cmd)) {
     return baseMessage;
   }
 
-  return `${baseMessage}. If this command should come from PATH, verify it is installed and visible to the server process. Shell built-ins such as dir or cd still require terminal_exec.`;
+  return `${baseMessage}. Verify it is installed and on PATH for the server process. For shell built-ins, pipes, or redirections, pass shell:true or use terminal_start + terminal_exec.`;
 }
 
 export function getStructuredParserHint({ cmd, args, ok, parseRequested, parsed, stdout }) {

package/src/tools.js

@@ -152,7 +152,7 @@ export function registerTools(server, manager) {
   // --- terminal_run ---
   tool(
     'terminal_run',
-    'Run a binary directly; no shell quoting needed.',
+    'Run a binary directly. shell=true for built-ins/pipes/redirects.',
     {
       cmd: z.string(),
       args: z.array(z.string()).default([]),
@@ -165,8 +165,9 @@ export function registerTools(server, manager) {
       successExitCode: z.number().int().nullable().default(0).describe('null=any'),
       successFile: z.string().optional(),
       successFilePattern: z.string().optional().describe('Regex'),
+      shell: z.boolean().default(false).describe('Run via system shell'),
     },
-    async ({ cmd, args, cwd, timeout, maxOutputBytes, parse, parseOnly, summary, successExitCode, successFile, successFilePattern }) => {
+    async ({ cmd, args, cwd, timeout, maxOutputBytes, parse, parseOnly, summary, successExitCode, successFile, successFilePattern, shell }) => {
       const result = await runCommand({
         cmd,
         args,
@@ -179,6 +180,7 @@ export function registerTools(server, manager) {
         successExitCode,
         successFile,
         successFilePattern,
+        shell,
       });
       return jsonContent(result);
     }

package/test/command-runner.test.js

@@ -288,18 +288,13 @@ test('runCommand handles explicit command paths with spaces', async () => {
   }
 });
 
-test('runCommand adds a helpful ENOENT hint on Windows for PATH commands', async (t) => {
-  if (process.platform !== 'win32') {
-    t.skip('Windows-only behavior');
-    return;
-  }
-
+test('runCommand adds a helpful ENOENT hint for PATH commands', async () => {
   await assert.rejects(
     runCommand({
       cmd: '__smart_terminal_missing_command__',
       parse: false,
     }),
-    /verify it is installed and visible to the server process/
+    /pass shell:true or use terminal_start \+ terminal_exec/
   );
 });
 

package/test/tools.test.js

@@ -288,6 +288,36 @@ test('terminal_run can re-evaluate success from a file pattern', async () => {
   }
 });
 
+test('terminal_run shell=true executes commands via the system shell', async () => {
+  const server = createFakeServer();
+  registerTools(server, {});
+
+  const result = await server.tools.get('terminal_run').handler({
+    cmd: 'echo shell-ok',
+    args: [],
+    shell: true,
+    parse: false,
+  });
+
+  const payload = JSON.parse(result.content[0].text);
+  assert.equal(payload.ok, true);
+  assert.match(payload.stdout.raw, /shell-ok/);
+});
+
+test('terminal_run ENOENT error hints at shell=true and terminal_exec fallback', async () => {
+  const server = createFakeServer();
+  registerTools(server, {});
+
+  await assert.rejects(
+    () => server.tools.get('terminal_run').handler({
+      cmd: 'smart-terminal-missing-binary-xyz',
+      args: [],
+      parse: false,
+    }),
+    /shell:true or use terminal_start \+ terminal_exec/
+  );
+});
+
 test('terminal_read rejects idleTimeout values that are not less than timeout', async () => {
   const server = createFakeServer();
   let getCalls = 0;