Remove source map annotation to prevent browser warnings (#1364)

This is necessary because we don't expose the sourcemap files to the browser in production

.changeset/tender-singers-prove.md

@@ -0,0 +1,5 @@
+---
+'@shopify/cli-hydrogen': patch
+---
+
+Remove sourcemap annotations from client bundles. This prevents errors showing up in the devtools when the sourcemaps fail to load.

packages/cli/src/commands/hydrogen/build.ts

@@ -11,8 +11,9 @@ import {
   copyFile,
   rmdir,
   glob,
-  removeFile,
   fileExists,
+  readFile,
+  writeFile,
 } from '@shopify/cli-kit/node/fs';
 import {resolvePath, relativePath, joinPath} from '@shopify/cli-kit/node/path';
 import {getPackageManager} from '@shopify/cli-kit/node/node-package-manager';
@@ -186,6 +187,10 @@ export async function runBuild({
     }
   }
 
+  if (process.env.NODE_ENV !== 'development') {
+    await cleanClientSourcemaps(buildPathClient);
+  }
+
   // The Remix compiler hangs due to a bug in ESBuild:
   // https://github.com/evanw/esbuild/issues/2727
   // The actual build has already finished so we can kill the process.
@@ -194,6 +199,20 @@ export async function runBuild({
   }
 }
 
+async function cleanClientSourcemaps(buildPathClient: string) {
+  const bundleFiles = await glob(joinPath(buildPathClient, '**/*.js'));
+
+  await Promise.all(
+    bundleFiles.map(async (filePath) => {
+      const file = await readFile(filePath);
+      return await writeFile(
+        filePath,
+        file.replace(/\/\/# sourceMappingURL=.+\.js\.map$/gm, ''),
+      );
+    }),
+  );
+}
+
 async function writeBundleAnalysis(
   buildPath: string,
   root: string,