aberlaas-lint 2.19.0 → 2.20.1

package/configs/eslint.js

@@ -1,16 +1,14 @@
-import globals from 'globals';
 import js from '@eslint/js';
+// @vitest/plugins requires @typescript-eslint/utils and typescripts as deps
+// See: https://github.com/vitest-dev/eslint-plugin-vitest/issues/543
+import pluginVitest from '@vitest/eslint-plugin';
+import { nodeVersion } from 'aberlaas-versions';
 import pluginImport from 'eslint-plugin-import';
 import pluginJsdoc from 'eslint-plugin-jsdoc';
 import pluginN from 'eslint-plugin-n';
 import pluginPrettierRecommended from 'eslint-plugin-prettier/recommended';
 import pluginReact from 'eslint-plugin-react';
-// Note:
-// It is currently required to manually add typescript and
-// @typescript-eslint/utils to aberlaas for the plugin to work
-// See: https://github.com/vitest-dev/eslint-plugin-vitest/issues/543
-import pluginVitest from '@vitest/eslint-plugin';
-import { nodeVersion } from 'aberlaas-versions';
+import globals from 'globals';
 
 export default [
   {
@@ -70,6 +68,16 @@ export default [
           property: 'contains',
           message: 'Typo: Use _.includes instead',
         },
+        {
+          object: '_',
+          property: 'contain',
+          message: 'Typo: Use _.includes instead',
+        },
+        {
+          object: '_',
+          property: 'include',
+          message: 'Typo: Use _.includes instead',
+        },
         {
           object: '_',
           property: 'padLeft',
@@ -108,12 +116,51 @@ export default [
       // Import
       'import/first': ['error'],
       'import/no-cycle': ['error', { ignoreExternal: true, disableScc: true }],
-      'import/order': ['error'],
+      // Put most common modules (firost, golgoth, etc) first
+      // Then other third parties
+      // Then internal files
+      'import/order': [
+        'error',
+        {
+          groups: [
+            'builtin',
+            'external',
+            'internal',
+            'parent',
+            'sibling',
+            'index',
+          ],
+          pathGroups: [
+            {
+              pattern: 'golgoth',
+              group: 'external',
+              position: 'before',
+            },
+            {
+              pattern: 'firost',
+              group: 'external',
+              position: 'before',
+            },
+          ],
+          pathGroupsExcludedImportTypes: [],
+          'newlines-between': 'never',
+          alphabetize: {
+            order: 'asc',
+            caseInsensitive: true,
+          },
+        },
+      ],
       'import/newline-after-import': ['error'],
-      // Ignoring this rule for lint-staged, as it seem to have a missing .main
-      // field in its package.json
-      // See: https://github.com/lint-staged/lint-staged/issues/1474
-      'import/no-unresolved': ['error', { ignore: ['lint-staged'] }],
+      // import/no-unresolved can only check for .main fields, not the more
+      // modern .exports fields.
+      // We keep here a list of exceptions for packages we use that don't have a .main field.
+      // See: https://github.com/import-js/eslint-plugin-import/issues/2132
+      'import/no-unresolved': [
+        'error',
+        {
+          ignore: ['changelogen', 'lint-staged', '@octokit/rest', 'stylelint'],
+        },
+      ],
 
       // JSDoc
       'jsdoc/check-param-names': ['warn'],

package/lib/circleci.js

@@ -1,17 +1,70 @@
+import { exists, firostError, run as firostRun, which } from 'firost';
+import { hostGitPath } from 'aberlaas-helper';
 import ciInfo from 'ci-info';
-import { exists, firostError, run, which } from 'firost';
-import helper from 'aberlaas-helper';
-import lintYml from './yml.js';
+import { fix as fixYml, run as runYml } from './yml.js';
 
-export default {
+export let __;
+
+/**
+ * Lint the file, both for yml issues and if possible circleci specifics
+ * @returns {boolean} True on success
+ */
+export async function run() {
+  const absoluteConfigPath = await __.getInputFile();
+  const isRunningOnCircleCi = __.isRunningOnCircleCi();
+
+  // Stop early if no config file, or if running on CircleCI
+  if (!absoluteConfigPath || isRunningOnCircleCi) {
+    return true;
+  }
+
+  // Lint as yml first
+  await runYml([absoluteConfigPath]);
+
+  // Stop early if no circleci bin available
+  if (!(await __.hasCircleCiBin())) {
+    return true;
+  }
+
+  // Validate the config
+  try {
+    await __.validateConfig();
+  } catch (error) {
+    const errorMessage = `CircleCI config error on ${__.configPath}\n${error.message}`;
+    throw firostError('ABERLAAS_LINT_CIRCLECI', errorMessage);
+  }
+
+  return true;
+}
+
+/**
+ * Autofix yml issues in file
+ * @returns {boolean} True on success
+ */
+export async function fix() {
+  const absoluteConfigPath = hostGitPath(__.configPath);
+  // Fix yml issues
+  await fixYml([absoluteConfigPath]);
+
+  // Check for file errors so it still fails if file is invalid
+  await run();
+}
+
+__ = {
   configPath: '.circleci/config.yml',
+
   /**
-   * Find all relevant files
+   * Find the CircleCI config file
    * @returns {Array} Array of files
    */
-  async getInputFiles() {
-    return await helper.findHostFiles([this.configPath]);
+  async getInputFile() {
+    const configFile = await hostGitPath(__.configPath);
+    if (!(await exists(configFile))) {
+      return false;
+    }
+    return configFile;
   },
+
   /**
    * Check if the code is currently running on CircleCI
    * @returns {boolean} True if running on CircleCI, false otherwise
@@ -19,6 +72,7 @@ export default {
   isRunningOnCircleCi() {
     return ciInfo.CIRCLE;
   },
+
   /**
    * Check if the circleci binary is available in the $PATH
    * @returns {boolean} True if available, false otherwise
@@ -27,55 +81,17 @@ export default {
     const binary = await which('circleci');
     return !!binary;
   },
+
   /**
    * Validate the CircleCI config file.
    * @returns {boolean} True if valid, throws an error if not
    */
   async validateConfig() {
-    await run('circleci config validate', { stdout: false });
+    await firostRun('circleci config validate', { stdout: false });
   },
-  /**
-   * Lint the file, both for yml issues and if possible circleci specifics
-   * @returns {boolean} True on success
-   */
-  async run() {
-    const absoluteConfigPath = helper.hostPath(this.configPath);
-    const hasConfigFile = await exists(absoluteConfigPath);
-    const isRunningOnCircleCi = this.isRunningOnCircleCi();
-
-    // Stop early if no config file, or if running on CircleCI
-    if (!hasConfigFile || isRunningOnCircleCi) {
-      return true;
-    }
-
-    // Lint as yml first
-    await lintYml.run([absoluteConfigPath]);
-
-    // Stop early if no circleci bin available
-    if (!(await this.hasCircleCiBin())) {
-      return true;
-    }
-
-    // Validate the config
-    try {
-      await this.validateConfig();
-    } catch (error) {
-      const errorMessage = `CircleCI config error on ${this.configPath}\n${error.message}`;
-      throw firostError('CircleCiLintError', errorMessage);
-    }
-
-    return true;
-  },
-  /**
-   * Autofix yml issues in file
-   * @returns {boolean} True on success
-   */
-  async fix() {
-    const absoluteConfigPath = helper.hostPath(this.configPath);
-    // Fix yml issues
-    await lintYml.fix([absoluteConfigPath]);
+};
 
-    // Check for file errors so it still fails if file is invalid
-    await this.run();
-  },
+export default {
+  run,
+  fix,
 };

package/lib/css.js

@@ -1,11 +1,68 @@
-import stylelint from 'stylelint';
 import { _ } from 'golgoth';
 import { firostError } from 'firost';
-import helper from 'aberlaas-helper';
+import { findHostPackageFiles, getConfig } from 'aberlaas-helper';
+import stylelint from 'stylelint';
 import stylelintConfig from '../configs/stylelint.js';
-import { fix as prettierFix } from './helpers/prettier.js';
+import { prettierFix } from './helpers/prettierFix.js';
 
-export default {
+export let __;
+
+/**
+ * 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
+ */
+export async function run(userPatterns, userConfigFile, userOptions = {}) {
+  // Options
+  const options = { fix: false, ...userOptions };
+
+  // Files
+  const files = await __.getInputFiles(userPatterns);
+  if (_.isEmpty(files)) {
+    return true;
+  }
+
+  // Config
+  const config = await getConfig(
+    userConfigFile,
+    'stylelint.config.js',
+    stylelintConfig,
+  );
+
+  const result = await stylelint.lint({
+    config,
+    files,
+    formatter: 'string',
+    ...options,
+  });
+
+  if (result.errored) {
+    throw firostError('ABERLAAS_LINT_CSS', result.report);
+  }
+  return true;
+}
+
+/**
+ * Autofix files in place
+ * @param {Array} userPatterns Patterns to narrow the search down
+ * @param {string} userConfigFile Custom config file to use
+ * @returns {boolean} True on success
+ */
+export async function fix(userPatterns, userConfigFile) {
+  const files = await __.getInputFiles(userPatterns);
+  if (_.isEmpty(files)) {
+    return true;
+  }
+  // Try to pretiffy as much as we can
+  await __.prettierFix(files);
+  // Still run a lint on it so it can fail if not everything is fixed
+  await run(userPatterns, userConfigFile, { fix: true });
+  return true;
+}
+
+__ = {
   /**
    * Find all relevant files
    * @param {Array} userPatterns Patterns to narrow the search down
@@ -15,59 +72,13 @@ export default {
     const filePatterns = _.isEmpty(userPatterns)
       ? ['./**/*.css']
       : userPatterns;
-    return await helper.findHostFiles(filePatterns, ['.css']);
+    return await findHostPackageFiles(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, userOptions = {}) {
-    // Options
-    const options = { fix: false, ...userOptions };
-
-    // Files
-    const files = await this.getInputFiles(userPatterns);
-    if (_.isEmpty(files)) {
-      return true;
-    }
-
-    // Config
-    const config = await helper.getConfig(
-      userConfigFile,
-      'stylelint.config.js',
-      stylelintConfig,
-    );
 
-    const result = await stylelint.lint({
-      config,
-      files,
-      formatter: 'string',
-      ...options,
-    });
+  prettierFix,
+};
 
-    if (result.errored) {
-      throw firostError('ERROR_CSS_LINT', result.report);
-    }
-    return true;
-  },
-  /**
-   * Autofix files in place
-   * @param {Array} userPatterns Patterns to narrow the search down
-   * @param {string} userConfigFile Custom config file to use
-   * @returns {boolean} True on success
-   */
-  async fix(userPatterns, userConfigFile) {
-    const files = await this.getInputFiles(userPatterns);
-    if (_.isEmpty(files)) {
-      return true;
-    }
-    // Try to pretiffy as much as we can
-    await prettierFix(files);
-    // Still run a lint on it so it can fail if not everything is fixed
-    await this.run(userPatterns, userConfigFile, { fix: true });
-    return true;
-  },
+export default {
+  run,
+  fix,
 };

package/lib/helpers/{prettier.js → prettierFix.js}

@@ -1,20 +1,16 @@
-import { format as prettierFormat } from 'prettier';
-import { firostError, read, write } from 'firost';
 import { _, pMap } from 'golgoth';
-import helper from 'aberlaas-helper';
+import { firostError, read, write } from 'firost';
+import { getConfig } from 'aberlaas-helper';
+import { format as prettierFormat } from 'prettier';
 import prettierConfig from '../../configs/prettier.js';
 
 /**
- * Fix all files using prettier
+ * Fix files using prettier
  * @param {Array} inputFiles Files to auto-fix
  */
-export async function fix(inputFiles) {
+export async function prettierFix(inputFiles) {
   // Config file
-  const config = await helper.getConfig(
-    null,
-    'prettier.config.js',
-    prettierConfig,
-  );
+  const config = await getConfig(null, 'prettier.config.js', prettierConfig);
 
   const errors = [];
 
@@ -36,19 +32,17 @@ export async function fix(inputFiles) {
     { 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:\n\n${formattedErrors}`,
-    );
+  if (_.isEmpty(errors)) {
+    return;
   }
-}
 
-export default {
-  fix,
-};
+  let formattedErrors = '';
+  _.each(errors, (error) => {
+    formattedErrors = `${formattedErrors}${error.filepath}\n\n${error.message}\n\n`;
+  });
+
+  throw firostError(
+    'ABERLAAS_LINT_PRETTIER_FIX',
+    `Some files could not be automatically fixed:\n\n${formattedErrors}`,
+  );
+}

package/lib/js.js

@@ -1,80 +1,85 @@
 import { _ } from 'golgoth';
 import { firostError } from 'firost';
+import { findHostPackageFiles, getConfig } from 'aberlaas-helper';
 import { ESLint } from 'eslint';
-import helper from 'aberlaas-helper';
 import eslintConfig from '../configs/eslint.js';
 
-export default {
-  /**
-   * Find all relevant files
-   * @param {Array} userPatterns Patterns to narrow the search down
-   * @returns {Array} Array of files
-   */
-  async getInputFiles(userPatterns) {
-    const filePatterns = _.isEmpty(userPatterns) ? ['./**/*.js'] : userPatterns;
+export let __;
 
-    return await helper.findHostFiles(filePatterns, [
-      '.js',
-      '.jsx',
-      '.ts',
-      '.tsx',
-    ]);
-  },
-  /**
-   * 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, userOptions = {}) {
-    // Options
-    const options = { fix: false, warnIgnored: false, ...userOptions };
+/**
+ * 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
+ */
+export async function run(userPatterns, userConfigFile, userOptions = {}) {
+  // Options
+  const options = { fix: false, warnIgnored: false, ...userOptions };
 
-    // Files to lint
-    const files = await this.getInputFiles(userPatterns);
-    if (_.isEmpty(files)) {
-      return true;
-    }
+  // Files to lint
+  const files = await __.getInputFiles(userPatterns);
+  if (_.isEmpty(files)) {
+    return true;
+  }
 
-    // Config file
-    const config = await helper.getConfig(
-      userConfigFile,
-      'eslint.config.js',
-      eslintConfig,
-    );
+  // Config file
+  const config = await getConfig(
+    userConfigFile,
+    'eslint.config.js',
+    eslintConfig,
+  );
 
-    // Run the actual lint
-    const eslint = new ESLint({
-      ...options,
-      overrideConfigFile: true,
-      overrideConfig: config,
-    });
-    const results = await eslint.lintFiles(files);
+  // Run the actual lint
+  const eslint = new ESLint({
+    ...options,
+    overrideConfigFile: true,
+    overrideConfig: config,
+  });
+  const results = await eslint.lintFiles(files);
 
-    // Fix
-    if (options.fix) {
-      await ESLint.outputFixes(results);
-    }
+  // Fix
+  if (options.fix) {
+    await ESLint.outputFixes(results);
+  }
 
-    // All good, we can stop
-    const errorCount = _.chain(results).map('errorCount').sum().value();
-    if (errorCount == 0) {
-      return true;
-    }
+  // All good, we can stop
+  const errorCount = _.chain(results).map('errorCount').sum().value();
+  if (errorCount == 0) {
+    return true;
+  }
 
-    // Format errors
-    const formatter = await eslint.loadFormatter('stylish');
-    const errorText = formatter.format(results);
-    throw firostError('JavaScriptLintError', errorText);
-  },
+  // Format errors
+  const formatter = await eslint.loadFormatter('stylish');
+  const errorText = formatter.format(results);
+  throw firostError('ABERLAAS_LINT_JS', errorText);
+}
+
+/**
+ * Autofix files in place
+ * @param {Array} userPatterns Patterns to narrow the search down
+ * @param {string} userConfigFile Custom config file to use
+ * @returns {boolean} True on success
+ */
+export async function fix(userPatterns, userConfigFile) {
+  return await run(userPatterns, userConfigFile, { fix: true });
+}
+
+__ = {
   /**
-   * Autofix files in place
+   * Find all relevant files
    * @param {Array} userPatterns Patterns to narrow the search down
-   * @param {string} userConfigFile Custom config file to use
-   * @returns {boolean} True on success
+   * @returns {Array} Array of files
    */
-  async fix(userPatterns, userConfigFile) {
-    return await this.run(userPatterns, userConfigFile, { fix: true });
+  async getInputFiles(userPatterns) {
+    const filePatterns = _.isEmpty(userPatterns) ? ['./**/*.js'] : userPatterns;
+    const allowedExtensions = ['.js', '.jsx', '.ts', '.tsx'];
+
+    return await findHostPackageFiles(filePatterns, allowedExtensions);
   },
 };
+
+export default {
+  run,
+  fix,
+};

package/lib/json.js

@@ -1,10 +1,55 @@
 import path from 'node:path';
 import { _, pMap } from 'golgoth';
 import { firostError, read } from 'firost';
-import helper from 'aberlaas-helper';
-import { fix as prettierFix } from './helpers/prettier.js';
+import { findHostPackageFiles, hostGitRoot } from 'aberlaas-helper';
+import { prettierFix } from './helpers/prettierFix.js';
 
-export default {
+export let __;
+
+/**
+ * Lint all files and display results.
+ * @param {Array} userPatterns Patterns to narrow the search down
+ * @returns {boolean} True on success
+ */
+export async function run(userPatterns) {
+  const files = await __.getInputFiles(userPatterns);
+  if (_.isEmpty(files)) {
+    return true;
+  }
+
+  let hasErrors = false;
+  const errorMessages = [];
+  await pMap(files, async (filepath) => {
+    try {
+      JSON.parse(await read(filepath));
+    } catch (error) {
+      hasErrors = true;
+      const relativePath = path.relative(hostGitRoot(), filepath);
+      errorMessages.push(`Invalid JSON: ${relativePath}`);
+      errorMessages.push(error.message);
+    }
+  });
+
+  if (hasErrors) {
+    throw firostError('ABERLAAS_LINT_JSON', errorMessages.join('\n'));
+  }
+  return true;
+}
+
+/**
+ * Autofix files in place
+ * @param {Array} userPatterns Patterns to narrow the search down
+ * @returns {boolean} True on success
+ */
+export async function fix(userPatterns) {
+  const files = await __.getInputFiles(userPatterns);
+  if (_.isEmpty(files)) {
+    return true;
+  }
+  await __.prettierFix(files);
+}
+
+__ = {
   /**
    * Find all relevant files
    * @param {Array} userPatterns Patterns to narrow the search down
@@ -14,47 +59,13 @@ export default {
     const filePatterns = _.isEmpty(userPatterns)
       ? ['./**/*.json']
       : userPatterns;
-    return await helper.findHostFiles(filePatterns, ['.json']);
+    return await findHostPackageFiles(filePatterns, ['.json']);
   },
-  /**
-   * Lint all files and display results.
-   * @param {Array} userPatterns Patterns to narrow the search down
-   * @returns {boolean} True on success
-   */
-  async run(userPatterns) {
-    const files = await this.getInputFiles(userPatterns);
-    if (_.isEmpty(files)) {
-      return true;
-    }
 
-    let hasErrors = false;
-    const errorMessages = [];
-    await pMap(files, async (filepath) => {
-      try {
-        JSON.parse(await read(filepath));
-      } catch (error) {
-        hasErrors = true;
-        const relativePath = path.relative(helper.hostRoot(), filepath);
-        errorMessages.push(`Invalid JSON: ${relativePath}`);
-        errorMessages.push(error.message);
-      }
-    });
-
-    if (hasErrors) {
-      throw firostError('JsonLintError', errorMessages.join('\n'));
-    }
-    return true;
-  },
-  /**
-   * Autofix files in place
-   * @param {Array} userPatterns Patterns to narrow the search down
-   * @returns {boolean} True on success
-   */
-  async fix(userPatterns) {
-    const files = await this.getInputFiles(userPatterns);
-    if (_.isEmpty(files)) {
-      return true;
-    }
-    await prettierFix(files);
-  },
+  prettierFix,
+};
+
+export default {
+  run,
+  fix,
 };

package/lib/main.js

@@ -1,7 +1,42 @@
 import { _, pMap } from 'golgoth';
 import { consoleError, firostError, firostImport } from 'firost';
 
-export default {
+export let __;
+
+/**
+ * Wrapper to lint all supported formats
+ * @param {object} cliArgs CLI Argument object, as created by minimist
+ * @returns {boolean} True on success
+ */
+export async function run(cliArgs = {}) {
+  const allTypesKeys = _.keys(__.linters);
+  const userTypes = _.intersection(_.keys(cliArgs), allTypesKeys);
+  const typesToLint = _.isEmpty(userTypes) ? allTypesKeys : userTypes;
+
+  let hasErrors = false;
+  await pMap(typesToLint, async (type) => {
+    const methodName = cliArgs.fix ? 'fix' : 'run';
+    try {
+      const linter = await __.getLinter(type);
+
+      const configFile = _.get(cliArgs, `config.${type}`);
+      const userPatterns = _.get(cliArgs, '_');
+
+      await linter[methodName](userPatterns, configFile);
+    } catch (error) {
+      __.consoleError(error.message);
+      hasErrors = true;
+    }
+  });
+
+  if (hasErrors) {
+    throw firostError('ABERLAAS_LINT_FAIL', 'Fail to lint files');
+  }
+
+  return true;
+}
+
+__ = {
   /**
    * List of all available linters, along with the --flag name
    **/
@@ -19,44 +54,17 @@ export default {
    * @returns {object} Linter
    **/
   async getLinter(linterType) {
-    const linterPath = this.linters[linterType];
+    const linterPath = __.linters[linterType];
     if (!linterPath) {
       return false;
     }
 
     return await firostImport(linterPath);
   },
-  /**
-   * Wrapper to lint all supported formats
-   * @param {object} cliArgs CLI Argument object, as created by minimist
-   * @returns {boolean} True on success
-   */
-  async run(cliArgs = {}) {
-    const allTypesKeys = _.keys(this.linters);
-    const userTypes = _.intersection(_.keys(cliArgs), allTypesKeys);
-    const typesToLint = _.isEmpty(userTypes) ? allTypesKeys : userTypes;
-
-    let hasErrors = false;
-    await pMap(typesToLint, async (type) => {
-      const methodName = cliArgs.fix ? 'fix' : 'run';
-      try {
-        const linter = await this.getLinter(type);
-
-        const configFile = _.get(cliArgs, `config.${type}`);
-        const userPatterns = _.get(cliArgs, '_');
-
-        await linter[methodName](userPatterns, configFile);
-      } catch (error) {
-        this.__consoleError(error.message);
-        hasErrors = true;
-      }
-    });
-
-    if (hasErrors) {
-      throw firostError('ERROR_LINT', 'Error while linting files');
-    }
 
-    return true;
-  },
-  __consoleError: consoleError,
+  consoleError,
+};
+
+export default {
+  run,
 };

package/lib/yml.js

@@ -1,11 +1,57 @@
 import path from 'node:path';
-import { firostError, read } from 'firost';
 import { _, pMap } from 'golgoth';
+import { firostError, read } from 'firost';
+import { findHostPackageFiles, hostGitRoot } from 'aberlaas-helper';
 import yamlLint from 'yaml-lint';
-import helper from 'aberlaas-helper';
-import { fix as prettierFix } from './helpers/prettier.js';
+import { prettierFix } from './helpers/prettierFix.js';
 
-export default {
+export let __;
+
+/**
+ * Lint all files and display results.
+ * @param {Array} userPatterns Patterns to narrow the search down
+ * @returns {boolean} True on success
+ */
+export async function run(userPatterns) {
+  const files = await __.getInputFiles(userPatterns);
+  if (_.isEmpty(files)) {
+    return true;
+  }
+
+  let hasErrors = false;
+  const errorMessages = [];
+  await pMap(files, async (filepath) => {
+    const input = await read(filepath);
+    try {
+      await yamlLint.lint(input);
+    } catch (error) {
+      hasErrors = true;
+      const relativePath = path.relative(hostGitRoot(), filepath);
+      errorMessages.push(`Invalid YAML: ${relativePath}`);
+      errorMessages.push(error.message);
+    }
+  });
+
+  if (hasErrors) {
+    throw firostError('ABERLAAS_LINT_YML', errorMessages.join('\n'));
+  }
+  return true;
+}
+
+/**
+ * Autofix files in place
+ * @param {Array} userPatterns Patterns to narrow the search down
+ * @returns {boolean} True on success
+ */
+export async function fix(userPatterns) {
+  const files = await __.getInputFiles(userPatterns);
+  if (_.isEmpty(files)) {
+    return true;
+  }
+  await __.prettierFix(files);
+}
+
+__ = {
   /**
    * Find all relevant files
    * @param {Array} userPatterns Patterns to narrow the search down
@@ -15,48 +61,13 @@ export default {
     const filePatterns = _.isEmpty(userPatterns)
       ? ['./**/*.yml', './**/*.yaml']
       : userPatterns;
-    return await helper.findHostFiles(filePatterns, ['.yml', '.yaml']);
+    return await findHostPackageFiles(filePatterns, ['.yml', '.yaml']);
   },
-  /**
-   * Lint all files and display results.
-   * @param {Array} userPatterns Patterns to narrow the search down
-   * @returns {boolean} True on success
-   */
-  async run(userPatterns) {
-    const files = await this.getInputFiles(userPatterns);
-    if (_.isEmpty(files)) {
-      return true;
-    }
 
-    let hasErrors = false;
-    const errorMessages = [];
-    await pMap(files, async (filepath) => {
-      const input = await read(filepath);
-      try {
-        await yamlLint.lint(input);
-      } catch (error) {
-        hasErrors = true;
-        const relativePath = path.relative(helper.hostRoot(), filepath);
-        errorMessages.push(`Invalid YAML: ${relativePath}`);
-        errorMessages.push(error.message);
-      }
-    });
-
-    if (hasErrors) {
-      throw firostError('YamlLintError', errorMessages.join('\n'));
-    }
-    return true;
-  },
-  /**
-   * Autofix files in place
-   * @param {Array} userPatterns Patterns to narrow the search down
-   * @returns {boolean} True on success
-   */
-  async fix(userPatterns) {
-    const files = await this.getInputFiles(userPatterns);
-    if (_.isEmpty(files)) {
-      return true;
-    }
-    await prettierFix(files);
-  },
+  prettierFix,
+};
+
+export default {
+  run,
+  fix,
 };

package/package.json

@@ -1,8 +1,9 @@
 {
   "name": "aberlaas-lint",
   "type": "module",
+  "sideEffects": false,
   "description": "aberlaas lint command: Lint files",
-  "version": "2.19.0",
+  "version": "2.20.1",
   "repository": "pixelastic/aberlaas",
   "homepage": "https://projects.pixelastic.com/aberlaas/",
   "author": "Tim Carry (@pixelastic)",
@@ -22,42 +23,39 @@
   "engines": {
     "node": ">=18.18.0"
   },
-  "scripts": {
-    "build": "../../scripts/local/build",
-    "build:prod": "../../scripts/local/build-prod",
-    "cms": "../../scripts/local/cms",
-    "serve": "../../scripts/local/serve",
-    "ci": "../../scripts/local/ci",
-    "release": "../../scripts/local/release",
-    "update-dependencies": "node ../../scripts/meta/update-dependencies.js",
-    "test:meta": "../../scripts/local/test-meta",
-    "test": "../../scripts/local/test",
-    "test:watch": "../../scripts/local/test-watch",
-    "compress": "../../scripts/local/compress",
-    "lint": "../../scripts/local/lint",
-    "lint:fix": "../../scripts/local/lint-fix"
-  },
   "dependencies": {
-    "@eslint/js": "9.15.0",
-    "@typescript-eslint/utils": "8.15.0",
-    "@vitest/eslint-plugin": "1.1.10",
-    "aberlaas-helper": "^2.18.1",
-    "aberlaas-versions": "^2.15.0",
-    "ci-info": "4.1.0",
-    "eslint": "9.15.0",
-    "eslint-config-prettier": "9.1.0",
-    "eslint-plugin-import": "2.31.0",
-    "eslint-plugin-jsdoc": "50.5.0",
-    "eslint-plugin-n": "17.14.0",
-    "eslint-plugin-prettier": "5.2.1",
+    "@eslint/js": "9.39.2",
+    "@typescript-eslint/utils": "8.54.0",
+    "@vitest/eslint-plugin": "1.6.6",
+    "aberlaas-helper": "workspace:*",
+    "aberlaas-versions": "workspace:*",
+    "ci-info": "4.3.1",
+    "eslint": "9.39.2",
+    "eslint-config-prettier": "10.1.8",
+    "eslint-plugin-import": "2.32.0",
+    "eslint-plugin-jsdoc": "62.4.1",
+    "eslint-plugin-n": "17.23.2",
+    "eslint-plugin-prettier": "5.5.5",
     "eslint-plugin-react": "7.37.5",
-    "firost": "5.2.1",
-    "globals": "15.14.0",
+    "firost": "5.5.1",
+    "globals": "17.2.0",
     "golgoth": "3.0.0",
-    "prettier": "3.3.3",
-    "stylelint": "16.10.0",
-    "typescript": "5.6.3",
+    "prettier": "3.8.1",
+    "stylelint": "17.0.0",
+    "typescript": "5.9.3",
     "yaml-lint": "1.7.0"
   },
-  "gitHead": "11175e8d126f535dc73a3aa942acd79f450d74c4"
+  "scripts": {
+    "build": "cd ../docs && yarn run build",
+    "build:prod": "cd ../docs && yarn run build:prod",
+    "cms": "cd ../docs && yarn run cms",
+    "serve": "cd ../docs && yarn run serve",
+    "release": "cd ../.. && ./scripts/release",
+    "test:meta": "cd ../.. && ./scripts/test-meta",
+    "test": "cd ../.. && ./scripts/test",
+    "test:watch": "cd ../.. && ./scripts/test-watch",
+    "compress": "cd ../.. && ./scripts/compress",
+    "lint": "cd ../.. && ./scripts/lint",
+    "lint:fix": "cd ../.. && ./scripts/lint-fix"
+  }
 }