@fruggr/zendesk-mcp-server 2.17.0 → 2.17.1

package/dist/index.js

@@ -6,6 +6,7 @@ import { homedir, release } from "node:os";
 import open from "open";
 import { dirname, join } from "node:path";
 import { fileURLToPath, pathToFileURL } from "node:url";
+import { parseArgs } from "node:util";
 import * as z from "zod/v4";
 import { McpServer, ResourceTemplate } from "@modelcontextprotocol/sdk/server/mcp.js";
 import * as cheerio from "cheerio";
@@ -706,97 +707,99 @@ const parsePort = (raw, label) => {
 	if (!DIGITS_ONLY.test(raw)) throw new Error(`Invalid ${label} value. Expected an integer 0-65535.`);
 	return Number(raw);
 };
-const parsePortEnv = (raw, label) => raw === void 0 || raw === "" ? void 0 : parsePort(raw, label);
-const appendTo = (key) => (result, value) => {
-	const list = result[key] ?? [];
-	list.push(value);
-	result[key] = list;
-};
-const STANDALONE_FLAGS = /* @__PURE__ */ new Map([
-	["--read-only", (result) => {
-		result.readOnly = true;
-	}],
-	["--no-topology", (result) => {
-		result.topology = false;
-	}],
-	["--no-promoted-articles", (result) => {
-		result.promotedArticles = false;
-	}],
-	["--dev", (result) => {
-		result.dev = true;
-	}]
+const parsePortEnv = (raw, label) => raw === void 0 ? void 0 : parsePort(raw, label);
+const requireNonEmptyEnv = (name) => {
+	const raw = process.env[name];
+	if (raw === "") throw new Error(`Empty ${name}. Set it to a value, or unset it entirely.`);
+	return raw;
+};
+const CLI_OPTIONS = {
+	mode: { type: "string" },
+	namespace: {
+		type: "string",
+		multiple: true
+	},
+	tool: {
+		type: "string",
+		multiple: true
+	},
+	"log-level": { type: "string" },
+	"hc-resource-scheme": { type: "string" },
+	transport: { type: "string" },
+	host: { type: "string" },
+	port: { type: "string" },
+	"public-url": { type: "string" },
+	"cors-origin": {
+		type: "string",
+		multiple: true
+	},
+	"callback-port": { type: "string" },
+	"read-only": { type: "boolean" },
+	"no-topology": { type: "boolean" },
+	"no-promoted-articles": { type: "boolean" },
+	dev: { type: "boolean" }
+};
+new Set(Object.entries(CLI_OPTIONS).filter(([, spec]) => spec.type === "string").map(([name]) => `--${name}`));
+const FIELD_BY_FLAG = /* @__PURE__ */ new Map([
+	["mode", "mode"],
+	["namespace", "namespaces"],
+	["tool", "tools"],
+	["log-level", "logLevel"],
+	["hc-resource-scheme", "hcResourceScheme"],
+	["transport", "transport"],
+	["host", "host"],
+	["public-url", "publicUrl"],
+	["cors-origin", "corsOrigins"]
 ]);
-const VALUED_FLAGS = /* @__PURE__ */ new Map([
-	["--mode", (result, value) => {
-		result.mode = value;
-	}],
-	["--hc-resource-scheme", (result, value) => {
-		result.hcResourceScheme = value;
-	}],
-	["--log-level", (result, value) => {
-		result.logLevel = value;
-	}],
-	["--transport", (result, value) => {
-		result.transport = value;
-	}],
-	["--host", (result, value) => {
-		result.host = value;
-	}],
-	["--public-url", (result, value) => {
-		result.publicUrl = value;
-	}],
-	["--port", (result, value) => {
-		result.port = parsePort(value, "--port");
-	}],
-	["--callback-port", (result, value) => {
-		result.callbackPort = parsePort(value, "--callback-port");
-	}],
-	["--namespace", appendTo("namespaces")],
-	["--tool", appendTo("tools")],
-	["--cors-origin", appendTo("corsOrigins")]
+const STANDALONE_EFFECTS = /* @__PURE__ */ new Map([
+	["read-only", { readOnly: true }],
+	["no-topology", { topology: false }],
+	["no-promoted-articles", { promotedArticles: false }],
+	["dev", { dev: true }]
 ]);
 const parseCliArgs = (args) => {
-	const result = {};
-	for (let i = 0; i < args.length; i++) {
-		const arg = args[i];
-		if (arg === void 0) continue;
-		const standalone = STANDALONE_FLAGS.get(arg);
-		if (standalone) {
-			standalone(result);
-			continue;
-		}
-		const valued = VALUED_FLAGS.get(arg);
-		const next = args[i + 1];
-		if (valued && next) {
-			valued(result, next);
-			i++;
+	const { values, positionals } = parseArgs({
+		args,
+		options: CLI_OPTIONS,
+		allowPositionals: true
+	});
+	if (positionals.length > 1) throw new Error(`Expected one positional argument (the subdomain), got ${positionals.length}. A repeatable flag has to be repeated (--namespace tickets --namespace help_center); it does not take a space-separated list.`);
+	const result = { subdomain: positionals[0] };
+	for (const [flag, value] of Object.entries(values)) {
+		if (Array.isArray(value) ? value.includes("") : value === "") throw new Error(`Empty value for --${flag}. Provide a value, or omit the flag.`);
+		const effect = STANDALONE_EFFECTS.get(flag);
+		if (effect) {
+			Object.assign(result, effect);
 			continue;
 		}
-		if (!arg.startsWith("-") && result.subdomain === void 0) result.subdomain = arg;
+		const field = FIELD_BY_FLAG.get(flag);
+		if (field) Object.assign(result, { [field]: value });
 	}
+	if (values.port !== void 0) result.port = parsePort(values.port, "--port");
+	if (values["callback-port"] !== void 0) result.callbackPort = parsePort(values["callback-port"], "--callback-port");
 	return result;
 };
 const resolveTransportSettings = (cli) => {
 	const corsFromEnv = (process.env["CORS_ORIGIN"] ?? "").split(",").map((s) => s.trim()).filter((s) => s.length > 0);
 	return {
-		transport: cli.transport ?? process.env["TRANSPORT"] ?? "stdio",
-		host: cli.host ?? process.env["HOST"] ?? "0.0.0.0",
-		port: cli.port ?? parsePortEnv(process.env["PORT"], "PORT") ?? 3e3,
-		publicUrl: cli.publicUrl ?? process.env["PUBLIC_URL"],
+		transport: cli.transport ?? requireNonEmptyEnv("TRANSPORT") ?? "stdio",
+		host: cli.host ?? requireNonEmptyEnv("HOST") ?? "0.0.0.0",
+		port: cli.port ?? parsePortEnv(requireNonEmptyEnv("PORT"), "PORT") ?? 3e3,
+		publicUrl: cli.publicUrl ?? requireNonEmptyEnv("PUBLIC_URL"),
 		corsOrigins: [...cli.corsOrigins ?? [], ...corsFromEnv]
 	};
 };
 const loadConfig = (argv = process.argv.slice(2)) => {
 	const cli = parseCliArgs(argv);
-	const subdomain = cli.subdomain ?? process.env["ZENDESK_SUBDOMAIN"] ?? "";
-	const oauthClientId = process.env["ZENDESK_OAUTH_CLIENT_ID"] ?? (subdomain ? `${subdomain}_zendesk` : "");
+	const subdomain = cli.subdomain ?? requireNonEmptyEnv("ZENDESK_SUBDOMAIN") ?? "";
+	const oauthClientId = requireNonEmptyEnv("ZENDESK_OAUTH_CLIENT_ID") ?? `${subdomain}_zendesk`;
 	const mode = cli.tools?.length ? "all" : cli.mode ?? "namespace";
-	const callbackPort = cli.callbackPort ?? parsePortEnv(process.env["ZENDESK_OAUTH_CALLBACK_PORT"], "ZENDESK_OAUTH_CALLBACK_PORT");
-	const hcResourceScheme = cli.hcResourceScheme ?? (process.env["HC_RESOURCE_SCHEME"] || void 0);
+	const callbackPort = cli.callbackPort ?? parsePortEnv(requireNonEmptyEnv("ZENDESK_OAUTH_CALLBACK_PORT"), "ZENDESK_OAUTH_CALLBACK_PORT");
+	const hcResourceScheme = cli.hcResourceScheme ?? requireNonEmptyEnv("HC_RESOURCE_SCHEME");
 	return ConfigSchema.parse({
 		subdomain,
 		oauthClientId,
-		logLevel: cli.logLevel ?? process.env["LOG_LEVEL"] ?? "info",
+		logLevel: cli.logLevel ?? requireNonEmptyEnv("LOG_LEVEL") ?? "info",
 		mode,
 		readOnly: cli.readOnly ?? false,
 		namespaces: cli.namespaces,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fruggr/zendesk-mcp-server",
-  "version": "2.17.0",
+  "version": "2.17.1",
   "mcpName": "io.github.fruggr/zendesk-mcp-server",
   "description": "Deep Zendesk MCP server for your AI assistant: search, draft, update and translate Help Center articles and manage Support tickets end to end — comments, triage and image attachments.",
   "type": "module",