@goke/mcp 0.1.1 → 0.1.2

package/README.md

@@ -95,6 +95,7 @@ const cli = goke("my-cli")
 cli
   .command("search", "Search pages")
   .option("--query <query>", z.string().describe("Search query"))
+  .required()
   .option("--limit [limit]", z.number().default(10).describe("Max results"))
   .action((options) => {
     return { results: findPages(options.query, options.limit) }
@@ -129,7 +130,7 @@ my-cli mcp
 
 When running as MCP, the server exposes `search` and `deploy` as tools. The `mcp` command itself is excluded. Options with Zod schemas (or any Standard Schema) become typed `inputSchema` properties in the MCP tool definition.
 
-**`inputSchema.required` is not the CLI `<value>` syntax.** `--query <query>` means the flag needs a value if it is present. The flag is still optional unless the schema rejects omit (`z.string()`, not `z.string().optional()`). Required **positionals** like `<env>` always go in `required`. Schema-required flags go in `required` too. Untyped flags and `wrapJsonSchema()` flags stay optional.
+**`inputSchema.required` is not the CLI `<value>` syntax.** `--query <query>` means the flag needs a value if it is present. The flag itself is still optional unless you chain `.required()`. Required **positionals** like `<env>` always go in `required`. `.required()` flags go in `required` too.
 
 ### Installing the MCP server in clients
 

package/dist/__test__/add-cli-tools-to-mcp.test.js

@@ -15,6 +15,7 @@ function createCli() {
     cli
         .command("say hi", "Say hello")
         .option("--name <name>", z.string().describe("Person to greet"))
+        .required()
         .option("--caps", z.boolean().default(false).describe("Uppercase output"))
         .action((options) => {
         const message = `Hello ${options.name}!`;

package/dist/__test__/create-mcp-action.test.js

@@ -29,7 +29,9 @@ describe("createMcpAction", () => {
         cli
             .command("add", "Add numbers")
             .option("--a <a>", z.number().describe("First"))
+            .required()
             .option("--b <b>", z.number().describe("Second"))
+            .required()
             .action((options) => ({ sum: options.a + options.b }));
         const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
         cli.command("mcp", "Start MCP server").action(createMcpAction({
@@ -186,7 +188,7 @@ describe("createMcpAction", () => {
             expect(searchTool.description).toBe("Search for items");
             expect(searchTool.inputSchema.properties).toHaveProperty("query");
             expect(searchTool.inputSchema.properties).toHaveProperty("limit");
-            expect(searchTool.inputSchema.required).toEqual(["query"]);
+            expect(searchTool.inputSchema.required).toBeUndefined();
             const deployTool = tools.tools.find((t) => t.name === "deploy");
             expect(deployTool.inputSchema.properties).toHaveProperty("env");
             expect(deployTool.inputSchema.properties).toHaveProperty("dryRun");
@@ -271,12 +273,14 @@ describe("createMcpAction", () => {
             await client.close();
         }
     });
-    it("puts schema-required --flag <value> in inputSchema.required", async () => {
+    it("puts .required() --flag <value> in inputSchema.required", async () => {
         const cli = goke("checks");
         cli
             .command("checks create", "Create a check")
             .option("--url <url>", z.string().describe("URL to check"))
+            .required()
             .option("--name <name>", z.string().describe("Check name"))
+            .required()
             .option("--timeout [ms]", z.number().optional().describe("Timeout"))
             .action((options) => options);
         const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();

package/dist/__test__/http-multi-tenant.test.js

@@ -90,6 +90,7 @@ function buildBaseCli() {
     cli
         .command("save <filename>", "Save content to a file in the tenant workspace")
         .option("--content <content>", z.string().describe("File content"))
+        .required()
         .action(async (filename, options, ctx) => {
         const full = path.posix.join(ctx.process.cwd, filename);
         await ctx.fs.writeFile(full, options.content);

package/dist/cli-to-mcp.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"cli-to-mcp.d.ts","sourceRoot":"","sources":["../src/cli-to-mcp.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AACzE,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,2CAA2C,CAAC;AACxE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,+CAA+C,CAAC;AAS/E,OAAO,EAML,KAAK,IAAI,EAIV,MAAM,MAAM,CAAC;AAwFd,MAAM,WAAW,uBAAuB;IACtC,GAAG,EAAE,IAAI,CAAC;IACV,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,aAAa,CAAC,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,OAAO,CAAC;IACjD,gBAAgB,CAAC,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,MAAM,CAAC;CACpD;AA4fD,MAAM,WAAW,sBAAsB;IACrC,mEAAmE;IACnE,GAAG,EAAE,IAAI,CAAC;IACV,iGAAiG;IACjG,aAAa,CAAC,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,OAAO,CAAC;IACjD,iCAAiC;IACjC,gBAAgB,CAAC,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,MAAM,CAAC;IACnD,oEAAoE;IACpE,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,8CAA8C;IAC9C,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,iFAAiF;IACjF,eAAe,CAAC,EAAE,MAAM,SAAS,GAAG,OAAO,CAAC,SAAS,CAAC,CAAC;CACxD;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,sBAAsB,GAAG,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,OAAO,CAAC,IAAI,CAAC,CAsClG;AAED,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,uBAAuB,GAAG,IAAI,CAyCvE"}
+{"version":3,"file":"cli-to-mcp.d.ts","sourceRoot":"","sources":["../src/cli-to-mcp.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,yCAAyC,CAAC;AACzE,OAAO,KAAK,EAAE,MAAM,EAAE,MAAM,2CAA2C,CAAC;AACxE,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,+CAA+C,CAAC;AAS/E,OAAO,EAKL,KAAK,IAAI,EAIV,MAAM,MAAM,CAAC;AAyFd,MAAM,WAAW,uBAAuB;IACtC,GAAG,EAAE,IAAI,CAAC;IACV,MAAM,EAAE,MAAM,GAAG,SAAS,CAAC;IAC3B,aAAa,CAAC,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,OAAO,CAAC;IACjD,gBAAgB,CAAC,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,MAAM,CAAC;CACpD;AAufD,MAAM,WAAW,sBAAsB;IACrC,mEAAmE;IACnE,GAAG,EAAE,IAAI,CAAC;IACV,iGAAiG;IACjG,aAAa,CAAC,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,OAAO,CAAC;IACjD,iCAAiC;IACjC,gBAAgB,CAAC,EAAE,CAAC,WAAW,EAAE,MAAM,KAAK,MAAM,CAAC;IACnD,oEAAoE;IACpE,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,8CAA8C;IAC9C,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,iFAAiF;IACjF,eAAe,CAAC,EAAE,MAAM,SAAS,GAAG,OAAO,CAAC,SAAS,CAAC,CAAC;CACxD;AAED;;;;;;;;;;;GAWG;AACH,wBAAgB,eAAe,CAAC,OAAO,EAAE,sBAAsB,GAAG,CAAC,GAAG,IAAI,EAAE,GAAG,EAAE,KAAK,OAAO,CAAC,IAAI,CAAC,CAsClG;AAED,wBAAgB,gBAAgB,CAAC,OAAO,EAAE,uBAAuB,GAAG,IAAI,CAyCvE"}

package/dist/cli-to-mcp.js

@@ -5,7 +5,7 @@
  * or a high-level McpServer by mounting tools/list + tools/call handlers.
  */
 import { CallToolRequestSchema, ErrorCode, ListToolsRequestSchema, McpError, } from "@modelcontextprotocol/sdk/types.js";
-import { coerceBySchema, extractJsonSchema, GokeProcessExit, schemaAcceptsOmittedValue, } from "goke";
+import { coerceBySchema, extractJsonSchema, GokeProcessExit, } from "goke";
 const CLI_TO_MCP_STATE = Symbol.for("@goke/mcp/cli-to-mcp-state");
 function createTextCaptureStream() {
     const chunks = [];
@@ -360,14 +360,11 @@ function createBinding(cli, command, toolName) {
         }
     }
     // `--days <days>` means "value required if the flag is present". The flag
-    // itself is required only when a non-optional schema rejects `undefined`.
+    // itself is required only when `.required()` set `flagRequired`.
     for (const option of options) {
         const normalized = normalizeOptionSchema(option);
         properties[option.name] = normalized.schema;
-        if (option.required
-            && option.default === undefined
-            && option.schema
-            && !schemaAcceptsOmittedValue(option.schema)) {
+        if (option.flagRequired && option.default === undefined) {
             requiredNames.push(option.name);
         }
         optionBindings.push({

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goke/mcp",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "type": "module",
   "description": "Dynamically generate CLI commands from MCP server tools",
   "repository": {
@@ -51,7 +51,7 @@
     "@types/node": "^22.19.7",
     "vitest": "^3.1.0",
     "zod": "^4.3.6",
-    "goke": "^6.16.0"
+    "goke": "^6.17.0"
   },
   "scripts": {
     "clean": "rm -rf dist",

package/src/__test__/add-cli-tools-to-mcp.test.ts

@@ -18,6 +18,7 @@ function createCli() {
   cli
     .command("say hi", "Say hello")
     .option("--name <name>", z.string().describe("Person to greet"))
+    .required()
     .option("--caps", z.boolean().default(false).describe("Uppercase output"))
     .action((options) => {
       const message = `Hello ${options.name}!`;

package/src/__test__/create-mcp-action.test.ts

@@ -35,7 +35,9 @@ describe("createMcpAction", () => {
     cli
       .command("add", "Add numbers")
       .option("--a <a>", z.number().describe("First"))
+      .required()
       .option("--b <b>", z.number().describe("Second"))
+      .required()
       .action((options) => ({ sum: options.a + options.b }));
 
     const [clientTransport, serverTransport] = InMemoryTransport.createLinkedPair();
@@ -238,7 +240,7 @@ describe("createMcpAction", () => {
       expect(searchTool.description).toBe("Search for items");
       expect(searchTool.inputSchema.properties).toHaveProperty("query");
       expect(searchTool.inputSchema.properties).toHaveProperty("limit");
-      expect(searchTool.inputSchema.required).toEqual(["query"]);
+      expect(searchTool.inputSchema.required).toBeUndefined();
 
       const deployTool = tools.tools.find((t) => t.name === "deploy")!;
       expect(deployTool.inputSchema.properties).toHaveProperty("env");
@@ -346,13 +348,15 @@ describe("createMcpAction", () => {
     }
   });
 
-  it("puts schema-required --flag <value> in inputSchema.required", async () => {
+  it("puts .required() --flag <value> in inputSchema.required", async () => {
     const cli = goke("checks");
 
     cli
       .command("checks create", "Create a check")
       .option("--url <url>", z.string().describe("URL to check"))
+      .required()
       .option("--name <name>", z.string().describe("Check name"))
+      .required()
       .option("--timeout [ms]", z.number().optional().describe("Timeout"))
       .action((options) => options);
 

package/src/__test__/http-multi-tenant.test.ts

@@ -104,6 +104,7 @@ function buildBaseCli(): Goke {
   cli
     .command("save <filename>", "Save content to a file in the tenant workspace")
     .option("--content <content>", z.string().describe("File content"))
+    .required()
     .action(async (filename: string, options: { content: string }, ctx) => {
       const full = path.posix.join(ctx.process.cwd, filename);
       await ctx.fs.writeFile(full, options.content);

package/src/cli-to-mcp.ts

@@ -20,7 +20,6 @@ import {
   coerceBySchema,
   extractJsonSchema,
   GokeProcessExit,
-  schemaAcceptsOmittedValue,
   type Command,
   type Goke,
   type GokeExecutionContext,
@@ -41,6 +40,7 @@ interface OptionLike {
   description: string;
   default?: unknown;
   required?: boolean;
+  flagRequired?: boolean;
   isBoolean?: boolean;
   schema?: StandardJSONSchemaV1;
 }
@@ -505,17 +505,12 @@ function createBinding(cli: Goke, command: Command, toolName: string): CliToolBi
   }
 
   // `--days <days>` means "value required if the flag is present". The flag
-  // itself is required only when a non-optional schema rejects `undefined`.
+  // itself is required only when `.required()` set `flagRequired`.
   for (const option of options) {
     const normalized = normalizeOptionSchema(option);
     properties[option.name] = normalized.schema;
 
-    if (
-      option.required
-      && option.default === undefined
-      && option.schema
-      && !schemaAcceptsOmittedValue(option.schema)
-    ) {
+    if (option.flagRequired && option.default === undefined) {
       requiredNames.push(option.name);
     }