moving prod sourcemap to separate files, fix #319

cli/plasmo/index.mjs

@@ -16,8 +16,7 @@ const commonConfig = {
 async function main() {
   const config = await fse.readJson("package.json")
   const define = {
-    "process.env.APP_VERSION": `"${config.version}"`,
-    "process.env.LAB": watch
+    "process.env.APP_VERSION": `"${config.version}"`
   }
 
   await build({

cli/plasmo/src/features/helpers/create-parcel-bundler.ts

@@ -1,8 +1,10 @@
 import { Parcel } from "@parcel/core"
 import ParcelFS from "@parcel/fs"
 import ParcelPM from "@parcel/package-manager"
-import { emptyDir, readJson } from "fs-extra"
-import { resolve } from "path"
+import { emptyDir, readJson, writeJson } from "fs-extra"
+import { dirname, join, resolve } from "path"
+
+import { hasFlag } from "@plasmo/utils/flags"
 
 import type { CommonPath } from "~features/extension-devtools/common-path"
 
@@ -35,16 +37,23 @@ export const createParcelBuilder = async (
   )
 
   const baseConfig = require.resolve("@plasmohq/parcel-config")
+
+  const runConfig = join(dirname(baseConfig), "run.json")
+
   const configJson = await readJson(baseConfig)
 
-  console.log(configJson)
+  if (hasFlag("--bundle-buddy")) {
+    configJson.reporters = ["...", "@parcel/reporter-bundle-buddy"]
+  }
+
+  await writeJson(runConfig, configJson)
 
   const bundler = new Parcel({
     inputFS,
     packageManager,
     entries: commonPath.entryManifestPath,
     cacheDir: resolve(commonPath.cacheDirectory, "parcel"),
-    config: require.resolve("@plasmohq/parcel-config"),
+    config: runConfig,
     shouldAutoInstall: true,
     ...options
   })

packages/parcel-config/.gitignore

@@ -0,0 +1 @@
+run.json

packages/parcel-transformer-manifest/src/index.ts

@@ -7,6 +7,7 @@
  */
 import { parse } from "@mischnic/json-sourcemap"
 import { Transformer } from "@parcel/plugin"
+import type { TargetSourceMapOptions } from "@parcel/types"
 import { validateSchema } from "@parcel/utils"
 
 import { vLog } from "@plasmo/utils/logging"
@@ -46,6 +47,18 @@ export default new Transformer({
     vLog("@plasmohq/parcel-transformer-manifest")
     // Set environment to browser, since web extensions are always used in
     // browsers, and because it avoids delegating extra config to the user
+
+    const sourceMapConfig: TargetSourceMapOptions =
+      options.mode === "development"
+        ? {
+            inline: true,
+            inlineSources: true
+          }
+        : {
+            inline: false,
+            inlineSources: false
+          }
+
     asset.setEnvironment({
       context: "browser",
       outputFormat:
@@ -57,8 +70,7 @@ export default new Transformer({
       },
       sourceMap: asset.env.sourceMap && {
         ...asset.env.sourceMap,
-        inline: true,
-        inlineSources: true
+        ...sourceMapConfig
       },
       includeNodeModules: asset.env.includeNodeModules,
       sourceType: asset.env.sourceType,