@fruggr/zendesk-mcp-server 3.4.0 → 3.5.0

package/dist/pkce-DE_v9NQy.js

@@ -0,0 +1,60 @@
+#!/usr/bin/env node
+import { createHash, randomBytes } from "node:crypto";
+//#region src/utils/html.ts
+/**
+* Escape a string for safe interpolation into HTML text/attribute context.
+* Both the stdio callback page and the HTTP consent page echo
+* attacker-controllable values (an OAuth `error_description`, a client name
+* from a fetched metadata document); without escaping these are an XSS sink.
+*/
+const escapeHtml = (value) => value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
+//#endregion
+//#region src/auth/oauth-scopes.ts
+/**
+* The OAuth scope the server asks Zendesk for, and the one predicate that
+* decides whether a cached token's grant is still good enough.
+*
+* Both scope strings live here rather than in `constants.ts` on purpose: they
+* are behavioural strings whose mutants must be killed by assertions
+* (`docs/decisions/mutation-testing.md`, "OAuth parameters and scopes"), and
+* `src/auth/**` is inside the mutation scope while `constants.ts` is not.
+* Keeping them next to the predicate that consumes them keeps both under the
+* same gate.
+*/
+const READ_SCOPE = "read";
+const READ_WRITE_SCOPE = "read write";
+const WHITESPACE = /\s+/;
+const scopeTokens = (scope) => scope.split(WHITESPACE).filter((s) => s.length > 0);
+/**
+* The scope to request for a given tool surface. `--read-only` already filters
+* every write tool out of the surface, so asking Zendesk for `write` on top of
+* that would be requesting an authority the server cannot even exercise — and
+* an OAuth client whose allowed scopes stop at `read` rejects the whole
+* authorize request with `invalid_scope`, minting no token at all (#283).
+*/
+const requestedScope = (readOnly) => readOnly ? READ_SCOPE : READ_WRITE_SCOPE;
+/**
+* The same decision as a list, for the RFC 9728 / RFC 8414 `scopes_supported`
+* metadata. Derived from `requestedScope` so what the HTTP transport advertises
+* provably cannot drift from what the stdio flow requests.
+*/
+const supportedScopes = (readOnly) => scopeTokens(requestedScope(readOnly));
+/**
+* Whether a grant still covers what this process needs: a flat subset test.
+* Coverage, not equality, so a token Zendesk granted wider than requested stays
+* usable. A non-string `granted` is a pre-#283 record, i.e. `read write` — only
+* reachable now through a `OAUTH_TOKEN_FILE` aimed at one by hand, since the
+* default layout no longer names those files. No scope hierarchy: granular
+* scopes (#284) replace this.
+*/
+const grantCovers = (granted, requested) => {
+	if (typeof granted !== "string") return true;
+	const held = new Set(scopeTokens(granted));
+	return scopeTokens(requested).every((token) => held.has(token));
+};
+//#endregion
+//#region src/auth/pkce.ts
+const generateCodeVerifier = () => randomBytes(32).toString("base64url");
+const generateCodeChallenge = (verifier) => createHash("sha256").update(verifier).digest("base64url");
+//#endregion
+export { supportedScopes as a, requestedScope as i, generateCodeVerifier as n, escapeHtml as o, grantCovers as r, generateCodeChallenge as t };

package/dist/sbom.cdx.json

@@ -1,15 +1,15 @@
 {
   "bomFormat": "CycloneDX",
   "specVersion": "1.6",
-  "serialNumber": "urn:uuid:a6053b0f-5ae6-5ce9-89d9-67565992fe89",
+  "serialNumber": "urn:uuid:4bd8e289-8e88-51ac-9e9e-efc17ba1d8a7",
   "version": 1,
   "metadata": {
     "component": {
       "type": "application",
-      "bom-ref": "pkg:npm/%40fruggr/zendesk-mcp-server@3.4.0",
+      "bom-ref": "pkg:npm/%40fruggr/zendesk-mcp-server@3.5.0",
       "name": "@fruggr/zendesk-mcp-server",
-      "version": "3.4.0",
-      "purl": "pkg:npm/%40fruggr/zendesk-mcp-server@3.4.0"
+      "version": "3.5.0",
+      "purl": "pkg:npm/%40fruggr/zendesk-mcp-server@3.5.0"
     }
   },
   "components": [
@@ -2115,10 +2115,10 @@
     },
     {
       "type": "library",
-      "bom-ref": "pkg:npm/zod@4.6.6",
+      "bom-ref": "pkg:npm/zod@4.6.7",
       "name": "zod",
-      "version": "4.6.6",
-      "purl": "pkg:npm/zod@4.6.6",
+      "version": "4.6.7",
+      "purl": "pkg:npm/zod@4.6.7",
       "licenses": [
         {
           "expression": "MIT"

package/dist/{token-store-DyO12sbo.js → token-store-CQx_KFe2.js}

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
-import { a as silentLogger, u as getOAuthUrls } from "./index.js";
-import { a as saveToken, c as grantCovers, d as escapeHtml, i as resolveTokenPath, l as requestedScope, o as generateCodeChallenge, r as loadToken, s as generateCodeVerifier, t as clearToken } from "./token-persistence-r0v9RO3q.js";
+import { d as saveToken, f as signInRequiredError, l as loadToken, m as silentLogger, s as clearToken, u as resolveTokenPath, y as getOAuthUrls } from "./index.js";
+import { i as requestedScope, n as generateCodeVerifier, o as escapeHtml, r as grantCovers, t as generateCodeChallenge } from "./pkce-DE_v9NQy.js";
 import { promisify } from "node:util";
 import path from "node:path";
 import { fileURLToPath } from "node:url";
@@ -510,7 +510,7 @@ const baseOpen = async (options) => {
 		});
 	});
 };
-const open = (target, options) => {
+const open$1 = (target, options) => {
 	if (typeof target !== "string") throw new TypeError("Expected a `target`");
 	return baseOpen({
 		...options,
@@ -736,7 +736,7 @@ const startBrowserAuth = (config, logger = silentLogger) => {
 			logger.debug("oauth_authorize_url", { url: authUrl });
 			console.error(`Opening browser for Zendesk authentication...`);
 			console.error(`If the browser doesn't open, visit: ${authUrl}`);
-			open(authUrl).then(() => {
+			open$1(authUrl).then(() => {
 				logger.debug("oauth_browser_opened");
 			}).catch((err) => {
 				logger.error("oauth_browser_open_failed", {
@@ -908,14 +908,17 @@ const createTokenStore = (config, logger = silentLogger) => {
 		});
 		return refreshing;
 	};
-	const getToken = async () => {
+	const tokenWithoutPrompt = async () => {
 		if (refreshing !== void 0) await refreshing;
 		if (token && !needsRefresh(token)) {
 			logger.debug("oauth_token_cache_hit");
 			return token.accessToken;
 		}
-		const refreshed = await refreshIfPossible();
-		if (refreshed) return refreshed;
+		return refreshIfPossible();
+	};
+	const getToken = async () => {
+		const silent = await tokenWithoutPrompt();
+		if (silent) return silent;
 		if (starting === void 0) starting = beginAuth();
 		const url = authorizeUrl ?? await starting;
 		throw createAuthRequiredError(url);
@@ -942,8 +945,14 @@ const createTokenStore = (config, logger = silentLogger) => {
 	}, SCHEDULED_REFRESH_MS);
 	scheduledRefresh.unref?.();
 	const dispose = () => clearInterval(scheduledRefresh);
+	const getTokenSilently = async () => {
+		const silent = await tokenWithoutPrompt();
+		if (silent) return silent;
+		throw signInRequiredError();
+	};
 	return {
 		getToken,
+		getTokenSilently,
 		setToken,
 		invalidate,
 		dispose

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@fruggr/zendesk-mcp-server",
-  "version": "3.4.0",
+  "version": "3.5.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",
@@ -72,7 +72,7 @@
   "engines": {
     "node": ">=22"
   },
-  "packageManager": "pnpm@12.12.1+sha512.b7f98934c0b1f29a65f5da52f2d7f994e7faad573ffdddea1497fb4dcb5aec051590ee7c7d95a38172b4bfae103a5b83edf2ee866e11731c2cafa96063610ac5",
+  "packageManager": "pnpm@12.14.0+sha512.c18ba6d87993d3d164cb5e8948f32b2c06ee854487e06cd8d86781342fb3e5bb4634367034711e86802baa9c37bea906ce1929ad08d70884c97d801df13389a6",
   "devDependencies": {
     "@biomejs/biome": "2.5.15",
     "@modelcontextprotocol/client": "2.3.1",
@@ -97,7 +97,7 @@
     "hast-util-to-html": "9.0.5",
     "hast-util-to-mdast": "10.1.2",
     "jose": "6.2.12",
-    "lefthook": "^2.1.10",
+    "lefthook": "^2.2.2",
     "msw": "^3.0.0",
     "oidc-provider": "~9.12.2",
     "open": "11.0.4",
@@ -117,6 +117,6 @@
     "typescript-legacy": "npm:typescript@6.0.3",
     "unified": "11.0.5",
     "vitest": "^5.0.0",
-    "zod": "4.6.6"
+    "zod": "4.6.7"
   }
 }

package/dist/token-persistence-r0v9RO3q.js

@@ -1,147 +0,0 @@
-#!/usr/bin/env node
-import { a as silentLogger, i as readPackageInfo, x as readEnv } from "./index.js";
-import { createHash, randomBytes } from "node:crypto";
-import { dirname, join } from "node:path";
-import { chmodSync, mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
-import { homedir } from "node:os";
-//#region src/utils/html.ts
-/**
-* Escape a string for safe interpolation into HTML text/attribute context.
-* Both the stdio callback page and the HTTP consent page echo
-* attacker-controllable values (an OAuth `error_description`, a client name
-* from a fetched metadata document); without escaping these are an XSS sink.
-*/
-const escapeHtml = (value) => value.replace(/&/g, "&amp;").replace(/</g, "&lt;").replace(/>/g, "&gt;").replace(/"/g, "&quot;").replace(/'/g, "&#39;");
-//#endregion
-//#region src/auth/oauth-scopes.ts
-/**
-* The OAuth scope the server asks Zendesk for, and the one predicate that
-* decides whether a cached token's grant is still good enough.
-*
-* Both scope strings live here rather than in `constants.ts` on purpose: they
-* are behavioural strings whose mutants must be killed by assertions
-* (`docs/decisions/mutation-testing.md`, "OAuth parameters and scopes"), and
-* `src/auth/**` is inside the mutation scope while `constants.ts` is not.
-* Keeping them next to the predicate that consumes them keeps both under the
-* same gate.
-*/
-const READ_SCOPE = "read";
-const READ_WRITE_SCOPE = "read write";
-const WHITESPACE = /\s+/;
-const scopeTokens = (scope) => scope.split(WHITESPACE).filter((s) => s.length > 0);
-/**
-* The scope to request for a given tool surface. `--read-only` already filters
-* every write tool out of the surface, so asking Zendesk for `write` on top of
-* that would be requesting an authority the server cannot even exercise — and
-* an OAuth client whose allowed scopes stop at `read` rejects the whole
-* authorize request with `invalid_scope`, minting no token at all (#283).
-*/
-const requestedScope = (readOnly) => readOnly ? READ_SCOPE : READ_WRITE_SCOPE;
-/**
-* The same decision as a list, for the RFC 9728 / RFC 8414 `scopes_supported`
-* metadata. Derived from `requestedScope` so what the HTTP transport advertises
-* provably cannot drift from what the stdio flow requests.
-*/
-const supportedScopes = (readOnly) => scopeTokens(requestedScope(readOnly));
-/**
-* Whether a grant still covers what this process needs: a flat subset test.
-* Coverage, not equality, so a token Zendesk granted wider than requested stays
-* usable. A non-string `granted` is a pre-#283 record, i.e. `read write` — only
-* reachable now through a `OAUTH_TOKEN_FILE` aimed at one by hand, since the
-* default layout no longer names those files. No scope hierarchy: granular
-* scopes (#284) replace this.
-*/
-const grantCovers = (granted, requested) => {
-	if (typeof granted !== "string") return true;
-	const held = new Set(scopeTokens(granted));
-	return scopeTokens(requested).every((token) => held.has(token));
-};
-//#endregion
-//#region src/auth/pkce.ts
-const generateCodeVerifier = () => randomBytes(32).toString("base64url");
-const generateCodeChallenge = (verifier) => createHash("sha256").update(verifier).digest("base64url");
-//#endregion
-//#region src/auth/token-persistence.ts
-const isWindows = process.platform === "win32";
-/**
-* Config-dir segments derived from the *scoped* package name
-* (`@fruggr/zendesk-mcp-server` → `fruggr` + `zendesk-mcp-server`) so the path is
-* vendor-namespaced and can't collide with another `zendesk-mcp-server`.
-*/
-const SCOPED_PACKAGE_NAME = /^@([^/]+)\/(.+)/;
-const appDirSegments = () => {
-	const { name } = readPackageInfo();
-	const scoped = SCOPED_PACKAGE_NAME.exec(name);
-	return scoped?.[1] && scoped[2] ? [scoped[1], scoped[2]] : [name];
-};
-const configDir = () => {
-	const segments = appDirSegments();
-	if (isWindows) {
-		const base = process.env["APPDATA"] ?? join(homedir(), "AppData", "Roaming");
-		return join(base, ...segments);
-	}
-	const base = process.env["XDG_CONFIG_HOME"] ?? join(homedir(), ".config");
-	return join(base, ...segments);
-};
-const safeName = (part) => part.replace(/[^a-z0-9-]/gi, "_");
-const READABLE_BUDGET = 120;
-const keyDigest = (key) => createHash("sha256").update(JSON.stringify([
-	key.subdomain,
-	key.oauthClientId,
-	key.scope
-])).digest("hex").slice(0, 8);
-/**
-* Path to the token file for one credential. Keyed on the whole `TokenKey`, so
-* a read-only server, a read-write one and one on another OAuth client each get
-* their own record instead of clobbering a shared file. Why that triple, and
-* why no migration from the old subdomain-only layout:
-* `docs/decisions/token-file-keying.md`.
-* `OAUTH_TOKEN_FILE` overrides with an explicit path — the way to separate
-* two Zendesk accounts that share a subdomain, client and scope.
-*/
-const resolveTokenPath = (key) => {
-	const { name, value: override } = readEnv("OAUTH_TOKEN_FILE");
-	if (override === "") throw new Error(`Empty ${name}. Set it to a value, or unset it entirely.`);
-	if (override) return override;
-	const readable = [
-		key.subdomain,
-		key.oauthClientId,
-		key.scope
-	].map(safeName).join("--").slice(0, READABLE_BUDGET);
-	return join(configDir(), `${readable}--${keyDigest(key)}.json`);
-};
-const writeFileAtomic = (path, record) => {
-	const dir = dirname(path);
-	mkdirSync(dir, { recursive: true });
-	const tmp = `${path}.${process.pid}.tmp`;
-	writeFileSync(tmp, JSON.stringify(record, null, 2), "utf8");
-	if (!isWindows) chmodSync(tmp, 384);
-	renameSync(tmp, path);
-	if (!isWindows) try {
-		chmodSync(dir, 448);
-	} catch {}
-};
-const loadToken = (path) => {
-	try {
-		const raw = JSON.parse(readFileSync(path, "utf8"));
-		if (typeof raw?.accessToken === "string") return raw;
-	} catch {}
-};
-const saveToken = (path, record, logger = silentLogger) => {
-	try {
-		writeFileAtomic(path, record);
-		logger.debug("token_persisted");
-	} catch (err) {
-		logger.warn("token_persist_failed", { error: err instanceof Error ? err.message : String(err) });
-	}
-};
-const clearToken = (path, logger = silentLogger) => {
-	try {
-		rmSync(path, { force: true });
-		logger.debug("token_cleared");
-	} catch (err) {
-		logger.warn("token_clear_failed", { error: err instanceof Error ? err.message : String(err) });
-	}
-};
-//#endregion
-export { saveToken as a, grantCovers as c, escapeHtml as d, resolveTokenPath as i, requestedScope as l, configDir as n, generateCodeChallenge as o, loadToken as r, generateCodeVerifier as s, clearToken as t, supportedScopes as u };