@kitql/eslint-config 0.4.0-next.1 → 0.4.0-next.3

package/.prettierrc.cjs

@@ -27,5 +27,4 @@ module.exports = {
     '',
     '^[./]', // inside
   ],
-  // importOrderSeparation: true,
 }

package/cmd.js

@@ -1,10 +1,11 @@
 #!/usr/bin/env node
 import { spawnSync } from 'node:child_process'
-import fs from 'node:fs'
 import { Option, program } from 'commander'
 
 import { Log, red } from '@kitql/helpers'
 
+import { findFileOrUp } from './helper/findFileOrUp.js'
+
 const log = new Log('kitql-lint')
 
 program.addOption(new Option('-f, --format', 'format'))
@@ -21,23 +22,6 @@ program.addOption(
 program.parse(process.argv)
 const options_cli = program.opts()
 
-const findFileOrUp = (fileName) => {
-  // Find file recursively 4 levels max up
-  for (let i = 0; i < 4; i++) {
-    try {
-      const path = '../'.repeat(i) + fileName
-      if (fs.statSync(path)) {
-        return path
-      }
-    } catch (error) {
-      // nothing to do
-    }
-  }
-
-  log.error(red(`${fileName} not found`))
-  process.exit(1)
-}
-
 const pathPrettierIgnore = findFileOrUp('.prettierignore')
 const pathPrettierCjs = findFileOrUp('.prettierrc.cjs')
 
@@ -86,8 +70,6 @@ function eslintRun() {
   const cmdEsLint =
     preToUse +
     `eslint` +
-    // ignore?
-    ` --ignore-pattern ${pathPrettierIgnore}` +
     // format or not
     `${format ? ' --fix' : ''}` +
     // exec

package/eslint.config.js

@@ -1,15 +1,28 @@
+import { includeIgnoreFile } from '@eslint/compat'
 import js from '@eslint/js'
 import svelte from 'eslint-plugin-svelte'
 import unusedImports from 'eslint-plugin-unused-imports'
 import globals from 'globals'
 import ts from 'typescript-eslint'
 
+import { findFileOrUp } from './helper/findFileOrUp.js'
+
+const pathPrettierIgnore = findFileOrUp('.prettierignore', { absolute: true })
+
 /** @type {import('eslint').Linter.Config[]} */
 export const config = [
-  js.configs.recommended,
+  {
+    name: '@kitql:prettier:ignores',
+    ignores: includeIgnoreFile(pathPrettierIgnore).ignores,
+  },
+  {
+    name: 'eslint/defaults/recommended',
+    ...js.configs.recommended, // TODO, would be nice to have a name by default?
+  },
   ...ts.configs.recommended,
   ...svelte.configs['flat/recommended'],
   {
+    name: '@kitql:languages',
     languageOptions: {
       globals: {
         ...globals.browser,
@@ -18,6 +31,7 @@ export const config = [
     },
   },
   {
+    name: '@kitql:svelte:languages',
     files: ['**/*.svelte'],
     languageOptions: {
       parserOptions: {
@@ -26,10 +40,34 @@ export const config = [
     },
   },
   {
-    ignores: ['build/', '.svelte-kit/', 'dist/'],
+    name: '@kitql:ignores',
+    ignores: ['build/', '.svelte-kit/', 'dist/', '**/build/', '**/.svelte-kit/', '**/dist/'],
   },
   {
-    name: '@kitql rules',
+    name: '@kitql:unused-imports',
+    plugins: {
+      'unused-imports': unusedImports,
+    },
+    rules: {
+      'no-unused-vars': 'off',
+      '@typescript-eslint/no-unused-vars': 'off',
+
+      'unused-imports/no-unused-imports': 'error',
+      'unused-imports/no-unused-vars': 'off',
+      // 'unused-imports/no-unused-vars': [
+      //   'warn',
+      //   {
+      //     vars: 'all',
+      //     varsIgnorePattern: '^_',
+      //     args: 'after-used',
+      //     argsIgnorePattern: '^_',
+      //   },
+      // ],
+      'no-empty': ['error', { allowEmptyCatch: true }],
+    },
+  },
+  {
+    name: '@kitql:rules',
     rules: {
       'no-console': [
         'error',
@@ -52,26 +90,6 @@ export const config = [
       'svelte/no-inner-declarations': 'off',
     },
   },
-  {
-    plugins: {
-      'unused-imports': unusedImports,
-    },
-    rules: {
-      'no-unused-vars': 'off', // or "@typescript-eslint/no-unused-vars": "off",
-      '@typescript-eslint/no-unused-vars': 'off',
-      'unused-imports/no-unused-imports': 'error',
-      'unused-imports/no-unused-vars': 'off',
-      // 'unused-imports/no-unused-vars': [
-      //   'warn',
-      //   {
-      //     vars: 'all',
-      //     varsIgnorePattern: '^_',
-      //     args: 'after-used',
-      //     argsIgnorePattern: '^_',
-      //   },
-      // ],
-    },
-  },
 ]
 
 export default config

package/helper/findFileOrUp.js

@@ -0,0 +1,22 @@
+import { statSync } from 'fs'
+import { resolve } from 'path'
+
+export const findFileOrUp = (fileName, options) => {
+  // Find file recursively 4 levels max up
+  for (let i = 0; i < 4; i++) {
+    try {
+      const pathFound = '../'.repeat(i) + fileName
+      if (statSync(pathFound)) {
+        if (options?.absolute) {
+          return resolve(pathFound)
+        }
+        return pathFound
+      }
+    } catch (error) {
+      // nothing to do
+    }
+  }
+
+  console.error(`"${fileName}" not found`)
+  return null
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kitql/eslint-config",
-  "version": "0.4.0-next.1",
+  "version": "0.4.0-next.3",
   "type": "module",
   "description": "opinionated linting and formatting for projects",
   "repository": {
@@ -19,7 +19,8 @@
     "cmd.js",
     "cmd.sh",
     "eslint.config.js",
-    "eslint.config.d.ts"
+    "eslint.config.d.ts",
+    "helper/findFileOrUp.js"
   ],
   "keywords": [
     "cli",
@@ -30,11 +31,13 @@
     "prettier": "^3.3.3"
   },
   "dependencies": {
+    "@eslint/compat": "^1.1.1",
     "@eslint/js": "^9.10.0",
     "@theguild/prettier-config": "2.0.7",
     "@types/eslint": "9.6.1",
+    "@typescript-eslint/parser": "^8.5.0",
     "commander": "12.1.0",
-    "eslint": "9.10.0",
+    "eslint": "^9.10.0",
     "eslint-plugin-svelte": "2.43.0",
     "eslint-plugin-unused-imports": "^4.1.3",
     "globals": "15.9.0",
@@ -52,6 +55,7 @@
     "format": "node ./cmd.js -f",
     "format:example": "kitql-lint --format",
     "lint": "node ./cmd.js --verbose -p none",
-    "lint:example": "kitql-lint"
+    "lint:example": "kitql-lint",
+    "inspector": "npx @eslint/config-inspector"
   }
 }