@togglhq/cli 1.8.0 → 1.8.1

package/build/cli.js

@@ -1,5 +1,5 @@
 #!/usr/bin/env node
-import { a as CommanderError, i as shutdownCliTelemetry, n as captureCliException, r as initCliTelemetry, t as createProgram } from "./program-B6EuV2vO.js";
+import { a as CommanderError, i as shutdownCliTelemetry, n as captureCliException, r as initCliTelemetry, t as createProgram } from "./program-CH5epPVt.js";
 import { readFileSync } from "node:fs";
 import { dirname, join } from "node:path";
 import { fileURLToPath } from "node:url";

package/build/index.js

@@ -1,2 +1,2 @@
-import { t as createProgram } from "./program-B6EuV2vO.js";
+import { t as createProgram } from "./program-CH5epPVt.js";
 export { createProgram };

package/build/{program-B6EuV2vO.js → program-CH5epPVt.js}

@@ -12618,6 +12618,51 @@ function switchCliWorkspace(workspaceId, profile) {
 	saveDeveloperToolsConfig(dev);
 	return target;
 }
+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
@@ -12629,15 +12674,14 @@ function switchCliWorkspace(workspaceId, 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
@@ -12719,7 +12763,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,
@@ -12727,7 +12771,9 @@ function captureToolInvocation(event) {
 		client,
 		session_id: sessionId,
 		package_version: packageVersion
-	}, 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);
@@ -24981,6 +25027,7 @@ function envEnabled(raw) {
 }
 let sentryEnabled = false;
 let productAnalyticsEnabled = false;
+let lastCommandError;
 function initCliTelemetry(packageVersion) {
 	productAnalyticsEnabled = initProductAnalytics({
 		client: "cli",
@@ -25014,17 +25061,17 @@ function initCliTelemetry(packageVersion) {
 */
 function recordCliCommand(commandName, action, success) {
 	if (!productAnalyticsEnabled) return;
+	const error = success ? void 0 : lastCommandError;
+	lastCommandError = void 0;
 	captureToolInvocation({
 		toolName: commandName,
 		action,
-		outcome: success ? "success" : "error"
+		outcome: success ? "success" : "error",
+		error
 	});
 }
 function classifyCliSentryError(error) {
-	const message = error instanceof Error ? error.message : String(error);
-	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.command_error";
+	return classifyErrorCategory("cli", error);
 }
 function toSentrySafeCliError(error) {
 	const category = classifyCliSentryError(error);
@@ -25034,6 +25081,7 @@ function toSentrySafeCliError(error) {
 	return safe;
 }
 function captureCliException(error) {
+	lastCommandError = error;
 	if (!sentryEnabled) return;
 	const category = classifyCliSentryError(error);
 	Sentry.withScope((scope) => {
@@ -25048,6 +25096,7 @@ async function shutdownCliTelemetry() {
 	if (tasks.length > 0) await Promise.allSettled(tasks);
 	sentryEnabled = false;
 	productAnalyticsEnabled = false;
+	lastCommandError = void 0;
 }
 //#endregion
 //#region src/output.ts
@@ -25624,4 +25673,4 @@ Common commands:
 //#endregion
 export { CommanderError as a, shutdownCliTelemetry as i, captureCliException as n, initCliTelemetry as r, createProgram as t };
 
-//# sourceMappingURL=program-B6EuV2vO.js.map
+//# sourceMappingURL=program-CH5epPVt.js.map

package/build/program.js

@@ -1,2 +1,2 @@
-import { t as createProgram } from "./program-B6EuV2vO.js";
+import { t as createProgram } from "./program-CH5epPVt.js";
 export { createProgram };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@togglhq/cli",
-  "version": "1.8.0",
+  "version": "1.8.1",
   "description": "Toggl CLI for Toggl 2.0 API operations.",
   "license": "UNLICENSED",
   "type": "module",