fix(cli): thread --dev-only into McpContext so the env guard fires (#32)

Author: d-cs

.changeset/mcp-thread-dev-only-flag.md

@@ -0,0 +1,5 @@
+---
+"trigger.dev": patch
+---
+
+fix(cli): honor the MCP server's `--dev-only` flag

packages/cli-v3/src/commands/mcp.ts

@@ -6,9 +6,9 @@ import { tryCatch } from "@trigger.dev/core/utils";
 import type { Command } from "commander";
 import { z } from "zod";
 import { CommonCommandOptions, commonOptions, wrapCommandAction } from "../cli/common.js";
-import { CLOUD_API_URL } from "../consts.js";
 import { serverMetadata } from "../mcp/config.js";
 import { McpContext } from "../mcp/context.js";
+import { toMcpContextOptions } from "../mcp/contextOptions.js";
 import { FileLogger } from "../mcp/logger.js";
 import { registerTools } from "../mcp/tools.js";
 import { printStandloneInitialBanner } from "../utilities/initialBanner.js";
@@ -95,13 +95,7 @@ export async function mcpCommand(options: McpCommandOptions) {
     ? new FileLogger(options.logFile, server)
     : undefined;
 
-  const context = new McpContext(server, {
-    projectRef: options.projectRef,
-    fileLogger,
-    apiUrl: options.apiUrl ?? CLOUD_API_URL,
-    profile: options.profile,
-    readonly: options.readonly,
-  });
+  const context = new McpContext(server, toMcpContextOptions(options, fileLogger));
 
   registerTools(context);
 

packages/cli-v3/src/mcp/contextOptions.test.ts

@@ -0,0 +1,31 @@
+import { describe, expect, it } from "vitest";
+import { toMcpContextOptions } from "./contextOptions.js";
+import type { McpCommandOptions } from "../commands/mcp.js";
+
+// The dev-only guards only work if `--dev-only` is threaded from the parsed
+// CLI options into the McpContext. These tests pin that wiring.
+const baseOptions = {
+  projectRef: "proj_123",
+  apiUrl: "https://api.example.com",
+  profile: "default",
+  readonly: false,
+  devOnly: false,
+} as unknown as McpCommandOptions;
+
+describe("toMcpContextOptions", () => {
+  it("threads devOnly=true through to the context options", () => {
+    expect(toMcpContextOptions({ ...baseOptions, devOnly: true }, undefined).devOnly).toBe(true);
+  });
+
+  it("threads devOnly=false through to the context options", () => {
+    expect(toMcpContextOptions({ ...baseOptions, devOnly: false }, undefined).devOnly).toBe(false);
+  });
+
+  it("carries the other relevant options across", () => {
+    const result = toMcpContextOptions(baseOptions, undefined);
+    expect(result.projectRef).toBe("proj_123");
+    expect(result.apiUrl).toBe("https://api.example.com");
+    expect(result.profile).toBe("default");
+    expect(result.readonly).toBe(false);
+  });
+});

packages/cli-v3/src/mcp/contextOptions.ts

@@ -0,0 +1,26 @@
+import { CLOUD_API_URL } from "../consts.js";
+import type { McpCommandOptions } from "../commands/mcp.js";
+import type { McpContextOptions } from "./context.js";
+import type { FileLogger } from "./logger.js";
+
+/**
+ * Map parsed CLI options onto the `McpContext` options. `devOnly` must be
+ * forwarded here or the context sees `undefined` and its dev-only guards
+ * never fire.
+ *
+ * Kept in its own module (type-only imports) so the wiring can be
+ * unit-tested without loading the full command/tool/build chain.
+ */
+export function toMcpContextOptions(
+  options: McpCommandOptions,
+  fileLogger: FileLogger | undefined
+): McpContextOptions {
+  return {
+    projectRef: options.projectRef,
+    fileLogger,
+    apiUrl: options.apiUrl ?? CLOUD_API_URL,
+    profile: options.profile,
+    readonly: options.readonly,
+    devOnly: options.devOnly,
+  };
+}