@mouse_484/oxc-config 0.8.11 → 0.8.12

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@mouse_484/oxc-config",
   "type": "module",
-  "version": "0.8.11",
+  "version": "0.8.12",
   "author": "mouse_484",
   "license": "MIT",
   "homepage": "https://github.com/mouse484/config/tree/main/packages/oxc",

package/scripts/typegen.js

@@ -77,12 +77,16 @@ dtsFile.addInterface({
       }
       return {
         name,
-        type: options ? `boolean | ${options.getText()}` : 'boolean',
+        type: options
+          ? `boolean | ${options.getText()}`
+          : 'boolean',
         hasQuestionToken: true,
-        docs: [{
-          description: `\`${name}\` option`,
-          tags,
-        }],
+        docs: [
+          {
+            description: `\`${name}\` option`,
+            tags,
+          },
+        ],
       }
     }),
 })

package/src/configs/eslint.js

@@ -8,13 +8,25 @@ export default createConfigs({
       plugins: ['eslint'],
       rules: {
         'id-length': 'off',
-        'func-style': ['error', 'declaration', { allowArrowFunctions: true }],
-        'no-magic-numbers': ['error', { ignore: [0, 1, -1] }],
+        'func-style': [
+          'error',
+          'declaration',
+          { allowArrowFunctions: true },
+        ],
+        'no-magic-numbers': [
+          'error',
+          {
+            ignore: [0, 1, -1],
+          },
+        ],
         'no-undefined': 'off',
         'no-ternary': 'off',
         'no-nested-ternary': 'error',
         'no-unneeded-ternary': 'error',
-        'no-void': ['error', { allowAsStatement: true }],
+        'no-void': [
+          'error',
+          { allowAsStatement: true },
+        ],
         'one-var': 'off',
         'no-warning-comments': 'off',
         'capitalized-comments': 'off',

package/src/configs/import.js

@@ -5,16 +5,27 @@ export default createConfigs({
   enable: true,
   configs: [
     {
-      jsPlugins: [{
-        name: 'perfectionist',
-        specifier: import.meta.resolve('eslint-plugin-perfectionist'),
-      }],
+      jsPlugins: [
+        {
+          name: 'perfectionist',
+          specifier: import.meta.resolve('eslint-plugin-perfectionist'),
+        },
+      ],
       rules: {
         // `sort-imports` is handled by oxfmt (`sortImports`)
         'perfectionist/sort-imports': 'off',
-        'perfectionist/sort-named-imports': ['error', { type: 'natural', order: 'asc' }],
-        'perfectionist/sort-named-exports': ['error', { type: 'natural', order: 'asc' }],
-        'perfectionist/sort-exports': ['error', { type: 'natural', order: 'asc' }],
+        'perfectionist/sort-named-imports': [
+          'error',
+          { type: 'natural', order: 'asc' },
+        ],
+        'perfectionist/sort-named-exports': [
+          'error',
+          { type: 'natural', order: 'asc' },
+        ],
+        'perfectionist/sort-exports': [
+          'error',
+          { type: 'natural', order: 'asc' },
+        ],
       },
     },
     {

package/src/configs/index.js

@@ -30,19 +30,26 @@ const CONFIGS = /** @type {const} */[
 @returns {import("oxlint").OxlintConfig[]} Generated configs for the enabled options
  */
 export default function buildConfigs(options) {
-  const context = { isPackageExists: createPackageDetector(), options }
+  const context = {
+    isPackageExists: createPackageDetector(),
+    options,
+  }
   return CONFIGS.flatMap(({ name, build, enable = true }) => {
     const option = options[name]
     if (option === false) {
       return []
     }
     if (option === undefined) {
-      const isEnabled = typeof enable === 'function' ? enable(context) : enable
+      const isEnabled = typeof enable === 'function'
+        ? enable(context)
+        : enable
       if (!isEnabled) {
         return []
       }
     }
     // @ts-expect-error - build configs with options
-    return build(context, typeof option === 'object' ? option : undefined)
+    return build(context, typeof option === 'object'
+      ? option
+      : undefined)
   })
 }

package/src/configs/opinionated.js

@@ -5,14 +5,19 @@ export default createConfigs({
   enable: true,
   configs: [
     {
-      jsPlugins: [{
-        name: 'antfu',
-        specifier: import.meta.resolve('eslint-plugin-antfu'),
-      }],
+      jsPlugins: [
+        {
+          name: 'antfu',
+          specifier: import.meta.resolve('eslint-plugin-antfu'),
+        },
+      ],
       rules: {
         'antfu/top-level-function': 'error',
         'antfu/import-dedupe': 'error',
-        'antfu/indent-unindent': ['error', { indent: 2 }],
+        'antfu/indent-unindent': [
+          'error',
+          { indent: 2 },
+        ],
         'antfu/no-ts-export-equal': 'error',
       },
     },

package/src/configs/react.js

@@ -9,10 +9,23 @@ export default createConfigs({
       rules: {
         'react/react-in-jsx-scope': 'off',
         'react/jsx-props-no-spreading': 'off',
-        'react/jsx-max-depth': ['error', { max: 5 }],
-        'react/jsx-filename-extension': ['error', { extensions: ['.jsx', '.tsx'] }],
+        'react/jsx-max-depth': [
+          'error',
+          { max: 5 },
+        ],
+        'react/jsx-filename-extension': [
+          'error',
+          {
+            extensions: ['.jsx', '.tsx'],
+          },
+        ],
         'react/jsx-no-literals': 'off',
-        'react/forbid-component-props': ['error', { forbid: ['style'] }],
+        'react/forbid-component-props': [
+          'error',
+          {
+            forbid: ['style'],
+          },
+        ],
       },
     },
     {

package/src/configs/tanstack.js

@@ -11,8 +11,16 @@ export default createConfigs({
             {
               files: ['src/routes/**/*.tsx'],
               rules: {
-                'no-use-before-define': ['error', { functions: false }],
-                'react/only-export-components': ['error', { customHOCs: ['createRootRoute', 'createFileRoute', 'createRootRouteWithContext'] }],
+                'no-use-before-define': [
+                  'error',
+                  { functions: false },
+                ],
+                'react/only-export-components': [
+                  'error',
+                  {
+                    customHOCs: ['createRootRoute', 'createFileRoute', 'createRootRouteWithContext'],
+                  },
+                ],
               },
             },
           ],

package/src/format.js

@@ -13,5 +13,8 @@ const DEFAULT_FORMAT = /** @type {import('oxfmt').OxfmtConfig} */ ({
  * @param {import('oxfmt').OxfmtConfig} [config]
  */
 export function format(config = {}) {
-  return { ...DEFAULT_FORMAT, ...config }
+  return {
+    ...DEFAULT_FORMAT,
+    ...config,
+  }
 }

package/src/lib/factory.js

@@ -7,10 +7,20 @@ export function createConfigs({ name, enable = true, options, configs }) {
     enable,
     options,
     build: (context, overriddenOptions) => {
-      const local = { ...context, options: { ...options, ...overriddenOptions } }
+      const local = {
+        ...context,
+        options: {
+          ...options,
+          ...overriddenOptions,
+        },
+      }
       return configs.flatMap((item) => {
-        const config = typeof item === 'function' ? item(local) : item
-        return config ? [config] : []
+        const config = typeof item === 'function'
+          ? item(local)
+          : item
+        return config
+          ? [config]
+          : []
       })
     },
   }

package/src/lib/packages.test.js

@@ -13,11 +13,21 @@ void test('detects packages from the consumer, including import-only and types-o
   const project = path.join(root, 'project')
   mkdirSync(project)
   const fixtures = {
-    'import-only': { exports: { import: './index.js' }, type: 'module' },
+    'import-only': {
+      exports: { import: './index.js' },
+      type: 'module',
+    },
     'hidden-manifest': { exports: './index.js', type: 'module' },
-    'types-only-export': { exports: { '.': { types: './index.d.ts' } } },
+    'types-only-export': {
+      exports: {
+        '.': { types: './index.d.ts' },
+      },
+    },
     '@types/example': { types: './index.d.ts' },
-    'conditional': { exports: { import: './index.js', require: './missing.cjs' }, type: 'module' },
+    'conditional': {
+      exports: { import: './index.js', require: './missing.cjs' },
+      type: 'module',
+    },
   }
   for (const [name, manifest] of Object.entries(fixtures)) {
     const directory = path.join(root, 'node_modules', name)
@@ -29,28 +39,28 @@ void test('detects packages from the consumer, including import-only and types-o
     writeFileSync(path.join(directory, 'index.d.ts'), 'export {}\n')
   }
 
-  const detect = createPackageDetector(project)
+  const isPackageInstalled = createPackageDetector(project)
   for (const name of Object.keys(fixtures)) {
-    assert.equal(detect(name), true, name)
+    assert.equal(isPackageInstalled(name), true, name)
   }
-  assert.equal(detect('missing-package'), false)
+  assert.equal(isPackageInstalled('missing-package'), false)
   // Packages installed beside the config must not enable consumer presets.
-  assert.equal(detect('eslint-plugin-antfu'), false)
+  assert.equal(isPackageInstalled('eslint-plugin-antfu'), false)
 
   symlinkSync(path.join(root, 'node_modules/import-only'), path.join(root, 'node_modules/linked-package'), 'junction')
-  assert.equal(detect('linked-package'), true)
+  assert.equal(isPackageInstalled('linked-package'), true)
 
   const broken = path.join(root, 'node_modules/broken-export')
   mkdirSync(broken)
   writeFileSync(path.join(broken, 'package.json'), JSON.stringify({ exports: './missing.js' }))
-  assert.equal(detect('broken-export'), false)
+  assert.equal(isPackageInstalled('broken-export'), false)
 })
 
 void test('a new detector observes the current project and newly installed packages', (context) => {
   const root = mkdtempSync(path.join(tmpdir(), 'oxc-package-detection-'))
   context.after(() => rmSync(root, { recursive: true, force: true }))
-  const detect = createPackageDetector(root)
-  assert.equal(detect('react'), false)
+  const isPackageInstalled = createPackageDetector(root)
+  assert.equal(isPackageInstalled('react'), false)
 
   const directory = path.join(root, 'node_modules/react')
   mkdirSync(directory, { recursive: true })

package/src/lint.js

@@ -10,22 +10,41 @@ const DEFAULT_CATEGORIES = {
   nursery: 'warn',
 }
 
-const OBJECT_KEYS = ['settings', 'env', 'globals', 'options']
+const OBJECT_KEYS = [
+  'settings',
+  'env',
+  'globals',
+  'options',
+]
 const ARRAY_KEYS = ['ignorePatterns']
-const HOISTED_KEYS = [...OBJECT_KEYS, ...ARRAY_KEYS, 'categories']
+const HOISTED_KEYS = [
+  ...OBJECT_KEYS,
+  ...ARRAY_KEYS,
+  'categories',
+]
 
 /**
 @type {import('.').lint}
  */
 export function lint(options, config = {}) {
-  const all = [...buildConfigs(options), config]
+  const all = [
+    ...buildConfigs(options),
+    config,
+  ]
 
   /**
   @type {Record<string, any>}
    */
   const root = {
-    categories: { ...DEFAULT_CATEGORIES, ...config.categories },
+    categories: {
+      ...DEFAULT_CATEGORIES,
+      ...config.categories,
+    },
   }
+  /**
+  @type {string[]}
+   */
+  let ignorePatterns = []
 
   for (const item of all) {
     for (const key of OBJECT_KEYS) {
@@ -34,18 +53,22 @@ export function lint(options, config = {}) {
       const value = item[key]
       if (value !== undefined) {
         // eslint-disable-next-line ts/no-unsafe-assignment
-        root[key] = { ...root[key], ...value }
+        root[key] = {
+          ...root[key],
+          ...value,
+        }
       }
     }
-    for (const key of ARRAY_KEYS) {
-      // @ts-expect-error - hoisted keys are not part of the OxlintConfig index signature
-      // eslint-disable-next-line ts/no-unsafe-assignment
-      const value = item[key]
-      if (value !== undefined) {
-        // eslint-disable-next-line ts/no-unsafe-assignment
-        root[key] = [...new Set([...(root[key] ?? []), ...value])]
-      }
+    if (item.ignorePatterns === undefined) {
+      continue
     }
+    ignorePatterns = [
+      ...new Set([
+        ...ignorePatterns,
+        ...item.ignorePatterns,
+      ]),
+    ]
+    root.ignorePatterns = ignorePatterns
   }
 
   return {

package/src/lint.test.js

@@ -0,0 +1,17 @@
+import assert from 'node:assert/strict'
+// eslint-disable-next-line test/no-import-node-test -- This package uses the built-in test runner.
+import { test } from 'node:test'
+import { lint } from './lint.js'
+
+void test('merges and deduplicates ignore patterns in order', () => {
+  const config = lint({ tanstack: true }, {
+    ignorePatterns: ['**/src/routeTree.gen.ts', 'custom/**'],
+  })
+  assert.deepEqual(config.ignorePatterns, ['**/src/routeTree.gen.ts', 'custom/**'])
+  assert.equal(config.extends?.some(item => item.ignorePatterns !== undefined), false)
+})
+
+void test('preserves an explicitly empty ignore list', () => {
+  const config = lint({ tanstack: false }, { ignorePatterns: [] })
+  assert.deepEqual(config.ignorePatterns, [])
+})