Enable Sentry Console integration (#8112)

* Enable Sentry Console integration

* Change message & fix tests

* Change message & fix tests

* Create few-bobcats-brake.md

Author: penalosa

.changeset/few-bobcats-brake.md

@@ -0,0 +1,5 @@
+---
+"wrangler": patch
+---
+
+When reporting errors to Sentry, Wrangler will now include the console output as additional metadata

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

@@ -64,7 +64,7 @@ describe("sentry", () => {
 				"Getting User settings...
 
 				If you think this is a bug then please create an issue at https://github.com/cloudflare/workers-sdk/issues/new/choose
-				? Would you like to report this error to Cloudflare?
+				? 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.
 				🤖 Using fallback value in non-interactive context: no"
 			`);
 			expect(sentryRequests?.length).toEqual(0);
@@ -103,7 +103,7 @@ describe("sentry", () => {
 				)
 			);
 			mockConfirm({
-				text: "Would you like to report this error to Cloudflare?",
+				text: "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.",
 				result: false,
 			});
 			await expect(runWrangler("whoami")).rejects.toMatchInlineSnapshot(
@@ -129,7 +129,7 @@ describe("sentry", () => {
 				)
 			);
 			mockConfirm({
-				text: "Would you like to report this error to Cloudflare?",
+				text: "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.",
 				result: true,
 			});
 			await expect(runWrangler("whoami")).rejects.toMatchInlineSnapshot(
@@ -381,6 +381,7 @@ describe("sentry", () => {
 				        "InboundFilters",
 				        "FunctionToString",
 				        "LinkedErrors",
+				        "Console",
 				        "OnUncaughtException",
 				        "OnUnhandledRejection",
 				        "ContextLines",

packages/wrangler/src/sentry/index.ts

@@ -18,9 +18,11 @@ const makeSentry10Transport = (options: BaseTransportOptions) => {
 	let eventQueue: [string, RequestInit][] = [];
 
 	const transportSentry10 = async (request: TransportRequest) => {
-		/* Adds helpful properties to the request body before we send it to our
-    proxy Worker. These properties can be parsed out from the NDJSON in
-    `request.body`, but it's easier and safer to just attach them here. */
+		/**
+		 * Adds helpful properties to the request body before we send it to our
+		 * proxy Worker. These properties can be parsed out from the NDJSON in
+		 * `request.body`, but it's easier and safer to just attach them here.
+		 */
 		const sentryWorkerPayload = {
 			envelope: request.body,
 			url: options.url,
@@ -84,7 +86,6 @@ const makeSentry10Transport = (options: BaseTransportOptions) => {
 };
 
 const disabledDefaultIntegrations = [
-	"Console", // Console logs may contain PII
 	"LocalVariables", // Local variables may contain tokens and PII
 	"Http", // Only captures method/URL/response status, but URL may contain PII
 	"Undici", // Same as "Http"
@@ -150,7 +151,7 @@ export function addBreadcrumb(
 export async function captureGlobalException(e: unknown) {
 	if (typeof SENTRY_DSN !== "undefined") {
 		sentryReportingAllowed = await confirm(
-			"Would you like to report this error to Cloudflare?",
+			"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 }
 		);