@vijayhardaha/dev-config 2.0.1 → 2.0.3

package/README.md

@@ -79,7 +79,7 @@ v2 drops FlatCompat and uses native ESLint 10 flat configs throughout.
 
 1. Install ESLint 10+: `bun add --dev eslint@10`
 2. Replace `eslint-plugin-import` with `eslint-plugin-import-x`
-3. Remove unused deps: `@eslint/compat`, `@eslint/eslintrc`, `@eslint/js`, `eslint-config-prettier`, `eslint-import-resolver-typescript`
+3. Remove unused deps: `@eslint/eslintrc`, `@eslint/js`, `eslint-config-prettier`, `eslint-import-resolver-typescript`
 
 ## Quick Start
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vijayhardaha/dev-config",
-  "version": "2.0.1",
+  "version": "2.0.3",
   "description": "Reusable development configurations for Next.js + TypeScript projects",
   "scripts": {
     "lint": "eslint .",
@@ -36,10 +36,17 @@
   },
   "exports": {
     ".": "./src/index.js",
+    "./package.json": "./package.json",
     "./eslint": "./src/eslint/index.js",
+    "./eslint/js": "./src/eslint/index.js",
+    "./eslint/base": "./src/eslint/index.js",
+    "./eslint/common": "./src/eslint/index.js",
     "./eslint/ts": "./src/eslint/typescript.js",
+    "./eslint/typescript": "./src/eslint/typescript.js",
     "./eslint/react": "./src/eslint/react.js",
+    "./eslint/reactjs": "./src/eslint/react.js",
     "./eslint/next": "./src/eslint/next.js",
+    "./eslint/nextjs": "./src/eslint/next.js",
     "./prettier": "./src/prettier/index.js",
     "./commitlint": "./src/commitlint/index.js",
     "./stylelint": "./src/stylelint/index.js",

package/src/eslint/lib/build-config.js

@@ -3,7 +3,6 @@ import { defineConfig } from 'eslint/config';
 import importX from 'eslint-plugin-import-x';
 import jsdocPlugin from 'eslint-plugin-jsdoc';
 import prettierRecommended from 'eslint-plugin-prettier/recommended';
-import tsEslint from 'typescript-eslint';
 
 import { globalIgnores } from './ignores.js';
 import { commonLanguageOptions } from './language-options.js';
@@ -11,7 +10,7 @@ import { commonRules } from './rules.js';
 import { commonParser } from './setup.js';
 
 /**
- * Filters and flattens conditional plugins based on user options.
+ * Filters conditional plugins based on user options.
  *
  * @param {object} conditionalPlugins - Plugin map keyed by option name.
  * @param {object} options - User-provided options.
@@ -24,7 +23,7 @@ const getEnabledPlugins = (conditionalPlugins, options) =>
     .flatMap(([, value]) => (Array.isArray(value) ? value : [value]));
 
 /**
- * Flattens a mixed list of config arrays and objects into a flat config array.
+ * Flattens a mixed list of config arrays and objects.
  *
  * @param {(Array|object)[]} plugins - Mixed list of flat config arrays and objects.
  *
@@ -45,7 +44,7 @@ const flattenPlugins = (plugins) => {
 };
 
 /**
- * Wraps plugin rules with fixupPluginRules for ESLint 10 backward compatibility.
+ * Wraps plugin rules with fixupPluginRules for backward compatibility.
  *
  * @param {object[]} flatConfigs - Flat config array.
  *
@@ -65,9 +64,61 @@ const fixPlugins = (flatConfigs) =>
   });
 
 /**
- * Merges user-provided global ignores with the common defaults.
+ * Removes centrally-registered plugins from individual configs to prevent
+ * "Cannot redefine plugin" errors.
  *
- * @param {string[]|undefined} userGlobalIgnores - User-provided global ignore patterns.
+ * @param {object[]} flatConfigs - Flat config array.
+ * @param {string[]} pluginNames - Plugin names to strip from configs.
+ *
+ * @returns {object[]} Config array with specified plugins removed.
+ */
+const stripPlugins = (flatConfigs, pluginNames) => {
+  if (pluginNames.length === 0) return flatConfigs;
+
+  const skip = new Set(pluginNames);
+
+  return flatConfigs.map((config) => {
+    if (!config.plugins) return config;
+
+    const plugins = { ...config.plugins };
+
+    for (const name of skip) {
+      delete plugins[name];
+    }
+
+    if (Object.keys(plugins).length === 0) {
+      const { plugins: _, ...rest } = config;
+
+      return rest;
+    }
+
+    return { ...config, plugins };
+  });
+};
+
+/**
+ * Removes the parser from individual configs when the main config provides one.
+ * Prevents parser conflicts (e.g., eslint-config-next/parser vs `@typescript-eslint/parser`).
+ *
+ * @param {object[]} flatConfigs - Flat config array.
+ *
+ * @returns {object[]} Config array with parsers removed.
+ */
+const stripParser = (flatConfigs) =>
+  flatConfigs.map((config) => {
+    if (!config.languageOptions?.parser) return config;
+
+    const { parser: _, ...languageOptions } = config.languageOptions;
+
+    return Object.keys(languageOptions).length > 0
+      ? { ...config, languageOptions }
+      : { ...config, languageOptions: undefined };
+  });
+
+/**
+ * Merges user-provided global ignores with common defaults.
+ *
+ * @param {string[]|undefined} userGlobalIgnores - User-provided ignore patterns.
  *
  * @returns {object} ESLint ignores config object.
  */
@@ -75,12 +126,14 @@ const mergeGlobalIgnores = (userGlobalIgnores) =>
   Array.isArray(userGlobalIgnores) ? globalIgnores(userGlobalIgnores) : globalIgnores();
 
 /**
- * Builds the main ESLint config object with language options, settings, and rules.
+ * Builds the main ESLint config object with language options, settings, rules,
+ * and centrally-registered plugins.
  *
  * @param {object} ctx - Context object with all config parameters.
  * @param {string[]} ctx.filePatterns - File patterns to apply the config to.
  * @param {object} ctx.opts - Resolved user options.
  * @param {boolean} ctx.typescript - Enable TypeScript support.
+ * @param {object} ctx.centralPlugins - Plugins to register on the main config.
  * @param {object} ctx.extraLanguageOptions - Additional language options.
  * @param {object} ctx.parserOptions - Parser options.
  * @param {object} ctx.extraSettings - Additional settings.
@@ -92,6 +145,7 @@ const buildConfigObject = ({
   filePatterns,
   opts,
   typescript,
+  centralPlugins = {},
   extraLanguageOptions,
   parserOptions,
   extraSettings,
@@ -102,12 +156,13 @@ const buildConfigObject = ({
   return {
     files: [...filePatterns],
     ...(ignores && { ignores }),
-    ...(typescript && { plugins: { '@typescript-eslint': tsEslint.plugin } }),
+    plugins: Object.fromEntries(
+      Object.entries(centralPlugins).map(([name, plugin]) => [name, fixupPluginRules(plugin)])
+    ),
     languageOptions: {
       ...commonLanguageOptions,
       ...(typescript && commonParser),
       ...extraLanguageOptions,
-      ...languageOptions,
       ...(typescript && { parserOptions: { tsconfigRootDir: process.cwd(), ...parserOptions } }),
     },
     settings: {
@@ -126,12 +181,13 @@ const buildConfigObject = ({
 };
 
 /**
- * Builds a flat ESLint configuration with support for various options.
+ * Builds a flat ESLint configuration.
  *
  * @param {object} config - Configuration options.
  * @param {string[]} config.files - File patterns to apply the config to.
  * @param {(Array|object)[]} config.builtinPlugins - Flat config arrays or objects to always include.
  * @param {object} config.conditionalPlugins - Conditional plugins based on options.
+ * @param {object} config.centralPlugins - Plugins registered on the main config object.
  * @param {object} [config.languageOptions] - Additional language options.
  * @param {object} [config.parserOptions] - Parser options.
  * @param {object} [config.settings] - Settings object.
@@ -145,6 +201,7 @@ export const buildConfig = ({
   files: filePatterns,
   builtinPlugins = [],
   conditionalPlugins = {},
+  centralPlugins = {},
   languageOptions: extraLanguageOptions = {},
   parserOptions = {},
   settings: extraSettings = {},
@@ -166,18 +223,21 @@ export const buildConfig = ({
   ].filter(Boolean);
 
   const flatConfigs = flattenPlugins(mergedPlugins);
-  const fixedConfigs = fixPlugins(flatConfigs);
+  const wrappedConfigs = fixPlugins(flatConfigs);
+  const strippedConfigs = stripPlugins(wrappedConfigs, Object.keys(centralPlugins));
+  const parsedConfigs = typescript ? stripParser(strippedConfigs) : strippedConfigs;
   const mergedGlobalIgnores = mergeGlobalIgnores(opts.globalIgnores);
 
   const configObject = buildConfigObject({
     filePatterns,
     opts,
     typescript,
+    centralPlugins,
     extraLanguageOptions,
     parserOptions,
     extraSettings,
     extraRules,
   });
 
-  return defineConfig([...mergedGlobalIgnores, ...fixedConfigs, configObject]);
+  return defineConfig([...mergedGlobalIgnores, ...parsedConfigs, configObject]);
 };

package/src/eslint/next.js

@@ -10,14 +10,18 @@
  */
 
 import nextCoreWebVitals from 'eslint-config-next/core-web-vitals';
+import reactPlugin from 'eslint-plugin-react';
+import reactHooks from 'eslint-plugin-react-hooks';
+import tsEslint from 'typescript-eslint';
 
 import { buildConfig, files } from './lib/index.js';
 
 /**
  * Removes the next/typescript item from the core-web-vitals config array.
  * The \@typescript-eslint plugin is registered on the main config object
- * instead (via build-config.js when typescript: true), which avoids plugin
- * redefinition errors when TypeScript rules are applied in the shared config.
+ * via centralPlugins, which avoids plugin redefinition errors. The
+ * next/typescript config is also removed to prevent parser conflicts with
+ * the centrally-managed TypeScript setup.
  *
  * @param {import('eslint').Linter.Config[]} configs - Config array to process.
  *
@@ -52,6 +56,7 @@ export const createConfig = (options = {}) => {
   return buildConfig({
     files: files.withTs,
     builtinPlugins: [...prepareNextConfig(nextCoreWebVitals)],
+    centralPlugins: { react: reactPlugin, 'react-hooks': reactHooks, '@typescript-eslint': tsEslint.plugin },
     parserOptions: { ecmaFeatures: { jsx: true } },
     settings: { react: { version: 'detect' } },
     rules: {

package/src/eslint/react.js

@@ -10,6 +10,7 @@
  */
 
 import jsxA11y from 'eslint-plugin-jsx-a11y';
+import reactPlugin from 'eslint-plugin-react';
 import reactRecommended from 'eslint-plugin-react/configs/recommended.js';
 import reactHooks from 'eslint-plugin-react-hooks';
 import tsEslint from 'typescript-eslint';
@@ -42,11 +43,12 @@ export const createConfig = (options = {}) => {
   return buildConfig({
     files: files.withTs,
     builtinPlugins: [
-      { ...reactRecommended, files: ['**/*.{jsx,tsx}'] },
-      { ...reactHooks.configs.flat.recommended, files: ['**/*.{jsx,tsx}'] },
+      { ...reactRecommended, files: files.withTs },
+      { ...reactHooks.configs.flat.recommended, files: files.withTs },
       ...tsEslint.configs.recommended,
     ],
-    conditionalPlugins: { a11y: { ...jsxA11y.flatConfigs.recommended, files: ['**/*.{jsx,tsx}'] } },
+    centralPlugins: { react: reactPlugin, 'react-hooks': reactHooks, '@typescript-eslint': tsEslint.plugin },
+    conditionalPlugins: { a11y: { ...jsxA11y.flatConfigs.recommended, files: files.withTs } },
     parserOptions: { ecmaFeatures: { jsx: true } },
     settings: { react: { version: 'detect' } },
     rules: { 'react/react-in-jsx-scope': 'off', 'react/no-unknown-property': ['error', { ignore: ['jsx', 'global'] }] },

package/src/eslint/typescript.js

@@ -37,6 +37,7 @@ export const createConfig = (options = {}) => {
   return buildConfig({
     files: files.withTs,
     builtinPlugins: [...tsEslint.configs.recommended],
+    centralPlugins: { '@typescript-eslint': tsEslint.plugin },
     typescript: true,
     options: { ...options, prettier, importOrder, jsdoc },
   });