@vijayhardaha/dev-config 2.1.0 → 2.1.1

package/README.md

@@ -6,7 +6,7 @@
 
 Reusable development configuration package for Next.js + TypeScript projects.
 
-> **v2.0.4** — Requires ESLint >=10. Native flat config only. No FlatCompat.
+> **v2.1.0** — Requires ESLint >=10. Native flat config only. No FlatCompat.
 
 ## Features
 
@@ -27,7 +27,7 @@ bun add --dev @vijayhardaha/dev-config
 ### Install Required Packages
 
 ```bash
-bun add --dev eslint @eslint/compat eslint-config-prettier prettier @prettier/plugin-xml eslint-plugin-prettier globals eslint-plugin-jsdoc eslint-plugin-import-x @typescript-eslint/eslint-plugin @typescript-eslint/parser typescript typescript-eslint husky
+bun add --dev eslint @eslint/compat @eslint/js eslint-config-prettier prettier @prettier/plugin-xml eslint-plugin-prettier globals eslint-plugin-jsdoc eslint-plugin-import-x @typescript-eslint/eslint-plugin @typescript-eslint/parser typescript typescript-eslint husky
 ```
 
 ### Install Optional Packages
@@ -56,6 +56,7 @@ bun add --dev eslint-plugin-react eslint-plugin-react-hooks eslint-plugin-jsx-a1
 
 ```bash
 bun add --dev @next/eslint-plugin-next eslint-config-next
+bun add --dev next
 ```
 
 #### Commitlint
@@ -79,15 +80,15 @@ v2 drops FlatCompat and uses native ESLint 10 flat configs throughout.
 - **ESLint 10 required** — no longer compatible with ESLint 8/9
 - **No FlatCompat** — all configs import flat config arrays/objects directly
 - **`eslint-plugin-import` → `eslint-plugin-import-x`** — the ESLint 10-compatible fork
-- **`eslint-config-prettier` removed** — bundled in `eslint-plugin-prettier/recommended`
-- **`@eslint/js` removed** — no longer needed without FlatCompat
+- **`eslint-config-prettier` restored** — required by `eslint-plugin-prettier/recommended`
+- **`@eslint/js` restored** — used for the JavaScript recommended rule set
 - **`plugins` option now accepts flat config arrays/objects** — no string-based plugin names
 
 ### Required updates
 
 1. Install ESLint 10+: `bun add --dev eslint@10`
 2. Replace `eslint-plugin-import` with `eslint-plugin-import-x`
-3. Remove unused deps: `@eslint/eslintrc`, `@eslint/js`
+3. Remove unused deps: `@eslint/eslintrc`, `eslint-plugin-import`
 
 ### What's new in v2.0.4
 
@@ -96,6 +97,13 @@ v2 drops FlatCompat and uses native ESLint 10 flat configs throughout.
 - **All packages declared as peer deps** — every config module's dependencies are declared with proper `peerDependenciesMeta`
 - **Flat config resolver fix** — switched from string-based `import-x/resolver` to `import-x/resolver-next` with `createTypeScriptImportResolver` for ESLint 10 compatibility
 
+### What's new in v2.1.0
+
+- **Root import no longer loads optional ESLint integrations** — use ESLint subpath imports for TypeScript, React, and Next.js presets.
+- **JavaScript recommended rules restored** — the base JavaScript preset now includes `@eslint/js` recommended rules.
+- **Next.js peer dependency declared** — the Next.js preset requires `next` because `eslint-config-next` resolves Next's bundled ESLint parser.
+- **Custom file patterns honored consistently** — TypeScript, React, and Next.js presets merge `options.files` into their generated file patterns.
+
 ## Quick Start
 
 ### ESLint

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vijayhardaha/dev-config",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "Reusable development configurations for Next.js + TypeScript projects",
   "scripts": {
     "lint": "eslint .",
@@ -65,28 +65,30 @@
     "typescript"
   ],
   "devDependencies": {
+    "@eslint/js": "^10.0.1",
     "@commitlint/cli": "^21.0.2",
     "@commitlint/config-conventional": "^21.0.2",
     "@commitlint/types": "^21.0.1",
-    "@typescript-eslint/parser": "^8.60.1",
-    "@typescript-eslint/eslint-plugin": "^8.60.1",
+    "@typescript-eslint/parser": "^8.61.0",
+    "@typescript-eslint/eslint-plugin": "^8.61.0",
     "@vitest/coverage-v8": "^4.1.8",
     "eslint": "^10.4.1",
-    "eslint-config-next": "^16.2.7",
+    "eslint-config-next": "^16.2.9",
     "eslint-config-prettier": "^10.1.8",
     "eslint-plugin-import-x": "^4.16.2",
-    "eslint-plugin-jsdoc": "^63.0.1",
+    "eslint-plugin-jsdoc": "^63.0.2",
     "eslint-plugin-jsx-a11y": "^6.10.2",
     "eslint-plugin-prettier": "^5.5.6",
     "eslint-plugin-react": "^7.37.5",
     "eslint-plugin-react-hooks": "^7.1.1",
     "globals": "^17.6.0",
     "husky": "^9.1.7",
-    "prettier": "^3.8.3",
+    "next": "^16.2.9",
+    "prettier": "^3.8.4",
     "@prettier/plugin-xml": "^3.4.2",
     "release-it": "^20.2.0",
     "typescript": "^6.0.3",
-    "typescript-eslint": "^8.60.1",
+    "typescript-eslint": "^8.61.0",
     "vitest": "^4.1.8"
   },
   "peerDependencies": {
@@ -94,6 +96,7 @@
     "@commitlint/config-conventional": ">=21",
     "@commitlint/types": ">=21",
     "@eslint/compat": ">=2",
+    "@eslint/js": ">=10",
     "@next/eslint-plugin-next": ">=16",
     "@prettier/plugin-xml": ">=3",
     "@typescript-eslint/eslint-plugin": ">=8",
@@ -111,6 +114,7 @@
     "globals": ">=17",
     "husky": ">=9",
     "next-sitemap": ">=4",
+    "next": ">=16",
     "prettier": ">=3",
     "stylelint": ">=17",
     "stylelint-config-property-sort-order-smacss": ">=11",
@@ -159,6 +163,9 @@
     "next-sitemap": {
       "optional": true
     },
+    "next": {
+      "optional": true
+    },
     "stylelint": {
       "optional": true
     },

package/src/eslint/index.js

@@ -9,6 +9,8 @@
  * =====================================================================
  */
 
+import js from '@eslint/js';
+
 import { buildConfig, files } from './lib/index.js';
 
 /**
@@ -34,6 +36,7 @@ export const createConfig = (options = {}) => {
 
   return buildConfig({
     files: [...files.withoutTs, ...(options.files || [])],
+    builtinPlugins: [js.configs.recommended],
     options: { ...options, prettier, importOrder, jsdoc },
   });
 };

package/src/eslint/index.test.js

@@ -1,3 +1,4 @@
+import { ESLint } from 'eslint';
 import { describe, it, expect } from 'vitest';
 
 // Test suite for the ESLint module's main entry point.
@@ -19,4 +20,17 @@ describe('eslint/index.js', () => {
     // Verify that the default export is defined (should be a base ESLint config object).
     expect(module.default).toBeDefined();
   });
+
+  // Test that the JavaScript config catches basic correctness issues.
+  it('should include recommended JavaScript rules', async () => {
+    const module = await import('./index.js');
+    const eslint = new ESLint({
+      overrideConfigFile: true,
+      overrideConfig: module.createConfig({ prettier: false, importOrder: false, jsdoc: false }),
+    });
+
+    const [result] = await eslint.lintText('const value = missing + 1;\n', { filePath: 'sample.js' });
+
+    expect(result.messages.some((message) => message.ruleId === 'no-undef')).toBe(true);
+  });
 });

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

@@ -97,7 +97,8 @@ const stripPlugins = (flatConfigs, pluginNames) => {
     }
 
     if (Object.keys(plugins).length === 0) {
-      const { plugins: _, ...rest } = config;
+      const rest = { ...config };
+      delete rest.plugins;
 
       return rest;
     }
@@ -118,7 +119,8 @@ const stripParser = (flatConfigs) =>
   flatConfigs.map((config) => {
     if (!config.languageOptions?.parser) return config;
 
-    const { parser: _, ...languageOptions } = config.languageOptions;
+    const languageOptions = { ...config.languageOptions };
+    delete languageOptions.parser;
 
     return Object.keys(languageOptions).length > 0
       ? { ...config, languageOptions }
@@ -161,7 +163,7 @@ const buildConfigObject = ({
   extraSettings,
   extraRules,
 }) => {
-  const { ignores, rules, settings, languageOptions, extend } = opts;
+  const { ignores, rules, settings, extend } = opts;
 
   return {
     files: [...filePatterns],

package/src/eslint/next.js

@@ -54,7 +54,7 @@ export const createConfig = (options = {}) => {
   const { prettier = true, react = true, a11y = true, importOrder = true, jsdoc = true } = options;
 
   return buildConfig({
-    files: files.withTs,
+    files: [...files.withTs, ...(options.files || [])],
     builtinPlugins: [...prepareNextConfig(nextCoreWebVitals)],
     centralPlugins: { react: reactPlugin, 'react-hooks': reactHooks, '@typescript-eslint': tsEslint.plugin },
     parserOptions: { ecmaFeatures: { jsx: true } },

package/src/eslint/next.test.js

@@ -4,29 +4,28 @@ import { describe, it, expect } from 'vitest';
 describe('eslint/next.js', () => {
   // Test that the module exports the createConfig function.
   it('should export createConfig function', async () => {
-    try {
-      // Dynamically import the next.js module to test its exports.
-      const module = await import('./next.js');
+    // Dynamically import the next.js module to test its exports.
+    const module = await import('./next.js');
 
-      // Verify that createConfig is a function (used to create ESLint config for Next.js).
-      expect(typeof module.createConfig).toBe('function');
-    } catch (error) {
-      // If import fails, expect an error to be defined.
-      expect(error).toBeDefined();
-    }
+    // Verify that createConfig is a function (used to create ESLint config for Next.js).
+    expect(typeof module.createConfig).toBe('function');
   });
 
   // Test that the module exports a default configuration object.
   it('should export default config', async () => {
-    try {
-      // Dynamically import the next.js module to test its exports.
-      const module = await import('./next.js');
+    // Dynamically import the next.js module to test its exports.
+    const module = await import('./next.js');
 
-      // Verify that the default export is defined (should be an ESLint config object).
-      expect(module.default).toBeDefined();
-    } catch (error) {
-      // If import fails, expect an error to be defined.
-      expect(error).toBeDefined();
-    }
+    // Verify that the default export is defined (should be an ESLint config object).
+    expect(module.default).toBeDefined();
+  });
+
+  // Test that additional file patterns are applied to the generated config.
+  it('should include additional file patterns from options', async () => {
+    const module = await import('./next.js');
+    const result = module.createConfig({ files: ['custom/**/*.tsx'] });
+    const configObject = result.at(-1);
+
+    expect(configObject.files).toContain('custom/**/*.tsx');
   });
 });

package/src/eslint/react.js

@@ -41,14 +41,16 @@ export const createConfig = (options = {}) => {
   const { prettier = true, a11y = true, importOrder = true, jsdoc = true } = options;
 
   return buildConfig({
-    files: files.withTs,
+    files: [...files.withTs, ...(options.files || [])],
     builtinPlugins: [
-      { ...reactRecommended, files: files.withTs },
-      { ...reactHooks.configs.flat.recommended, files: files.withTs },
+      { ...reactRecommended, files: [...files.withTs, ...(options.files || [])] },
+      { ...reactHooks.configs.flat.recommended, files: [...files.withTs, ...(options.files || [])] },
       ...tsEslint.configs.recommended,
     ],
     centralPlugins: { react: reactPlugin, 'react-hooks': reactHooks, '@typescript-eslint': tsEslint.plugin },
-    conditionalPlugins: { a11y: { ...jsxA11y.flatConfigs.recommended, files: files.withTs } },
+    conditionalPlugins: {
+      a11y: { ...jsxA11y.flatConfigs.recommended, files: [...files.withTs, ...(options.files || [])] },
+    },
     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/react.test.js

@@ -19,4 +19,13 @@ describe('eslint/react.js', () => {
     // Verify that the default export is defined (should be an ESLint config object).
     expect(module.default).toBeDefined();
   });
+
+  // Test that additional file patterns are applied to the generated config.
+  it('should include additional file patterns from options', async () => {
+    const module = await import('./react.js');
+    const result = module.createConfig({ files: ['custom/**/*.tsx'] });
+    const configObject = result.at(-1);
+
+    expect(configObject.files).toContain('custom/**/*.tsx');
+  });
 });

package/src/eslint/typescript.js

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

package/src/eslint/typescript.test.js

@@ -19,4 +19,13 @@ describe('eslint/typescript.js', () => {
     // Verify that the default export is defined (should be an ESLint config object).
     expect(module.default).toBeDefined();
   });
+
+  // Test that additional file patterns are applied to the generated config.
+  it('should include additional file patterns from options', async () => {
+    const module = await import('./typescript.js');
+    const result = module.createConfig({ files: ['custom/**/*.ts'] });
+    const configObject = result.at(-1);
+
+    expect(configObject.files).toContain('custom/**/*.ts');
+  });
 });

package/src/index.js

@@ -15,27 +15,6 @@
  */
 export { default as eslint, createConfig as createEslintConfig } from './eslint/index.js';
 
-/**
- * ESLint configuration for TypeScript files.
- *
- * @type {import('eslint').Linter.Config}
- */
-export { default as eslintTs, createConfig as createEslintTsConfig } from './eslint/typescript.js';
-
-/**
- * ESLint configuration for React files.
- *
- * @type {import('eslint').Linter.Config}
- */
-export { default as eslintReact, createConfig as createEslintReactConfig } from './eslint/react.js';
-
-/**
- * ESLint configuration for Next.js files.
- *
- * @type {import('eslint').Linter.Config}
- */
-export { default as eslintNext, createConfig as createEslintNextConfig } from './eslint/next.js';
-
 /**
  * Prettier configuration.
  *

package/src/index.test.js

@@ -3,34 +3,23 @@ import { describe, it, expect } from 'vitest';
 // Test suite for the main index.js exports.
 describe('index.js exports', () => {
   // Test that all expected functions and objects are exported from the module.
-  it('should export functions', async () => {
-    try {
-      // Dynamically import the module to test its exports.
-      const module = await import('./index.js');
+  it('should export functions without requiring optional integrations', async () => {
+    // Dynamically import the module to test its exports.
+    const module = await import('./index.js');
 
-      // Verify ESLint-related exports are functions.
-      expect(typeof module.eslint).toBe('function');
-      expect(typeof module.createEslintConfig).toBe('function');
-      expect(typeof module.eslintTs).toBe('function');
-      expect(typeof module.createEslintTsConfig).toBe('function');
-      expect(typeof module.eslintReact).toBe('function');
-      expect(typeof module.createEslintReactConfig).toBe('function');
-      expect(typeof module.eslintNext).toBe('function');
-      expect(typeof module.createEslintNextConfig).toBe('function');
+    // Verify base ESLint exports are available without optional integrations.
+    expect(Array.isArray(module.eslint)).toBe(true);
+    expect(typeof module.createEslintConfig).toBe('function');
 
-      // Verify config objects are exported.
-      expect(typeof module.prettier).toBe('object');
-      expect(typeof module.commitlint).toBe('object');
-      expect(typeof module.nextSitemap).toBe('object');
+    // Verify config objects are exported.
+    expect(typeof module.prettier).toBe('object');
+    expect(typeof module.commitlint).toBe('object');
+    expect(typeof module.nextSitemap).toBe('object');
 
-      // Verify sitemap config function is exported.
-      expect(typeof module.createSitemapConfig).toBe('function');
+    // Verify sitemap config function is exported.
+    expect(typeof module.createSitemapConfig).toBe('function');
 
-      // Verify Stylelint config object is exported.
-      expect(typeof module.stylelint).toBe('object');
-    } catch (error) {
-      // If import fails, expect an error to be defined.
-      expect(error).toBeDefined();
-    }
+    // Verify Stylelint config object is exported.
+    expect(typeof module.stylelint).toBe('object');
   });
 });

package/src/prettier/index.js

@@ -71,7 +71,10 @@ const config = {
 
     // ---- XML ----
     // Use 2-space indentation for XML files
-    { files: ['.xml', '.xsd', '.xsl', '.xslt'], options: { tabWidth: 2 } },
+    {
+      files: ['**/*.xml', '**/*.xsd', '**/*.xsl', '**/*.xslt'],
+      options: { tabWidth: 2, xmlWhitespaceSensitivity: 'preserve' },
+    },
   ],
 };
 

package/src/prettier/index.test.js

@@ -36,4 +36,12 @@ describe('prettier/index.js', () => {
     expect(Array.isArray(config.overrides)).toBe(true);
     expect(config.overrides.length).toBeGreaterThan(0);
   });
+
+  // Test that XML-like extensions are matched as normal file globs.
+  it('should use XML glob patterns that match normal files', async () => {
+    const module = await import('./index.js');
+    const xmlOverride = module.default.overrides.find((override) => override.options?.xmlWhitespaceSensitivity);
+
+    expect(xmlOverride.files).toEqual(['**/*.xml', '**/*.xsd', '**/*.xsl', '**/*.xslt']);
+  });
 });