mcp-prompt-optimizer 3.7.3 → 3.7.4

package/CHANGELOG.md

@@ -5,6 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [3.7.4] - 2026-07-16
+
+### Fixed
+- **CLI commands were mostly non-functional**: the README documented 8 `mcp-prompt-optimizer <command>` subcommands, but the argv dispatcher only ever recognized `connect`. Running any of `check-status`, `validate-key`, `diagnose`, `clear-cache`, `help`, or `version` silently started the blocking stdio MCP server instead, hanging the terminal. All six are now wired to their existing `lib/*.js` implementations.
+- **Unrecognized commands no longer fall through to the server.** Any argument that isn't a known command now prints an error and usage to stderr and exits 1, instead of silently starting the MCP server. This closes the bug class for future commands too, not just the six fixed here.
+- **Dropped `test` from the documented CLI commands.** It mapped to `tests/test-runner.js`, a maintainer-only pre-publish validation script that isn't included in the published npm package (`tests/` is not in `files`), so it could never have worked for an end user.
+
 ## [3.7.3] - 2026-07-16
 
 ### Fixed

package/README.md

@@ -368,7 +368,6 @@ All plans include AI context detection, template management, personal model conf
 npx mcp-prompt-optimizer connect        # Interactive wizard: add API key to Claude Desktop config
 mcp-prompt-optimizer check-status       # Check API key and quota status
 mcp-prompt-optimizer validate-key       # Validate API key with backend
-mcp-prompt-optimizer test               # Test backend integration
 mcp-prompt-optimizer diagnose           # Run comprehensive diagnostic
 mcp-prompt-optimizer clear-cache        # Clear validation cache
 mcp-prompt-optimizer help               # Show help and setup instructions

package/index.js

@@ -2198,12 +2198,50 @@ async function runConnectWizard() {
   });
 }
 
+function printCliHelp() {
+  console.log(`MCP Prompt Optimizer v${packageJson.version}
+
+Usage:
+  mcp-prompt-optimizer                    Start the MCP server (used by MCP clients)
+  mcp-prompt-optimizer connect            Interactive wizard: add API key to Claude Desktop config
+  mcp-prompt-optimizer check-status       Check API key and quota status
+  mcp-prompt-optimizer validate-key       Validate API key with backend
+  mcp-prompt-optimizer diagnose           Run comprehensive diagnostic
+  mcp-prompt-optimizer clear-cache        Clear validation cache
+  mcp-prompt-optimizer help               Show this help
+  mcp-prompt-optimizer version            Show version information`);
+}
+
+function printCliVersion() {
+  console.log(packageJson.version);
+}
+
+const CLI_COMMANDS = {
+  connect: runConnectWizard,
+  'check-status': () => require('./lib/check-status')().then(() => process.exit(0)),
+  'validate-key': () => require('./lib/validate-key')().then(() => process.exit(0)),
+  'clear-cache': () => require('./lib/clear-cache')().then(() => process.exit(0)),
+  diagnose: () => require('./lib/diagnose')().then(() => process.exit(0)),
+  help: printCliHelp,
+  '--help': printCliHelp,
+  '-h': printCliHelp,
+  version: printCliVersion,
+  '--version': printCliVersion,
+  '-v': printCliVersion,
+};
+
 if (require.main === module) {
   const args = process.argv.slice(2);
-  if (args[0] === 'connect') {
-    runConnectWizard();
-  } else {
+  const cmd = args[0];
+
+  if (!cmd) {
     startValidatedMCPServer();
+  } else if (CLI_COMMANDS[cmd]) {
+    CLI_COMMANDS[cmd]();
+  } else {
+    console.error(`❌ Unknown command: ${cmd}\n`);
+    printCliHelp();
+    process.exit(1);
   }
 }
 

package/lib/api-key-manager.js

@@ -241,7 +241,11 @@ class CloudApiKeyManager {
             // SECURITY: Offline mode removed - backend validation required
             // No cache fallback beyond 2 hours
 
-            throw new Error(`API key validation failed: ${error.message}. Please check your internet connection.`);
+            // A 4xx means the backend rejected the key itself, not a connectivity
+            // problem — telling the user to check their internet is wrong there.
+            const isClientError = error.statusCode >= 400 && error.statusCode < 500;
+            const hint = isClientError ? '' : ' Please check your internet connection.';
+            throw new Error(`API key validation failed: ${error.message}.${hint}`);
         }
     }
 
@@ -268,7 +272,17 @@ class CloudApiKeyManager {
             } catch (error) {
                 lastError = error;
                 this.log(`Attempt ${attempt} failed: ${error.message}`, 'warn');
-                
+
+                // A 4xx means the backend definitively rejected the request (bad/expired
+                // key, malformed request) — retrying with the same key can't change that.
+                // Only network errors and 5xx (transient/server-side) are worth retrying.
+                // 429 is the exception: rate limits are exactly what backoff is for.
+                const isClientError = error.statusCode >= 400 && error.statusCode < 500 && error.statusCode !== 429;
+                if (isClientError) {
+                    this.log('Non-retryable client error — not retrying', 'error');
+                    break;
+                }
+
                 if (attempt < this.maxRetries) {
                     const delay = this.calculateRetryDelay(attempt);
                     this.log(`Retrying in ${delay}ms...`);
@@ -690,7 +704,9 @@ class CloudApiKeyManager {
                             } catch {
                                 errorMessage = `HTTP ${res.statusCode}: ${responseData}`;
                             }
-                            reject(new Error(errorMessage));
+                            const httpError = new Error(errorMessage);
+                            httpError.statusCode = res.statusCode;
+                            reject(httpError);
                         }
                     } catch (parseError) {
                         reject(new Error(`Invalid response format: ${parseError.message}`));

package/lib/check-status.js

@@ -25,7 +25,13 @@ async function checkStatus() {
         console.log('='.repeat(50));
         console.log(`🎯 Subscription Tier: ${apiKeyInfo.tier}`);
         console.log(`🔑 API Key Type: ${apiKeyInfo.keyType}`);
-        console.log(`✅ Key Status: ${apiKeyInfo.isValid ? 'Valid' : 'Invalid'}`);
+        console.log(`${apiKeyInfo.isValid ? '✅' : '❌'} Key Status: ${apiKeyInfo.isValid ? 'Valid' : 'Invalid'}`);
+        if (!apiKeyInfo.isValid) {
+            // getApiKeyInfo() reports failure via isValid:false rather than throwing,
+            // so this catch block never sees it — exit(1) has to happen here instead.
+            if (apiKeyInfo.error) console.log(`   ${apiKeyInfo.error}`);
+            process.exit(1);
+        }
     } catch (error) {
         console.error(`❌ Status check failed: ${error.message}\n`);
         process.exit(1);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "mcp-prompt-optimizer",
-  "version": "3.7.3",
+  "version": "3.7.4",
   "description": "Professional cloud-based MCP server for AI-powered prompt optimization with intelligent context detection, Bayesian optimization, AG-UI real-time optimization, template auto-save, optimization insights, personal model configuration via WebUI, team collaboration, enterprise-grade features, production resilience, and startup validation. Universal compatibility with Claude Desktop, Cursor, Windsurf, and 17+ MCP clients.",
   "main": "index.js",
   "bin": {
@@ -195,6 +195,15 @@
     "last_sync": "2026-06-01T00:00:00Z"
   },
   "release_notes": {
+    "v3.7.4": {
+      "major_features": [
+        "Fixed: 6 of 8 documented CLI commands (check-status, validate-key, diagnose, clear-cache, help, version) were not implemented in the argv dispatcher and silently started the blocking MCP server instead, hanging the terminal. Now wired to their lib/*.js implementations.",
+        "Fixed: unrecognized CLI arguments now print an error + usage to stderr and exit(1), instead of silently falling through to server startup. Removes the entire hung-terminal bug class, not just the known commands.",
+        "Removed 'test' from documented CLI commands: it mapped to a maintainer-only pre-publish script (tests/test-runner.js) that isn't included in the published package."
+      ],
+      "breaking_changes": [],
+      "migration_guide": "No migration required. Only affects users who ran documented CLI subcommands other than 'connect', which previously hung."
+    },
     "v3.7.3": {
       "major_features": [
         "Docs only: corrected stale free-tier quota in README (7 optimizations/month -> 20), matching the backend enforcement fix"