fixup! allow WRANGLER_SEND_METRICS to override whether to report Wrangler crashes to Sentry

Author: petebacondarwin

.changeset/angry-apes-share.md

@@ -2,4 +2,4 @@
 "wrangler": patch
 ---
 
-allow WRANGLER_SEND_METRICS to override whether to report Wrangler crashes to Sentry
+Allow WRANGLER_SEND_ERROR_REPORTS env var to override whether to report Wrangler crashes to Sentry

packages/wrangler/src/__tests__/sentry.test.ts

@@ -123,7 +123,7 @@ describe("sentry", () => {
 			expect(sentryRequests?.length).toEqual(0);
 		});
 
-		it("should not hit sentry (or even ask) after reportable error if WRANGLER_SEND_METRICS is explicitly false", async () => {
+		it("should not hit sentry (or even ask) after reportable error if WRANGLER_SEND_ERROR_REPORTS is explicitly false", async () => {
 			// Trigger an API error
 			msw.use(
 				http.get(
@@ -138,7 +138,7 @@ describe("sentry", () => {
 				})
 			);
 			await expect(
-				runWrangler("whoami", { WRANGLER_SEND_METRICS: "false" })
+				runWrangler("whoami", { WRANGLER_SEND_ERROR_REPORTS: "false" })
 			).rejects.toMatchInlineSnapshot(`[TypeError: Failed to fetch]`);
 			expect(std.out).toMatchInlineSnapshot(`
 			"Getting User settings...
@@ -454,7 +454,7 @@ describe("sentry", () => {
 			});
 		});
 
-		it("should hit sentry after reportable error (without confirmation) if WRANGLER_SEND_METRICS is explicitly true", async () => {
+		it("should hit sentry after reportable error (without confirmation) if WRANGLER_SEND_ERROR_REPORTS is explicitly true", async () => {
 			// Trigger an API error
 			msw.use(
 				http.get(
@@ -469,7 +469,7 @@ describe("sentry", () => {
 				})
 			);
 			await expect(
-				runWrangler("whoami", { WRANGLER_SEND_METRICS: "true" })
+				runWrangler("whoami", { WRANGLER_SEND_ERROR_REPORTS: "true" })
 			).rejects.toMatchInlineSnapshot(`[TypeError: Failed to fetch]`);
 			expect(std.out).toMatchInlineSnapshot(`
 			"Getting User settings...

packages/wrangler/src/environment-variables/factory.ts

@@ -88,6 +88,8 @@ type VariableNames =
 	| "WRANGLER_C3_COMMAND"
 	/** Enable/disable telemetry data collection. */
 	| "WRANGLER_SEND_METRICS"
+	/** Enable/disable error reporting to Sentry. */
+	| "WRANGLER_SEND_ERROR_REPORTS"
 	/** CI branch name (internal use). */
 	| "WORKERS_CI_BRANCH"
 	/** CI tag matching configuration (internal use). */

packages/wrangler/src/environment-variables/misc-variables.ts

@@ -45,6 +45,14 @@ export const getWranglerSendMetricsFromEnv =
 		variableName: "WRANGLER_SEND_METRICS",
 	});
 
+/**
+ * `WRANGLER_SEND_ERROR_REPORTS` can override whether we attempt to send error reports to Sentry.
+ */
+export const getWranglerSendErrorReportsFromEnv =
+	getBooleanEnvironmentVariableFactory({
+		variableName: "WRANGLER_SEND_ERROR_REPORTS",
+	});
+
 /**
  * Set `WRANGLER_API_ENVIRONMENT` environment variable to "staging" to tell Wrangler to hit the staging APIs rather than production.
  */

packages/wrangler/src/sentry/index.ts

@@ -3,7 +3,7 @@ import { rejectedSyncPromise } from "@sentry/utils";
 import { fetch } from "undici";
 import { version as wranglerVersion } from "../../package.json";
 import { confirm } from "../dialogs";
-import { getWranglerSendMetricsFromEnv } from "../environment-variables/misc-variables";
+import { getWranglerSendErrorReportsFromEnv } from "../environment-variables/misc-variables";
 import { logger } from "../logger";
 import type { BaseTransportOptions, TransportRequest } from "@sentry/types";
 import type { RequestInit } from "undici";
@@ -151,10 +151,10 @@ export function addBreadcrumb(
 // consent if not already granted.
 export async function captureGlobalException(e: unknown) {
 	if (typeof SENTRY_DSN !== "undefined") {
-		const sendMetricsEnvVar = getWranglerSendMetricsFromEnv();
+		const sendErrorReportsEnvVar = getWranglerSendErrorReportsFromEnv();
 		sentryReportingAllowed =
-			sendMetricsEnvVar !== undefined
-				? sendMetricsEnvVar
+			sendErrorReportsEnvVar !== undefined
+				? sendErrorReportsEnvVar
 				: await confirm(
 						"Would you like to report this error to Cloudflare? Wrangler's output and the error details will be shared with the Wrangler team to help us diagnose and fix the issue.",
 						{ fallbackValue: false }