chore: minify bundle, don't ship sourcemaps

We haven't found much use for sourcemaps in production, and we should probably minify the bundle anyway. This will also remove an dev only warnings react used to log.

.changeset/cold-pianos-bow.md

@@ -0,0 +1,7 @@
+---
+"wrangler": patch
+---
+
+chore: minify bundle, don't ship sourcemaps
+
+we haven't found much use for sourcemaps in production, and we should probably minify the bundle anyway. This will also remove an dev only warnings react used to log.

packages/wrangler/package.json

@@ -119,7 +119,7 @@
     "bundle": "node -r esbuild-register scripts/bundle.ts",
     "build": "npm run clean && npm run bundle",
     "prepublishOnly": "npm run build",
-    "start": "npm run bundle && NODE_OPTIONS=--enable-source-maps ./bin/wrangler.js",
+    "start": "SOURCEMAPS=true npm run bundle && NODE_OPTIONS=--enable-source-maps ./bin/wrangler.js",
     "test": "jest --silent=false --verbose=true",
     "test-watch": "npm run test -- --runInBand --testTimeout=50000 --watch"
   },

packages/wrangler/scripts/bundle.ts

@@ -11,7 +11,7 @@ async function run() {
     outdir: "./wrangler-dist",
     platform: "node",
     format: "cjs",
-    // minify: true, // TODO: enable this again
+    minify: true,
     external: [
       "fsevents",
       "esbuild",
@@ -23,7 +23,7 @@ async function run() {
       "@esbuild-plugins/node-globals-polyfill",
       "@esbuild-plugins/node-modules-polyfill",
     ],
-    sourcemap: true,
+    sourcemap: process.env.SOURCEMAPS === "true",
     inject: [path.join(__dirname, "../import_meta_url.js")],
     define: {
       "import.meta.url": "import_meta_url",
@@ -39,7 +39,7 @@ async function run() {
     format: "esm",
     minify: true,
     external: ["miniflare", "@miniflare/core"],
-    sourcemap: true,
+    sourcemap: process.env.SOURCEMAPS === "true",
   });
 }