@fruggr/zendesk-mcp-server 2.18.0 → 2.19.0

package/dist/index.js

@@ -281,7 +281,6 @@ const startBrowserAuth = (config, logger = silentLogger) => {
 		});
 		const requestedPort = config.callbackPort ?? 27439;
 		const onStartError = (err) => {
-			clearTimeout(authTimeout);
 			const code = err.code;
 			logger.error("oauth_callback_listen_failed", {
 				port: requestedPort,
@@ -4544,6 +4543,10 @@ const registerReloadTool = (server, reload, logger = silentLogger) => {
 * `reload_tools` tool that hot-reloads edited tool code on demand. stdio only —
 * HTTP builds a per-session server per request, so there is no long-lived
 * server to hot-swap.
+*
+* Returns the running server so the caller can close it on shutdown: dev mode
+* runs over the same stdio transport as normal mode and must exit the same way
+* when the client disconnects.
 */
 /* v8 ignore start -- runtime bootstrap: binds the reload tool to a real stdio
 transport; the reload machinery it wires up is covered by dev-reload.test.ts */
@@ -4552,6 +4555,7 @@ const startDevServer = async (config, getToken, logger = silentLogger, onUnautho
 	registerReloadTool(server, reload, logger);
 	await startStdioTransport(server, logger);
 	logger.info("dev_mode_enabled");
+	return server;
 };
 /* v8 ignore stop */
 //#endregion
@@ -4907,27 +4911,125 @@ const startHttpTransport = async (config, logger = silentLogger, options = {}) =
 	};
 };
 //#endregion
+//#region src/utils/shutdown.ts
+/**
+* How long a shutdown may take before the watchdog forces the exit.
+*
+* Generous enough for `server.close()` and an HTTP session drain, short enough
+* to stay inside the tightest common supervisor grace — `docker stop` allows 10s
+* and Kubernetes 30s before their own SIGKILL (systemd is far laxer at 90s). A
+* process killed by its supervisor is exactly the unclean exit this removes.
+*/
+const SHUTDOWN_GRACE_MS = 3e3;
+/**
+* Adapt a `process` to a {@link ShutdownRuntime}. Takes the process as an
+* argument rather than closing over the global so the adapter itself is
+* testable — otherwise the one part of this module that touches the real
+* process would be the one part no test can reach.
+*/
+const createRuntime = (proc) => ({
+	on: (event, listener) => {
+		proc.on(event, listener);
+	},
+	stdin: { on: (event, listener) => {
+		proc.stdin.on(event, listener);
+	} },
+	exit: (code) => {
+		proc.exit(code);
+	},
+	setTimer: (fn, ms) => {
+		const timer = setTimeout(fn, ms);
+		return {
+			unref: () => void timer.unref(),
+			clear: () => clearTimeout(timer)
+		};
+	}
+});
+const defaultRuntime = createRuntime(process);
+/**
+* Install the process's one shutdown path and return its trigger.
+*
+* Registering a `SIGTERM` handler *removes* Node's default terminate, which
+* makes the exit our responsibility: a cleanup that stalls on an in-flight
+* request would otherwise leave a process SIGTERM cannot kill — the very
+* symptom this exists to remove. Hence the watchdog, which is load-bearing
+* rather than defensive, and the unconditional `exit` on every path.
+*
+* The exit is explicit rather than a drained event loop because the OAuth
+* callback server (`auth/browser-oauth.ts`) is a listening socket that is not
+* `unref()`'d: letting the loop drain would keep a disconnected session alive
+* for up to the 5-minute auth timeout.
+*/
+const installShutdown = (options) => {
+	const { cleanup, logger, watchStdin, graceMs = SHUTDOWN_GRACE_MS } = options;
+	const runtime = options.runtime ?? defaultRuntime;
+	let started = false;
+	let exited = false;
+	const exitOnce = (code) => {
+		if (exited) return;
+		exited = true;
+		runtime.exit(code);
+	};
+	const shutdown = async (reason) => {
+		if (started) return;
+		started = true;
+		logger.info("shutdown_started", { reason });
+		const watchdog = runtime.setTimer(() => {
+			logger.warn("shutdown_forced", { graceMs });
+			exitOnce(0);
+		}, graceMs);
+		watchdog.unref();
+		try {
+			await cleanup();
+			logger.info("shutdown_complete", { reason });
+		} catch (err) {
+			logger.warn("shutdown_cleanup_failed", { error: err instanceof Error ? err.message : String(err) });
+		} finally {
+			watchdog.clear();
+			exitOnce(0);
+		}
+	};
+	runtime.on("SIGINT", () => void shutdown("SIGINT"));
+	runtime.on("SIGTERM", () => void shutdown("SIGTERM"));
+	if (watchStdin) runtime.stdin.on("end", () => void shutdown("stdin_eof"));
+	return shutdown;
+};
+//#endregion
 //#region src/index.ts
 const buildStdioTokenStore = (config, logger) => createTokenStore({
 	subdomain: config.subdomain,
 	oauthClientId: config.oauthClientId,
 	callbackPort: config.callbackPort
 }, logger);
+const connectStdio = async (config, tokenStore, logger) => {
+	if (config.dev) return startDevServer(config, tokenStore.getToken, logger, tokenStore.invalidate);
+	const server = createMcpServer(config, tokenStore.getToken, logger, tokenStore.invalidate);
+	await startStdioTransport(server, logger);
+	return server;
+};
 const main = async () => {
 	const config = loadConfig();
 	const logger = createLogger(config.logLevel);
 	if (config.transport === "stdio") {
 		const tokenStore = buildStdioTokenStore(config, logger);
-		if (config.dev) {
-			await startDevServer(config, tokenStore.getToken, logger, tokenStore.invalidate);
-			return;
-		}
-		const server = createMcpServer(config, tokenStore.getToken, logger, tokenStore.invalidate);
-		await startStdioTransport(server, logger);
+		const server = await connectStdio(config, tokenStore, logger);
+		installShutdown({
+			watchStdin: true,
+			logger,
+			cleanup: async () => {
+				await server.close();
+				tokenStore.dispose();
+			}
+		});
 		return;
 	}
 	if (config.dev) logger.warn("dev_mode_ignored_http");
-	await startHttpTransport(config, logger);
+	const http = await startHttpTransport(config, logger);
+	installShutdown({
+		watchStdin: false,
+		logger,
+		cleanup: http.close
+	});
 };
 main().catch((error) => {
 	console.error("Fatal error:", error);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fruggr/zendesk-mcp-server",
-  "version": "2.18.0",
+  "version": "2.19.0",
   "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",
@@ -69,13 +69,13 @@
   "engines": {
     "node": ">=20"
   },
-  "packageManager": "pnpm@11.21.0+sha512.521705bce689924eac72f5a3587122f362689ef6571e55ba80076fd637c11132ecffada26fad4ea79c485bfddbfd3d5a2a5b05805a77e893de71ec8a6cca3bb1",
+  "packageManager": "pnpm@11.22.0+sha512.1ff870c4c6133dfd88fb2afc46dd13d47f09c9794b438c6fdb47ca98caf3bc16381ee0be93a091b8e3824cf01f889f46d7d9e20910fb0be1ab0fb5baa80dd621",
   "dependencies": {
     "@modelcontextprotocol/sdk": "1.30.0",
     "cheerio": "1.2.0",
     "hast-util-to-html": "9.0.5",
     "hast-util-to-mdast": "10.1.2",
-    "open": "11.0.0",
+    "open": "11.0.1",
     "rehype-parse": "9.0.1",
     "rehype-raw": "7.0.0",
     "rehype-remark": "10.0.1",
@@ -88,15 +88,15 @@
     "zod": "4.4.3"
   },
   "devDependencies": {
-    "@biomejs/biome": "2.5.7",
+    "@biomejs/biome": "2.5.8",
     "@semantic-release/changelog": "^7.0.0",
     "@semantic-release/exec": "^7.1.0",
     "@semantic-release/git": "^11.0.0",
     "@semantic-release/github": "^12.0.6",
     "@semantic-release/npm": "^13.1.5",
     "@semantic-release/release-notes-generator": "^14.1.1",
-    "@stryker-mutator/core": "^9.6.1",
-    "@stryker-mutator/vitest-runner": "^9.6.1",
+    "@stryker-mutator/core": "^10.0.0",
+    "@stryker-mutator/vitest-runner": "^10.0.0",
     "@tsconfig/node20": "^20.1.9",
     "@tsconfig/strictest": "^2.0.8",
     "@types/hast": "^3.0.4",