@od-oneapp/config 2026.2.2001-canary.1

package/eslint/storybook.mjs

@@ -0,0 +1,81 @@
+/**
+ * @fileoverview ESLint configuration for Storybook v10 projects
+ *
+ * Extends Next.js config and adds Storybook-specific rules for story files.
+ * Optimized for Storybook v10 with Next.js integration.
+ *
+ * **Features**:
+ * - Storybook v10 recommended rules
+ * - Story file patterns (.stories.tsx, .stories.mdx)
+ * - Storybook config file support (.storybook/**)
+ * - Type-aware linting disabled for performance
+ *
+ * **Use cases**:
+ * - Storybook documentation sites
+ * - Component development with Storybook
+ * - Storybook stories and addons
+ *
+ * **Note**: Should be combined with Next.js config and test config:
+ * ```javascript
+ * import nextConfig from '@od-oneapp/config/eslint/next';
+ * import storybookConfig from '@od-oneapp/config/eslint/storybook';
+ * import testConfig from '@od-oneapp/config/eslint/test';
+ * export default [...nextConfig, ...storybookConfig, ...testConfig];
+ * ```
+ *
+ * @example
+ * ```javascript
+ * // eslint.config.mjs
+ * import nextConfig from '@od-oneapp/config/eslint/next';
+ * import storybookConfig from '@od-oneapp/config/eslint/storybook';
+ * import testConfig from '@od-oneapp/config/eslint/test';
+ *
+ * export default [...nextConfig, ...storybookConfig, ...testConfig];
+ * ```
+ */
+
+import storybook from 'eslint-plugin-storybook';
+import tseslint from 'typescript-eslint';
+
+import nextConfig from './next.mjs';
+import { DISABLED_TYPE_AWARE_RULES } from './shared/disabled-type-aware-rules.mjs';
+import { createValidatedConfig } from './utils/create-validated-config.mjs';
+
+/**
+ * Storybook ESLint configuration
+ *
+ * @type {import('typescript-eslint').ConfigArray}
+ */
+const config = tseslint.config(
+  ...nextConfig,
+  // Storybook-specific configuration for story files and Storybook config files
+  // This config extends Next.js and adds Storybook v10 specific rules
+  {
+    files: [
+      '**/*.stories.@(ts|tsx|js|jsx|mjs|mdx)',
+      '**/.storybook/**/*.{ts,tsx,js,jsx,mjs}',
+      // Test files in stories directory
+      '**/stories/**/*.test.{ts,tsx}',
+      '**/stories/**/*.spec.{ts,tsx}',
+    ],
+    languageOptions: {
+      parserOptions: {
+        // Disable type-aware linting for Storybook config files to avoid project service errors
+        projectService: false,
+      },
+    },
+    plugins: {
+      storybook,
+    },
+    rules: {
+      // Storybook v10 recommended rules
+      ...storybook.configs.recommended.rules,
+      // Disable type-aware rules that require project service
+      ...DISABLED_TYPE_AWARE_RULES,
+      // Allow console.log in Storybook files for demonstration purposes
+      'no-console': 'off',
+    },
+  },
+);
+
+export default createValidatedConfig(config, 'storybook');

package/eslint/test.mjs

@@ -0,0 +1,298 @@
+/**
+ * @fileoverview ESLint configuration for test files
+ *
+ * Extends base config and adds test framework rules for Vitest, Cypress, and Playwright.
+ * Optimized for test file patterns and test-specific best practices.
+ *
+ * **Features**:
+ * - Vitest rules (recommended + custom)
+ * - Cypress E2E test rules
+ * - Playwright E2E test rules
+ * - Testing Library rules
+ * - Jest-DOM matcher rules
+ * - Type-aware linting disabled for performance
+ *
+ * **Supported test frameworks**:
+ * - Vitest (unit and integration tests)
+ * - Cypress (E2E tests)
+ * - Playwright (E2E tests)
+ * - Testing Library (component tests)
+ *
+ * **Note**: Automatically applies to test file patterns. Include alongside your framework config:
+ * ```javascript
+ * import nextConfig from '@od-oneapp/config/eslint/next';
+ * import testConfig from '@od-oneapp/config/eslint/test';
+ * export default [...nextConfig, ...testConfig];
+ * ```
+ *
+ * @example
+ * ```javascript
+ * // eslint.config.mjs
+ * import nextConfig from '@od-oneapp/config/eslint/next';
+ * import testConfig from '@od-oneapp/config/eslint/test';
+ *
+ * export default [...nextConfig, ...testConfig];
+ * ```
+ */
+
+import vitestPlugin from '@vitest/eslint-plugin';
+import cypressPlugin from 'eslint-plugin-cypress';
+import jestDom from 'eslint-plugin-jest-dom';
+import playwrightPlugin from 'eslint-plugin-playwright';
+import testingLibrary from 'eslint-plugin-testing-library';
+import tseslint from 'typescript-eslint';
+
+import baseConfig from './base.mjs';
+import { DISABLED_TYPE_AWARE_RULES } from './shared/disabled-type-aware-rules.mjs';
+import { createValidatedConfig } from './utils/create-validated-config.mjs';
+
+/**
+ * Test files ESLint configuration
+ *
+ * @type {import('typescript-eslint').ConfigArray}
+ */
+const config = tseslint.config(
+  ...baseConfig,
+  // Test file configuration for Vitest, Cypress, and Playwright
+  {
+    files: [
+      // Vitest test files
+      '**/*.test.{ts,tsx,js,jsx,mjs}',
+      '**/*.spec.{ts,tsx,js,jsx,mjs}',
+      '**/test/**/*.{ts,tsx,js,jsx,mjs}',
+      '**/tests/**/*.{ts,tsx,js,jsx,mjs}',
+      '**/__tests__/**/*.{ts,tsx,js,jsx,mjs}',
+      // Cypress test files
+      '**/cypress/**/*.{ts,tsx,js,jsx}',
+      '**/cypress.config.{ts,js,mjs}',
+      '**/cypress.env.{ts,js,mjs}',
+      // Playwright test files
+      '**/playwright/**/*.{ts,tsx,js,jsx}',
+      '**/playwright.config.{ts,js,mjs}',
+      '**/*.playwright.{ts,tsx,js,jsx}',
+      // Vitest config files (excluding Storybook-specific ones)
+      '**/vitest.config.{ts,js,mjs}',
+      // Vitest setup files
+      '**/vitest.setup.{ts,js,mjs}',
+      '**/vitest/setup/**/*.{ts,tsx,js,jsx,mjs}',
+      // General test utilities and helpers
+      '**/*.test-utils.{ts,tsx,js,jsx}',
+      '**/*.test-helpers.{ts,tsx,js,jsx}',
+      '**/vitest/utils/**/*.{ts,tsx,js,jsx,mjs}',
+      '**/test-utils/**/*.{ts,tsx,js,jsx,mjs}',
+      '**/test-helpers/**/*.{ts,tsx,js,jsx,mjs}',
+    ],
+    // Explicitly exclude Storybook files - handled by Storybook config
+    ignores: [
+      '**/.storybook/**',
+      '**/*.stories.{ts,tsx,js,jsx,mjs}',
+      // Exclude test files in stories directory - they're handled by Storybook config
+      '**/stories/**/*.test.{ts,tsx}',
+      '**/stories/**/*.spec.{ts,tsx}',
+    ],
+    plugins: {
+      vitest: vitestPlugin,
+      cypress: cypressPlugin,
+      playwright: playwrightPlugin,
+      'testing-library': testingLibrary,
+      'jest-dom': jestDom,
+    },
+    languageOptions: {
+      globals: {
+        // Vitest globals
+        describe: 'readonly',
+        it: 'readonly',
+        test: 'readonly',
+        expect: 'readonly',
+        vi: 'readonly',
+        beforeEach: 'readonly',
+        afterEach: 'readonly',
+        beforeAll: 'readonly',
+        afterAll: 'readonly',
+        // Playwright globals (when using @playwright/test)
+        // Note: expect is also provided by Playwright, but Vitest's expect takes precedence
+        page: 'readonly',
+        context: 'readonly',
+        browser: 'readonly',
+        request: 'readonly',
+        chromium: 'readonly',
+        firefox: 'readonly',
+        webkit: 'readonly',
+        // Cypress globals
+        cy: 'readonly',
+        Cypress: 'readonly',
+      },
+      parserOptions: {
+        // Disable type-aware linting for test files to improve performance
+        // Test files often don't need full type checking and this speeds up linting
+        projectService: false,
+      },
+    },
+    rules: {
+      // ============================================================================
+      // Vitest Rules
+      // ============================================================================
+      ...vitestPlugin.configs.recommended.rules,
+      'vitest/consistent-test-it': ['error', { fn: 'test', withinDescribe: 'it' }],
+      'vitest/no-alias-methods': 'error',
+      'vitest/no-commented-out-tests': 'warn',
+      'vitest/no-conditional-expect': 'off', // Allow conditionals in mocks and test utilities
+      'vitest/no-conditional-in-test': 'off', // Allow conditionals in mocks and test utilities
+      'vitest/no-disabled-tests': 'warn',
+      'vitest/no-done-callback': 'error',
+      'vitest/no-duplicate-hooks': 'error',
+      'vitest/no-focused-tests': 'error',
+      'vitest/no-identical-title': 'error',
+      'vitest/no-interpolation-in-snapshots': 'error',
+      'vitest/no-large-snapshots': 'warn',
+      'vitest/no-mocks-import': 'error',
+      'vitest/no-standalone-expect': 'error',
+      'vitest/no-test-prefixes': 'error',
+      'vitest/no-test-return-statement': 'error',
+      'vitest/prefer-called-with': 'warn',
+      'vitest/prefer-comparison-matcher': 'warn',
+      'vitest/prefer-each': 'warn',
+      'vitest/prefer-equality-matcher': 'warn',
+      'vitest/prefer-expect-assertions': 'off', // Can be noisy
+      'vitest/prefer-expect-resolves': 'error',
+      'vitest/prefer-hooks-in-order': 'error',
+      'vitest/prefer-hooks-on-top': 'error',
+      'vitest/prefer-lowercase-title': ['error', { ignoreTopLevelDescribe: true }],
+      'vitest/prefer-mock-promise-shorthand': 'warn',
+      'vitest/prefer-snapshot-hint': 'warn',
+      'vitest/prefer-spy-on': 'warn',
+      'vitest/prefer-strict-equal': 'warn',
+      'vitest/prefer-todo': 'warn',
+      'vitest/require-hook': 'off', // Allow top-level operations in test factories and setup
+      'vitest/require-to-throw-message': 'warn',
+      'vitest/require-top-level-describe': 'off', // Flexible for test organization
+      'vitest/valid-describe-callback': 'error',
+      'vitest/valid-expect': 'error',
+      'vitest/valid-title': 'error',
+
+      // ============================================================================
+      // Cypress Rules
+      // ============================================================================
+      ...cypressPlugin.configs.recommended.rules,
+      'cypress/assertion-before-screenshot': 'warn',
+      'cypress/no-assigning-return-values': 'error',
+      'cypress/no-async-tests': 'error',
+      'cypress/no-force': 'warn',
+      'cypress/no-pause': 'error',
+      'cypress/no-unnecessary-waiting': 'error',
+
+      // ============================================================================
+      // Playwright Rules
+      // ============================================================================
+      'playwright/expect-expect': 'warn',
+      'playwright/max-nested-describe': ['warn', { max: 5 }],
+      'playwright/missing-playwright-await': 'error',
+      'playwright/no-conditional-expect': 'off', // Allow conditionals in test utilities
+      'playwright/no-conditional-in-test': 'off', // Allow conditionals in test utilities
+      'playwright/no-element-handle': 'warn',
+      'playwright/no-eval': 'error',
+      'playwright/no-focused-test': 'error',
+      'playwright/no-force-option': 'warn',
+      'playwright/no-page-pause': 'error',
+      'playwright/no-useless-await': 'error',
+      'playwright/no-wait-for-timeout': 'warn',
+      'playwright/prefer-web-first-assertions': 'error',
+      'playwright/valid-expect': 'error',
+
+      // ============================================================================
+      // Testing Library Rules
+      // ============================================================================
+      'testing-library/await-async-queries': 'error',
+      'testing-library/no-await-sync-queries': 'error',
+      'testing-library/no-debugging-utils': 'warn',
+      'testing-library/prefer-screen-queries': 'error',
+      'testing-library/no-container': 'warn',
+      'testing-library/no-node-access': 'warn',
+
+      // ============================================================================
+      // Jest-DOM Rules
+      // ============================================================================
+      'jest-dom/prefer-checked': 'warn',
+      'jest-dom/prefer-enabled-disabled': 'warn',
+      'jest-dom/prefer-in-document': 'warn',
+      'jest-dom/prefer-required': 'warn',
+      'jest-dom/prefer-to-have-attribute': 'warn',
+
+      // ============================================================================
+      // Test-Specific Overrides
+      // ============================================================================
+      // Disable type-aware rules for test files (type checking disabled for performance)
+      ...DISABLED_TYPE_AWARE_RULES,
+
+      // Allow console in tests
+      'no-console': 'off',
+
+      // Allow explicit undefined in mocks for type safety (e.g., mockResolvedValue(undefined) for Promise<void>)
+      'unicorn/no-useless-undefined': 'off',
+
+      // Allow explicit any in tests (mocks, test utilities, etc.)
+      '@typescript-eslint/no-explicit-any': 'off',
+
+      // Allow non-null assertions in tests (common for accessing mock properties)
+      // Note: Already disabled via DISABLED_TYPE_AWARE_RULES, but explicit here for clarity
+      '@typescript-eslint/no-non-null-assertion': 'off',
+
+      // Allow array callback references in tests (common pattern for test data)
+      // This is 'error' in react config, but tests often need function references
+      'unicorn/no-array-callback-reference': 'off',
+
+      // Relax security and promise rules in tests
+      'security/detect-object-injection': 'off',
+      'promise/always-return': 'off',
+      'promise/catch-or-return': 'off',
+
+      // Note: unused-imports/no-unused-vars from base config handles unused vars with _ prefix pattern
+      // No need to re-enable @typescript-eslint/no-unused-vars (would cause double reporting)
+
+      // Note: import/no-extraneous-dependencies is handled in the import plugin override below
+      // (only applies when test.mjs is combined with react.mjs or next.mjs which have the import plugin)
+    },
+  },
+  {
+    // Import plugin override - only applies when combined with react.mjs/next.mjs
+    // This config block is safely ignored if import plugin isn't loaded
+    files: [
+      '**/*.test.{ts,tsx,js,jsx,mjs}',
+      '**/*.spec.{ts,tsx,js,jsx,mjs}',
+      '**/test/**/*.{ts,tsx,js,jsx,mjs}',
+      '**/tests/**/*.{ts,tsx,js,jsx,mjs}',
+      '**/__tests__/**/*.{ts,tsx,js,jsx,mjs}',
+    ],
+    rules: {
+      'import/no-extraneous-dependencies': 'off', // Test files can import dev dependencies
+    },
+  },
+  // Separate config for test config files (vitest.config.ts, cypress.config.ts, etc.)
+  // Excludes .storybook/** files which are handled by Storybook config
+  {
+    files: [
+      '**/vitest.config.{ts,js,mjs}',
+      '**/cypress.config.{ts,js,mjs}',
+      '**/playwright.config.{ts,js,mjs}',
+      // Vitest setup files (excluding .storybook/** which is ignored)
+      '**/vitest.setup.{ts,js,mjs}',
+      '**/setupTests.{ts,js,mjs}',
+      '**/test-setup.{ts,js,mjs}',
+    ],
+    ignores: ['**/.storybook/**'],
+    languageOptions: {
+      parserOptions: {
+        // Disable type-aware linting for test config files
+        projectService: false,
+      },
+    },
+    rules: {
+      // Config files don't need type-aware linting
+      ...DISABLED_TYPE_AWARE_RULES,
+      // Setup files need to run code at top level
+      'vitest/require-hook': 'off',
+    },
+  },
+);
+
+export default createValidatedConfig(config, 'test');

package/eslint/utils/__tests__/create-validated-config.test.mjs

@@ -0,0 +1,42 @@
+import { describe, expect, it, vi, beforeEach } from 'vitest';
+
+import { createValidatedConfig } from '../create-validated-config.mjs';
+
+describe('createValidatedConfig', () => {
+  beforeEach(() => {
+    vi.resetModules();
+  });
+
+  it('should return config directly and cache it', () => {
+    const config = [{ rules: {} }];
+    const result = createValidatedConfig(config, 'test');
+    expect(result).toBe(config);
+
+    // Calling again should hit cache
+    const result2 = createValidatedConfig(config, 'test');
+    expect(result2).toBe(config);
+  });
+
+  it('should throw if config is invalid in development', () => {
+    const originalEnv = process.env.NODE_ENV;
+    process.env.NODE_ENV = 'development';
+
+    // Passing a non-object to trigger validation failure
+    // Note: WeakMap will throw if we try to set it later, but validation should throw first
+    expect(() => createValidatedConfig(null, 'test')).toThrow();
+
+    process.env.NODE_ENV = originalEnv;
+  });
+
+  it('should skip validation if not in development', () => {
+    const originalEnv = process.env.NODE_ENV;
+    process.env.NODE_ENV = 'production';
+
+    // An object that would fail validation if it was enabled
+    const invalidConfig = { rules: 'not-an-object' };
+    const result = createValidatedConfig(invalidConfig, 'test');
+    expect(result).toBe(invalidConfig);
+
+    process.env.NODE_ENV = originalEnv;
+  });
+});

package/eslint/utils/__tests__/filter-ts-eslint-configs.test.mjs

@@ -0,0 +1,57 @@
+import { describe, expect, it } from 'vitest';
+
+import {
+  filterTypeScriptESLintConfigs,
+  filterPluginsForNextJs,
+} from '../filter-ts-eslint-configs.mjs';
+
+describe('filter-ts-eslint-configs', () => {
+  describe('filterTypeScriptESLintConfigs', () => {
+    it('should filter configs by name', () => {
+      const configs = [
+        { name: 'typescript-eslint/recommended', rules: {} },
+        { name: 'my-config', rules: {} },
+      ];
+      const result = filterTypeScriptESLintConfigs(configs);
+      expect(result).toHaveLength(1);
+      expect(result[0].name).toBe('my-config');
+    });
+
+    it('should filter configs by plugin', () => {
+      const configs = [
+        { plugins: { '@typescript-eslint': {} }, rules: {} },
+        { plugins: { other: {} }, rules: {} },
+      ];
+      const result = filterTypeScriptESLintConfigs(configs);
+      expect(result).toHaveLength(1);
+      expect(result[0].plugins).toHaveProperty('other');
+    });
+
+    it('should keep null/undefined configs', () => {
+      const configs = [null, undefined, { name: 'test' }];
+      const result = filterTypeScriptESLintConfigs(configs);
+      expect(result).toHaveLength(3);
+    });
+  });
+
+  describe('filterPluginsForNextJs', () => {
+    it('should filter conflicting plugins', () => {
+      const configs = [{ plugins: { import: {}, other: {} }, rules: {} }];
+      const result = filterPluginsForNextJs(configs);
+      expect(result[0].plugins).toHaveProperty('other');
+      expect(result[0].plugins).not.toHaveProperty('import');
+    });
+
+    it('should remove plugins property if empty after filtering', () => {
+      const configs = [{ plugins: { import: {} }, rules: {} }];
+      const result = filterPluginsForNextJs(configs);
+      expect(result[0]).not.toHaveProperty('plugins');
+    });
+
+    it('should return as-is if no plugins or no conflicts', () => {
+      const configs = [{ rules: {} }, { plugins: { other: {} }, rules: {} }];
+      const result = filterPluginsForNextJs(configs);
+      expect(result).toEqual(configs);
+    });
+  });
+});

package/eslint/utils/__tests__/validate-config.test.mjs

@@ -0,0 +1,104 @@
+/**
+ * @fileoverview validate-config.test.mjs
+ */
+
+import { describe, expect, it } from 'vitest';
+
+import { validateAllConfigs, validateESLintConfig } from '../validate-config.mjs';
+
+describe('validateESLintConfig', () => {
+  it('should validate a valid config object', () => {
+    const config = {
+      files: ['**/*.ts'],
+      rules: {
+        'no-console': 'warn',
+      },
+    };
+    expect(validateESLintConfig(config, 'test')).toBe(true);
+  });
+
+  it('should validate a valid config array', () => {
+    const config = [
+      { files: ['**/*.ts'], rules: {} },
+      { files: ['**/*.js'], rules: {} },
+    ];
+    expect(validateESLintConfig(config, 'test')).toBe(true);
+  });
+
+  it('should reject null config', () => {
+    expect(() => validateESLintConfig(null, 'test')).toThrow(
+      '[test] Config must be an object or array',
+    );
+  });
+
+  it('should reject non-object config', () => {
+    expect(() => validateESLintConfig('invalid', 'test')).toThrow(
+      '[test] Config must be an object or array',
+    );
+    expect(() => validateESLintConfig(123, 'test')).toThrow(
+      '[test] Config must be an object or array',
+    );
+  });
+
+  it('should validate config with plugins', () => {
+    const config = {
+      plugins: {
+        react: {},
+      },
+      rules: {},
+    };
+    expect(validateESLintConfig(config, 'test')).toBe(true);
+  });
+
+  it('should reject config with invalid plugins type', () => {
+    const config = {
+      plugins: 'invalid', // Should be object
+    };
+    expect(() => validateESLintConfig(config, 'test')).toThrow('[test] plugins must be an object');
+  });
+
+  it('should reject config with invalid rules type', () => {
+    const config = {
+      rules: 'invalid', // Should be object
+    };
+    expect(() => validateESLintConfig(config, 'test')).toThrow('[test] rules must be an object');
+  });
+
+  it('should reject config with invalid languageOptions type', () => {
+    const config = {
+      languageOptions: 'invalid', // Should be object
+    };
+    expect(() => validateESLintConfig(config, 'test')).toThrow(
+      '[test] languageOptions must be an object',
+    );
+  });
+
+  it('should validate nested array configs', () => {
+    const config = [
+      { files: ['**/*.ts'], rules: {} },
+      [
+        { files: ['**/*.js'], rules: {} },
+        { files: ['**/*.mjs'], rules: {} },
+      ],
+    ];
+    expect(validateESLintConfig(config, 'test')).toBe(true);
+  });
+});
+
+describe('validateAllConfigs', () => {
+  it('should validate all configs successfully', () => {
+    const configs = {
+      node: { files: ['**/*.ts'], rules: {} },
+      react: { files: ['**/*.tsx'], rules: {} },
+    };
+    expect(validateAllConfigs(configs)).toBe(true);
+  });
+
+  it('should throw if any config is invalid', () => {
+    const configs = {
+      valid: { files: ['**/*.ts'], rules: {} },
+      invalid: { plugins: 'invalid' },
+    };
+    expect(() => validateAllConfigs(configs)).toThrow('[invalid] plugins must be an object');
+  });
+});

package/eslint/utils/create-validated-config.mjs

@@ -0,0 +1,69 @@
+/**
+ * @fileoverview Utility for creating validated ESLint configurations
+ *
+ * Provides caching and development-mode validation for ESLint configs.
+ * Helps catch configuration errors early during development without impacting production builds.
+ */
+
+import { validateESLintConfig } from './validate-config.mjs';
+
+// Use console.error to avoid cross-package dependency cycles in @od-oneapp/config.
+const logError = console.error;
+
+/**
+ * Cache for validated configs to avoid redundant validation on repeated imports.
+ *
+ * Uses WeakMap to allow garbage collection of config objects when they're no longer referenced.
+ * This prevents memory leaks when configs are recreated or replaced.
+ *
+ * @type {WeakMap<object, boolean>}
+ */
+const validatedConfigsCache = new WeakMap();
+
+/**
+ * Creates a validated ESLint config that checks structure in development mode.
+ * This helps catch configuration errors early during development.
+ *
+ * Includes caching to avoid redundant validation when configs are imported
+ * multiple times (e.g., base -> react -> next chain).
+ *
+ * @param {import('typescript-eslint').ConfigArray} config - The ESLint config array from tseslint.config()
+ * @param {string} configName - Name of the config for error messages (e.g., 'react', 'node')
+ * @returns {import('typescript-eslint').ConfigArray} The validated config array
+ * @throws {Error} If config is invalid in development mode
+ *
+ * @example
+ * import tseslint from 'typescript-eslint';
+ * import { createValidatedConfig } from './utils/create-validated-config.mjs';
+ *
+ * const config = tseslint.config(...);
+ * export default createValidatedConfig(config, 'react');
+ */
+export function createValidatedConfig(config, configName = 'config') {
+  // Skip if already validated (cache hit)
+  if (validatedConfigsCache.has(config)) {
+    return config;
+  }
+
+  // Only validate in development mode to avoid impacting production builds
+  // NODE_ENV is commonly used for development/production distinction
+  const isDevelopment = process.env.NODE_ENV === 'development' || !process.env.NODE_ENV;
+
+  if (isDevelopment) {
+    try {
+      validateESLintConfig(config, configName);
+    } catch (error) {
+      // Provide helpful error message with context
+      const message = `\n❌ ESLint config validation failed for "${configName}": ${error.message}\nThis error is only shown in development mode.\n`;
+      // logError is either the logger function (takes message, context) or console.error (takes message, ...args)
+      // Both will work - logger will use context, console will log all args
+      logError(message, { configName, error });
+      throw error;
+    }
+  }
+
+  // Mark as validated in cache
+  validatedConfigsCache.set(config, true);
+
+  return config;
+}