@sarj/eslint-plugin 4.1.0 → 5.0.0

package/README.md

@@ -105,6 +105,7 @@ Both distilled from two years of PR-review comments across ~1,065 PRs.
 |---|---|---|
 | `no-zod-native-enum` | `z.nativeEnum(...)` and `z.enum(SomeTsEnum)` — the schema-layer back door around `no-enum`. Autofixes an inline string-literal object to `z.enum([...])`. | warn / error |
 | `prefer-zod-enum` | `z.union([z.literal("a"), z.literal("b")])` — autofixes closed string choices to the shorter, equivalent `z.enum(["a", "b"])`. | warn / error |
+| `prefer-zod-infer` | An `interface`/`type` that restates a Zod schema declared in the same module instead of deriving it with `z.infer`. Options: `ignoreTypeNames`, `requireIdenticalShape` (default `true`). | warn / error |
 | `prefer-module-level-constant` | A literal-only `const` collection (array, object, `Set`, `Map`, `Object.freeze`) or non-global regex declared inside a function body, never mutated and never escaping — hoist it to module scope. Options: `minElements` (default 3), `checkRegex`, `ignoreTestFiles`. | warn / error |
 | `prefer-non-nullable-collection` | An array type explicitly combined with `null`/`undefined`, creating two equivalent empty states. | warn / error |
 
@@ -146,6 +147,26 @@ already recognises. Set `"prefix"` or `"suffix"` to pin one:
 "@sarj/zod-naming-convention": ["error", { convention: "suffix" }]
 ```
 
+### `prefer-zod-infer`: `requireIdenticalShape` / `ignoreTypeNames`
+
+By default the rule only reports a type whose members match its schema's keys
+one for one — same names, same optionality, same nullability, no `.transform()`
+anywhere in the pair. On a 30,759-file, 17-repo sweep that is 5 reports and 5
+true positives. Drop the shape comparison to report on name correlation alone
+(8 reports on the same corpus, 1 of them noise, and it catches twins that have
+already drifted):
+
+```js
+"@sarj/prefer-zod-infer": ["error", { requireIdenticalShape: false }]
+```
+
+`ignoreTypeNames` takes regex sources matched against the declared type name,
+for the pair that is genuinely meant to be maintained by hand:
+
+```js
+"@sarj/prefer-zod-infer": ["error", { ignoreTypeNames: ["^LegacyUser$"] }]
+```
+
 ### `prefer-string-literal-union`: `ignoreFields`
 
 Field names whose value set is owned by a vendor and genuinely open (a Slack
@@ -168,7 +189,7 @@ and every repo gets to name its own.
 
 | Rule | Option | Default | Effect |
 |---|---|---|---|
-| `no-raw-fetch-outside-clients` | `allow` | client/test path patterns | Files exempt from the "no bare `fetch`" rule |
+| `no-raw-fetch-outside-clients` | `allow` | client / vendor-wrapper path patterns | Extra files exempt from the "no bare `fetch`" rule (test files are exempt unconditionally) |
 | `no-dynamic-sql` | `methods` | `["prepare", "exec", "query"]` | Statement-taking methods to inspect |
 | `no-storage-in-stateless-modules` | `modules` | `[]` (rule off) | Directories declared stateless |
 | `no-storage-in-stateless-modules` | `methods` | `["prepare", "put", "getWithMetadata"]` | Storage methods to flag |
@@ -177,7 +198,10 @@ and every repo gets to name its own.
 
 The path options on the first three rules are **regular-expression sources
 matched against the absolute filename**, not globs — so they can express both
-path separators. `allowIn` is the exception, on `no-hand-rolled-sleep` as on
+path separators. Test files no longer need to appear there: the rule delegates
+to the shared `isTestFile` predicate, so supplying `allow` replaces only the
+production exemptions and cannot accidentally un-exempt a test tree.
+`allowIn` is the exception, on `no-hand-rolled-sleep` as on
 `require-fetch-timeout`: it takes minimatch-ish **globs**, also matched against
 the absolute path, so anchor them with a `**/` prefix. Supplying an option
 **replaces** the default rather than extending it.
@@ -198,7 +222,7 @@ export default [
       // This repo keeps its HTTP layer in `lib/api/`, not `clients/`.
       "@sarj/no-raw-fetch-outside-clients": [
         "error",
-        { allow: ["[\\\\/]lib[\\\\/]api[\\\\/]", "\\.test\\.", "\\.spec\\."] },
+        { allow: ["[\\\\/]lib[\\\\/]api[\\\\/]"] },
       ],
       // Declare which modules must stay stateless.
       "@sarj/no-storage-in-stateless-modules": [