@tmcp/transport-cli 0.0.2 → 0.0.3

package/README.md

@@ -123,6 +123,7 @@ node my-cli.js get-user '{"id":"1"}' --output structured --fields user.name,user
 - Output is written to stdout.
 - JSON outputs are pretty-printed.
 - Errors are written to stderr and set `process.exitCode` to `1`.
+- Running the CLI without a command prints help output.
 - The CLI initializes an MCP session, sends `notifications/initialized`, and paginates through `tools/list` automatically.
 - The program name shown in help output comes from `McpServer`'s `name`.
 

package/package.json

@@ -1,10 +1,13 @@
 {
   "name": "@tmcp/transport-cli",
-  "version": "0.0.2",
+  "version": "0.0.3",
   "description": "CLI transport for TMCP with static JSON-first tool commands",
   "type": "module",
   "main": "src/index.js",
   "types": "src/types/index.d.ts",
+  "engines": {
+    "node": ">=18.0.0"
+  },
   "files": [
     "src"
   ],

package/src/index.js

@@ -3,6 +3,7 @@
  * @import { ListToolsResult, Tool } from "./internal.js";
  */
 import process from 'node:process';
+import { randomUUID } from 'node:crypto';
 import { AsyncLocalStorage } from 'node:async_hooks';
 import sade from 'sade';
 
@@ -20,6 +21,7 @@ const CLIENT_INFO = {
 };
 
 const RESERVED_COMMANDS = new Set(['call', 'schema', 'tools']);
+const UNSAFE_ALIAS_NAME = /[<>[\]\s]/;
 
 /**
  * @param {unknown} value
@@ -233,7 +235,11 @@ function filter_fields(value, paths) {
 		if (segments.length === 0) {
 			throw new Error('`--fields` only supports dot-separated paths');
 		}
-		set_path_value(result, segments, get_path_value(value, segments));
+		set_path_value(
+			result,
+			segments,
+			structuredClone(get_path_value(value, segments)),
+		);
 	}
 
 	return result;
@@ -301,6 +307,36 @@ function format_tool_result(result, options) {
 	return format_json(filter_fields(selected, parse_fields(options.fields)));
 }
 
+/**
+ * @param {Record<string, unknown>} result
+ * @returns {string}
+ */
+function format_tool_error(result) {
+	const content = result.content;
+
+	if (Array.isArray(content)) {
+		const text_blocks = content
+			.map((item) => {
+				if (
+					is_record(item) &&
+					item.type === 'text' &&
+					typeof item.text === 'string'
+				) {
+					return item.text;
+				}
+
+				return undefined;
+			})
+			.filter((item) => typeof item === 'string');
+
+		if (text_blocks.length > 0) {
+			return text_blocks.join('\n');
+		}
+	}
+
+	return JSON.stringify(result, null, 2);
+}
+
 /**
  * @template {Record<string, unknown> | undefined} [TCustom=undefined]
  */
@@ -323,7 +359,7 @@ export class CliTransport {
 	/**
 	 * @type {string}
 	 */
-	#session_id = crypto.randomUUID();
+	#session_id = randomUUID();
 
 	/**
 	 * @param {McpServer<any, TCustom>} server
@@ -332,6 +368,10 @@ export class CliTransport {
 		this.#server = server;
 	}
 
+	#exit() {
+		process.exit(process.exitCode ?? 0);
+	}
+
 	/**
 	 * @param {string} method
 	 * @param {Record<string, unknown>} [params]
@@ -473,6 +513,12 @@ export class CliTransport {
 		const args = await resolve_tool_input(input);
 		const result = await this.#call_tool(tool.name, args, ctx);
 
+		if (result?.isError) {
+			process.stderr.write(`Error: ${format_tool_error(result)}\n`);
+			process.exitCode = 1;
+			return;
+		}
+
 		process.stdout.write(format_tool_result(result, options));
 	}
 
@@ -481,39 +527,25 @@ export class CliTransport {
 	 * @param {Array<string>} [argv]
 	 */
 	async run(ctx, argv) {
-		const init_result = await this.#initialize(ctx);
-		const tools = await this.#list_tools(ctx);
-		const script_name = init_result?.serverInfo?.name ?? 'tmcp';
-		const prog = sade(script_name);
+		try {
+			const init_result = await this.#initialize(ctx);
+			const tools = await this.#list_tools(ctx);
+			const script_name = init_result?.serverInfo?.name ?? 'tmcp';
+			const prog = sade(script_name);
 
-		/** @type {Map<string, Tool>} */
-		const tool_map = new Map();
+			/** @type {Map<string, Tool>} */
+			const tool_map = new Map();
 
-		for (const tool of tools) {
-			tool_map.set(tool.name, tool);
-		}
+			for (const tool of tools) {
+				tool_map.set(tool.name, tool);
+			}
 
-		prog.command('tools', 'List available tools').action(() => {});
-		prog.command('schema <tool>', 'Print a tool schema').action(() => {});
-
-		const call_command = prog
-			.command('call <tool> [input]', 'Call a tool with JSON input')
-			.option(
-				'--output',
-				'Select full, structured, content, or text output',
-				'full',
-			)
-			.option(
-				'--fields',
-				'Select comma-separated dot paths from the chosen output',
+			prog.command('tools', 'List available tools').action(() => {});
+			prog.command('schema <tool>', 'Print a tool schema').action(
+				() => {},
 			);
 
-		call_command.action(() => {});
-
-		for (const tool of tools) {
-			if (RESERVED_COMMANDS.has(tool.name)) continue;
-
-			prog.command(`${tool.name} [input]`, tool.description ?? '')
+			prog.command('call <tool> [input]', 'Call a tool with JSON input')
 				.option(
 					'--output',
 					'Select full, structured, content, or text output',
@@ -524,19 +556,54 @@ export class CliTransport {
 					'Select comma-separated dot paths from the chosen output',
 				)
 				.action(() => {});
-		}
 
-		const args = argv ? ['node', script_name, ...argv] : process.argv;
-		const parsed = prog.parse(args, { lazy: true });
+			for (const tool of tools) {
+				if (RESERVED_COMMANDS.has(tool.name)) {
+					process.stderr.write(
+						`Warning: skipping bare alias for tool "${tool.name}" because its name conflicts with a built-in command. Use \`call ${tool.name}\` instead.\n`,
+					);
+					continue;
+				}
+
+				if (UNSAFE_ALIAS_NAME.test(tool.name)) {
+					process.stderr.write(
+						`Warning: skipping bare alias for tool "${tool.name}" because its name contains CLI syntax characters. Use \`call ${tool.name}\` instead.\n`,
+					);
+					continue;
+				}
 
-		if (!parsed) return;
+				prog.command(`${tool.name} [input]`, tool.description ?? '')
+					.option(
+						'--output',
+						'Select full, structured, content, or text output',
+						'full',
+					)
+					.option(
+						'--fields',
+						'Select comma-separated dot paths from the chosen output',
+					)
+					.action(() => {});
+			}
 
-		const { name, args: handler_args } = parsed;
-		const { positionals, options } = extract_command_args(handler_args);
+			const command_args = argv ?? process.argv.slice(2);
+			const args = [
+				'node',
+				script_name,
+				...(command_args.length === 0 ? ['--help'] : command_args),
+			];
+			const parsed = prog.parse(args, { lazy: true });
+
+			if (!parsed) {
+				this.#exit();
+				return;
+			}
+
+			const { name, args: handler_args } = parsed;
+			const { positionals, options } = extract_command_args(handler_args);
 
-		try {
 			if (name === 'tools') {
 				process.stdout.write(format_json(tools));
+				this.#exit();
 				return;
 			}
 
@@ -551,6 +618,7 @@ export class CliTransport {
 				process.stdout.write(
 					format_json(this.#get_tool(tool_map, tool_name)),
 				);
+				this.#exit();
 				return;
 			}
 
@@ -573,6 +641,7 @@ export class CliTransport {
 					normalize_tool_options(options),
 					ctx,
 				);
+				this.#exit();
 				return;
 			}
 
@@ -587,12 +656,17 @@ export class CliTransport {
 					normalize_tool_options(options),
 					ctx,
 				);
+				this.#exit();
+				return;
 			}
+
+			this.#exit();
 		} catch (err) {
 			process.stderr.write(
 				`Error: ${err instanceof Error ? err.message : String(err)}\n`,
 			);
 			process.exitCode = 1;
+			this.#exit();
 		}
 	}
 }

package/src/types/index.d.ts.map

@@ -12,6 +12,6 @@
 	"sourcesContent": [
 		null
 	],
-	"mappings": ";;cAkTaA,YAAYA;;;;;;;aAzSgCC,UAAUA;aAIZC,WAAWA",
+	"mappings": ";;cAsVaA,YAAYA;;;;;;;aA5UgCC,UAAUA;aAIZC,WAAWA",
 	"ignoreList": []
 }