OmniRoute/eslint.config.mjs

import nextVitals from "eslint-config-next/core-web-vitals";
import tseslint from "typescript-eslint";

// #7879: bar NEW local `toNumber` definitions outside the canonical helper.
// Pre-existing definitions (~51 across the codebase) are frozen via
// config/quality/eslint-suppressions.json and migrated tier-by-tier; only a
// genuinely NEW `function toNumber`/`const toNumber = ...` should fail.
const TO_NUMBER_RESTRICTION = {
  selector: "FunctionDeclaration[id.name='toNumber'], VariableDeclarator[id.name='toNumber']",
  message:
    "New local `toNumber` definitions are barred — import `toNumber` from " +
    "`@/shared/utils/numeric` instead (#7879). See that module's JSDoc for the " +
    "canonical coercion shape and the `toNumberOrNull`/`toNumberArray` variants.",
};

/** @type {import("eslint").Linter.Config[]} */
const eslintConfig = [
  ...nextVitals,
  // Pacote 4 (plano mestre testes+CI, 2026-07-04) — zero-warning policy: TODA regra roda
  // como "error" e a dívida pré-existente vive congelada por arquivo+regra em
  // config/quality/eslint-suppressions.json (ESLint bulk suppressions nativo). Violação
  // NOVA = vermelho no ato (lint-staged no pre-commit + job lint-guard no fast path);
  // o drift de +41/+88 warnings/ciclo que era rebaselinado às cegas na release morre no
  // PR que o introduz. Aperto do baseline: npx eslint . --prune-suppressions
  // --suppressions-location config/quality/eslint-suppressions.json (na release).
  {
    // Escopo = onde os presets do next registram estes plugins (bloco global sem `files`
    // atingiria scripts/*.mjs sem o plugin react-hooks e explodiria o flat config).
    files: ["src/**/*.{ts,tsx,js,jsx}"],
    rules: {
      "react-hooks/exhaustive-deps": "error",
      "@next/next/no-img-element": "error",
      "import/no-anonymous-default-export": "error",
    },
  },
  // FASE-02: Security rules (strict everywhere)
  {
    rules: {
      "no-eval": "error",
      "no-implied-eval": "error",
      "no-new-func": "error",
      "no-restricted-imports": [
        "error",
        {
          paths: [
            {
              name: "prop-types",
              message: "PropTypes are deprecated. Use TypeScript types/interfaces instead.",
            },
          ],
        },
      ],
    },
  },
  // i18n: ham toLowerCase().includes() arama pattern'ini engelle
  // (Türkçe İ/ı karakterlerini bozar — matchesSearch kullanılmalı).
  // "warn" (error değil): kuralın eklendiği anda kod tabanında zaten bu pattern'i
  // kullanan ~19 satır var; aşamalı temizlik için uyarı seviyesinde tutuluyor
  // (proje politikası: 0 error, warning'ler tolere edilir).
  {
    files: ["src/app/**/*.{ts,tsx}", "src/components/**/*.{ts,tsx}"],
    rules: {
      "no-restricted-syntax": [
        "error",
        {
          selector:
            "CallExpression[callee.property.name='includes'][callee.object.callee.property.name='toLowerCase']",
          message:
            "Türkçe-güvenli arama için matchesSearch() kullan (@/shared/utils/turkishText). Ham toLowerCase().includes() İ/ı karakterlerini bozar.",
        },
        TO_NUMBER_RESTRICTION,
      ],
    },
  },
  // #7879: same toNumber restriction for the rest of src/ and open-sse/ — kept
  // as a separate block (via `ignores`) so it does not clobber the
  // app/components-scoped rule array above (flat config replaces a rule's
  // options entirely per matching file, it does not merge arrays).
  {
    files: ["src/**/*.ts", "open-sse/**/*.ts"],
    ignores: ["src/app/**", "src/components/**"],
    rules: {
      "no-restricted-syntax": ["error", TO_NUMBER_RESTRICTION],
    },
  },
  // Canonical helper module itself is exempt from its own restriction.
  {
    files: ["src/shared/utils/numeric.ts"],
    rules: {
      "no-restricted-syntax": "off",
    },
  },
  // Relaxed rules for open-sse and tests (incremental adoption)
  {
    files: ["open-sse/**/*.ts", "tests/**/*.mjs", "tests/**/*.ts"],
    plugins: {
      "@typescript-eslint": tseslint.plugin,
    },
    rules: {
      "@typescript-eslint/no-explicit-any": "error",
      "@next/next/no-assign-module-variable": "off",
      "react-hooks/rules-of-hooks": "off",
    },
  },
  // Global ignores — keep ESLint scoped to source files only
  {
    ignores: [
      // Next.js build output (distDir now .build/next; keep .next for legacy)
      ".next/**",
      ".build/**",
      "src/.next/**",
      "out/**",
      "build/**",
      "dist/**",
      "coverage/**",
      "next-env.d.ts",
      // Scripts and binaries
      "scripts/**",
      "bin/**",
      // Dependencies
      "node_modules/**",
      ".worktrees/**",
      // Nested git worktrees created by review/resolve skills live under
      // .claude/ (gitignored). They hold other sessions' in-progress work and
      // their files move mid-scan, so never lint them from the main checkout.
      ".claude/**",
      ".omnivscodeagent/**",
      // VS Code extension and its large test fixtures
      "vscode-extension/**",
      "_references/**",
      "_mono_repo/**",
      // Electron app
      "electron/**",
      // Docs
      "docs/**",
      // Open-SSE compiled/bundled output
      "open-sse/mcp-server/dist/**",
      // Playwright test output
      "playwright-report/**",
      "test-results/**",
      // Legacy app/ and QA backup dirs (renamed to dist/ in Layer 1)
      "app/**",
      "app.__qa_backup/**",
      // CLI package copy directory
      "clipr/**",
    ],
  },
];

export default eslintConfig;