@vijayhardaha/dev-config 2.4.1 → 2.4.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vijayhardaha/dev-config",
-  "version": "2.4.1",
+  "version": "2.4.2",
   "description": "Reusable development configurations for Next.js + TypeScript projects",
   "scripts": {
     "lint": "eslint .",

package/src/eslint/__tests__/next.test.js

@@ -29,12 +29,27 @@ describe('eslint/next.js', () => {
     expect(configObject.files).toContain('custom/**/*.tsx');
   });
 
-  // Test that Tailwind plugins are registered on the main config by default.
-  it('should register Tailwind plugins by default', async () => {
+  // Test that Tailwind plugins and rules are omitted by default.
+  it('should omit Tailwind plugins and rules by default', async () => {
     const module = await import('../next.js');
     const result = module.createConfig();
     const configObject = result.at(-1);
 
+    // Verify that no Tailwind plugins are centrally registered.
+    expect(configObject.plugins['better-tailwindcss']).toBeUndefined();
+    expect(configObject.plugins.tailwindcss).toBeUndefined();
+
+    // Verify that no Tailwind rules are present in the config.
+    expect(Object.keys(configObject.rules).filter((rule) => rule.startsWith('better-tailwindcss/'))).toHaveLength(0);
+    expect(Object.keys(configObject.rules).filter((rule) => rule.startsWith('tailwindcss/'))).toHaveLength(0);
+  });
+
+  // Test that Tailwind plugins, rules, and settings register when opted in.
+  it('should register Tailwind plugins when tailwind option is true', async () => {
+    const module = await import('../next.js');
+    const result = module.createConfig({ tailwind: true });
+    const configObject = result.at(-1);
+
     // Verify that the better-tailwindcss plugin is centrally registered.
     expect(configObject.plugins['better-tailwindcss']).toBeDefined();
 
@@ -48,25 +63,6 @@ describe('eslint/next.js', () => {
     expect(configObject.rules['tailwindcss/no-unnecessary-arbitrary-value']).toBe('warn');
   });
 
-  // Test that Tailwind plugins, rules, and settings are omitted when disabled.
-  it('should omit Tailwind plugins and rules when tailwind option is false', async () => {
-    const module = await import('../next.js');
-    const result = module.createConfig({ tailwind: false });
-    const configObject = result.at(-1);
-
-    // Verify that no Tailwind plugins are centrally registered.
-    expect(configObject.plugins['better-tailwindcss']).toBeUndefined();
-    expect(configObject.plugins.tailwindcss).toBeUndefined();
-
-    // Verify that no Tailwind rules are present in the config.
-    expect(Object.keys(configObject.rules).filter((rule) => rule.startsWith('better-tailwindcss/'))).toHaveLength(0);
-    expect(Object.keys(configObject.rules).filter((rule) => rule.startsWith('tailwindcss/'))).toHaveLength(0);
-
-    // Verify that no Tailwind settings are present in the config.
-    expect(configObject.settings.tailwindcss).toBeUndefined();
-    expect(configObject.settings['better-tailwindcss']).toBeUndefined();
-  });
-
   // Test that user settings override the generated Tailwind settings.
   it('should let user settings override Tailwind settings', async () => {
     const module = await import('../next.js');

package/src/eslint/__tests__/react.test.js

@@ -29,12 +29,27 @@ describe('eslint/react.js', () => {
     expect(configObject.files).toContain('custom/**/*.tsx');
   });
 
-  // Test that Tailwind plugins are registered on the main config by default.
-  it('should register Tailwind plugins by default', async () => {
+  // Test that Tailwind plugins and rules are omitted by default.
+  it('should omit Tailwind plugins and rules by default', async () => {
     const module = await import('../react.js');
     const result = module.createConfig();
     const configObject = result.at(-1);
 
+    // Verify that no Tailwind plugins are centrally registered.
+    expect(configObject.plugins['better-tailwindcss']).toBeUndefined();
+    expect(configObject.plugins.tailwindcss).toBeUndefined();
+
+    // Verify that no Tailwind rules are present in the config.
+    expect(Object.keys(configObject.rules).filter((rule) => rule.startsWith('better-tailwindcss/'))).toHaveLength(0);
+    expect(Object.keys(configObject.rules).filter((rule) => rule.startsWith('tailwindcss/'))).toHaveLength(0);
+  });
+
+  // Test that Tailwind plugins, rules, and settings register when opted in.
+  it('should register Tailwind plugins when tailwind option is true', async () => {
+    const module = await import('../react.js');
+    const result = module.createConfig({ tailwind: true });
+    const configObject = result.at(-1);
+
     // Verify that the better-tailwindcss plugin is centrally registered.
     expect(configObject.plugins['better-tailwindcss']).toBeDefined();
 
@@ -47,23 +62,4 @@ describe('eslint/react.js', () => {
     // Verify that the arbitrary value replacement rule is enabled.
     expect(configObject.rules['tailwindcss/no-unnecessary-arbitrary-value']).toBe('warn');
   });
-
-  // Test that Tailwind plugins, rules, and settings are omitted when disabled.
-  it('should omit Tailwind plugins and rules when tailwind option is false', async () => {
-    const module = await import('../react.js');
-    const result = module.createConfig({ tailwind: false });
-    const configObject = result.at(-1);
-
-    // Verify that no Tailwind plugins are centrally registered.
-    expect(configObject.plugins['better-tailwindcss']).toBeUndefined();
-    expect(configObject.plugins.tailwindcss).toBeUndefined();
-
-    // Verify that no Tailwind rules are present in the config.
-    expect(Object.keys(configObject.rules).filter((rule) => rule.startsWith('better-tailwindcss/'))).toHaveLength(0);
-    expect(Object.keys(configObject.rules).filter((rule) => rule.startsWith('tailwindcss/'))).toHaveLength(0);
-
-    // Verify that no Tailwind settings are present in the config.
-    expect(configObject.settings.tailwindcss).toBeUndefined();
-    expect(configObject.settings['better-tailwindcss']).toBeUndefined();
-  });
 });

package/src/eslint/__tests__/tailwind.test.js

@@ -63,7 +63,7 @@ describe('eslint tailwind autofix', () => {
       '',
     ].join('\n');
 
-    const result = await lintAndFix(projectDir, nextModule, code);
+    const result = await lintAndFix(projectDir, nextModule, code, { tailwind: true });
 
     expect(result.output).toContain('aspect-3/4');
     expect(result.output).toContain('max-w-70');
@@ -81,7 +81,10 @@ describe('eslint tailwind autofix', () => {
       '',
     ].join('\n');
 
-    const result = await lintAndFix(projectDir, nextModule, code, { rules: { 'import-x/order': 'off' } });
+    const result = await lintAndFix(projectDir, nextModule, code, {
+      tailwind: true,
+      rules: { 'import-x/order': 'off' },
+    });
 
     expect(result.output).toContain('data-[side=bottom]:translate-y-10');
     expect(result.output).toContain('dark:hover:-translate-x-10');
@@ -94,7 +97,7 @@ describe('eslint tailwind autofix', () => {
       + 'transition-all hover:-translate-y-1 hover:shadow-lg dark:border-slate-800 dark:bg-slate-900';
     const code = ['const Card = () => (', `  <div className="${longClasses}">Card</div>`, ');', ''].join('\n');
 
-    const result = await lintAndFix(projectDir, nextModule, code);
+    const result = await lintAndFix(projectDir, nextModule, code, { tailwind: true });
 
     expect(result.output).toMatch(/className="\n\s+flex/);
   }, 60_000);

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

@@ -127,7 +127,7 @@ const buildConfigObject = ({
  * @param {object} [config.parserOptions] - Parser options.
  * @param {object} [config.settings] - Settings object.
  * @param {object} [config.rules] - Additional rules.
- * @param {object} [config.options] - User-provided options. Tailwind rules are enabled by default and can be disabled with `tailwind: false`.
+ * @param {object} [config.options] - User-provided options. Tailwind rules are disabled by default and can be enabled with `tailwind: true`.
  * @param {boolean} [config.typescript] - Enable TypeScript support.
  *
  * @returns {import('eslint').Linter.Config[]} ESLint configuration array.
@@ -144,7 +144,7 @@ export const buildConfig = ({
   options = {},
   typescript = false,
 }) => {
-  const opts = { prettier: true, importOrder: true, jsdoc: true, tailwind: true, ...options };
+  const opts = { prettier: true, importOrder: true, jsdoc: true, tailwind: false, ...options };
 
   const conditionalPluginList = getEnabledPlugins(conditionalPlugins, opts);
 

package/src/eslint/lib/rules.js

@@ -125,10 +125,12 @@ export const tailwindSettings = () => {
  * consumer enables `prettier-plugin-tailwindcss`, ordering and wrapping are
  * omitted so the formatter owns those concerns without circular fixes.
  *
+ * @param {boolean} [tailwind] - Enable Tailwind CSS class rules.
+ *
  * @returns {object} Tailwind-related ESLint rules (may be partial when plugins are missing).
  */
-export const tailwindRules = () => {
-  if (!betterTailwindcssPlugin) {
+export const tailwindRules = (tailwind = true) => {
+  if (!tailwind || !betterTailwindcssPlugin) {
     return {};
   }
 
@@ -304,7 +306,7 @@ const importOrderRules = (importOrder = true) =>
  * @returns {object} ESLint rules object.
  */
 export const commonRules = (options = {}) => {
-  const { typescript = true, importOrder = true, prettier = true, jsdoc = true, tailwind = true } = options;
+  const { typescript = true, importOrder = true, prettier = true, jsdoc = true, tailwind = false } = options;
 
   return {
     // ---- TypeScript Rules ----
@@ -320,6 +322,6 @@ export const commonRules = (options = {}) => {
     ...jsdocRules(jsdoc),
 
     // ---- Tailwind Integration ----
-    ...(tailwind && tailwindRules()),
+    ...tailwindRules(tailwind),
   };
 };

package/src/eslint/next.js

@@ -39,7 +39,7 @@ const prepareNextConfig = (configs) => configs.filter((c) => c.name !== 'next/ty
  * @param {boolean} [options.a11y] - Enable accessibility rules.
  * @param {boolean} [options.importOrder] - Enable import order rules.
  * @param {boolean} [options.jsdoc] - Enable JSDoc rules for public/exported APIs.
- * @param {boolean} [options.tailwind] - Enable Tailwind CSS class rules (default: true).
+ * @param {boolean} [options.tailwind] - Enable Tailwind CSS class rules (default: false).
  * @param {string[]} [options.ignores] - Additional ignore patterns.
  * @param {object} [options.rules] - Additional or overridden rules.
  * @param {object} [options.settings] - Additional settings.
@@ -52,7 +52,7 @@ const prepareNextConfig = (configs) => configs.filter((c) => c.name !== 'next/ty
  * @returns {import('eslint').Linter.Config[]} ESLint configuration array.
  */
 export const createConfig = (options = {}) => {
-  const { prettier = true, react = true, a11y = true, importOrder = true, jsdoc = true, tailwind = true } = options;
+  const { prettier = true, react = true, a11y = true, importOrder = true, jsdoc = true, tailwind = false } = options;
 
   return buildConfig({
     files: [...files.withTs, ...(options.files || [])],

package/src/eslint/react.js

@@ -26,7 +26,7 @@ import { buildConfig, files, getTailwindCentralPlugins } from './lib/index.js';
  * @param {boolean} [options.a11y] - Enable accessibility rules.
  * @param {boolean} [options.importOrder] - Enable import order rules.
  * @param {boolean} [options.jsdoc] - Enable JSDoc rules for public/exported APIs.
- * @param {boolean} [options.tailwind] - Enable Tailwind CSS class rules (default: true).
+ * @param {boolean} [options.tailwind] - Enable Tailwind CSS class rules (default: false).
  * @param {string[]} [options.ignores] - Additional ignore patterns.
  * @param {object} [options.rules] - Additional or overridden rules.
  * @param {object} [options.settings] - Additional settings.
@@ -39,7 +39,7 @@ import { buildConfig, files, getTailwindCentralPlugins } from './lib/index.js';
  * @returns {import('eslint').Linter.Config[]} ESLint configuration array.
  */
 export const createConfig = (options = {}) => {
-  const { prettier = true, a11y = true, importOrder = true, jsdoc = true, tailwind = true } = options;
+  const { prettier = true, a11y = true, importOrder = true, jsdoc = true, tailwind = false } = options;
 
   return buildConfig({
     files: [...files.withTs, ...(options.files || [])],