fix: use sourceMapFilename in source map plugins (#676)

Author: zamotany

packages/haul-core/src/preset/__tests__/__snapshots__/makeConfigFactory.test.ts.snap

@@ -104,6 +104,7 @@ Object {
         "libraryTarget": "this",
         "path": "<<REPLACED>>/packages/haul-core/src/preset/__tests__",
         "publicPath": "http://localhost:8081/",
+        "sourceMapFilename": "[file].map",
       },
       "plugins": Array [
         DefinePlugin {
@@ -174,6 +175,7 @@ Object {
         "libraryTarget": "this",
         "path": "<<REPLACED>>/packages/haul-core/src/preset/__tests__",
         "publicPath": "http://localhost:8081/",
+        "sourceMapFilename": "[file].map",
       },
       "plugins": Array [
         DefinePlugin {
@@ -239,6 +241,7 @@ Object {
         "globalObject": "this",
         "path": "<<REPLACED>>/packages/haul-core/src/preset/__tests__",
         "publicPath": "http://localhost:8081/",
+        "sourceMapFilename": "[file].map",
       },
       "plugins": Array [
         DefinePlugin {
@@ -397,6 +400,7 @@ Object {
         "libraryTarget": "this",
         "path": "<<REPLACED>>/packages/haul-core/src/preset/__tests__",
         "publicPath": "http://localhost:8081/",
+        "sourceMapFilename": "[file].map",
       },
       "plugins": Array [
         DefinePlugin {
@@ -461,6 +465,7 @@ Object {
         "libraryTarget": "this",
         "path": "<<REPLACED>>/packages/haul-core/src/preset/__tests__",
         "publicPath": "http://localhost:8081/",
+        "sourceMapFilename": "[file].map",
       },
       "plugins": Array [
         DefinePlugin {
@@ -520,6 +525,7 @@ Object {
         "globalObject": "this",
         "path": "<<REPLACED>>/packages/haul-core/src/preset/__tests__",
         "publicPath": "http://localhost:8081/",
+        "sourceMapFilename": "[file].map",
       },
       "plugins": Array [
         DefinePlugin {
@@ -628,6 +634,7 @@ Object {
         "globalObject": "this",
         "path": "<<REPLACED>>/packages/haul-core/src/preset/__tests__",
         "publicPath": "http://localhost:8081/",
+        "sourceMapFilename": "[file].map",
       },
       "plugins": Array [
         DefinePlugin {

packages/haul-core/src/preset/makeConfigFactory.ts

@@ -189,12 +189,15 @@ export default function makeConfigFactory(getDefaultConfig: GetDefaultConfig) {
                 webpackConfig.output!.path!,
                 path.join(normalizedBundleConfig.root, env.sourcemapOutput)
               );
+        } else {
+          webpackConfig.output!.sourceMapFilename = '[file].map';
         }
 
         webpackConfig.plugins = (webpackConfig.plugins || [])
           .concat(getSourceMapPlugin(
             normalizedBundleConfig,
-            normalizedServerConfig
+            normalizedServerConfig,
+            webpackConfig.output!.sourceMapFilename
           ) as webpack.Plugin)
           .filter(Boolean);
 

packages/haul-core/src/preset/utils/getSourceMapPlugin.ts

@@ -6,7 +6,8 @@ import {
 
 export default function getSourceMapPlugin(
   bundleConfig: NormalizedBundleConfig,
-  serverConfig: NormalizedServerConfig
+  serverConfig: NormalizedServerConfig,
+  sourceMapFilename: string
 ) {
   const baseOptions = {
     test: /\.(js|jsx|css|ts|tsx|(js)?bundle)($|\?)/i,
@@ -21,7 +22,7 @@ export default function getSourceMapPlugin(
   } else if (bundleConfig.sourceMap) {
     return new webpack.SourceMapDevToolPlugin({
       ...baseOptions,
-      filename: '[file].map',
+      filename: sourceMapFilename,
       moduleFilenameTemplate: '[absolute-resource-path]',
     });
   }