@yasainet/eslint 0.0.73 → 0.0.74

package/src/deno/boundaries/entry-point.mjs

@@ -0,0 +1,44 @@
+import { denoLocalPlugin } from "../local-plugins/index.mjs";
+
+const FUNCTIONS_ROOT = "supabase/functions";
+
+export const denoEntryPointConfigs = [
+  {
+    name: "deno/entry-point",
+    files: [`${FUNCTIONS_ROOT}/**/*.ts`],
+    ignores: [`${FUNCTIONS_ROOT}/_*/**`],
+    rules: {
+      "no-restricted-imports": [
+        "error",
+        {
+          patterns: [
+            {
+              group: ["**/services/*", "**/services"],
+              message:
+                "Top-level files must not import services directly. Import from entries instead.",
+            },
+            {
+              group: ["**/queries/*", "**/queries"],
+              message:
+                "Top-level files must not import queries directly. Import from entries instead.",
+            },
+            {
+              group: ["*/_lib/*", "*/_lib/**"],
+              message:
+                "Top-level files must not import _lib/ directly. Import from entries instead.",
+            },
+          ],
+        },
+      ],
+    },
+  },
+  {
+    name: "deno/flat-entry-point",
+    files: [`${FUNCTIONS_ROOT}/**/*.ts`],
+    ignores: [`${FUNCTIONS_ROOT}/_*/**`],
+    plugins: { "deno-local": denoLocalPlugin },
+    rules: {
+      "deno-local/flat-entry-point": "error",
+    },
+  },
+];

package/src/deno/boundaries/lib.mjs

@@ -0,0 +1,28 @@
+const FUNCTIONS_ROOT = "supabase/functions";
+const FEATURE_ROOT = "supabase/functions/_features";
+
+export const denoLibBoundaryConfigs = [
+  {
+    name: "deno/lib-boundary",
+    files: [`${FUNCTIONS_ROOT}/**/*.ts`],
+    ignores: [
+      `${FUNCTIONS_ROOT}/_lib/**`,
+      `${FEATURE_ROOT}/**/queries/**`,
+      `${FEATURE_ROOT}/**/types/**`,
+    ],
+    rules: {
+      "no-restricted-imports": [
+        "error",
+        {
+          patterns: [
+            {
+              group: ["*/_lib/*", "*/_lib/**"],
+              message:
+                "_lib/ can only be imported from queries (lib-boundary violation)",
+            },
+          ],
+        },
+      ],
+    },
+  },
+];

package/src/deno/boundaries/utils.mjs

@@ -0,0 +1,25 @@
+const FUNCTIONS_ROOT = "supabase/functions";
+
+export const denoUtilsBoundaryConfigs = [
+  {
+    name: "deno/utils-boundary",
+    files: [`${FUNCTIONS_ROOT}/_utils/**/*.ts`],
+    rules: {
+      "no-restricted-imports": [
+        "error",
+        {
+          patterns: [
+            {
+              group: ["*/_features/*", "*/_features/**"],
+              message: "_utils/ cannot import _features/",
+            },
+            {
+              group: ["*/_lib/*", "*/_lib/**"],
+              message: "_utils/ cannot import _lib/",
+            },
+          ],
+        },
+      ],
+    },
+  },
+];

package/src/deno/index.mjs

@@ -1,29 +1,25 @@
-import { createEntryPointConfigs } from "../common/entry-points.mjs";
+import { createEntryPointConfigs } from "../common/boundaries/entry-point.mjs";
 import { createCommonConfigs } from "../common/index.mjs";
-import { denoImportsConfigs } from "./imports.mjs";
+
+import { denoEntryPointConfigs } from "./boundaries/entry-point.mjs";
+import { denoLibBoundaryConfigs } from "./boundaries/lib.mjs";
+import { denoUtilsBoundaryConfigs } from "./boundaries/utils.mjs";
 
 const FEATURE_ROOT = "supabase/functions/_features";
 
-const denoEntryPointConfigs = createEntryPointConfigs(
+const denoNamespaceImportConfigs = createEntryPointConfigs(
   ["supabase/functions/**/*.ts"],
   ["supabase/functions/_*/**"],
 );
 
-// Deno files are not covered by the consumer's project tsconfig
-// (Supabase functions live outside Next.js's tsconfig), so type-aware
-// rules cannot consult the type checker. Disable them here and rely on
-// `deno check` / `deno lint` for type-level guarantees.
-//
-// `rulesFiles` is narrowed so the override only affects Deno files. When this
-// entry is combined with `next`/`node`, those entries keep their type-aware
-// settings on their own files.
-/** Deno ESLint flat config entry point. */
 export const eslintConfig = [
   ...createCommonConfigs(FEATURE_ROOT, {
     banAliasImports: true,
     typeAware: false,
     rulesFiles: ["supabase/functions/**/*.ts"],
   }),
-  ...denoImportsConfigs,
+  ...denoLibBoundaryConfigs,
   ...denoEntryPointConfigs,
+  ...denoUtilsBoundaryConfigs,
+  ...denoNamespaceImportConfigs,
 ];

package/src/deno/local-plugins/flat-entry-point.mjs

@@ -1,4 +1,3 @@
-/** Ensure Edge Function entry points are directly under supabase/functions/. */
 export const flatEntryPointRule = {
   meta: {
     type: "problem",
@@ -18,14 +17,10 @@ export const flatEntryPointRule = {
         const relative = filename.slice(idx + "supabase/functions/".length);
         const segments = relative.split("/").filter(Boolean);
 
-        // _prefix directories are shared code, not entry points
         if (segments[0]?.startsWith("_")) return;
 
-        // Root-level files (deno.json, .env, etc.) are not entry points
         if (segments.length <= 1) return;
 
-        // Valid: <name>/index.ts (exactly 2 segments)
-        // Invalid: <name>/<nested>/index.ts (3+ segments)
         if (segments.length > 2) {
           context.report({
             node,

package/src/deno/local-plugins/index.mjs

@@ -1,6 +1,5 @@
 import { flatEntryPointRule } from "./flat-entry-point.mjs";
 
-/** Deno-specific local plugin (independent from common localPlugin). */
 export const denoLocalPlugin = {
   rules: {
     "flat-entry-point": flatEntryPointRule,

package/src/next/boundaries/components.mjs

@@ -0,0 +1,36 @@
+import {
+  LIB_BOUNDARY_PATTERNS,
+  MAPPING_PATTERNS,
+} from "../../common/_internal/import-patterns.mjs";
+
+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)",
+  },
+];
+
+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,
+          ],
+        },
+      ],
+    },
+  },
+];

package/src/next/boundaries/hooks.mjs

@@ -0,0 +1,36 @@
+import {
+  LIB_BOUNDARY_PATTERNS,
+  MAPPING_PATTERNS,
+} from "../../common/_internal/import-patterns.mjs";
+
+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)",
+  },
+];
+
+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,
+          ],
+        },
+      ],
+    },
+  },
+];

package/src/next/boundaries/lib.mjs

@@ -0,0 +1,23 @@
+import {
+  LIB_BOUNDARY_PATTERNS,
+  MAPPING_PATTERNS,
+} from "../../common/_internal/import-patterns.mjs";
+
+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] },
+      ],
+    },
+  },
+];

package/src/next/boundaries/page.mjs

@@ -0,0 +1,36 @@
+import {
+  LIB_BOUNDARY_PATTERNS,
+  MAPPING_PATTERNS,
+} from "../../common/_internal/import-patterns.mjs";
+
+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)",
+  },
+];
+
+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,
+          ],
+        },
+      ],
+    },
+  },
+];

package/src/next/boundaries/route.mjs

@@ -0,0 +1,36 @@
+import {
+  LIB_BOUNDARY_PATTERNS,
+  MAPPING_PATTERNS,
+} from "../../common/_internal/import-patterns.mjs";
+
+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)",
+  },
+];
+
+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,
+          ],
+        },
+      ],
+    },
+  },
+];

package/src/next/boundaries/sitemap.mjs

@@ -0,0 +1,36 @@
+import {
+  LIB_BOUNDARY_PATTERNS,
+  MAPPING_PATTERNS,
+} from "../../common/_internal/import-patterns.mjs";
+
+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)",
+  },
+];
+
+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,
+          ],
+        },
+      ],
+    },
+  },
+];

package/src/next/directives.mjs

@@ -1,4 +1,3 @@
-/** Enforce "use server" / "use client" directives per file convention. */
 export const directivesConfigs = [
   {
     name: "directives/server-entry",

package/src/next/imports.mjs

@@ -1,6 +1,5 @@
 import { localPlugin } from "../common/local-plugins/index.mjs";
 
-/** Enforce relative paths within same feature, @/ for cross-feature in Next.js. */
 export const importPathStyleConfigs = [
   {
     name: "imports/path-style",

package/src/next/index.mjs

@@ -1,28 +1,24 @@
-import { createEntryPointConfigs } from "../common/entry-points.mjs";
+import { createEntryPointConfigs } from "../common/boundaries/entry-point.mjs";
 import { createCommonConfigs } from "../common/index.mjs";
-import {
-  componentsBoundaryConfigs,
-  hooksBoundaryConfigs,
-  libBoundaryConfigs,
-  pageBoundaryConfigs,
-  routeBoundaryConfigs,
-  sitemapBoundaryConfigs,
-} from "../common/imports.mjs";
 
+import { componentsBoundaryConfigs } from "./boundaries/components.mjs";
+import { hooksBoundaryConfigs } from "./boundaries/hooks.mjs";
+import { libBoundaryConfigs } from "./boundaries/lib.mjs";
+import { pageBoundaryConfigs } from "./boundaries/page.mjs";
+import { routeBoundaryConfigs } from "./boundaries/route.mjs";
+import { sitemapBoundaryConfigs } from "./boundaries/sitemap.mjs";
 import { directivesConfigs } from "./directives.mjs";
 import { importPathStyleConfigs } from "./imports.mjs";
-import { layoutsConfigs } from "./layouts.mjs";
-import { namingConfigs } from "./naming.mjs";
+import { componentsLayerConfigs } from "./layers/components.mjs";
+import { hooksLayerConfigs } from "./layers/hooks.mjs";
+import { layoutsConfigs } from "./layers/layouts.mjs";
 import { tailwindcssConfigs } from "./tailwindcss.mjs";
 
 const nextEntryPointConfigs = createEntryPointConfigs(
   ["src/app/**/*.ts", "src/app/**/*.tsx"],
 );
 
-/** Next.js ESLint flat config entry point. */
 export const eslintConfig = [
-  // shadcn/ui generated components live directly under `src/components/shared/ui/`.
-  // Files in `custom/` are user-authored and remain linted.
   {
     name: "rules/ignore-shadcn-ui",
     ignores: ["src/components/shared/ui/*.{ts,tsx}"],
@@ -34,7 +30,8 @@ export const eslintConfig = [
   ...sitemapBoundaryConfigs,
   ...hooksBoundaryConfigs,
   ...componentsBoundaryConfigs,
-  ...namingConfigs,
+  ...hooksLayerConfigs,
+  ...componentsLayerConfigs,
   ...directivesConfigs,
   ...importPathStyleConfigs,
   ...layoutsConfigs,

package/src/next/layers/components.mjs

@@ -0,0 +1,30 @@
+import { checkFile } from "../../common/_internal/plugins.mjs";
+
+export const componentsLayerConfigs = [
+  {
+    name: "naming/components-tsx-only",
+    files: ["src/components/**/*.ts"],
+    rules: {
+      "no-restricted-syntax": [
+        "error",
+        {
+          selector: "Program",
+          message:
+            "components/ must only contain .tsx files. Logic belongs in src/features/.",
+        },
+      ],
+    },
+  },
+  {
+    name: "naming/components-pascal-case",
+    files: ["src/components/**/*.tsx"],
+    ignores: ["src/components/shared/ui/**", "src/components/**/index.tsx"],
+    plugins: { "check-file": checkFile },
+    rules: {
+      "check-file/filename-naming-convention": [
+        "error",
+        { "**/*.tsx": "PASCAL_CASE" },
+      ],
+    },
+  },
+];

package/src/next/layers/hooks.mjs

@@ -0,0 +1,31 @@
+import { checkFile } from "../../common/_internal/plugins.mjs";
+
+export const hooksLayerConfigs = [
+  {
+    name: "naming/hooks",
+    files: ["src/features/**/hooks/*.ts"],
+    plugins: { "check-file": checkFile },
+    rules: {
+      "check-file/filename-naming-convention": [
+        "error",
+        { "**/*.ts": "use-+([a-z0-9])*(-+([a-z0-9]))" },
+        { ignoreMiddleExtensions: true },
+      ],
+    },
+  },
+  {
+    name: "naming/hooks-export",
+    files: ["src/features/**/hooks/*.ts"],
+    rules: {
+      "no-restricted-syntax": [
+        "error",
+        {
+          selector:
+            "ExportNamedDeclaration > FunctionDeclaration[id.name!=/^use[A-Z]/]",
+          message:
+            "Exported functions in hooks must start with 'use' (e.g., useAuth).",
+        },
+      ],
+    },
+  },
+];

package/src/next/layers/layouts.mjs

@@ -0,0 +1,12 @@
+import { localPlugin } from "../../common/local-plugins/index.mjs";
+
+export const layoutsConfigs = [
+  {
+    name: "layouts/main-structural-only",
+    files: ["src/app/**/layout.tsx"],
+    plugins: { local: localPlugin },
+    rules: {
+      "local/layout-main-structural-only": "error",
+    },
+  },
+];

package/src/next/tailwindcss.mjs

@@ -3,14 +3,6 @@ import { dirname, join, sep } from "node:path";
 
 import betterTailwindcss from "eslint-plugin-better-tailwindcss";
 
-// `eslint-plugin-better-tailwindcss` resolves a relative `entryPoint` against
-// the linter's `cwd`, which under LSP servers (vscode-eslint, Zed) is often
-// the edited file's directory rather than the consumer's project root and
-// breaks resolution. Mirror the `findProjectRoot` pattern from
-// `common/rules.mjs` and `common/constants.mjs`: walk up from this module
-// outside of `node_modules` and locate `src/app/globals.css`, then pass the
-// absolute path so resolution is cwd-independent. Falls back to the relative
-// path when not found, preserving the previous CLI-only behavior.
 const findEntryPoint = (start) => {
   let dir = start;
   while (dir !== dirname(dir)) {
@@ -27,19 +19,6 @@ const findEntryPoint = (start) => {
 
 const entryPoint = findEntryPoint(import.meta.dirname);
 
-/**
- * Tailwind CSS v4 lint rules:
- *
- * - margin is forbidden; spacing is controlled by padding/gap on the parent
- * - `space-x-*` / `space-y-*` are also banned because they apply margin to
- *   children under the hood. Use `flex/grid + gap` instead. Negative variants
- *   (`-space-x-2`) remain allowed for intentional overlap
- * - class order, deprecated classes, conflicts, duplicates, and whitespace
- *   are enforced via `eslint-plugin-better-tailwindcss`
- * - `entryPoint` is auto-resolved to the consuming project's
- *   `src/app/globals.css` via walk-up from this module. Override in the
- *   project's eslint.config.mjs if the file lives elsewhere.
- */
 export const tailwindcssConfigs = [
   {
     name: "tailwindcss/rules",

package/src/node/index.mjs

@@ -1,11 +1,10 @@
-import { createEntryPointConfigs } from "../common/entry-points.mjs";
+import { createEntryPointConfigs } from "../common/boundaries/entry-point.mjs";
 import { createCommonConfigs } from "../common/index.mjs";
 
 const nodeEntryPointConfigs = createEntryPointConfigs(
   ["scripts/commands/*.ts"],
 );
 
-/** Node.js ESLint flat config entry point. */
 export const eslintConfig = [
   ...createCommonConfigs("scripts/features", { banAliasImports: true }),
   ...nodeEntryPointConfigs,