oxlint-plugin-arch 0.7.0 → 0.8.0

package/dist/rules/dillon-anti-slop/rules/no-module-mocking.d.ts

@@ -1 +1,8 @@
+/**
+ * Disallow Vitest and Jest module mocking, plus any other configured test-double APIs.
+ *
+ * Example: `vi.mock('./mailer')` fails; with `deny: ['*.spyOn']`,
+ * `vi.spyOn(mailer, 'send')` fails too. `checkDestructured` also catches
+ * `const { spyOn } = vi; spyOn(mailer, 'send')`.
+ */
 export declare const noModuleMockingRule: import("@oxlint/plugins").Rule;

package/dist/rules/module-policy/index.js

@@ -790,6 +790,11 @@ function layoutNewline(text) {
 ` : `
 `;
 }
+// src/utils/helpers/mocking.ts
+function mockingApiPatternsAssert(patterns) {
+  if (patterns.some((pattern) => !/^(?:\*|(?:vi|jest|\*)\.(?:[A-Za-z_$][\w$]*|\*))$/.test(pattern)))
+    throw new Error("Test mocking APIs must be `*` or `<vi|jest|*>.<method|*>`.");
+}
 // src/utils/helpers/options.ts
 function optionsFirst(context, fallback) {
   return context.options[0] ?? fallback;

package/dist/types/preset.types.d.ts

@@ -70,6 +70,25 @@ export interface PresetTestsOptions {
     profile?: 'standard' | 'fixtures';
     additionalTestFunctions?: string[];
     additionalAssertionFunctions?: string[];
+    /** Test-double APIs banned in test files; `modules` (module mocking only) when omitted. */
+    mocking?: PresetTestMockingPreset | PresetTestMockingOptions;
+}
+export type PresetTestMockingPreset = 'modules' | 'strict';
+export interface PresetTestMockingApi {
+    api: string;
+    message?: string;
+}
+export interface PresetTestMockingException {
+    files: string[];
+    /** API patterns allowed in these files; `*` allows every API. */
+    apis: string[];
+}
+export interface PresetTestMockingOptions {
+    preset?: PresetTestMockingPreset;
+    /** API patterns added to the preset list, such as `vi.*` or `jest.spyOn`. */
+    deny?: Array<string | PresetTestMockingApi>;
+    /** Per-file exceptions, added after the preset's own. */
+    allow?: PresetTestMockingException[];
 }
 export interface PresetImportsOptions {
     backendEntryPoints?: string[];

package/dist/utils/helpers/index.d.ts

@@ -5,6 +5,7 @@ export * from './exports.ts';
 export * from './factories.ts';
 export * from './guards.ts';
 export * from './layout.ts';
+export * from './mocking.ts';
 export * from './naming.ts';
 export * from './options.ts';
 export * from './orpc.ts';

package/dist/utils/helpers/mocking.d.ts

@@ -0,0 +1,2 @@
+/** Throws unless every pattern is `*` or `<vi|jest|*>.<method|*>`. */
+export declare function mockingApiPatternsAssert(patterns: string[]): void;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "oxlint-plugin-arch",
-  "version": "0.7.0",
+  "version": "0.8.0",
   "description": "Configurable architecture rules for Oxlint JavaScript plugins.",
   "type": "module",
   "main": "./dist/index.js",

package/src/rules/dillon-anti-slop/UPSTREAM.md

@@ -17,3 +17,9 @@ vendoring and does not publish an official npm package.
 Local behavior fixes recognize `SAFETY:` comments on exported declarations and
 export-default assertions, while retaining statement boundaries for other
 assertions. Covered by `src/tests/safety-comment.test.ts` and preset fixtures.
+
+Local option changes let `no-module-mocking` take `deny` and `allow` API
+patterns and `checkDestructured`, so the preset can also ban stubs such as
+`vi.spyOn`. With no options
+it reports the same calls as before. Covered by
+`src/tests/no-module-mocking.test.ts` and preset tests.