@togglhq/mcp 1.8.0 → 1.8.1

package/build/cli.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { a as initProductAnalytics, c as shutdownProductAnalytics, i as getProductAnalyticsDistinctId, n as runMcpCli, o as redactUnsafeSentryMessages, r as captureToolInvocation, s as resolveProductAnalyticsDistinctId } from "./src-BHdIWjE4.js";
+import { a as initProductAnalytics, c as shutdownProductAnalytics, i as getProductAnalyticsDistinctId, n as runMcpCli, o as redactUnsafeSentryMessages, r as captureToolInvocation, s as resolveProductAnalyticsDistinctId } from "./src-Dh4qJKIU.js";
 import * as Sentry from "@sentry/node";
 //#region src/analytics-tool-metadata.ts
 /**
@@ -135,12 +135,15 @@ function wrapServerWithProductAnalytics(server) {
 			const action = readToolAction(handlerArgs);
 			let distinctId = resolveProductAnalyticsDistinctId("mcp") ?? getProductAnalyticsDistinctId();
 			let outcome = "error";
+			let result;
+			let thrown;
 			try {
-				const result = await toolHandler(...handlerArgs);
+				result = await toolHandler(...handlerArgs);
 				outcome = classifyToolResult(result);
 				if (IDENTITY_CHANGING_MCP_TOOLS.has(name) && outcome === "success") distinctId = resolveProductAnalyticsDistinctId("mcp") ?? distinctId;
 				return result;
 			} catch (error) {
+				thrown = error;
 				outcome = "error";
 				throw error;
 			} finally {
@@ -148,7 +151,8 @@ function wrapServerWithProductAnalytics(server) {
 					toolName: name,
 					action,
 					outcome,
-					distinctId
+					distinctId,
+					error: outcome === "error" ? thrown ?? result : void 0
 				});
 			}
 		};
@@ -161,7 +165,7 @@ function wrapServerWithProductAnalytics(server) {
 }
 //#endregion
 //#region src/cli.ts
-const packageVersion = "1.8.0";
+const packageVersion = "1.8.1";
 let sentryEnabled = false;
 let productAnalyticsEnabled = false;
 runMcpCli({

package/build/index.js

@@ -1,3 +1,3 @@
 #!/usr/bin/env node
-import { n as runMcpCli, t as bootstrapServer } from "./src-BHdIWjE4.js";
+import { n as runMcpCli, t as bootstrapServer } from "./src-Dh4qJKIU.js";
 export { bootstrapServer, runMcpCli };

package/build/{src-BHdIWjE4.js → src-Dh4qJKIU.js}

@@ -32689,6 +32689,51 @@ function listCachedMcpWorkspaces(profile) {
 		active: workspace.workspace_id === auth.workspace_id
 	}));
 }
+const CLI_GENERIC_ERROR_CATEGORY = "cli.command_error";
+const MCP_GENERIC_ERROR_CATEGORY = "mcp.tool_error";
+const ERROR_CATEGORY_SET = new Set([
+	"cli.validation_error",
+	"cli.api_error",
+	CLI_GENERIC_ERROR_CATEGORY,
+	"mcp.validation_error",
+	"mcp.api_error",
+	"mcp.auth_error",
+	MCP_GENERIC_ERROR_CATEGORY
+]);
+function isDeclaredErrorCategory(value) {
+	return typeof value === "string" && ERROR_CATEGORY_SET.has(value);
+}
+function firstToolResultText(source) {
+	if (!source || typeof source !== "object") return null;
+	const content = source.content;
+	if (!Array.isArray(content)) return null;
+	const first = content[0];
+	if (first?.type === "text" && typeof first.text === "string") return first.text;
+	return null;
+}
+function errorTextForClassification(source) {
+	if (source instanceof Error) return source.message;
+	if (typeof source === "string") return source;
+	const toolText = firstToolResultText(source);
+	if (toolText !== null) return toolText;
+	return "";
+}
+function classifyCli(message) {
+	if (message.startsWith("Invalid input:") || message.startsWith("Invalid input ")) return "cli.validation_error";
+	if (/^\d{3}:\s/.test(message)) return "cli.api_error";
+	return CLI_GENERIC_ERROR_CATEGORY;
+}
+function classifyMcp(message) {
+	if (message.startsWith("Invalid input:") || message.startsWith("Invalid input ")) return "mcp.validation_error";
+	if (message.startsWith("API Error (")) return "mcp.api_error";
+	if (message.startsWith("Authentication failed:") || message.startsWith("Not authenticated.")) return "mcp.auth_error";
+	return MCP_GENERIC_ERROR_CATEGORY;
+}
+function classifyErrorCategory(client, source) {
+	const message = errorTextForClassification(source);
+	if (isDeclaredErrorCategory(message) && message.startsWith(`${client}.`)) return message;
+	return client === "cli" ? classifyCli(message) : classifyMcp(message);
+}
 /**
 * Exception messages are the one channel `sendDefaultPii: false` does not cover,
 * and this repo's errors put user data in them: validation failures embed the
@@ -32700,15 +32745,14 @@ function listCachedMcpWorkspaces(profile) {
 * Shared between the CLI and the public MCP build so the two cannot drift.
 */
 /**
-* Fixed categories are `<surface>.<snake_case>` — anchored at both ends, because
-* a prefix test would pass anything that merely *starts* that way. A real message
-* like `cli.ts:12: unexpected token "secret"` begins with `cli.` and would
-* otherwise sail through unredacted.
+* Fixed categories are the seven declared `<surface>.<snake_case>` values, not
+* a `cli|mcp` prefix regex. A prefix or regex test would pass a caught message
+* like `cli.customer_name`. A real filename message such as
+* `cli.ts:12: unexpected token "secret"` would also begin with `cli.`.
 */
-const SAFE_CATEGORY_PATTERN = /^(?:cli|mcp)\.[a-z][a-z0-9_]*$/;
 const REDACTED_SENTRY_MESSAGE = "redacted";
 function isSafeSentryMessage(value) {
-	return typeof value === "string" && SAFE_CATEGORY_PATTERN.test(value);
+	return isDeclaredErrorCategory(value);
 }
 /**
 * Replace every exception message that is not a known category, and any top-level
@@ -32793,7 +32837,7 @@ function normalizeAction(action) {
 /** Strict allowlist — never accept args/results. */
 function captureToolInvocation(event) {
 	if (!enabled || !apiKey) return;
-	const task = sendCapture(TOOL_INVOCATION_EVENT, {
+	const properties = {
 		tool_name: event.toolName,
 		action: normalizeAction(event.action),
 		outcome: event.outcome,
@@ -32801,7 +32845,9 @@ function captureToolInvocation(event) {
 		client,
 		session_id: sessionId,
 		package_version: packageVersion$1
-	}, event.distinctId).finally(() => {
+	};
+	if (event.outcome === "error") properties.error_category = classifyErrorCategory(client, event.error);
+	const task = sendCapture(TOOL_INVOCATION_EVENT, properties, event.distinctId).finally(() => {
 		pending.delete(task);
 	});
 	pending.add(task);
@@ -45440,7 +45486,7 @@ import_main.default.config({
 	quiet: true,
 	ignore: ["MISSING_ENV_FILE"]
 });
-const packageVersion = "1.8.0";
+const packageVersion = "1.8.1";
 function resolvePublicUrls() {
 	return {
 		focusApiUrl: process.env.TOGGL_FOCUS_API_URL ?? "https://focus.toggl.com",
@@ -45504,4 +45550,4 @@ function bootstrapServer(options = {}) {
 //#endregion
 export { initProductAnalytics as a, shutdownProductAnalytics as c, getProductAnalyticsDistinctId as i, runMcpCli as n, redactUnsafeSentryMessages as o, captureToolInvocation as r, resolveProductAnalyticsDistinctId as s, bootstrapServer as t };
 
-//# sourceMappingURL=src-BHdIWjE4.js.map
+//# sourceMappingURL=src-Dh4qJKIU.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@togglhq/mcp",
-  "version": "1.8.0",
+  "version": "1.8.1",
   "description": "Toggl 2.0 MCP server for Claude Code, Claude Desktop, and other MCP clients.",
   "license": "UNLICENSED",
   "type": "module",