@naturalcycles/dev-lib 20.0.9 → 20.1.0

package/cfg/eslint-rules.js

@@ -193,14 +193,15 @@ export default {
     'import-x/no-duplicates': [2, { 'prefer-inline': false }],
     'import-x/export': 2,
     'import-x/no-empty-named-blocks': 2,
-    'import-x/no-cycle': [
-      2,
-      {
-        ignoreExternal: true,
-        allowUnsafeDynamicCyclicDependency: true,
-        maxDepth: 10,
-      },
-    ],
+    // Disabling no-cycle as the slowest rule. Oxlint is to be adopted instead
+    // 'import-x/no-cycle': [
+    //   2,
+    //   {
+    //     ignoreExternal: true,
+    //     allowUnsafeDynamicCyclicDependency: true,
+    //     maxDepth: 10,
+    //   },
+    // ],
     'import-x/no-useless-path-segments': 2,
     'import-x/no-default-export': 0, // biome
     // 'import-x/no-commonjs': 2, // biome `noCommonJs`

package/cfg/eslint.config.js

@@ -1,5 +1,5 @@
 /**
- * @naturalcycles/dev-lib/cfg/eslint.config.js
+ * `@naturalcycles/dev-lib/cfg/eslint.config.js`
  *
  * Shared eslint FLAT config.
  */

package/cfg/lint-staged.config.js

@@ -1,7 +1,6 @@
 /*
   Default config for `lint-staged`.
   Extendable.
-  Supports default configs for `prettier`, `stylelint`, `eslint`, if they are not found in target project.
 */
 
 const {
@@ -28,6 +27,8 @@ const stylelintConfigPath = [`stylelint.config.js`].find(fs.existsSync)
 
 const eslintConfigPath = ['eslint.config.js'].find(fs.existsSync)
 
+const oxlintConfigPath = ['.oxlintrc.json'].find(fs.existsSync)
+
 let prettierCmd = undefined
 
 if (prettierConfigPath) {
@@ -61,6 +62,12 @@ if (eslintConfigPath) {
     .join(' ')
 }
 
+let oxlintCmd = undefined
+
+if (oxlintConfigPath) {
+  oxlintCmd = ['oxlint', '--fix', '--max-warnings=1'].filter(Boolean).join(' ')
+}
+
 const stylelintExists =
   !!stylelintConfigPath &&
   fs.existsSync('node_modules/stylelint') &&
@@ -71,7 +78,7 @@ const biomeConfigPath = ['biome.jsonc'].find(p => fs.existsSync(p))
 const biomeCmd = biomeConfigPath && `biome lint --write --unsafe --no-errors-on-unmatched`
 
 const linters = {
-  // biome, eslint, stylelint, prettier
+  // biome, oxlint, eslint, stylelint, prettier
   [`./{src,scripts,e2e}/**/*.{${prettierExtensionsAll}}`]: match =>
     runBiomeEslintStylelintPrettier(match),
 
@@ -88,11 +95,17 @@ export function runBiomeEslintStylelintPrettier(match) {
   const filesList = getFilesList(match)
   if (!filesList) return []
 
-  return [biomeCmd, eslintCmd, stylelintCmd, prettierCmd]
+  return [biomeCmd, oxlintCmd, eslintCmd, stylelintCmd, prettierCmd]
     .filter(Boolean)
     .map(s => `${s} ${filesList}`)
 }
 
+export function runOxlintPrettier(match) {
+  const filesList = getFilesList(match)
+  if (!filesList) return []
+  return [oxlintCmd, prettierCmd].filter(Boolean).map(s => `${s} ${filesList}`)
+}
+
 export function runPrettier(match) {
   const filesList = getFilesList(match)
   if (!filesList || !prettierCmd) return []

package/cfg/oxlint.config.json

@@ -0,0 +1,84 @@
+{
+  "plugins": [
+    "oxc",
+    "eslint",
+    "typescript",
+    "unicorn",
+    "import",
+    "jsdoc",
+    "node",
+    "promise",
+    "vitest"
+  ],
+  "rules": {
+    "no-this-alias": 0,
+    "no-async-promise-executor": 0,
+    "no-standalone-expect": 0,
+    "no-conditional-expect": 0,
+    "expect-expect": 0,
+    "no-disabled-tests": 0,
+    "valid-expect": 0,
+    "valid-title": 0,
+    "valid-describe-callback": 0,
+    "no-useless-call": 2,
+    "no-accumulating-spread": 2,
+    "prefer-array-find": 2,
+    "prefer-array-flat-map": 2,
+    "prefer-set-has": 2,
+    "no-bitwise": 2,
+    "no-empty": [2, { "allowEmptyCatch": true }],
+    "no-regex-spaces": 2,
+    "no-restricted-globals": [2, "event", "__dirname", "__filename"],
+    "no-unused-expressions": 2,
+    "no-var": 2,
+    "unicode-bom": 2,
+    "import/extensions": [2, "always"],
+    "import/no-amd": 2,
+    "import/no-commonjs": 2,
+    "import/no-cycle": 2,
+    "import/no-default-export": 2,
+    "import/no-webpack-loader-syntax": 2,
+    "oxc/bad-bitwise-operator": 2,
+    "oxc/no-const-enum": 2,
+    "promise/spec-only": 2,
+    "typescript/explicit-function-return-type": [2, { "allowExpressions": true }],
+    // "typescript/explicit-module-boundary-types": [2, {"allowArgumentsExplicitlyTypedAsAny": true}]
+    "typescript/no-empty-object-type": [2, { "allowInterfaces": "always" }],
+    "typescript/no-import-type-side-effects": 2,
+    // "typescript/no-namespace": 2,
+    "typescript/no-non-null-asserted-nullish-coalescing": 2,
+    "typescript/no-require-imports": 2,
+    "typescript/no-var-requires": 2,
+    "typescript/non-nullable-type-assertion-style": 2,
+    "typescript/prefer-literal-enum-member": 2,
+    "typescript/promise-function-async": [
+      2,
+      {
+        "checkArrowFunctions": false,
+        "checkFunctionDeclarations": true,
+        "checkFunctionExpressions": true,
+        "checkMethodDeclarations": true
+      }
+    ],
+    "unicorn/no-anonymous-default-export": 2,
+    "unicorn/no-array-reduce": [
+      2,
+      {
+        "allowSimpleOperations": true
+      }
+    ],
+    "unicorn/no-document-cookie": 2,
+    "unicorn/no-length-as-slice-end": 2,
+    "unicorn/no-magic-array-flat-depth": 2,
+    "unicorn/prefer-modern-math-apis": 2,
+    "unicorn/prefer-node-protocol": 2,
+    "vue/no-multiple-slot-args": 2
+  },
+  "settings": {
+    "jsdoc": {
+      "tagNamePreference": {
+        "experimental": "experimental"
+      }
+    }
+  }
+}

package/dist/bin/dev-lib.js

@@ -5,7 +5,7 @@ import { _assert } from '@naturalcycles/js-lib/error/assert.js';
 import { fs2 } from '@naturalcycles/nodejs-lib/fs2';
 import { runScript } from '@naturalcycles/nodejs-lib/runScript';
 import { buildCopy, buildProd, runTSCInFolders } from '../build.util.js';
-import { eslintAll, lintAllCommand, lintStagedCommand, runBiome, runCommitlintCommand, runPrettier, stylelintAll, } from '../lint.util.js';
+import { eslintAll, lintAllCommand, lintStagedCommand, runBiome, runCommitlintCommand, runOxlint, runPrettier, stylelintAll, } from '../lint.util.js';
 import { runTest } from '../test.util.js';
 const commands = [
     new Separator(), // build
@@ -65,13 +65,21 @@ const commands = [
         desc: 'Run "lint-staged", which runs linter on git staged files.',
     },
     { name: 'eslint', fn: eslintAll, desc: 'Run eslint on all files.' },
-    { name: 'eslint-no-fix', deprecated: true, fn: async () => await eslintAll({ fix: false }) },
+    { name: 'eslint-no-fix', deprecated: true, fn: () => eslintAll({ fix: false }) },
     {
         name: 'eslint --no-fix',
-        fn: async () => await eslintAll({ fix: false }),
+        fn: () => eslintAll({ fix: false }),
         desc: 'Run eslint on all files with "auto-fix" disabled. Useful for debugging.',
         interactiveOnly: true,
     },
+    { name: 'oxlint', fn: runOxlint, desc: 'Run oxlint on all files.' },
+    { name: 'oxlint-no-fix', deprecated: true, fn: () => runOxlint(false) },
+    {
+        name: 'oxlint --no-fix',
+        fn: () => runOxlint(false),
+        desc: 'Run oxlint on all files with "auto-fix" disabled. Useful for debugging.',
+        interactiveOnly: true,
+    },
     {
         name: 'biome',
         fn: () => runBiome(),

package/dist/lint.util.d.ts

@@ -10,7 +10,8 @@ interface EslintAllOptions {
 /**
  * Runs `eslint` command for all predefined paths (e.g /src, /scripts, etc).
  */
-export declare function eslintAll(opt?: EslintAllOptions): Promise<void>;
+export declare function eslintAll(opt?: EslintAllOptions): void;
+export declare function runOxlint(fix?: boolean): void;
 interface RunPrettierOptions {
     experimentalCli?: boolean;
     fix?: boolean;

package/dist/lint.util.js

@@ -42,10 +42,11 @@ export async function lintAllCommand() {
     const fix = !CI;
     // Fast linters (that run in <1 second) go first
     runActionLint();
+    runOxlint(fix);
     runBiome(fix);
     // From this point we start the "slow" linters, with ESLint leading the way
     // We run eslint BEFORE Prettier, because eslint can delete e.g unused imports.
-    await eslintAll({
+    eslintAll({
         fix,
     });
     if (existsSync(`node_modules/stylelint`) &&
@@ -78,7 +79,7 @@ export async function lintAllCommand() {
 /**
  * Runs `eslint` command for all predefined paths (e.g /src, /scripts, etc).
  */
-export async function eslintAll(opt) {
+export function eslintAll(opt) {
     const { argv } = _yargs().options({
         ext: {
             type: 'string',
@@ -94,9 +95,9 @@ export async function eslintAll(opt) {
         ...opt,
     };
     const extensions = ext.split(',');
-    await runESLint(extensions, fix);
+    runESLint(extensions, fix);
 }
-async function runESLint(extensions = eslintExtensions.split(','), fix = true) {
+function runESLint(extensions = eslintExtensions.split(','), fix = true) {
     const eslintConfigPath = `eslint.config.js`;
     if (!existsSync(eslintConfigPath)) {
         // faster to bail-out like this
@@ -107,10 +108,10 @@ async function runESLint(extensions = eslintExtensions.split(','), fix = true) {
     const cacheLocation = `node_modules/.cache/eslint`;
     const cacheFound = existsSync(cacheLocation);
     console.log(dimGrey(`${check(cacheFound)}eslint cache found: ${cacheFound}`));
-    await exec2.spawnAsync(eslintPath, {
+    exec2.spawn(eslintPath, {
         name: ['eslint', !fix && '--no-fix'].filter(Boolean).join(' '),
         args: [
-            `--config`,
+            '--config',
             eslintConfigPath,
             `{src,scripts,e2e}/**/*.{${extensions.join(',')}}`,
             // `--parser-options=project:${tsconfigPath}`,
@@ -120,8 +121,8 @@ async function runESLint(extensions = eslintExtensions.split(','), fix = true) {
             '--cache',
             '--cache-location',
             cacheLocation,
-            `--no-error-on-unmatched-pattern`,
-            fix ? `--fix` : '--no-fix',
+            '--no-error-on-unmatched-pattern',
+            fix ? '--fix' : '--no-fix',
         ].filter(_isTruthy),
         shell: false,
         env: _filterFalsyValues({
@@ -130,6 +131,23 @@ async function runESLint(extensions = eslintExtensions.split(','), fix = true) {
         }),
     });
 }
+export function runOxlint(fix = true) {
+    const oxlintConfigPath = `.oxlintrc.json`;
+    if (!existsSync(oxlintConfigPath)) {
+        return;
+    }
+    const oxlintPath = findPackageBinPath('oxlint', 'oxlint');
+    exec2.spawn(oxlintPath, {
+        name: ['oxlintPath', !fix && '--no-fix'].filter(Boolean).join(' '),
+        args: [
+            // '--report-unused-disable-directives',
+            '--max-warnings=1',
+            fix ? '--fix' : '--no-fix',
+        ].filter(_isTruthy),
+        logFinish: false,
+        shell: false,
+    });
+}
 const prettierPaths = [
     // Everything inside these folders
     `./{${prettierDirs}}/**/*.{${prettierExtensionsAll}}`,

package/dist/testing/testOffline.js

@@ -1,7 +1,7 @@
 import { AppError } from '@naturalcycles/js-lib/error/error.util.js';
 import { red } from '@naturalcycles/nodejs-lib/colors';
 import createMitm from 'mitm';
-const LOCAL_HOSTS = ['localhost', '127.0.0.1'];
+const LOCAL_HOSTS = new Set(['localhost', '127.0.0.1']);
 const detectLeaks = process.argv.some(a => a.includes('detectLeaks'));
 let mitm;
 /**
@@ -17,7 +17,7 @@ export function testOffline(opt) {
     mitm = createMitm();
     mitm.on('connect', (socket, socketOptions) => {
         const { host } = socketOptions;
-        if (!LOCAL_HOSTS.includes(host)) {
+        if (!LOCAL_HOSTS.has(host)) {
             process.stderr.write(red(`Network request forbidden by testOffline: ${host}\n`));
             opt?.onForbiddenRequest?.(host);
             throw new AppError(`Network request forbidden by testOffline: ${host}`, {

package/package.json

@@ -1,11 +1,11 @@
 {
   "name": "@naturalcycles/dev-lib",
   "type": "module",
-  "version": "20.0.9",
+  "version": "20.1.0",
   "dependencies": {
     "@biomejs/biome": "^2",
-    "@commitlint/cli": "^19",
-    "@commitlint/config-conventional": "^19",
+    "@commitlint/cli": "^20",
+    "@commitlint/config-conventional": "^20",
     "@eslint/js": "^9",
     "@inquirer/prompts": "^7",
     "@naturalcycles/js-lib": "^15",
@@ -23,6 +23,7 @@
     "lint-staged": "^16",
     "micromatch": "^4",
     "mitm": "^1",
+    "oxlint": "^1",
     "prettier": "^3",
     "typescript-eslint": "^8",
     "vue-eslint-parser": "^10"
@@ -42,14 +43,15 @@
     }
   },
   "devDependencies": {
-    "@types/node": "^24",
-    "@types/mitm": "^1"
+    "@types/mitm": "^1",
+    "@types/node": "^24"
   },
   "exports": {
     "./cfg/": "./cfg/",
     "./cfg/biome.jsonc": "./cfg/biome.jsonc",
     "./cfg/commitlint.config.js": "./cfg/commitlint.config.js",
     "./cfg/eslint.config.js": "./cfg/eslint.config.js",
+    "./cfg/oxlint.config.json": "./cfg/oxlint.config.json",
     "./cfg/prettier.config.js": "./cfg/prettier.config.js",
     "./cfg/stylelint.config.js": "./cfg/stylelint.config.js",
     "./cfg/tsconfig.src.json": "./cfg/tsconfig.src.json",
@@ -93,8 +95,8 @@
   "scripts": {
     "tsx-debug": "tsx scripts/testScript.ts",
     "dev-lib": "tsx ./src/bin/dev-lib.ts",
-    "bt": "tsx ./src/bin/dev-lib.ts bt && tsx scripts/eslintPrintConfig.script.ts",
-    "check": "tsx ./src/bin/dev-lib.ts check && tsx scripts/eslintPrintConfig.script.ts",
+    "bt": "tsx ./src/bin/dev-lib.ts bt && tsx scripts/eslintPrintConfig.script.ts && tsx scripts/oxlintPrintConfig.script.ts",
+    "check": "tsx ./src/bin/dev-lib.ts check && tsx scripts/eslintPrintConfig.script.ts && tsx scripts/oxlintPrintConfig.script.ts",
     "typecheck": "tsx ./src/bin/dev-lib.ts typecheck",
     "clean": "tsx ./src/bin/dev-lib.ts clean",
     "build": "tsx ./src/bin/dev-lib.ts build",
@@ -103,10 +105,12 @@
     "test-leaks": "tsx ./src/bin/dev-lib.ts test-leaks",
     "test-integration": "tsx ./src/bin/dev-lib.ts test-integration",
     "test-manual": "tsx ./src/bin/dev-lib.ts test-manual",
-    "lint": "tsx scripts/eslintPrintConfig.script.ts && tsx ./src/bin/dev-lib.ts lint",
+    "lint": "tsx scripts/eslintPrintConfig.script.ts && tsx scripts/oxlintPrintConfig.script.ts && tsx ./src/bin/dev-lib.ts lint",
     "eslint": "tsx ./src/bin/dev-lib.ts eslint",
     "eslint-no-fix": "tsx ./src/bin/dev-lib.ts eslint-no-fix",
     "biome-no-fix": "tsx ./src/bin/dev-lib.ts biome-no-fix",
+    "oxlint": "tsx ./src/bin/dev-lib.ts oxlint",
+    "oxlint-no-fix": "tsx ./src/bin/dev-lib.ts oxlint-no-fix",
     "lint-staged-debug": "tsx ./src/bin/dev-lib.ts lint-staged"
   }
 }