Fix many tests, add hot-patching to get the ball rolling

package.json

@@ -33,8 +33,5 @@
   },
   "engines": {
     "node": ">=12"
-  },
-  "resolutions": {
-    "**/eslint": "8.0.0-beta.1"
   }
 }

packages/crafty-preset-babel/package.json

@@ -21,7 +21,7 @@
     "babel-loader": "8.2.2",
     "gulp-babel": "8.0.0",
     "gulp-concat": "2.6.1",
-    "gulp-eslint7": "0.3.2",
+    "gulp-eslint-new": "0.4.0",
     "gulp-newer": "1.4.0",
     "gulp-sourcemaps": "3.0.0",
     "gulp-terser": "2.1.0"

packages/crafty-preset-babel/src/gulp.js

@@ -18,8 +18,12 @@ module.exports = function createTask(crafty, bundle, StreamHandler) {
     const {
       toTempFile
     } = require("@swissquote/crafty-preset-eslint/src/eslintConfigurator");
-    const eslint = require("gulp-eslint7");
-    stream.add(eslint(toTempFile(crafty.config.eslint))).add(eslint.format());
+    const eslint = require("gulp-eslint-new");
+    stream.add(eslint(
+      {
+        overrideConfigFile: toTempFile(crafty.config.eslint)
+      }
+    )).add(eslint.format());
 
     // Fail the build if we have linting
     // errors and we build directly

packages/crafty-preset-eslint/package.json

@@ -17,7 +17,7 @@
     "@swissquote/rollup-plugin-eslint": "1.16.3",
     "copy-anything": "2.0.3",
     "debug": "^4.1.0",
-    "eslint": "8.0.0-beta.1",
+    "eslint": "8.0.1",
     "eslint-webpack-plugin": "3.0.1",
     "merge-anything": "4.0.1",
     "resolve-from": "5.0.0",

packages/crafty-preset-eslint/src/commands/jsLint.js

@@ -20,4 +20,4 @@ process.argv.push(require.resolve("../resolver.js"))
 process.argv.push("--config");
 process.argv.push(tmpfile);
 
-require("eslint/bin/eslint");
+require(".bin/eslint");

packages/crafty-preset-eslint/src/index.js

@@ -97,8 +97,7 @@ module.exports = {
     chain.plugin("lint-js").use(require.resolve("eslint-webpack-plugin"), [
       {
         extensions: crafty.config.eslintExtensions,
-        overrideConfigFile: toTempFile(crafty.config.eslint),
-        resolver: require.resolve("@swissquote/crafty-preset-eslint/src/resolver.js")
+        overrideConfigFile: toTempFile(crafty.config.eslint)
       }
     ]);
   }

packages/crafty-preset-eslint/src/resolver.js

@@ -3,7 +3,7 @@ const {
 } = require("@eslint/eslintrc");
 
 function resolve(moduleName, relativeToPath) {
-  console.log("resolving", { moduleName, relativeToPath });
+  //console.log({moduleName, relativeToPath});
   try {
     // First check for the module relative to the current location
     return ModuleResolver.resolve(moduleName, __filename);

packages/crafty-preset-swc/package.json

@@ -20,7 +20,7 @@
     "debug": "^4.1.0",
     "find-up": "5.0.0",
     "gulp-concat": "2.6.1",
-    "gulp-eslint7": "0.3.2",
+    "gulp-eslint-new": "0.4.0",
     "gulp-newer": "1.4.0",
     "gulp-sourcemaps": "3.0.0",
     "gulp-swc": "1.1.1",

packages/crafty-preset-swc/src/gulp.js

@@ -18,8 +18,12 @@ module.exports = function createTask(crafty, bundle, StreamHandler) {
     const {
       toTempFile
     } = require("@swissquote/crafty-preset-eslint/src/eslintConfigurator");
-    const eslint = require("gulp-eslint7");
-    stream.add(eslint(toTempFile(crafty.config.eslint))).add(eslint.format());
+    const eslint = require("gulp-eslint-new");
+    stream.add(eslint(
+      {
+        overrideConfigFile: toTempFile(crafty.config.eslint)
+      }
+    )).add(eslint.format());
 
     // Fail the build if we have linting
     // errors and we build directly

packages/crafty-preset-typescript/package.json

@@ -21,7 +21,7 @@
     "find-up": "5.0.0",
     "fork-ts-checker-webpack-plugin": "6.3.4",
     "gulp-babel": "8.0.0",
-    "gulp-eslint7": "0.3.2",
+    "gulp-eslint-new": "0.4.0",
     "gulp-sourcemaps": "3.0.0",
     "gulp-typescript": "5.0.1",
     "tmp": "0.2.1",

packages/crafty-preset-typescript/src/gulp.js

@@ -18,9 +18,13 @@ module.exports = function createTask(crafty, bundle, StreamHandler) {
     const {
       toTempFile
     } = require("@swissquote/crafty-preset-eslint/src/eslintConfigurator");
-    const eslint = require("gulp-eslint7");
+    const eslint = require("gulp-eslint-new");
     stream
-      .add(eslint({ configFile: toTempFile(crafty.config.eslint) }))
+      .add(eslint(
+        {
+          overrideConfigFile: toTempFile(crafty.config.eslint)
+        }
+      ))
       .add(eslint.format());
 
     // Fail the build if we have linting

packages/eslint-plugin-swissquote/package.json

@@ -37,7 +37,7 @@
     "eslint": "^6.0.0 | ^7.0.0 | ^8.0.0"
   },
   "devDependencies": {
-    "eslint": "8.0.0-beta.1",
+    "eslint": "8.0.1",
     "jest": "27.2.5"
   },
   "engines": {

packages/integration/__tests__/__snapshots__/crafty-preset-babel.js.snap

@@ -17,9 +17,6 @@ exports[`Generates IDE Helper 2`] = `
 // This file is generated to improve IDE Integration
 // You don't need to commit this file, nor need it to run \`crafty build\`
 
-// Fix module resolution
-require(\\"__PATH__/packages/crafty-preset-eslint/src/patchModuleResolver.js\\");
-
 module.exports = {
     \\"plugins\\": [
         \\"@swissquote/swissquote\\"

packages/integration/__tests__/__snapshots__/crafty-preset-swc.js.snap

@@ -17,9 +17,6 @@ exports[`Generates IDE Helper 2`] = `
 // This file is generated to improve IDE Integration
 // You don't need to commit this file, nor need it to run \`crafty build\`
 
-// Fix module resolution
-require(\\"__PATH__/packages/crafty-preset-eslint/src/patchModuleResolver.js\\");
-
 module.exports = {
     \\"plugins\\": [
         \\"@swissquote/swissquote\\"

packages/rollup-plugin-eslint/package.json

@@ -35,6 +35,6 @@
   },
   "dependencies": {
     "@rollup/pluginutils": "4.1.1",
-    "eslint": "8.0.0-beta.1"
+    "eslint": "8.0.1"
   }
 }