Skip to content

Commit b65fade

Browse files
author
Lasim
committed
refactor(gateway): remove unnecessary hints from MCP command error messages
1 parent 6a3332b commit b65fade

File tree

1 file changed

+29
-30
lines changed
  • services/gateway/src/commands

1 file changed

+29
-30
lines changed

‎services/gateway/src/commands/mcp.ts‎

Lines changed: 29 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -46,43 +46,43 @@ export function registerMCPCommand(program: Command) {
4646

4747
if (isNaN(serverNumber) || serverNumber < 1) {
4848
console.log(chalk.red(`❌ Invalid server number "${options.tools}". Please use a positive number.`));
49-
console.log(chalk.gray('💡 Use "deploystack mcp" to see available servers'));
49+
console.log(chalk.gray('Use "deploystack mcp" to see available servers'));
5050
process.exit(1);
5151
}
5252

5353
// CRITICAL: Check if gateway is running first
5454
if (!isGatewayRunning()) {
5555
console.log(chalk.red('❌ Gateway is not running'));
56-
console.log(chalk.gray('💡 Use "deploystack start" to start the gateway first'));
57-
console.log(chalk.gray('💡 The --tools command only works with running MCP servers'));
56+
console.log(chalk.gray('Use "deploystack start" to start the gateway first'));
57+
console.log(chalk.gray('The --tools command only works with running MCP servers'));
5858
process.exit(1);
5959
}
6060

6161
// Check authentication for tools mode
6262
if (!await storage.isAuthenticated()) {
6363
console.log(chalk.red('❌ Not authenticated'));
64-
console.log(chalk.gray(`💡 Run 'deploystack login' to authenticate`));
64+
console.log(chalk.gray(`Run 'deploystack login' to authenticate`));
6565
process.exit(1);
6666
}
6767

6868
const credentials = await storage.getCredentials();
6969
if (!credentials) {
7070
console.log(chalk.red('❌ No stored credentials found'));
71-
console.log(chalk.gray(`💡 Run 'deploystack login' to authenticate`));
71+
console.log(chalk.gray(`Run 'deploystack login' to authenticate`));
7272
process.exit(1);
7373
}
7474

7575
if (!credentials.selectedTeam) {
7676
console.log(chalk.red('❌ No team selected'));
77-
console.log(chalk.gray(`💡 Run 'deploystack teams --switch <team-number>' to select a team`));
77+
console.log(chalk.gray(`Run 'deploystack teams --switch <team-number>' to select a team`));
7878
process.exit(1);
7979
}
8080

8181
// Get MCP configuration
8282
const config = await mcpService.getMCPConfig();
8383
if (!config || config.servers.length === 0) {
8484
console.log(chalk.yellow('⚠️ No MCP servers configured'));
85-
console.log(chalk.gray('💡 Run "deploystack mcp --refresh" to download configuration'));
85+
console.log(chalk.gray('Run "deploystack mcp --refresh" to download configuration'));
8686
process.exit(1);
8787
}
8888

@@ -111,8 +111,8 @@ export function registerMCPCommand(program: Command) {
111111

112112
if (gatewayTools.length === 0) {
113113
console.log(chalk.yellow('⚠️ No tools found in this MCP server'));
114-
console.log(chalk.gray('💡 The server may not expose any tools or may not be running properly'));
115-
console.log(chalk.gray('💡 Check "deploystack status" to see server status'));
114+
console.log(chalk.gray('The server may not expose any tools or may not be running properly'));
115+
console.log(chalk.gray('Check "deploystack status" to see server status'));
116116
return;
117117
}
118118

@@ -121,8 +121,8 @@ export function registerMCPCommand(program: Command) {
121121

122122
} catch (error) {
123123
console.error(chalk.red(`❌ Failed to communicate with running MCP server: ${error instanceof Error ? error.message : String(error)}`));
124-
console.log(chalk.gray('💡 Check "deploystack status" to see if the server is running'));
125-
console.log(chalk.gray('💡 Try restarting the gateway with "deploystack stop && deploystack start"'));
124+
console.log(chalk.gray('Check "deploystack status" to see if the server is running'));
125+
console.log(chalk.gray('Try restarting the gateway with "deploystack stop && deploystack start"'));
126126
process.exit(1);
127127
}
128128

@@ -179,35 +179,35 @@ export function registerMCPCommand(program: Command) {
179179
});
180180
});
181181

182-
console.log(chalk.gray('\n💡 * = Required parameter'));
182+
console.log(chalk.gray('\n* = Required parameter'));
183183
}
184184

185-
console.log(chalk.gray(`\n💡 Server: ${selectedServer.installation_name}`));
186-
console.log(chalk.gray(`💡 Runtime: ${selectedServer.runtime}`));
187-
console.log(chalk.gray(`💡 Command: ${selectedServer.command} ${selectedServer.args.join(' ')}`));
188-
console.log(chalk.blue('💡 Tools retrieved from running MCP server (not spawned)'));
185+
console.log(chalk.gray(`\nServer: ${selectedServer.installation_name}`));
186+
console.log(chalk.gray(`Runtime: ${selectedServer.runtime}`));
187+
console.log(chalk.gray(`Command: ${selectedServer.command} ${selectedServer.args.join(' ')}`));
188+
console.log(chalk.blue('Tools retrieved from running MCP server (not spawned)'));
189189

190190
return;
191191
}
192192

193193
// Check authentication
194194
if (!await storage.isAuthenticated()) {
195195
console.log(chalk.red('❌ Not authenticated'));
196-
console.log(chalk.gray(`💡 Run 'deploystack login' to authenticate`));
196+
console.log(chalk.gray(`Run 'deploystack login' to authenticate`));
197197
process.exit(1);
198198
}
199199

200200
const credentials = await storage.getCredentials();
201201
if (!credentials) {
202202
console.log(chalk.red('❌ No stored credentials found'));
203-
console.log(chalk.gray(`💡 Run 'deploystack login' to authenticate`));
203+
console.log(chalk.gray(`Run 'deploystack login' to authenticate`));
204204
process.exit(1);
205205
}
206206

207207
// Check if team is selected
208208
if (!credentials.selectedTeam) {
209209
console.log(chalk.red('❌ No team selected'));
210-
console.log(chalk.gray(`💡 Run 'deploystack teams --switch <team-number>' to select a team`));
210+
console.log(chalk.gray(`Run 'deploystack teams --switch <team-number>' to select a team`));
211211
process.exit(1);
212212
}
213213

@@ -235,7 +235,7 @@ export function registerMCPCommand(program: Command) {
235235

236236
if (!hasConfig) {
237237
console.log(chalk.yellow('⚠️ No MCP configuration found'));
238-
console.log(chalk.gray('💡 Run with --refresh to download configuration'));
238+
console.log(chalk.gray('Run with --refresh to download configuration'));
239239
return;
240240
}
241241

@@ -256,7 +256,7 @@ export function registerMCPCommand(program: Command) {
256256
console.log(` Status: ${chalk.green('Fresh ✅')}`);
257257
} else {
258258
console.log(` Status: ${chalk.yellow('Stale ⚠️')} (>24h old)`);
259-
console.log(chalk.gray('💡 Consider running with --refresh to update'));
259+
console.log(chalk.gray('Consider running with --refresh to update'));
260260
}
261261

262262
// Show MCP servers table
@@ -282,12 +282,11 @@ export function registerMCPCommand(program: Command) {
282282
console.log(table.toString());
283283
}
284284

285-
console.log(chalk.gray('\n💡 Use this configuration with "deploystack start" to run the gateway'));
286-
console.log(chalk.gray('💡 Use "deploystack mcp --refresh" to update from the cloud'));
287-
console.log(chalk.gray('💡 Use "deploystack mcp --tools <server-number>" to discover available tools'));
285+
console.log(chalk.gray('Use "deploystack mcp --refresh" to update from the cloud'));
286+
console.log(chalk.gray('Use "deploystack mcp --tools <server-number>" to discover available tools'));
288287
} else {
289288
console.log(chalk.yellow('\n⚠️ No MCP servers configured for this team'));
290-
console.log(chalk.gray('💡 Install MCP servers via the web interface'));
289+
console.log(chalk.gray('Install MCP servers via the web interface'));
291290
}
292291

293292
// Check for remote configuration updates
@@ -346,7 +345,7 @@ export function registerMCPCommand(program: Command) {
346345
}
347346
} catch (restartError) {
348347
console.log(chalk.red(`❌ Failed to restart gateway: ${restartError instanceof Error ? restartError.message : String(restartError)}`));
349-
console.log(chalk.gray('💡 You can restart manually with "deploystack restart"'));
348+
console.log(chalk.gray('You can restart manually with "deploystack restart"'));
350349
}
351350
});
352351

@@ -364,8 +363,8 @@ export function registerMCPCommand(program: Command) {
364363
if (refreshError instanceof Error) {
365364
console.log(chalk.gray(` Error: ${refreshError.message}`));
366365
}
367-
console.log(chalk.gray('💡 This may be due to network connectivity or backend issues'));
368-
console.log(chalk.gray('💡 Your local configuration is still available and functional'));
366+
console.log(chalk.gray('This may be due to network connectivity or backend issues'));
367+
console.log(chalk.gray('Your local configuration is still available and functional'));
369368
}
370369

371370
} catch (error) {
@@ -377,9 +376,9 @@ export function registerMCPCommand(program: Command) {
377376
console.log(chalk.red(`❌ Failed to access MCP configuration: ${error.message}`));
378377

379378
if (error.code === 'TOKEN_EXPIRED') {
380-
console.log(chalk.gray(`💡 Run 'deploystack login' to refresh your authentication`));
379+
console.log(chalk.gray(`Run 'deploystack login' to refresh your authentication`));
381380
} else if (error.code === 'NETWORK_ERROR') {
382-
console.log(chalk.gray('💡 Check your internet connection and try again'));
381+
console.log(chalk.gray('Check your internet connection and try again'));
383382
}
384383
} else {
385384
console.log(chalk.red(`❌ Unexpected error: ${error instanceof Error ? error.message : String(error)}`));

0 commit comments

Comments
 (0)