fix: Warning of missing source maps in website Chrome DevTools logs

Loading source maps has never worked for the content scripts. It failed silently until Chrome 80, but now a warning is displayed. It's currently unclear if/when source map loading would be supported. See https://chromium-review.googlesource.com/c/chromium/src/+/2141899 It would be possible, with additional rollup plugins, to inline the TS source maps, but that would increase the compiled .js file sizes a lot (over 100kb for content.js). Turning off source map for the content script. Signed-off-by: Tanel Metsar <[email protected]>
web-eid · Oct 24, 2022 · dfa7169 · dfa7169
1 parent ea16b9f
commit dfa7169
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 6 deletions.
diff --git a/rollup.config.js b/rollup.config.js
@@ -23,7 +23,7 @@ export default [
       {
         file:      `dist/firefox/${name}.js`,
         format:    "iife",
-        sourcemap: true,
+        sourcemap: name === "background",
       },
     ],
 
@@ -57,7 +57,7 @@ export default [
       {
         file:      `dist/safari/${name}.js`,
         format:    "iife",
-        sourcemap: true,
+        sourcemap: name === "background",
       },
     ],
 
@@ -88,9 +88,8 @@ export default [
 
     output: [
       {
-        file:      "dist/firefox/token-signing-page-script.js",
-        format:    "iife",
-        sourcemap: true,
+        file:   "dist/firefox/token-signing-page-script.js",
+        format: "iife",
       },
     ],
 

diff --git a/tsconfig.json b/tsconfig.json
@@ -6,7 +6,6 @@
     "declaration": true,
     "declarationMap": true,
     "moduleResolution": "node",
-    "sourceMap": true,
     "strict": true,
     "outDir": "dist",
     "baseUrl": ".",