From 106484b58dc498999aff34d5e0c7fad65e3ac878 Mon Sep 17 00:00:00 2001
From: hiroki osame <hiroki.osame@gmail.com>
Date: Thu, 12 Jan 2023 21:04:51 -0500
Subject: [PATCH] fix: use older webpack-sources for WP4 and built-in one for
 WP5 (#300)

---
 package.json         |  4 ++--
 pnpm-lock.yaml       | 25 ++++++++++++-------------
 src/minify-plugin.ts | 13 +++++++++++--
 3 files changed, 25 insertions(+), 17 deletions(-)

diff --git a/package.json b/package.json
index ab6499c9..7a465068 100644
--- a/package.json
+++ b/package.json
@@ -36,14 +36,14 @@
 		"json5": "^2.2.0",
 		"loader-utils": "^2.0.0",
 		"tapable": "^2.2.0",
-		"webpack-sources": "^2.2.0"
+		"webpack-sources": "^1.4.3"
 	},
 	"devDependencies": {
 		"@pvtnbr/eslint-config": "^0.33.0",
 		"@types/loader-utils": "^2.0.3",
 		"@types/node": "^18.11.18",
 		"@types/webpack": "^4.41.33",
-		"@types/webpack-sources": "^3.2.0",
+		"@types/webpack-sources": "^0.1.9",
 		"css-loader": "^5.2.7",
 		"eslint": "^8.31.0",
 		"manten": "^0.6.0",
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml
index 5b367a43..e3072029 100644
--- a/pnpm-lock.yaml
+++ b/pnpm-lock.yaml
@@ -5,7 +5,7 @@ specifiers:
   '@types/loader-utils': ^2.0.3
   '@types/node': ^18.11.18
   '@types/webpack': ^4.41.33
-  '@types/webpack-sources': ^3.2.0
+  '@types/webpack-sources': ^0.1.9
   css-loader: ^5.2.7
   esbuild: ^0.16.17
   eslint: ^8.31.0
@@ -19,7 +19,7 @@ specifiers:
   tsx: ^3.12.2
   typescript: ^4.9.4
   webpack: ^4.44.2
-  webpack-sources: ^2.2.0
+  webpack-sources: ^1.4.3
   webpack-test-utils: ^2.1.0
   webpack5: npm:webpack@^5.0.0
 
@@ -29,14 +29,14 @@ dependencies:
   json5: 2.2.0
   loader-utils: 2.0.0
   tapable: 2.2.0
-  webpack-sources: 2.3.1
+  webpack-sources: 1.4.3
 
 devDependencies:
   '@pvtnbr/eslint-config': 0.33.0_iukboom6ndih5an6iafl45j2fe
   '@types/loader-utils': 2.0.3
   '@types/node': 18.11.18
   '@types/webpack': 4.41.33
-  '@types/webpack-sources': 3.2.0
+  '@types/webpack-sources': 0.1.9
   css-loader: 5.2.7_webpack@4.46.0
   eslint: 8.31.0
   manten: 0.6.0
@@ -561,6 +561,14 @@ packages:
     resolution: {integrity: sha512-PBjIUxZHOuj0R15/xuwJYjFi+KZdNFrehocChv4g5hu6aFroHue8m0lBP0POdK2nKzbw0cgV1mws8+V/JAcEkQ==}
     dev: true
 
+  /@types/webpack-sources/0.1.9:
+    resolution: {integrity: sha512-bvzMnzqoK16PQIC8AYHNdW45eREJQMd6WG/msQWX5V2+vZmODCOPb4TJcbgRljTZZTwTM4wUMcsI8FftNA7new==}
+    dependencies:
+      '@types/node': 18.11.18
+      '@types/source-list-map': 0.1.2
+      source-map: 0.6.1
+    dev: true
+
   /@types/webpack-sources/3.2.0:
     resolution: {integrity: sha512-Ft7YH3lEVRQ6ls8k4Ff1oB4jN6oy/XmU6tQISKdhfh+1mR+viZFphS6WL0IrtDOzvefmJg5a0s7ZQoRXwqTEFg==}
     dependencies:
@@ -5575,15 +5583,6 @@ packages:
     dependencies:
       source-list-map: 2.0.1
       source-map: 0.6.1
-    dev: true
-
-  /webpack-sources/2.3.1:
-    resolution: {integrity: sha512-y9EI9AO42JjEcrTJFOYmVywVZdKVUfOvDUPsJea5GIr1JOEGFVqwlY2K098fFoIjOkDzHn2AjRvM8dsBZu+gCA==}
-    engines: {node: '>=10.13.0'}
-    dependencies:
-      source-list-map: 2.0.1
-      source-map: 0.6.1
-    dev: false
 
   /webpack-sources/3.2.3:
     resolution: {integrity: sha512-/DyMEOrDgLKKIG0fmvtz+4dUX/3Ghozwgm6iPp8KRhvn+eQf9+Q7GWxVNMk3+uCPWfdXYC4ExGBckIXdFEfH1w==}
diff --git a/src/minify-plugin.ts b/src/minify-plugin.ts
index 2b73d5a9..f296bcd9 100644
--- a/src/minify-plugin.ts
+++ b/src/minify-plugin.ts
@@ -1,5 +1,8 @@
 import { transform as defaultEsbuildTransform } from 'esbuild';
-import { RawSource, SourceMapSource } from 'webpack-sources';
+import {
+	RawSource as WP4RawSource,
+	SourceMapSource as WP4SourceMapSource,
+} from 'webpack-sources';
 import webpack from 'webpack';
 import type {
 	SyncHook, SyncBailHook, AsyncSeriesHook, HookMap,
@@ -107,7 +110,13 @@ class ESBuildMinifyPlugin {
 	private async transformAssets(
 		compilation: Compilation,
 	): Promise<void> {
-		const { options: { devtool } } = compilation.compiler;
+		const { compiler } = compilation;
+		const { options: { devtool } } = compiler;
+
+		// @ts-expect-error Only exists on Webpack 5
+		const sources = compiler.webpack?.sources;
+		const SourceMapSource = (sources ? sources.SourceMapSource : WP4SourceMapSource);
+		const RawSource = (sources ? sources.RawSource : WP4RawSource);
 
 		const sourcemap = (
 			// TODO: drop support for esbuild sourcemap in future so it all goes through WP API