diff --git a/src/output.ts b/src/output.ts index 69a6195..e85c0ac 100644 --- a/src/output.ts +++ b/src/output.ts @@ -115,12 +115,8 @@ export function formatServerDetails( if (isHttpServer(config)) { lines.push(`${color('Transport:', colors.bold)} HTTP`); - lines.push(`${color('URL:', colors.bold)} ${config.url}`); } else { lines.push(`${color('Transport:', colors.bold)} stdio`); - lines.push( - `${color('Command:', colors.bold)} ${config.command} ${(config.args || []).join(' ')}`, - ); } if (instructions) { diff --git a/tests/output.test.ts b/tests/output.test.ts index 518713f..e7cc1a1 100644 --- a/tests/output.test.ts +++ b/tests/output.test.ts @@ -4,6 +4,7 @@ import { describe, test, expect } from 'bun:test'; import { + formatServerDetails, formatServerList, formatSearchResults, formatToolSchema, @@ -101,6 +102,41 @@ describe('output', () => { }); }); + describe('formatServerDetails', () => { + test('shows transport without stdio command details', () => { + const output = formatServerDetails( + 'filesystem', + { + command: 'npx', + args: ['-y', '@modelcontextprotocol/server-filesystem', '.'], + }, + [], + ); + + expect(output).toContain('Server:'); + expect(output).toContain('filesystem'); + expect(output).toContain('Transport:'); + expect(output).toContain('stdio'); + expect(output).not.toContain('Command:'); + expect(output).not.toContain('@modelcontextprotocol/server-filesystem'); + }); + + test('shows transport without http endpoint details', () => { + const output = formatServerDetails( + 'deepwiki', + { url: 'https://mcp.deepwiki.com/mcp' }, + [], + ); + + expect(output).toContain('Server:'); + expect(output).toContain('deepwiki'); + expect(output).toContain('Transport:'); + expect(output).toContain('HTTP'); + expect(output).not.toContain('URL:'); + expect(output).not.toContain('https://mcp.deepwiki.com/mcp'); + }); + }); + describe('formatToolSchema', () => { test('formats tool with schema', () => { const tool = {