aberlaas 2.1.0 → 2.2.0

package/README.md

@@ -192,6 +192,129 @@ userland allows user to change the files if they want to change the behavior.
 configuration files for `aberlaas` itself. They eat their own dog food by
 referencing the same configs as above.
 
+## Tools used and their future
+
+### ESLint
+
+ESLint doesn't yet support ESM config files. We'll upgrade to the latest ESLint
+when it does. This should also help fix the issue with Yarn PnP (see below).
+
+### Yarn
+
+**tl;dr; We'll move to Yarn PnP once ESLint has support for Flat Configs, and
+default `yarn run` doesn't add a ~1s delay overhead**
+
+#### PnP
+
+
+Aberlaas is using Yarn Berry (v2+) as its main package management tool.
+
+Yarn Berry comes with a Plug And Play (PnP) feature that replaces the usage of
+`node_modules` in favor of a `.pnp.cjs` file. Instead of having a very large
+`node_modules` folder, a unique copy of each dependency is stored in the user
+home folder and the `.pnp.cjs` file only keeps references to those folder. This
+makes installing dependencies faster as it needs way less I/O.
+
+By ditching the whole `node_modules` principle, it also removes concepts like
+hoisting of dependencies in a monorepo. This, unfortunately, breaks ESLint.
+
+ESLint expect all its plugins to be defined as `peerDependencies` at the root of
+a mono-repo, as it will always try to include them from there. It works more or
+less correctly at the best of times, and `aberlaas` already has some hacks
+(including `resolvePluginsRelativeTo`) to work around that.
+
+But with PnP, there is no way to make it work correctly, so I will need to wait
+for a better compatibility between ESLint and Yarn 2 before using it.
+
+Sources:
+- [#8](https://github.com/yarnpkg/berry/issues/8), the initial case study of Yarn + ESLint
+- [GitHub issue that explicitly explain the problem](https://github.com/yarnpkg/berry/discussions/3909)
+- [`eslint-patch`, an ESLint plugin that hacks around this issue](https://yarnpkg.com/package?name=@rushstack/eslint-patch)
+- [The `resolvePluginsRelativeTo` bandaid from ESLint](https://eslint.org/docs/latest/use/command-line-interface#--resolve-plugins-relative-to)
+- [A nice postmortem of moving to Yarn 2](https://www.dolthub.com/blog/2022-03-18-migrating-to-yarn-2/)
+- [The Flat Config feature in ESLint that should solve the problem](https://eslint.org/docs/latest/use/configure/configuration-files-new)
+
+#### Calling binaries from the host
+
+In yarn v1, if you install `aberlaas` in your project, all of `aberlaas`
+dependencies (including binaries) were hoisted to the root. This was a design
+flaw, as if several dependencies defined binaries by the same name, they would
+fight in a race condition to take the slot in `node_modules/.bin`.
+
+This has been fixed in Yarn Berry, but it also means it's no longer possible to
+call `yarn run eslint` from a repository that includes `aberlaas`, because
+`eslint` is not a direct dependency of the repo.
+
+It might be possible to define proxy binaries inside of `aberlaas`, but those
+will incur performance issues as they will need to spawn one more yarn context
+(see below).
+
+#### Calling binaries from aberlaas
+
+In Yarn V1, I was calling binaries that `aberlaas` depends on (`eslint`, `vitest`,
+etc) by calling `yarn run`. This was adding some overhead but was acceptable.
+Yarn Berry adds even more overhead and it is becoming noticeable.
+
+Now, my prefered way it to use the NodeJS API of the dependencies instead of
+their CLI, to limit the overhead. I managed to move most tools to this new
+approach, but sometimes I still need to use the CLI (`vitest` for example has
+awesome live watching and display reloading that I don't think I can easily
+replicate through code).
+
+I ran some performance tests to see what would be the fastest way to call
+`vitest` from `aberlaas`
+
+```
+hyperfine \
+    "zsh -i -c 'yarn bin vitest && /home/tim/local/www/projects/aberlaas/node_modules/vitest/vitest.mjs --version'" \
+    "zsh -i -c 'yarn run vitest --version'" \
+    "/home/tim/local/www/projects/aberlaas/node_modules/vitest/vitest.mjs --version" \
+    "/home/tim/local/www/projects/aberlaas/node_modules/.bin/vitest --version" 
+Benchmark 1: zsh -i -c 'yarn run vitest --version'
+  Time (mean ± σ):      1.945 s ±  0.051 s    [User: 1.986 s, System: 0.850 s]
+  Range (min … max):    1.859 s …  2.018 s    10 runs
+ 
+Benchmark 2: zsh -i -c 'yarn bin vitest && /home/tim/local/www/projects/aberlaas/node_modules/vitest/vitest.mjs --version'
+  Time (mean ± σ):      2.108 s ±  0.150 s    [User: 2.108 s, System: 0.843 s]
+  Range (min … max):    1.930 s …  2.289 s    10 runs
+ 
+Benchmark 3: /home/tim/local/www/projects/aberlaas/node_modules/vitest/vitest.m
+js --version
+  Time (mean ± σ):     482.5 ms ±  40.9 ms    [User: 448.4 ms, System: 327.2 ms]
+  Range (min … max):   442.1 ms … 553.3 ms    10 runs
+ 
+Benchmark 4: /home/tim/local/www/projects/aberlaas/node_modules/.bin/vitest --version
+  Time (mean ± σ):     491.9 ms ±  29.6 ms    [User: 454.1 ms, System: 331.2 ms]
+  Range (min … max):   453.8 ms … 535.4 ms    10 runs
+```
+
+Finding the binary through `yarn bin` then calling it is the slowest, but `yarn
+run` isn't much faster. Directly calling the binary is the fastest, but it's
+path can't be easily guessed (apart from reading and parsing a `package.json`).
+But using the symlinks in `node_modules/.bin` is consistent, barely slower than
+calling the binary directly and much faster than using yarn.
+
+This is what I'll be using. Of course, this will break when I'll move to PnP as
+the `node_modules` folder won't be there, but hopefully Yarn will be faster by
+then and I can use `yarn run` reliably.
+
+#### Speed
+
+With Yarn 2+, calling `yarn run` seem to add ~1s of overhead each time. This is
+a known issue (due to the fact Yarn 2 needs to spawn yarn 1, node and a few
+other layers). 
+
+1s is not much in itself, but grows quickly when you have nested `yarn run
+aberlaas` calls that call `yarn run eslint`, and even more when you need to deal
+with monorepos and multiple packages that each have their own yarn scope.
+
+There are open issues on the topic, but nothing merged yet:
+
+- [#3732](https://github.com/yarnpkg/berry/issues/3732), where it is discussed to make yarn run aware that it's running yarn run
+  and keep the same state without spawning new yarns
+- [#2575](https://github.com/yarnpkg/berry/issues/2575), which is the main issue about Yarn performance, with benchmarks against
+  npm/npx/yarn v1.
+
 ## Where does the name Aberlaas come from?
 
 Aberlaas is the base camp from which all great expedition start in the _La Horde
@@ -205,4 +328,4 @@ For your convenience, `aberlass` and `aberlas` are added as aliases by default.
 
 ## Documentation
 
-The complete documentation can be found on https://projects.pixelastic.com/aberlaas/
+The complete documentation can be found on https://projects.pixelastic.com/aberlaas/

package/commands/compress/dummy.js

@@ -0,0 +1,11 @@
+/**
+ * This is a temporary, dummy file to be able to test the top-level
+ * compress.run() function. I assume I'll add more compression types in addition
+ * to png, so this dummy compress is just to test that all compress are
+ * correctly called, but it doesn't do anything
+ **/
+export default {
+  async run() {
+    return true;
+  },
+};

package/commands/compress/index.js

@@ -2,11 +2,13 @@ import _ from 'golgoth/lodash.js';
 import pMap from 'golgoth/pMap.js';
 import consoleError from 'firost/consoleError.js';
 import firostError from 'firost/error.js';
-import helper from '../../helper.js';
+import compressPng from './png.js';
+import compressDummy from './dummy.js';
 
 export default {
   types: {
-    png: './png.js',
+    png: compressPng,
+    dummy: compressDummy,
   },
   /**
    * Wrapper to compress all supported formats
@@ -22,7 +24,7 @@ export default {
     await pMap(typesToCompress, async (type) => {
       try {
         const userPatterns = _.get(cliArgs, '_');
-        const compresser = require(this.types[type]);
+        const compresser = this.types[type];
 
         await compresser.run(userPatterns);
       } catch (error) {
@@ -37,16 +39,5 @@ export default {
 
     return true;
   },
-  /**
-   * Find all relevant files of the specified extension in the host
-   * Note: Should be used by child classes
-   * @param {Array} safeListExtension List of allowed extensions to keep
-   * @param {Array} userPatterns Patterns to narrow the search down
-   * @returns {Array} Array of files
-   **/
-  async getInputFiles(safeListExtension, userPatterns = []) {
-    const inputPatterns = _.isEmpty(userPatterns) ? '.' : userPatterns;
-    return await helper.findHostFiles(inputPatterns, safeListExtension);
-  },
   __consoleError: consoleError,
 };

package/commands/compress/png.js

@@ -1,4 +1,5 @@
-import lint from './index.js';
+import helper from '../../helper.js';
+import _ from 'golgoth/lodash.js';
 import run from 'firost/run.js';
 import which from 'firost/which.js';
 import firostError from 'firost/error.js';
@@ -10,15 +11,19 @@ export default {
    * @returns {Array} Array of files
    **/
   async getInputFiles(userPatterns) {
-    return await lint.getInputFiles(['.png'], userPatterns);
+    const filePatterns = _.isEmpty(userPatterns)
+      ? ['./**/*.png']
+      : userPatterns;
+    return await helper.findHostFiles(filePatterns, ['.png']);
   },
+
   /**
-   * Check if the binary is available in the $PATH
-   * @returns {boolean} True if available, false otherwise
+   
+   * Returns path to the binary to execute
+   * @returns {string|boolean} Path to the binary, or false if not found
    **/
-  async hasBin() {
-    const binary = await which('pngmin');
-    return !!binary;
+  async getBinaryPath() {
+    return await this.__which('pngmin');
   },
   /**
    * Compress files
@@ -27,16 +32,20 @@ export default {
    **/
   async run(userPatterns) {
     // Stop early if no bin
-    if (!(await this.hasBin())) {
+    const binaryPath = await this.getBinaryPath();
+    if (!binaryPath) {
       return true;
     }
 
     try {
       const files = await this.getInputFiles(userPatterns);
-      const command = `pngmin ${files.join(' ')}`;
+      const command = `${binaryPath} ${files.join(' ')}`;
       await run(command, { stdout: false });
     } catch (error) {
       throw firostError('PngCompressError', error.message);
     }
+
+    return true;
   },
+  __which: which,
 };

package/commands/init/index.js

@@ -45,6 +45,9 @@ export default {
    * default rules and overwrite them as they see fit
    **/
   async addConfigFiles() {
+    // Yarn
+    await this.copyToHost('templates/_yarnrc.yml', '.yarnrc.yml');
+
     // ESLint
     await this.copyToHost('templates/_eslintrc.cjs', '.eslintrc.cjs');
     await this.copyToHost('templates/_eslintignore.conf', '.eslintignore');
@@ -106,6 +109,7 @@ export default {
   async addScripts() {
     const defaultScripts = [
       { key: 'ci', value: 'scripts/ci' },
+      { key: 'compress', value: 'scripts/compress' },
       { key: 'lint', value: 'scripts/lint' },
       { key: 'lint:fix', value: 'scripts/lint-fix' },
       { key: 'release', value: 'scripts/release' },

package/commands/lint/css.js

@@ -1,8 +1,8 @@
 import helper from '../../helper.js';
 import { fix as prettierFix } from './helpers/prettier.js';
+import stylelint from 'stylelint';
 import _ from 'golgoth/lodash.js';
 import firostError from 'firost/error.js';
-import run from 'firost/run.js';
 
 export default {
   /**
@@ -11,37 +11,45 @@ export default {
    * @returns {Array} Array of files
    **/
   async getInputFiles(userPatterns) {
-    return await helper.findHostFiles(userPatterns, ['.css']);
+    const filePatterns = _.isEmpty(userPatterns)
+      ? ['./**/*.css']
+      : userPatterns;
+    return await helper.findHostFiles(filePatterns, ['.css']);
   },
   /**
    * Lint all files and display results.
    * @param {Array} userPatterns Patterns to narrow the search down
    * @param {string} userConfigFile Custom config file to use
+   * @param {object} userOptions Options to pass to ESLint, including fix
    * @returns {boolean} True on success
    **/
-  async run(userPatterns, userConfigFile) {
+  async run(userPatterns, userConfigFile, userOptions = {}) {
+    // Options
+    const options = { fix: false, ...userOptions };
+
+    // Files
     const files = await this.getInputFiles(userPatterns);
     if (_.isEmpty(files)) {
       return true;
     }
 
+    // Config
     const configFile = await helper.configFile(
       userConfigFile,
       '.stylelintrc.cjs',
       'configs/stylelint.cjs',
     );
-    const binary = await helper.which('stylelint');
-    const options = [...files, '--color', '--config', configFile];
-    try {
-      await run(`${binary} ${options.join(' ')}`, { stdout: false });
-    } catch (error) {
-      // If it fails because no files passed actually exists, it's not really
-      // a failure
-      const errorMessage = error.stdout;
-      if (_.startsWith(errorMessage, 'Error: No files matching the pattern')) {
-        return true;
-      }
-      throw firostError('ERROR_CSS_LINT', error.stdout);
+    const config = await helper.import(configFile);
+
+    const result = await stylelint.lint({
+      config,
+      files,
+      formatter: 'string',
+      ...options,
+    });
+
+    if (result.errored) {
+      throw firostError('ERROR_CSS_LINT', result.output);
     }
     return true;
   },
@@ -57,11 +65,9 @@ export default {
       return true;
     }
     // Try to pretiffy as much as we can
-    await this.__prettierFix(files);
+    await prettierFix(files);
     // Still run a lint on it so it can fail if not everything is fixed
-    await this.run(userPatterns, userConfigFile);
+    await this.run(userPatterns, userConfigFile, { fix: true });
     return true;
   },
-  __prettierFix: prettierFix,
-  __run: run,
 };

package/commands/lint/helpers/prettier.js

@@ -1,25 +1,53 @@
+import * as prettier from 'prettier';
 import helper from '../../../helper.js';
-import run from 'firost/run.js';
 import firostError from 'firost/error.js';
+import write from 'firost/write.js';
+import read from 'firost/read.js';
+import pMap from 'golgoth/pMap.js';
+import _ from 'golgoth/lodash.js';
 
 /**
  * Fix all files using prettier
- * Note: Will be called by child classes
- * Note: Prettier does not output any information as to why it failed, so
- * we'll manually run the command on each file individually so we can catch
- * the file that errors and display it
  * @param {Array} inputFiles Files to auto-fix
  **/
 export async function fix(inputFiles) {
-  const binary = await helper.which('prettier');
-  const options = ['--write', ...inputFiles];
-  try {
-    const command = `${binary} ${options.join(' ')}`;
-    await run(command, { stdout: false });
-  } catch (err) {
+  // Config file
+  const configFile = await helper.configFile(
+    null,
+    '.prettierrc.cjs',
+    'lib/configs/prettier.cjs',
+  );
+  const config = await prettier.resolveConfig(configFile);
+
+  const errors = [];
+
+  // Read all files, run them through the formatter and save them back to disk
+  // If any emits error, store the errors and display them all in one output
+  await pMap(
+    inputFiles,
+    async (filepath) => {
+      try {
+        const content = await read(filepath);
+        const options = { ...config, filepath };
+        const result = await prettier.format(content, options);
+        await write(result, filepath);
+      } catch (error) {
+        const message = error.toString();
+        errors.push({ filepath, message });
+      }
+    },
+    { concurrency: 10 },
+  );
+
+  if (!_.isEmpty(errors)) {
+    let formattedErrors = '';
+    _.each(errors, (error) => {
+      formattedErrors = `${formattedErrors}${error.filepath}\n\n${error.message}\n\n`;
+    });
+
     throw firostError(
       'LINT_ERROR_FIX_PRETTIER',
-      'Some files could not be automatically fixed.\nPlease run `yarn run lint` to further debug',
+      `Some files could not be automatically fixed:\n\n${formattedErrors}`,
     );
   }
 }

package/commands/lint/js.js

@@ -1,7 +1,6 @@
 import helper from '../../helper.js';
 import _ from 'golgoth/lodash.js';
 import firostError from 'firost/error.js';
-import run from 'firost/run.js';
 import { ESLint } from 'eslint';
 
 export default {
@@ -11,7 +10,8 @@ export default {
    * @returns {Array} Array of files
    **/
   async getInputFiles(userPatterns) {
-    return await helper.findHostFiles(userPatterns, ['.js']);
+    const filePatterns = _.isEmpty(userPatterns) ? ['./**/*.js'] : userPatterns;
+    return await helper.findHostFiles(filePatterns, ['.js']);
   },
   /**
    * Lint all files and display results.
@@ -70,5 +70,4 @@ export default {
   async fix(userPatterns, userConfigFile) {
     return await this.run(userPatterns, userConfigFile, { fix: true });
   },
-  __run: run,
 };

package/commands/lint/json.js

@@ -13,7 +13,10 @@ export default {
    * @returns {Array} Array of files
    **/
   async getInputFiles(userPatterns) {
-    return await helper.findHostFiles(userPatterns, ['.json']);
+    const filePatterns = _.isEmpty(userPatterns)
+      ? ['./**/*.json']
+      : userPatterns;
+    return await helper.findHostFiles(filePatterns, ['.json']);
   },
   /**
    * Lint all files and display results.
@@ -30,7 +33,7 @@ export default {
     const errorMessages = [];
     await pMap(files, async (filepath) => {
       try {
-        this.__parse(await read(filepath));
+        JSON.parse(await read(filepath));
       } catch (error) {
         hasErrors = true;
         const relativePath = path.relative(helper.hostRoot(), filepath);
@@ -54,8 +57,6 @@ export default {
     if (_.isEmpty(files)) {
       return true;
     }
-    await this.__prettierFix(files);
+    await prettierFix(files);
   },
-  __prettierFix: prettierFix,
-  __parse: JSON.parse,
 };

package/commands/lint/yml.js

@@ -14,7 +14,10 @@ export default {
    * @returns {Array} Array of files
    **/
   async getInputFiles(userPatterns) {
-    return await helper.findHostFiles(userPatterns, ['.yml', '.yaml']);
+    const filePatterns = _.isEmpty(userPatterns)
+      ? ['./**/*.yml', './**/*.yaml']
+      : userPatterns;
+    return await helper.findHostFiles(filePatterns, ['.yml', '.yaml']);
   },
   /**
    * Lint all files and display results.
@@ -32,7 +35,7 @@ export default {
     await pMap(files, async (filepath) => {
       const input = await read(filepath);
       try {
-        await this.__lint(input);
+        await yamlLint.lint(input);
       } catch (error) {
         hasErrors = true;
         const relativePath = path.relative(helper.hostRoot(), filepath);
@@ -56,8 +59,6 @@ export default {
     if (_.isEmpty(files)) {
       return true;
     }
-    await this.__prettierFix(files);
+    await prettierFix(files);
   },
-  __prettierFix: prettierFix,
-  __lint: yamlLint.lint,
 };

package/commands/test/index.js

@@ -17,9 +17,8 @@ export default {
    **/
   async run(cliArgs) {
     const options = await this.vitestCliOptions(cliArgs);
-    const binary = await helper.which('vitest');
 
-    await run(`${binary} ${options.join(' ')}`, { stdin: true });
+    await run(`yarn run vitest ${options.join(' ')}`, { stdin: true });
     return true;
   },
 

package/configs/eslint.cjs

@@ -15,6 +15,23 @@ module.exports = {
     'plugin:import/recommended',
     'plugin:prettier/recommended',
   ],
+  settings: {
+    // eslint-plugin-import doesn't currently support the "exports" syntax in
+    // package.json. This allow mapping between custom entrypoints and
+    // files on disk.
+    // For example, it doesn't understand "import * from 'vitest/config';" as
+    // "vitest/config/" isn't really an existing filepath, but a mapping defined
+    // in vitest package.json
+    //
+    // Until this is fixed (see
+    // https://github.com/import-js/eslint-plugin-import/issues/2430)
+    // we manually define the most common extensions
+    'import/resolver': {
+      node: {
+        extensions: ['.js', '.cjs', '.mjs', '.d.ts'],
+      },
+    },
+  },
   plugins: ['jsdoc', 'prettier'],
   rules: {
     'dot-notation': ['error'],
@@ -98,10 +115,8 @@ module.exports = {
         testName: false,
         // Shorter method names
         fit: false,
-        ftest: false,
         fdescribe: false,
         xit: false,
-        xtest: false,
         xdescribe: false,
 
         captureOutput: false,
@@ -113,7 +128,7 @@ module.exports = {
       ],
       rules: {
         'no-restricted-globals': [
-          'warn',
+          'error',
           {
             name: 'fit',
             message: 'No focused test',

package/configs/lintstaged.cjs

@@ -18,6 +18,12 @@ if (pkg.scripts && pkg.scripts.lint) {
   result['*.json'] = ['yarn run lint:fix --json'];
   result['*.js'] = ['yarn run lint:fix --js'];
 }
+
+// Compressing images
+if (pkg.scripts && pkg.scripts.compress) {
+  result['*.png'] = ['yarn run compress --png'];
+}
+
 // Testing changed js files
 if (pkg.scripts && pkg.scripts.test) {
   result['./lib/**/*.js'] = ['yarn run test --failFast --related'];

package/helper.js

@@ -159,4 +159,17 @@ export default {
   async yarnRun(scriptName) {
     return await run(`yarn run ${scriptName}`, { cwd: this.hostRoot() });
   },
+  /**
+   * Dynamically import a file
+   * This is a wrapper around the default import, but bypasses the cache
+   * It makes sure importing a file several times always loads the last version
+   * TODO: This should probably be moved in firost once firost is moved to ESM
+   * @param {string} filepath Path to the file to load
+   * @returns {object} Content of the loaded file
+   **/
+  async import(filepath) {
+    const content = await import(`${filepath}?cacheBusting=${Date.now()}`);
+
+    return content.default || content;
+  },
 };

package/package.json

@@ -2,7 +2,7 @@
   "name": "aberlaas",
   "type": "module",
   "description": "Scaffold your JavaScript projects with tests, lint and release scripts",
-  "version": "2.1.0",
+  "version": "2.2.0",
   "repository": "pixelastic/aberlaas",
   "homepage": "https://projects.pixelastic.com/aberlaas/",
   "author": "Tim Carry (@pixelastic)",
@@ -33,15 +33,14 @@
     "configs/",
     "templates/"
   ],
-  "bin": {
-    "aberlaas": "bin/aberlaas.js"
-  },
+  "bin": "bin/aberlaas.js",
   "dependencies": {
     "@octokit/rest": "18.12.0",
     "ci-info": "3.9.0",
     "dedent": "1.5.1",
     "eslint": "8.54.0",
     "eslint-config-prettier": "9.0.0",
+    "eslint-plugin-import": "2.29.0",
     "eslint-plugin-jsdoc": "46.9.0",
     "eslint-plugin-n": "16.3.1",
     "eslint-plugin-prettier": "5.0.1",
@@ -77,9 +76,5 @@
     "test:watch": "../scripts/lib/test-watch",
     "postinstall": "./scripts/postinstall"
   },
-  "devDependencies": {
-    "eslint-plugin-import": "2.29.0",
-    "eslint-plugin-vitest": "0.3.10"
-  },
-  "gitHead": "5d021891b8ba9c030747a3bc7bf20d86c85422db"
+  "gitHead": "bbf9f5024d007deca00abf451ddf879c876d80aa"
 }

package/templates/_yarnrc.yml

@@ -0,0 +1,9 @@
+compressionLevel: 0
+
+defaultSemverRangePrefix: ''
+
+enableGlobalCache: true
+
+nmMode: hardlinks-local
+
+nodeLinker: node-modules

package/templates/scripts/compress

@@ -0,0 +1,4 @@
+#!/usr/bin/env sh
+set -e
+
+aberlaas compress "$@"

package/configs/jest/index.cjs

@@ -1,52 +0,0 @@
-const path = require('path');
-const jestExtendedPath = path.resolve(__dirname, './jest-extended.cjs');
-const setupFileAfterEnv = path.resolve(__dirname, './setupFileAfterEnv.cjs');
-const testEnvironment = path.resolve(__dirname, './testEnvironment.cjs');
-module.exports = {
-  // Custom environment that automatically set testName and allow for --failFast
-  testEnvironment,
-
-  // Use additional Jest plugins
-  setupFilesAfterEnv: [jestExtendedPath, setupFileAfterEnv],
-
-  // We need to set rootDir to the host directory, otherwise Jest will assume
-  // test are to be looked for in the directory that contains the config (ie.
-  // aberlaas).
-  rootDir: process.cwd(),
-
-  // Tests should be in a __tests__ folder
-  testMatch: ['**/__tests__/**/*.js?(x)'],
-
-  // By default watch mode watches for changes in all directories, and whenever
-  // a test file or associated code file changes, it re-runs tests.
-  // This can cause useless re-render of tests when it catches changes in
-  // directories like ./tmp, etc, so we exclude such directories
-  // watchPathIgnorePatterns only accept RegExp and not globs, so we have to
-  // deal with ((.*)/)? to express any depth of folders
-  watchPathIgnorePatterns: [
-    '<rootDir>/((.*)/)?node_modules/',
-    '<rootDir>/((.*)/)?tmp/',
-    '<rootDir>/((.*)/)?templates/',
-  ],
-  testPathIgnorePatterns: [
-    '<rootDir>/((.*)/)?fixtures/',
-    '<rootDir>/((.*)/)?tmp/',
-    '<rootDir>/((.*)/)?templates/',
-  ],
-
-  // moduleNameMapper: {
-  //   // When using lodash-es (which is treeshakeable, thus preferable in front-end
-  //   // envs), it will fail in tests as it isn't compiled to ES5.
-  //   // So, we make jest load the full lodash instead. Note that the lib still
-  //   // needs to be added as a devDependency
-  //   '^lodash-es$': 'lodash',
-  // },
-
-  // Make sure we don't transform source code and use the default ESM code
-  transform: {},
-
-  bail: true,
-
-  resetMocks: true,
-  restoreMocks: true,
-};

package/configs/jest/jest-extended.cjs

@@ -1,3 +0,0 @@
-// This is a proxy to make sure we load the plugin no matter where it is
-// installed
-module.exports = require('jest-extended');

package/configs/jest/setupFileAfterEnv.cjs

@@ -1,13 +0,0 @@
-/* eslint-disable jest/no-jasmine-globals,no-global-assign */
-const captureOutput = require('firost/captureOutput');
-const dedent = require('dedent');
-
-/**
- * Add new globals to each test file:
- * - captureOutput accepts a callback that will be executed with all output
- *   silenced and returned instead
- * - dedent (https://github.com/dmnd/dedent) to enter multiline strings without
- *   the indentation
- **/
-global.captureOutput = captureOutput;
-global.dedent = dedent;

package/configs/jest/sharedState.cjs

@@ -1,14 +0,0 @@
-/**
- * Jest environments are sandboxed by test suite. To be able to fail fast and
- * stop all tests as soon as one of them failed, we are using this file as an
- * external singleton.
- **/
-module.exports = {
-  __skipAllTests: false,
-  skipAllTests() {
-    this.__skipAllTests = true;
-  },
-  shouldSkipAllTests() {
-    return this.__skipAllTests;
-  },
-};

package/configs/jest/testEnvironment.cjs

@@ -1,46 +0,0 @@
-const NodeEnvironment = require('jest-environment-node');
-const shouldFailFast = process.env.ABERLAAS_TEST_FAIL_FAST;
-const sharedState = require('./sharedState.cjs');
-
-class AberlaasEnvironment extends NodeEnvironment {
-  async handleTestEvent(event, _state) {
-    this.failFast(event);
-    this.setTestName(event);
-  }
-  /**
-   * If one test fails, we skip all other tests
-   * @param {object} event As fired by handleTestEvent
-   */
-  failFast(event) {
-    // Do nothing if --failFast is not passed
-    if (!shouldFailFast) {
-      return;
-    }
-
-    // Whenever a test is failing, we update the shared state to skip all other
-    // tests
-    const eventName = event.name;
-    const isTestFailing = eventName == 'test_fn_failure';
-    const isTestStarting = eventName === 'test_start';
-
-    if (isTestFailing) {
-      sharedState.skipAllTests();
-    }
-    if (isTestStarting && sharedState.shouldSkipAllTests()) {
-      event.test.mode = 'skip';
-    }
-  }
-  /**
-   * When a test starts, we set the global variable testName to the name of the
-   * test
-   * @param {object} event As fired by handleTestEvent
-   **/
-  setTestName(event) {
-    const eventName = event.name;
-    if (eventName === 'test_start') {
-      this.global.testName = event.test.name;
-    }
-  }
-}
-
-module.exports = AberlaasEnvironment;

package/configs/jest.cjs

@@ -1 +0,0 @@
-module.exports = require('./jest/index.cjs');

package/templates/jest.config.cjs

@@ -1,4 +0,0 @@
-const config = require('aberlaas/configs/jest.cjs');
-module.exports = {
-  ...config,
-};