oxlint-plugin-arch 0.5.0 → 0.6.1

package/dist/presets/index.d.ts

@@ -1,2 +1,2 @@
 export { default as tanstackStartReactModulesPreset } from './tanstack-start-react-modules-preset.ts';
-export type { PresetOptions, PresetArchitecture, PresetArchitectureEntry, PresetFolderLayout, PresetTanstackStartOptions, PresetLimitsOptions, PresetModulesOptions, PresetImportsOptions, PresetOrpcOptions, PresetSqlOptions, PresetFormsOptions, PresetReactOptions, PresetPolicyOptions, PresetPolicy, PresetPolicyName, PresetTailwindOptions, PresetShadcnOptions, } from '../types/preset.types.ts';
+export type { PresetOptions, PresetArchitecture, PresetArchitectureEntry, PresetFolderLayout, PresetTanstackStartOptions, PresetLimitsOptions, PresetModulesOptions, PresetImportsOptions, PresetOrpcOptions, PresetSqlOptions, PresetFormsOptions, PresetReactOptions, PresetTestsOptions, PresetPolicyOptions, PresetPolicy, PresetPolicyName, PresetTailwindOptions, PresetShadcnOptions, } from '../types/preset.types.ts';

package/dist/presets/index.js

@@ -30,6 +30,8 @@ var presetPolicyOrder = [
   "shadcn"
 ];
 var presetCommonRules = {
+  "vitest/expect-expect": "off",
+  "vitest/no-standalone-expect": "off",
   "jsx-a11y/no-noninteractive-tabindex": "error",
   "no-case-declarations": "error",
   "no-empty": "error",
@@ -413,18 +415,7 @@ var presetTestRules = {
   "jest/no-export": "off",
   "jest/no-standalone-expect": "off",
   "jest/require-to-throw-message": "off",
-  "vitest/no-standalone-expect": [
-    "error",
-    {
-      additionalTestBlockFunctions: ["test"]
-    }
-  ],
-  "vitest/expect-expect": [
-    "error",
-    {
-      additionalTestBlockFunctions: ["test"]
-    }
-  ],
+  "vitest/no-conditional-expect": "error",
   "vitest/require-mock-type-parameters": "error"
 };
 var presetExportPrefix = {
@@ -589,6 +580,7 @@ function presetTypesConfig(context) {
         }
       ]
     }, [...ui, ...testFiles]),
+    ...options.react?.componentProps === false ? [] : [presetOverride(components, { "arch/component-props": ["error", { filePrefix: true }] }, [...ui, ...testFiles])],
     presetOverride(presetScopes(app, "**/*.types.ts"), {
       "arch/no-restricted-files": ["error", { message: "Domain types belong in src/types." }]
     }, [...types, ...ui]),
@@ -679,6 +671,7 @@ function presetTypesConfig(context) {
 function presetModulesConfig(context) {
   const { web, backend, app, services, utilities, ui, testFiles, components, hooks, options } = context;
   const conceptFiles = (options.modules?.customFileRoles ?? []).map((role) => `**/*.${role}.ts`);
+  const migrations = options.modules?.migrationFiles ?? [];
   const frontendEntries = [
     ...presetScopes(web, "services/**/*.{client,server,rsc}.ts"),
     ...presetScopes(web, "services/**/*.rsc.tsx")
@@ -696,7 +689,7 @@ function presetModulesConfig(context) {
             message: mode === "flat" ? "This folder uses a flat layout; subfolders are forbidden." : "This folder uses exactly one owning domain directory; loose and deeper files are forbidden."
           }
         ]
-      }),
+      }, migrations),
       ...mode === "domain" ? [
         presetOverride([`${path}/*/${presetExtensions}`], {
           "arch/folder-prefix": [
@@ -740,7 +733,7 @@ function presetModulesConfig(context) {
           minMembers: 1
         }
       ]
-    }, [...utilities, ...testFiles, ...conceptFiles]),
+    }, [...utilities, ...testFiles, ...conceptFiles, ...migrations]),
     presetOverride(frontendEntries, {
       "modules/service-functions": ["error", { frontend: true }]
     }, testFiles),
@@ -792,7 +785,10 @@ function presetModulesConfig(context) {
       ]
     }, testFiles),
     presetOverride(presetScopes(app, "services/**/{actions,queries}/**/*.ts"), {
-      "arch/no-restricted-files": ["error", { message: "Use domain-action.name.ts or domain-query.name.ts." }]
+      "arch/no-restricted-files": [
+        "error",
+        { message: "Use domain-action.name.ts or domain-query.name.ts." }
+      ]
     })
   ];
   policies.naming = [
@@ -814,7 +810,10 @@ function presetModulesConfig(context) {
       ]
     }, testFiles),
     presetOverride(presetScopes(web, "services/**/*.constants.ts"), {
-      "arch/export-file-prefix": ["error", { ...presetExportPrefix, allDeclarations: true, allowPattern: "^\\*$" }]
+      "arch/export-file-prefix": [
+        "error",
+        { ...presetExportPrefix, allDeclarations: true, allowPattern: "^\\*$" }
+      ]
     }, testFiles),
     presetOverride(presetScopes(app, "**/*.utils.ts"), {
       "arch/export-file-prefix": [
@@ -843,7 +842,10 @@ function presetModulesConfig(context) {
     presetOverride(presetScopes(web.filter((path) => !context.folders.some((scope) => scope.path === path + "/components")), "components/**/*.tsx"), {
       "arch/folder-prefix": ["error", { singularize: "trailing-s", separators: ["-"] }]
     }, [...ui, ...testFiles]),
-    ...["action", "query"].map((role) => presetOverride([...presetScopes(app, `services/**/*-${role}.*.ts`), ...presetScopes(web, `services/**/*-${role}.*.rsc.tsx`)], {
+    ...["action", "query"].map((role) => presetOverride([
+      ...presetScopes(app, `services/**/*-${role}.*.ts`),
+      ...presetScopes(web, `services/**/*-${role}.*.rsc.tsx`)
+    ], {
       "arch/filename-match": [
         "error",
         {
@@ -897,29 +899,53 @@ function presetModulesConfig(context) {
 
 // src/configs/tests.ts
 function presetTestsConfig(context) {
-  const { backend, testFiles, baseImports } = context;
+  const { backend, testFiles, baseImports, options } = context;
+  const profile = options.tests?.profile ?? "fixtures";
+  if (profile !== "standard" && profile !== "fixtures")
+    throw new Error("Unknown test profile.");
+  const testFunctions = ["test", "it", ...options.tests?.additionalTestFunctions ?? []];
+  const assertionFunctions = options.tests?.additionalAssertionFunctions ?? [];
+  if (testFunctions.some((name) => !/^[A-Za-z_$][\w$]*(?:\.[A-Za-z_$][\w$]*)*$/.test(name)))
+    throw new Error("Additional test functions must be literal identifiers or dotted names.");
+  if (assertionFunctions.some((name) => !/^[A-Za-z_$][\w$]*(?:\.[\w$*]+)*$/.test(name)))
+    throw new Error("Assertion functions must be rooted dotted names or patterns.");
   const policies = {};
   policies.tests = [
     presetOverride(testFiles, {
       ...presetTestRules,
       "dillon-anti-slop/no-module-mocking": "error",
-      "modules/test-modifiers": "error",
-      "arch/test-title-pattern": ["error", { forbid: "^should\\b", flags: "i" }]
-    }),
-    presetOverride(presetScopes(backend.map((path) => path.replace(/\/src$/, "")), "tests/**/*.ts"), {
-      "modules/concurrent-db": "error",
-      "no-restricted-imports": [
+      "modules/test-modifiers": [
+        "error",
+        { additionalTestFunctions: options.tests?.additionalTestFunctions ?? [] }
+      ],
+      "modules/test-assertions": [
         "error",
         {
-          patterns: [
-            ...baseImports,
-            {
-              group: ["**/helpers/**"],
-              message: "Use named fixtures or tests/support."
-            }
-          ]
+          additionalTestFunctions: options.tests?.additionalTestFunctions ?? [],
+          additionalAssertionFunctions: assertionFunctions
         }
+      ],
+      "arch/test-title-pattern": [
+        "error",
+        { callees: ["describe", ...testFunctions], forbid: "^should\\b", flags: "i" }
       ]
+    }),
+    presetOverride(presetScopes(backend.map((path) => path.replace(/\/src$/, "")), "tests/**/*.ts"), {
+      "modules/concurrent-db": "error",
+      ...profile === "fixtures" ? {
+        "no-restricted-imports": [
+          "error",
+          {
+            patterns: [
+              ...baseImports,
+              {
+                group: ["**/helpers/**"],
+                message: "Use named fixtures or tests/support."
+              }
+            ]
+          }
+        ]
+      } : {}
     })
   ];
   return policies;
@@ -1016,15 +1042,35 @@ function presetReactConfig(context) {
     ]))));
   }
   if (Object.keys(css).length && options.shadcn !== false) {
+    const allow = ["layout", "opacity", "truncate"];
+    const padding = ["p", "px", "py", "ps", "pe", "pt", "pr", "pb", "pl"];
+    const gap = ["gap", "gap-x", "gap-y"];
+    const safeArea = padding.flatMap((group) => ["top", "right", "bottom", "left"].flatMap((side) => [
+      `${group}-[env(safe-area-inset-${side})]`,
+      ...group === "p" ? [] : [`${group}-[max(--spacing(*),env(safe-area-inset-${side}))]`]
+    ]));
     policies.shadcn = [
       presetOverride(webFiles, {
-        "shadcn/no-restyle": ["error", { allow: ["layout", "opacity"] }],
-        "shadcn/no-arbitrary-values": ["error", { allow: ["layout"] }],
+        "shadcn/no-restyle": [
+          "error",
+          {
+            allow,
+            contracts: [
+              { pattern: "^(CardContent|PopoverContent)$", allow: [...allow, ...padding] },
+              { pattern: "^TabsContent$", allow: [...allow, ...gap, ...padding] },
+              { pattern: "^(Tabs|HoverCardContent|SidebarHeader|BreadcrumbList)$", allow: [...allow, ...gap] }
+            ]
+          }
+        ],
+        "shadcn/no-arbitrary-values": ["error", { allow: ["layout", ...safeArea] }],
         "shadcn/require-static-classes": "error",
         "modules/dynamic-classes": "error"
       }, ui),
       presetOverride(webFiles, {
         "shadcn/no-unknown-classes": ["error", { allow: [] }]
+      }),
+      presetOverride(presetScopes(web, "components/ui/sonner.tsx"), {
+        "shadcn/no-unknown-classes": ["error", { allow: ["toaster", "toast"] }]
       })
     ];
   }

package/dist/presets/tanstack-start-react-modules-preset.js

@@ -30,6 +30,8 @@ var presetPolicyOrder = [
   "shadcn"
 ];
 var presetCommonRules = {
+  "vitest/expect-expect": "off",
+  "vitest/no-standalone-expect": "off",
   "jsx-a11y/no-noninteractive-tabindex": "error",
   "no-case-declarations": "error",
   "no-empty": "error",
@@ -413,18 +415,7 @@ var presetTestRules = {
   "jest/no-export": "off",
   "jest/no-standalone-expect": "off",
   "jest/require-to-throw-message": "off",
-  "vitest/no-standalone-expect": [
-    "error",
-    {
-      additionalTestBlockFunctions: ["test"]
-    }
-  ],
-  "vitest/expect-expect": [
-    "error",
-    {
-      additionalTestBlockFunctions: ["test"]
-    }
-  ],
+  "vitest/no-conditional-expect": "error",
   "vitest/require-mock-type-parameters": "error"
 };
 var presetExportPrefix = {
@@ -589,6 +580,7 @@ function presetTypesConfig(context) {
         }
       ]
     }, [...ui, ...testFiles]),
+    ...options.react?.componentProps === false ? [] : [presetOverride(components, { "arch/component-props": ["error", { filePrefix: true }] }, [...ui, ...testFiles])],
     presetOverride(presetScopes(app, "**/*.types.ts"), {
       "arch/no-restricted-files": ["error", { message: "Domain types belong in src/types." }]
     }, [...types, ...ui]),
@@ -679,6 +671,7 @@ function presetTypesConfig(context) {
 function presetModulesConfig(context) {
   const { web, backend, app, services, utilities, ui, testFiles, components, hooks, options } = context;
   const conceptFiles = (options.modules?.customFileRoles ?? []).map((role) => `**/*.${role}.ts`);
+  const migrations = options.modules?.migrationFiles ?? [];
   const frontendEntries = [
     ...presetScopes(web, "services/**/*.{client,server,rsc}.ts"),
     ...presetScopes(web, "services/**/*.rsc.tsx")
@@ -696,7 +689,7 @@ function presetModulesConfig(context) {
             message: mode === "flat" ? "This folder uses a flat layout; subfolders are forbidden." : "This folder uses exactly one owning domain directory; loose and deeper files are forbidden."
           }
         ]
-      }),
+      }, migrations),
       ...mode === "domain" ? [
         presetOverride([`${path}/*/${presetExtensions}`], {
           "arch/folder-prefix": [
@@ -740,7 +733,7 @@ function presetModulesConfig(context) {
           minMembers: 1
         }
       ]
-    }, [...utilities, ...testFiles, ...conceptFiles]),
+    }, [...utilities, ...testFiles, ...conceptFiles, ...migrations]),
     presetOverride(frontendEntries, {
       "modules/service-functions": ["error", { frontend: true }]
     }, testFiles),
@@ -792,7 +785,10 @@ function presetModulesConfig(context) {
       ]
     }, testFiles),
     presetOverride(presetScopes(app, "services/**/{actions,queries}/**/*.ts"), {
-      "arch/no-restricted-files": ["error", { message: "Use domain-action.name.ts or domain-query.name.ts." }]
+      "arch/no-restricted-files": [
+        "error",
+        { message: "Use domain-action.name.ts or domain-query.name.ts." }
+      ]
     })
   ];
   policies.naming = [
@@ -814,7 +810,10 @@ function presetModulesConfig(context) {
       ]
     }, testFiles),
     presetOverride(presetScopes(web, "services/**/*.constants.ts"), {
-      "arch/export-file-prefix": ["error", { ...presetExportPrefix, allDeclarations: true, allowPattern: "^\\*$" }]
+      "arch/export-file-prefix": [
+        "error",
+        { ...presetExportPrefix, allDeclarations: true, allowPattern: "^\\*$" }
+      ]
     }, testFiles),
     presetOverride(presetScopes(app, "**/*.utils.ts"), {
       "arch/export-file-prefix": [
@@ -843,7 +842,10 @@ function presetModulesConfig(context) {
     presetOverride(presetScopes(web.filter((path) => !context.folders.some((scope) => scope.path === path + "/components")), "components/**/*.tsx"), {
       "arch/folder-prefix": ["error", { singularize: "trailing-s", separators: ["-"] }]
     }, [...ui, ...testFiles]),
-    ...["action", "query"].map((role) => presetOverride([...presetScopes(app, `services/**/*-${role}.*.ts`), ...presetScopes(web, `services/**/*-${role}.*.rsc.tsx`)], {
+    ...["action", "query"].map((role) => presetOverride([
+      ...presetScopes(app, `services/**/*-${role}.*.ts`),
+      ...presetScopes(web, `services/**/*-${role}.*.rsc.tsx`)
+    ], {
       "arch/filename-match": [
         "error",
         {
@@ -897,29 +899,53 @@ function presetModulesConfig(context) {
 
 // src/configs/tests.ts
 function presetTestsConfig(context) {
-  const { backend, testFiles, baseImports } = context;
+  const { backend, testFiles, baseImports, options } = context;
+  const profile = options.tests?.profile ?? "fixtures";
+  if (profile !== "standard" && profile !== "fixtures")
+    throw new Error("Unknown test profile.");
+  const testFunctions = ["test", "it", ...options.tests?.additionalTestFunctions ?? []];
+  const assertionFunctions = options.tests?.additionalAssertionFunctions ?? [];
+  if (testFunctions.some((name) => !/^[A-Za-z_$][\w$]*(?:\.[A-Za-z_$][\w$]*)*$/.test(name)))
+    throw new Error("Additional test functions must be literal identifiers or dotted names.");
+  if (assertionFunctions.some((name) => !/^[A-Za-z_$][\w$]*(?:\.[\w$*]+)*$/.test(name)))
+    throw new Error("Assertion functions must be rooted dotted names or patterns.");
   const policies = {};
   policies.tests = [
     presetOverride(testFiles, {
       ...presetTestRules,
       "dillon-anti-slop/no-module-mocking": "error",
-      "modules/test-modifiers": "error",
-      "arch/test-title-pattern": ["error", { forbid: "^should\\b", flags: "i" }]
-    }),
-    presetOverride(presetScopes(backend.map((path) => path.replace(/\/src$/, "")), "tests/**/*.ts"), {
-      "modules/concurrent-db": "error",
-      "no-restricted-imports": [
+      "modules/test-modifiers": [
+        "error",
+        { additionalTestFunctions: options.tests?.additionalTestFunctions ?? [] }
+      ],
+      "modules/test-assertions": [
         "error",
         {
-          patterns: [
-            ...baseImports,
-            {
-              group: ["**/helpers/**"],
-              message: "Use named fixtures or tests/support."
-            }
-          ]
+          additionalTestFunctions: options.tests?.additionalTestFunctions ?? [],
+          additionalAssertionFunctions: assertionFunctions
         }
+      ],
+      "arch/test-title-pattern": [
+        "error",
+        { callees: ["describe", ...testFunctions], forbid: "^should\\b", flags: "i" }
       ]
+    }),
+    presetOverride(presetScopes(backend.map((path) => path.replace(/\/src$/, "")), "tests/**/*.ts"), {
+      "modules/concurrent-db": "error",
+      ...profile === "fixtures" ? {
+        "no-restricted-imports": [
+          "error",
+          {
+            patterns: [
+              ...baseImports,
+              {
+                group: ["**/helpers/**"],
+                message: "Use named fixtures or tests/support."
+              }
+            ]
+          }
+        ]
+      } : {}
     })
   ];
   return policies;
@@ -1016,15 +1042,35 @@ function presetReactConfig(context) {
     ]))));
   }
   if (Object.keys(css).length && options.shadcn !== false) {
+    const allow = ["layout", "opacity", "truncate"];
+    const padding = ["p", "px", "py", "ps", "pe", "pt", "pr", "pb", "pl"];
+    const gap = ["gap", "gap-x", "gap-y"];
+    const safeArea = padding.flatMap((group) => ["top", "right", "bottom", "left"].flatMap((side) => [
+      `${group}-[env(safe-area-inset-${side})]`,
+      ...group === "p" ? [] : [`${group}-[max(--spacing(*),env(safe-area-inset-${side}))]`]
+    ]));
     policies.shadcn = [
       presetOverride(webFiles, {
-        "shadcn/no-restyle": ["error", { allow: ["layout", "opacity"] }],
-        "shadcn/no-arbitrary-values": ["error", { allow: ["layout"] }],
+        "shadcn/no-restyle": [
+          "error",
+          {
+            allow,
+            contracts: [
+              { pattern: "^(CardContent|PopoverContent)$", allow: [...allow, ...padding] },
+              { pattern: "^TabsContent$", allow: [...allow, ...gap, ...padding] },
+              { pattern: "^(Tabs|HoverCardContent|SidebarHeader|BreadcrumbList)$", allow: [...allow, ...gap] }
+            ]
+          }
+        ],
+        "shadcn/no-arbitrary-values": ["error", { allow: ["layout", ...safeArea] }],
         "shadcn/require-static-classes": "error",
         "modules/dynamic-classes": "error"
       }, ui),
       presetOverride(webFiles, {
         "shadcn/no-unknown-classes": ["error", { allow: [] }]
+      }),
+      presetOverride(presetScopes(web, "components/ui/sonner.tsx"), {
+        "shadcn/no-unknown-classes": ["error", { allow: ["toaster", "toast"] }]
       })
     ];
   }