@yasainet/eslint 0.0.72 → 0.0.74

package/src/common/imports.mjs

@@ -1,457 +0,0 @@
-const LAYER_PATTERNS = {
-  queries: [
-    {
-      group: ["**/services/*", "**/services"],
-      message: "queries cannot import services (layer violation)",
-    },
-    {
-      group: ["**/entries/*", "**/entries"],
-      message: "queries cannot import entries (layer violation)",
-    },
-    {
-      group: ["**/hooks/*", "**/hooks"],
-      message: "queries cannot import hooks (layer violation)",
-    },
-  ],
-  services: [
-    {
-      group: ["**/entries/*", "**/entries"],
-      message: "services cannot import entries (layer violation)",
-    },
-    {
-      group: ["**/hooks/*", "**/hooks"],
-      message: "services cannot import hooks (layer violation)",
-    },
-  ],
-  entries: [
-    {
-      group: ["**/queries/*", "**/queries"],
-      message: "entries cannot import queries (layer violation)",
-    },
-    {
-      group: ["**/hooks/*", "**/hooks"],
-      message: "entries cannot import hooks (layer violation)",
-    },
-  ],
-};
-
-const LATERAL_PATTERNS = {
-  queries: [
-    {
-      group: ["@/features/*/queries/*", "@/features/*/queries"],
-      message:
-        "queries cannot import other feature's queries (lateral violation)",
-    },
-  ],
-  services: [
-    {
-      group: ["@/features/*/services/*", "@/features/*/services"],
-      message:
-        "services cannot import other feature's services (lateral violation)",
-    },
-  ],
-  entries: [
-    {
-      group: ["@/features/*/entries/*", "@/features/*/entries"],
-      message:
-        "entries cannot import other feature's entries (lateral violation)",
-    },
-    {
-      group: [
-        "@/features/*/services/*",
-        "@/features/*/services",
-        "!@/features/shared/services/*",
-        "!@/features/shared/services",
-      ],
-      message:
-        "entries cannot import other feature's services. Use the same feature's service (1:1) or move orchestration into the service layer. `shared/services/*` is exempt for cross-cutting side effects (notifications etc.).",
-    },
-  ],
-};
-
-const CARDINALITY_PATTERNS = {
-  server: [
-    {
-      group: ["**/services/client", "**/services/admin"],
-      message:
-        "server entry can only import server service (cardinality violation)",
-    },
-  ],
-  client: [
-    {
-      group: ["**/services/server", "**/services/admin"],
-      message:
-        "client entry can only import client service (cardinality violation)",
-    },
-  ],
-  admin: [
-    {
-      group: ["**/services/server", "**/services/client"],
-      message:
-        "admin entry can only import admin service (cardinality violation)",
-    },
-  ],
-};
-
-function prefixLibPatterns(prefix, mapping) {
-  const prefixes = Object.keys(mapping);
-  const allowedLib = mapping[prefix];
-  return prefixes
-    .filter((p) => p !== prefix)
-    .map((p) => ({
-      group: [`**/lib/${mapping[p]}`, `**/lib/${mapping[p]}/*`],
-      message: `queries/${prefix}.ts can only import from lib/${allowedLib}. Use the correct query file for this lib.`,
-    }));
-}
-
-const LIB_BOUNDARY_PATTERNS = [
-  {
-    group: ["@/lib/*", "@/lib/**"],
-    message:
-      "lib/* can only be imported from queries (lib-boundary violation)",
-  },
-];
-
-const MAPPING_PATTERNS = [
-  {
-    group: ["@/utils/mapping.util"],
-    importNames: ["mapSnakeToCamel", "mapCamelToSnake"],
-    message:
-      "Mapping functions are only allowed in services. Snake/camel conversion belongs at the service boundary.",
-  },
-];
-
-const PAGE_BOUNDARY_PATTERNS = [
-  {
-    group: ["**/queries/*", "**/queries"],
-    message:
-      "page.tsx can only import entries, not queries (page-boundary violation)",
-  },
-  {
-    group: ["**/services/*", "**/services"],
-    message:
-      "page.tsx can only import entries, not services (page-boundary violation)",
-  },
-];
-
-const ROUTE_BOUNDARY_PATTERNS = [
-  {
-    group: ["**/queries/*", "**/queries"],
-    message:
-      "route.ts can only import entries, not queries (route-boundary violation)",
-  },
-  {
-    group: ["**/services/*", "**/services"],
-    message:
-      "route.ts can only import entries, not services (route-boundary violation)",
-  },
-];
-
-const SITEMAP_BOUNDARY_PATTERNS = [
-  {
-    group: ["**/queries/*", "**/queries"],
-    message:
-      "sitemap.ts can only import entries, not queries (sitemap-boundary violation)",
-  },
-  {
-    group: ["**/services/*", "**/services"],
-    message:
-      "sitemap.ts can only import entries, not services (sitemap-boundary violation)",
-  },
-];
-
-const HOOKS_BOUNDARY_PATTERNS = [
-  {
-    group: ["**/queries/*", "**/queries"],
-    message:
-      "hooks can only import entries, not queries (hooks-boundary violation)",
-  },
-  {
-    group: ["**/services/*", "**/services"],
-    message:
-      "hooks can only import entries, not services (hooks-boundary violation)",
-  },
-];
-
-const COMPONENTS_BOUNDARY_PATTERNS = [
-  {
-    group: ["**/queries/*", "**/queries"],
-    message:
-      "components can only import entries or hooks, not queries (components-boundary violation)",
-  },
-  {
-    group: ["**/services/*", "**/services"],
-    message:
-      "components can only import entries or hooks, not services (components-boundary violation)",
-  },
-];
-
-function makeConfig(name, files, ...patternArrays) {
-  const patterns = patternArrays.flat();
-  if (patterns.length === 0) return null;
-  return {
-    name: `imports/${name}`,
-    files,
-    rules: {
-      "no-restricted-imports": ["error", { patterns }],
-    },
-  };
-}
-
-// In ESLint flat config, when multiple matching configs set the same rule
-// (`no-restricted-imports`), the later config's options REPLACE the earlier
-// ones — patterns are not merged. Page / hooks / components boundary configs
-// run after libBoundaryConfigs and would silently drop lib + mapping bans
-// unless we re-include those patterns explicitly.
-/** Next.js-only: restrict page.tsx to only import entries. */
-export const pageBoundaryConfigs = [
-  {
-    name: "imports/page-boundary",
-    files: ["src/app/**/page.tsx"],
-    rules: {
-      "no-restricted-imports": [
-        "error",
-        {
-          patterns: [
-            ...PAGE_BOUNDARY_PATTERNS,
-            ...LIB_BOUNDARY_PATTERNS,
-            ...MAPPING_PATTERNS,
-          ],
-        },
-      ],
-    },
-  },
-];
-
-/** Next.js-only: restrict route.ts to only import entries. */
-export const routeBoundaryConfigs = [
-  {
-    name: "imports/route-boundary",
-    files: ["src/app/**/route.ts"],
-    rules: {
-      "no-restricted-imports": [
-        "error",
-        {
-          patterns: [
-            ...ROUTE_BOUNDARY_PATTERNS,
-            ...LIB_BOUNDARY_PATTERNS,
-            ...MAPPING_PATTERNS,
-          ],
-        },
-      ],
-    },
-  },
-];
-
-/** Next.js-only: restrict sitemap.ts to only import entries. */
-export const sitemapBoundaryConfigs = [
-  {
-    name: "imports/sitemap-boundary",
-    files: ["src/app/sitemap.ts", "src/app/**/sitemap.ts"],
-    rules: {
-      "no-restricted-imports": [
-        "error",
-        {
-          patterns: [
-            ...SITEMAP_BOUNDARY_PATTERNS,
-            ...LIB_BOUNDARY_PATTERNS,
-            ...MAPPING_PATTERNS,
-          ],
-        },
-      ],
-    },
-  },
-];
-
-/** Next.js-only: restrict hooks to only import entries (not queries or services). */
-export const hooksBoundaryConfigs = [
-  {
-    name: "imports/hooks-boundary",
-    files: ["src/features/**/hooks/*.ts"],
-    rules: {
-      "no-restricted-imports": [
-        "error",
-        {
-          patterns: [
-            ...HOOKS_BOUNDARY_PATTERNS,
-            ...LIB_BOUNDARY_PATTERNS,
-            ...MAPPING_PATTERNS,
-          ],
-        },
-      ],
-    },
-  },
-];
-
-/** Next.js-only: restrict components to only import entries or hooks (not queries or services). */
-export const componentsBoundaryConfigs = [
-  {
-    name: "imports/components-boundary",
-    files: ["src/components/**/*.{ts,tsx}"],
-    rules: {
-      "no-restricted-imports": [
-        "error",
-        {
-          patterns: [
-            ...COMPONENTS_BOUNDARY_PATTERNS,
-            ...LIB_BOUNDARY_PATTERNS,
-            ...MAPPING_PATTERNS,
-          ],
-        },
-      ],
-    },
-  },
-];
-
-/**
- * Next.js-only: restrict @/lib imports and mapping imports outside features.
- * Per-feature subdir rules live in createImportsConfigs to avoid clobbering each other.
- */
-export const libBoundaryConfigs = [
-  {
-    name: "imports/lib-boundary",
-    files: ["src/**/*.{ts,tsx}"],
-    ignores: [
-      "src/lib/**",
-      "src/proxy.ts",
-      "src/app/**/route.ts",
-      "src/features/**",
-    ],
-    rules: {
-      "no-restricted-imports": [
-        "error",
-        { patterns: [...LIB_BOUNDARY_PATTERNS, ...MAPPING_PATTERNS] },
-      ],
-    },
-  },
-];
-
-/** Scope import restriction rules to the given feature root. */
-export function createImportsConfigs(
-  featureRoot,
-  prefixLibMapping,
-  { banAliasImports = false } = {},
-) {
-  const configs = [];
-
-  if (banAliasImports) {
-    configs.push({
-      name: "imports/ban-alias",
-      files: [`${featureRoot}/**/*.ts`],
-      rules: {
-        "no-restricted-imports": [
-          "error",
-          {
-            patterns: [
-              {
-                group: ["@/*", "@/**"],
-                message:
-                  "Alias imports (@/) are not available in this environment. Use relative paths.",
-              },
-            ],
-          },
-        ],
-      },
-    });
-  }
-
-  configs.push(
-    makeConfig(
-      "queries",
-      [`${featureRoot}/**/queries/*.ts`],
-      LAYER_PATTERNS.queries,
-      LATERAL_PATTERNS.queries,
-      MAPPING_PATTERNS,
-    ),
-  );
-
-  for (const prefix of Object.keys(prefixLibMapping)) {
-    const patterns = prefixLibPatterns(prefix, prefixLibMapping);
-    if (patterns.length === 0) continue;
-    configs.push(
-      makeConfig(
-        `queries/${prefix}`,
-        [`${featureRoot}/**/queries/${prefix}.ts`],
-        LAYER_PATTERNS.queries,
-        LATERAL_PATTERNS.queries,
-        patterns,
-        MAPPING_PATTERNS,
-      ),
-    );
-  }
-
-  configs.push(
-    makeConfig(
-      "services",
-      [`${featureRoot}/**/services/*.ts`],
-      LAYER_PATTERNS.services,
-      LATERAL_PATTERNS.services,
-      LIB_BOUNDARY_PATTERNS,
-    ),
-  );
-
-  configs.push(
-    makeConfig(
-      "entries",
-      [`${featureRoot}/**/entries/*.ts`],
-      LAYER_PATTERNS.entries,
-      LATERAL_PATTERNS.entries,
-      LIB_BOUNDARY_PATTERNS,
-      MAPPING_PATTERNS,
-    ),
-  );
-
-  configs.push(
-    makeConfig(
-      "utils",
-      [`${featureRoot}/**/utils/*.ts`],
-      LIB_BOUNDARY_PATTERNS,
-      MAPPING_PATTERNS,
-    ),
-  );
-
-  // Catch-all for feature subdirs without a per-layer config
-  // (constants, hooks, schemas). lib-boundary + mapping ban.
-  configs.push({
-    name: "imports/feature-other",
-    files: [`${featureRoot}/**/*.ts`],
-    ignores: [
-      `${featureRoot}/**/services/*.ts`,
-      `${featureRoot}/**/queries/*.ts`,
-      `${featureRoot}/**/entries/*.ts`,
-      `${featureRoot}/**/utils/*.ts`,
-      `${featureRoot}/**/types/*.ts`,
-    ],
-    rules: {
-      "no-restricted-imports": [
-        "error",
-        { patterns: [...LIB_BOUNDARY_PATTERNS, ...MAPPING_PATTERNS] },
-      ],
-    },
-  });
-
-  // Types: mapping ban only. lib is allowed (Database/Tables type imports).
-  configs.push({
-    name: "imports/feature-types",
-    files: [`${featureRoot}/**/types/*.ts`],
-    rules: {
-      "no-restricted-imports": ["error", { patterns: MAPPING_PATTERNS }],
-    },
-  });
-
-  for (const prefix of ["server", "client", "admin"]) {
-    configs.push(
-      makeConfig(
-        `entries/${prefix}`,
-        [`${featureRoot}/**/entries/${prefix}.ts`],
-        LAYER_PATTERNS.entries,
-        LATERAL_PATTERNS.entries,
-        CARDINALITY_PATTERNS[prefix],
-        LIB_BOUNDARY_PATTERNS,
-        MAPPING_PATTERNS,
-      ),
-    );
-  }
-
-  return configs.filter(Boolean);
-}

package/src/common/layers.mjs

@@ -1,158 +0,0 @@
-import { localPlugin } from "./local-plugins/index.mjs";
-
-/**
- * Scope layer rules to the given feature root:
- *
- * - `typeAware: true` (default) includes `layers/no-any-return`, which uses
- *   the TypeScript checker to inspect inferred return types
- * - `typeAware: false` skips it for environments where the checker cannot run
- *   (e.g., Deno files outside the project tsconfig)
- */
-export function createLayersConfigs(featureRoot, { typeAware = true } = {}) {
-  const loggerSelector = "CallExpression[callee.object.name='logger']";
-  const loggerMessage =
-    "logger is not allowed outside entries. Logging belongs in entries.";
-
-  const aliasDynamicImportSelector =
-    "ImportExpression[source.type='Literal'][source.value=/^@\\//]";
-  const aliasDynamicImportMessage =
-    "Dynamic imports of `@/` aliased paths are not allowed in features layers. They bypass prefix-lib and lateral cardinality (e.g. `await import('@/lib/supabase/admin')` from queries/server.ts escapes the lib-boundary check). Create the correct queries/<prefix>.ts or services/<prefix>.ts file instead. External npm packages can still be lazy-loaded for cold-start optimization.";
-
-  const noAnyReturnConfig = {
-    name: "layers/no-any-return",
-    files: [
-      `${featureRoot}/**/queries/*.ts`,
-      `${featureRoot}/**/services/*.ts`,
-    ],
-    plugins: { local: localPlugin },
-    rules: {
-      "local/no-any-return": "error",
-    },
-  };
-
-  return [
-    // Logger/console: all features except entries
-    {
-      name: "layers/logger",
-      files: [`${featureRoot}/**/*.ts`],
-      ignores: [`${featureRoot}/**/entries/*.ts`],
-      rules: {
-        "no-console": "error",
-        "no-restricted-syntax": [
-          "error",
-          { selector: loggerSelector, message: loggerMessage },
-        ],
-      },
-    },
-    // Queries: try-catch + if + loops + logger
-    {
-      name: "layers/queries",
-      files: [`${featureRoot}/**/queries/*.ts`],
-      rules: {
-        "no-restricted-syntax": [
-          "error",
-          {
-            selector: "TryStatement",
-            message:
-              "try-catch is not allowed in queries. Error handling belongs in entries.",
-          },
-          {
-            selector: "IfStatement",
-            message:
-              "if statements are not allowed in queries. Conditional logic belongs in services.",
-          },
-          {
-            selector: "ForStatement",
-            message:
-              "Loops are not allowed in queries. Queries should be thin CRUD wrappers — iteration belongs in services.",
-          },
-          {
-            selector: "ForOfStatement",
-            message:
-              "Loops are not allowed in queries. Queries should be thin CRUD wrappers — iteration belongs in services.",
-          },
-          {
-            selector: "ForInStatement",
-            message:
-              "Loops are not allowed in queries. Queries should be thin CRUD wrappers — iteration belongs in services.",
-          },
-          {
-            selector: "WhileStatement",
-            message:
-              "Loops are not allowed in queries. Queries should be thin CRUD wrappers — iteration belongs in services.",
-          },
-          {
-            selector: "DoWhileStatement",
-            message:
-              "Loops are not allowed in queries. Queries should be thin CRUD wrappers — iteration belongs in services.",
-          },
-          {
-            selector: "ThrowStatement",
-            message:
-              "throw is not allowed in queries. Queries must return Supabase's { data, error } shape as-is. Error handling belongs in entries.",
-          },
-          { selector: loggerSelector, message: loggerMessage },
-          {
-            selector: aliasDynamicImportSelector,
-            message: aliasDynamicImportMessage,
-          },
-        ],
-      },
-    },
-    // Boundary type safety: queries & services must not leak `any`
-    // into their public API. Uses type-aware inspection of the inferred
-    // return type so unannotated functions are still checked.
-    ...(typeAware ? [noAnyReturnConfig] : []),
-    // Services: try-catch + logger + throw + dead error fallbacks
-    {
-      name: "layers/services",
-      files: [`${featureRoot}/**/services/*.ts`],
-      rules: {
-        "no-restricted-syntax": [
-          "error",
-          {
-            selector: "TryStatement",
-            message:
-              "try-catch is not allowed in services. Error handling belongs in entries.",
-          },
-          {
-            selector: "ThrowStatement",
-            message:
-              "throw is not allowed in services. Communicate failures via T | null / { data, error } / empty default. Native exceptions from libs auto-propagate to entry's catch.",
-          },
-          { selector: loggerSelector, message: loggerMessage },
-          {
-            selector:
-              "LogicalExpression[operator='??'][left.type='ChainExpression'][left.expression.property.name='message'][right.type='Literal']",
-            message:
-              "Dead fallback for error message. If you reached this branch the error is known — return the error directly. Unhandled exceptions belong in entries.",
-          },
-          {
-            selector:
-              "LogicalExpression[operator='??'][left.type='MemberExpression'][left.property.name='error'][right.type='ObjectExpression']",
-            message:
-              "Dead fallback for nullable error. Check `if (error)` and return the error directly. Unhandled exceptions belong in entries.",
-          },
-          {
-            selector: aliasDynamicImportSelector,
-            message: aliasDynamicImportMessage,
-          },
-        ],
-      },
-    },
-    // Entries: ban dynamic `@/` imports that bypass cardinality / lateral rules
-    {
-      name: "layers/entries",
-      files: [`${featureRoot}/**/entries/*.ts`],
-      rules: {
-        "no-restricted-syntax": [
-          "error",
-          {
-            selector: aliasDynamicImportSelector,
-            message: aliasDynamicImportMessage,
-          },
-        ],
-      },
-    },
-  ];
-}