eslint.config.mjs

// @ts-check

import eslint from '@eslint/js';
import tseslint from 'typescript-eslint';
import importPlugin from 'eslint-plugin-import';
import {jsdoc} from 'eslint-plugin-jsdoc';
import tsdoc from "eslint-plugin-tsdoc";
import noComplexInlineReturnType from './tools/eslint-rules/no-complex-inline-return-type.js';

export default tseslint.config(
  eslint.configs.recommended,
  tseslint.configs.recommended,
  jsdoc({
    config: 'flat/requirements-typescript',
  }),
  {
    ignores: [
      '**/generated/*',
      '**/dist/*',
      '**/build/*',
      '**/coverage/*',
      'docs/**/*',
      '**/.react-router/**',
      '**/fixtures/**',
    ],
  },
  {
    plugins : {
      "tsdoc": tsdoc,
      "custom": {
        rules: {
          "no-complex-inline-return-type": noComplexInlineReturnType
        }
      }
    },
    files: ['**/*.{ts,tsx}'],
    extends: [importPlugin.flatConfigs.recommended, importPlugin.flatConfigs.typescript],
    rules: {
      "tsdoc/syntax": "warn",
      'import/no-unresolved': 'off',
      'jsdoc/require-example': 'off',
      'jsdoc/require-throws-type': 'off',
      'jsdoc/require-param': ['warn', { checkDestructured: false }],
      'max-lines': ['error', { max: 400, skipBlankLines: false, skipComments: false }],
      '@typescript-eslint/explicit-member-accessibility': 'warn',
      "complexity": ["warn", { "max": 15 }],
      "max-lines-per-function": ["warn", {
        max: 80,
        skipBlankLines: true,
        skipComments: true
      }],
      "@typescript-eslint/prefer-as-const": "error",
      "@typescript-eslint/ban-ts-comment": [
        "error",
        {
          "ts-expect-error": "allow-with-description",
          "minimumDescriptionLength": 10
        }
      ],
      "custom/no-complex-inline-return-type": ["error", { "maxProperties": 1 }]

    },
    "settings": {
      "import/resolver": {
        "node": {
          "extensions": [
            ".js",
            ".jsx"
          ]
        }
      }
    }
  },
  {
    files: [
      '**/*.test.{ts,tsx}',
      '**/test/**/*.ts',
      '**/test-*.ts',
      '**/__tests__/**/*.ts',
    ],
    rules: {
      '@typescript-eslint/explicit-member-accessibility': 'off',
      'jsdoc/require-yields': 'off',
      "complexity": 'off',
      "max-lines-per-function": 'off'
    },
  },
  {
    rules: {
      'no-console': [
        'error',
        {
          allow: ['warn', 'error', 'debug', 'info'], // Allow console.warn, console.error, console.debug, and console.info
        },
      ],
      '@typescript-eslint/no-namespace': [
        'error',
        {
          allowDeclarations: true,
        },
      ],
      // Handle unused variables and parameters more intelligently
      '@typescript-eslint/no-unused-vars': [
        'error',
        {
          args: 'after-used', // Allow unused params if they come after used ones
          argsIgnorePattern: '^_', // Ignore parameters starting with underscore
          varsIgnorePattern: '^_', // Ignore variables starting with underscore
          ignoreRestSiblings: true, // Ignore unused vars in object destructuring rest
          destructuredArrayIgnorePattern: '^_', // Ignore unused array destructuring elements starting with _
          caughtErrors: 'all', // Check caught error arguments
          caughtErrorsIgnorePattern: '^_', // Ignore caught errors starting with _
        },
      ],
    },
  },
  {
    // Override for scripts, snippet, and validate files - allow more console methods but still restrict console.log
    files: ['scripts/**/*', 'services/snippet/src/snippet.ts', 'validate.ts'],
    rules: {
      'no-console': [
        'error',
        {
          allow: ['warn', 'error', 'debug', 'info'], // Allow all except console.log
        },
      ],
    },
  },
  {
    // Override for test files, build scripts, and demos
    files: [
      '**/*.test.ts',
      '**/*.spec.ts',
      '**/build.ts',
      '**/demo.ts',
      '**/__tests__/**',
      '**/test/**',
    ],
    rules: {
      'no-console': 'off', // Allow console usage in tests and build scripts
    },
  },
  {
    // Explicit override for synthetic loader fixtures (CommonJS modules used by e2e loader tests)
    // We lint them with CommonJS globals and script sourceType instead of ignoring them.
    // Rationale: keep fixtures under lint while avoiding noisy no-undef for CJS patterns
    files: ['packages/agents/core/test-fixtures/**/*.js'],
    languageOptions: {
      sourceType: 'script',
      globals: {
        module: 'readonly',
        exports: 'readonly',
        require: 'readonly',
        __dirname: 'readonly',
        __filename: 'readonly',
        process: 'readonly',
      },
    },
    rules: {
      // Allow console in fixtures if used for test signaling
      'no-console': 'off',
    },
  },
  {
    files: ['packages/commands/js-debugger/test/fixtures/browser/**/*.js'],
    languageOptions: {
      globals: {
        window: 'readonly',
        document: 'readonly',
        console: 'readonly',
        setTimeout: 'readonly',
      },
    },
    rules: {
      'no-console': 'off',
      'no-debugger': 'off',
    },
  },
  {
    files: ['packages/commands/js-debugger/test/fixtures/node/**/*.js'],
    languageOptions: {
      globals: {
        console: 'readonly',
        process: 'readonly',
        setTimeout: 'readonly',
        setInterval: 'readonly',
        clearInterval: 'readonly',
        require: 'readonly',
        module: 'readonly',
      },
    },
    rules: {
      'no-console': 'off',
      'no-debugger': 'off',
    },
  },
);