@naturalcycles/dev-lib 20.1.3 → 20.1.5

package/cfg/eslint-rules.js

@@ -278,7 +278,7 @@ export default {
       'rxjs/observable/interval',
       'rxjs/observable/forkJoin',
     ],
-    curly: [2, 'multi-line'], // this one is an exception that can conflict with prettier
+    curly: [0, 'multi-line'], // this one is an exception that can conflict with prettier, oxlint
     'no-self-assign': 2,
     'no-setter-return': 2,
     'no-shadow': 0, // it is buggy with TypeScript enums
@@ -319,7 +319,7 @@ export default {
     'valid-typeof': 2,
     'no-await-in-loop': 0, // it's actually often ok
     'no-extend-native': 0, // oxlint
-    'guard-for-in': 2,
+    'guard-for-in': 0, // oxlint
     '@typescript-eslint/no-namespace': [
       2,
       {
@@ -446,7 +446,7 @@ export default {
         enforceForIfStatements: true,
       },
     ],
-    'max-params': [2, { max: 5 }],
+    'max-params': [0, { max: 5 }], // oxlint
     'no-else-return': 2,
     'no-sequences': 2,
     'no-useless-concat': 0, // oxlint

package/cfg/eslint-vitest-rules.js

@@ -14,6 +14,7 @@ export default {
     'vitest/no-commented-out-tests': 0,
     'vitest/no-duplicate-hooks': 2,
     'vitest/no-test-return-statement': 2,
+    'vitest/no-import-node-test': 0, // oxlint
     'vitest/padding-around-before-all-blocks': 2,
     'vitest/padding-around-after-all-blocks': 2,
     'vitest/padding-around-before-each-blocks': 2,
@@ -25,6 +26,7 @@ export default {
     'vitest/prefer-spy-on': 2,
     'vitest/prefer-strict-boolean-matchers': 2,
     'vitest/prefer-to-be': 2,
+    'vitest/prefer-called-exactly-once-with': 0, // todo: enable
     'vitest/prefer-to-contain': 2,
     'vitest/prefer-to-have-length': 2,
     'vitest/prefer-vi-mocked': 2,

package/cfg/lint-staged.config.js

@@ -65,7 +65,9 @@ if (eslintConfigPath) {
 let oxlintCmd = undefined
 
 if (oxlintConfigPath) {
-  oxlintCmd = ['oxlint', '--fix', '--fix-suggestions', '--max-warnings=1'].filter(Boolean).join(' ')
+  oxlintCmd = ['oxlint', '--fix', '--fix-suggestions', '--fix-dangerously', '--max-warnings=0']
+    .filter(Boolean)
+    .join(' ')
 }
 
 const stylelintExists =

package/cfg/oxlint.config.json

@@ -42,6 +42,9 @@
     "no-useless-constructor": 2,
     "preserve-caught-error": 2,
     "eqeqeq": [2, "smart"],
+    "curly": [2, "multi-line"],
+    "guard-for-in": 2,
+    "max-params": [2, { "max": 5 }],
     "import/extensions": [2, "always"],
     "import/no-amd": 2,
     "import/no-commonjs": 2,
@@ -96,6 +99,7 @@
       }
     ],
     "unicorn/no-accessor-recursion": 2,
+    "vitest/no-import-node-test": 2,
     "vue/no-multiple-slot-args": 2
   },
   "settings": {

package/dist/lint.util.js

@@ -42,8 +42,8 @@ export async function lintAllCommand() {
     const fix = !CI;
     // Fast linters (that run in <1 second) go first
     runActionLint();
-    runOxlint(fix);
     runBiome(fix);
+    runOxlint(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.
     eslintAll({
@@ -141,9 +141,10 @@ export function runOxlint(fix = true) {
         name: ['oxlint', !fix && '--no-fix'].filter(Boolean).join(' '),
         args: [
             // '--report-unused-disable-directives',
-            '--max-warnings=1',
+            '--max-warnings=0',
             fix && '--fix',
             fix && '--fix-suggestions',
+            fix && '--fix-dangerously',
         ].filter(_isTruthy),
         shell: false,
     });

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@naturalcycles/dev-lib",
   "type": "module",
-  "version": "20.1.3",
+  "version": "20.1.5",
   "dependencies": {
     "@biomejs/biome": "^2",
     "@commitlint/cli": "^20",