fix: respect sourceMap option (#141)

* fix: default esbuild sourceMap option * refactor: set `sourceMap` default to true
nitrojs · Apr 22, 2022 · e5ab741 · e5ab741
1 parent 05c4104
commit e5ab741
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/src/options.ts b/src/options.ts
@@ -62,6 +62,7 @@ const NitroDefaults: NitroConfig = {
   analyze: false,
   moduleSideEffects: ['unenv/runtime/polyfill/'],
   replace: {},
+  sourceMap: true,
 
   // Advanced
   typescript: {

diff --git a/src/rollup/config.ts b/src/rollup/config.ts
@@ -151,7 +151,7 @@ export const getRollupConfig = (nitro: Nitro) => {
   // esbuild
   rollupConfig.plugins.push(esbuild({
     target: 'es2019',
-    sourceMap: true,
+    sourceMap: nitro.options.sourceMap,
     ...nitro.options.esbuild?.options
   }))
 

diff --git a/src/rollup/plugins/esbuild.ts b/src/rollup/plugins/esbuild.ts
@@ -86,7 +86,7 @@ export function esbuild (options: Options = {}): Plugin {
         loader,
         target,
         define: options.define,
-        sourcemap: options.sourceMap !== false,
+        sourcemap: options.sourceMap,
         sourcefile: id
       })