refactor: next (#78)

BREAKING CHANGE: minimum supported webpack version is `5`

Author: alexander-akait

.github/workflows/nodejs.yml

@@ -56,7 +56,7 @@ jobs:
       matrix:
         os: [ubuntu-latest, windows-latest, macos-latest]
         node-version: [10.x, 12.x, 14.x]
-        webpack-version: [4, latest]
+        webpack-version: [latest]
 
     runs-on: ${{ matrix.os }}
 

package-lock.json

@@ -39,36 +39,32 @@
   ],
   "peerDependencies": {
     "coffeescript": ">= 2.0.0",
-    "webpack": "^4.0.0 || ^5.0.0"
-  },
-  "dependencies": {
-    "loader-utils": "^2.0.0",
-    "schema-utils": "^3.0.0"
+    "webpack": "^5.0.0"
   },
   "devDependencies": {
-    "@babel/cli": "^7.11.6",
-    "@babel/core": "^7.11.6",
-    "@babel/preset-env": "^7.11.5",
+    "@babel/cli": "^7.12.10",
+    "@babel/core": "^7.12.10",
+    "@babel/preset-env": "^7.12.11",
     "@commitlint/cli": "^11.0.0",
     "@commitlint/config-conventional": "^11.0.0",
     "@webpack-contrib/defaults": "^6.3.0",
     "@webpack-contrib/eslint-config-webpack": "^3.0.0",
-    "babel-jest": "^26.5.2",
+    "babel-jest": "^26.6.3",
     "coffeescript": "^2.5.1",
-    "cross-env": "^7.0.2",
+    "cross-env": "^7.0.3",
     "del": "^6.0.0",
     "del-cli": "^3.0.1",
-    "eslint": "^7.10.0",
-    "eslint-config-prettier": "^6.12.0",
+    "eslint": "^7.16.0",
+    "eslint-config-prettier": "^7.1.0",
     "eslint-plugin-import": "^2.22.1",
-    "husky": "^4.3.0",
-    "jest": "^26.5.2",
-    "lint-staged": "^10.4.0",
+    "husky": "^4.3.6",
+    "jest": "^26.6.3",
+    "lint-staged": "^10.5.3",
     "memfs": "^3.2.0",
     "npm-run-all": "^4.1.5",
-    "prettier": "^2.1.2",
+    "prettier": "^2.2.1",
     "standard-version": "^9.0.0",
-    "webpack": "^4.44.2"
+    "webpack": "^5.11.0"
   },
   "keywords": [
     "webpack"

package.json

@@ -2,21 +2,13 @@
 	MIT License http://www.opensource.org/licenses/mit-license.php
 	Author Tobias Koppers @sokra
 */
-import { validate } from 'schema-utils';
-import { getOptions } from 'loader-utils';
 import coffeescript from 'coffeescript';
 
 import schema from './options.json';
 import CoffeeScriptError from './CoffeeScriptError';
 
 export default function loader(source) {
-  const options = getOptions(this);
-
-  validate(schema, options, {
-    name: 'CoffeeScript Loader',
-    baseDataPath: 'options',
-  });
-
+  const options = this.getOptions(schema);
   const callback = this.async();
   const useSourceMap =
     typeof options.sourceMap === 'boolean' ? options.sourceMap : this.sourceMap;

src/index.js

@@ -12,7 +12,7 @@ exports[`loader should generate an error on broken code: warnings 1`] = `Array [
 exports[`loader should throw an error on invalid CoffeeScript options: errors 1`] = `
 Array [
   "ModuleBuildError: Module build failed (from \`replaced original path\`):
-CoffeeScriptError: Error: Cannot find module '@babel/preset-env1' from ''",
+CoffeeScriptError: Error: Cannot resolve module '@babel/preset-env1' from paths [''] from /node_modules/@babel/core/lib/config/files/plugins.js",
 ]
 `;
 

test/__snapshots__/loader.test.js.snap

@@ -1,5 +1,5 @@
-export default (compiler) => {
-  return new Promise((resolve, reject) => {
+export default (compiler) =>
+  new Promise((resolve, reject) => {
     compiler.run((error, stats) => {
       if (error) {
         return reject(error);
@@ -8,4 +8,3 @@ export default (compiler) => {
       return resolve(stats);
     });
   });
-};

test/helpers/compile.js

@@ -1,5 +1,3 @@
 import normalizeErrors from './normalizeErrors';
 
-export default (stats) => {
-  return normalizeErrors(stats.compilation.errors);
-};
+export default (stats) => normalizeErrors(stats.compilation.errors);

test/helpers/getErrors.js

@@ -1,5 +1,3 @@
 import normalizeErrors from './normalizeErrors';
 
-export default (stats) => {
-  return normalizeErrors(stats.compilation.warnings);
-};
+export default (stats) => normalizeErrors(stats.compilation.warnings);

test/helpers/getWarnings.js

@@ -14,8 +14,7 @@ function removeCWD(str) {
     .replace(new RegExp(cwd, 'g'), '');
 }
 
-export default (errors) => {
-  return errors.map((error) =>
+export default (errors) =>
+  errors.map((error) =>
     removeCWD(error.toString().split('\n').slice(0, 2).join('\n'))
   );
-};