@sarj/eslint-plugin 19.18.0 → 19.19.0

package/dist/catalog.js

@@ -2,7 +2,7 @@ import {
   publicDocumentation,
   ruleSpec,
   src_default
-} from "./chunk-E2O4LN7G.js";
+} from "./chunk-UI4ADSAA.js";
 
 // src/docs/directive-disable-enable-pair.ts
 var DIRECTIVE_DISABLE_ENABLE_PAIR_DOCUMENTATION = {
@@ -6441,6 +6441,57 @@ var PRODUCTION_DERIVED_TEST_CASES_DOCUMENTATION = {
   ]
 };
 
+// src/docs/react-prefer-updater-function.ts
+var CART = "import { useState } from 'react'; export function Cart({ load }: { load: () => Promise<string> }) { const [items, setItems] = useState<string[]>([]); const add = async () => { const item = await load(); ";
+var CART_END = ' }; return <button type="button" onClick={add}>{items.length}</button>; }';
+var RENDER_PHASE = "import { useState } from 'react'; export function Panel({ request }: { request: boolean }) { const [dialog, setDialog] = useState({ open: request, request }); if (dialog.request !== request) { setDialog({ open: request || dialog.open, request }); } return <p>{String(dialog.open)}</p>; }";
+var REACT_PREFER_UPDATER_FUNCTION_DOCUMENTATION = {
+  defaultLevel: "warning",
+  summary: "Use the updater form when the next React state is built from the current state.",
+  rationale: "A state variable holds the snapshot from the render that created the callback. After an await, in a timer, promise callback, subscription or effect, or when two updates run before React re-renders, that snapshot is stale. `setItems([...items, item])` then drops whatever another update added in the meantime. An updater receives the latest queued state, so each update builds on the one before it.",
+  remediation: "Pass an updater instead. Rewrite `setItems([...items, item])` as `setItems((previous) => [...previous, item])`, and read `previous` wherever the argument read the state or a local derived from it. Compute IDs, timestamps and other impure values before the call, because React runs updaters twice in Strict Mode development builds.",
+  category: "correctness",
+  limitations: [
+    "The rule is advisory and has no automatic fix. In a synchronous event handler that updates once, the snapshot is still current, so the rewrite keeps today's behavior and protects the handler against a later await or a second update.",
+    "The rule pairs a setter with its state through one array destructure of a React useState call, including renamed, default and namespace React imports, and resolves every read by scope. It ignores setters reached through props, aliases or custom hooks that return state tuples. It also ignores object-destructured state, useReducer dispatch and reassigned bindings.",
+    "The rule reports only state reads that the setter argument itself evaluates. It ignores reads inside nested functions, type-only reads, a bare identity write and next values first stored in a local. It also ignores setter calls made directly in the function that declares the state, because they run during render, where the state belongs to the render in progress."
+  ],
+  references: [
+    "https://react.dev/reference/react/useState#updating-state-based-on-the-previous-state",
+    "https://react.dev/learn/queueing-a-series-of-state-updates",
+    "https://bun.com/docs/lint/rules/bun/react-prefer-updater-function"
+  ],
+  examples: [
+    {
+      id: "snapshot-after-await",
+      title: "Append to the render snapshot after an await",
+      outcome: "match",
+      files: [{ path: "src/cart.tsx", source: `${CART}setItems([...items, item]);${CART_END}` }],
+      focusPath: "src/cart.tsx",
+      expectedCount: 1,
+      public: true
+    },
+    {
+      id: "updater-after-await",
+      title: "Append to the latest state through an updater",
+      outcome: "no-match",
+      files: [{ path: "src/cart.tsx", source: `${CART}setItems((previous) => [...previous, item]);${CART_END}` }],
+      focusPath: "src/cart.tsx",
+      expectedCount: 0,
+      public: true
+    },
+    {
+      id: "render-phase-update",
+      title: "Adjust state from the render in progress",
+      outcome: "no-match",
+      files: [{ path: "src/panel.tsx", source: RENDER_PHASE }],
+      focusPath: "src/panel.tsx",
+      expectedCount: 0,
+      public: false
+    }
+  ]
+};
+
 // src/docs/repeated-static-call-cases.ts
 var REPEATED_STATIC_CALL_CASES_DOCUMENTATION = {
   summary: "Review three or more consecutive static-input call assertions as potential named cases.",
@@ -7578,6 +7629,7 @@ var RULE_DOCUMENTATION = {
   "prefer-zod-infer": PREFER_ZOD_INFER_DOCUMENTATION,
   "prefer-zod-parse-output-type": PREFER_ZOD_PARSE_OUTPUT_TYPE_DOCUMENTATION,
   "production-derived-test-cases": PRODUCTION_DERIVED_TEST_CASES_DOCUMENTATION,
+  "react-prefer-updater-function": REACT_PREFER_UPDATER_FUNCTION_DOCUMENTATION,
   "repeated-static-call-cases": REPEATED_STATIC_CALL_CASES_DOCUMENTATION,
   "require-assert-never": REQUIRE_ASSERT_NEVER_DOCUMENTATION,
   "require-button-accessible-name": REQUIRE_BUTTON_ACCESSIBLE_NAME_DOCUMENTATION,

package/dist/{chunk-E2O4LN7G.js → chunk-UI4ADSAA.js}

@@ -16467,6 +16467,9 @@ var VIEWS = /* @__PURE__ */ new WeakMap();
 var DIRECT_EVAL = /* @__PURE__ */ new WeakMap();
 var REACT_IMPORTS = /* @__PURE__ */ new WeakMap();
 var PARAMETER_BINDINGS = /* @__PURE__ */ new WeakMap();
+function importsReactValue(source) {
+  return source.ast.body.some((statement) => statement.type === "ImportDeclaration" && statement.source.value === "react" && statement.importKind !== "type");
+}
 function renderFunction(node) {
   let parent = parentOf(node);
   while (parent !== null) {
@@ -16644,7 +16647,7 @@ var no_readonly_prop_state_default = defineRule({
         tsrs = null;
         pending = [];
         const source = context.sourceCode;
-        if (isGeneratedFile({ filename: context.filename, cwd: context.cwd }, source.text) || isTestFile({ filename: context.filename, cwd: context.cwd }) || !source.ast.body.some((statement) => statement.type === "ImportDeclaration" && statement.source.value === "react" && statement.importKind !== "type")) return false;
+        if (isGeneratedFile({ filename: context.filename, cwd: context.cwd }, source.text) || isTestFile({ filename: context.filename, cwd: context.cwd }) || !importsReactValue(source)) return false;
         tsrs = tsrsServices(context);
         return true;
       },
@@ -31917,8 +31920,62 @@ var production_derived_test_cases_default = defineRule({
   }
 });
 
+// src/rules/react-prefer-updater-function.ts
+var NAME131 = "react-prefer-updater-function";
+function stateForSetter(source, setter) {
+  const declarator = setter.defs[0]?.node;
+  if (setter.defs.length !== 1 || declarator?.type !== "VariableDeclarator" || declarator.id.type !== "ArrayPattern" || declarator.init === null) return null;
+  const [value, update] = declarator.id.elements;
+  const call = unwrapExpression(declarator.init);
+  if (value?.type !== "Identifier" || update?.type !== "Identifier" || !setter.identifiers.includes(update) || call.type !== "CallExpression" || !reactHook(source, call.callee, "useState") || !stableValueBinding(setter)) return null;
+  const state = source.getDeclaredVariables(declarator).find((variable) => variable.identifiers.includes(value));
+  return state !== void 0 && stableValueBinding(state) ? { state, owner: enclosingFunction(declarator) } : null;
+}
+function readsStateEagerly(state, argument) {
+  return state.references.some((reference) => reference.isRead() && !typeOnlyReference(reference) && evaluatedBy(reference.identifier, argument));
+}
+function evaluatedBy(node, argument) {
+  for (let current = node; current !== null && !isFunction(current); current = parentOf(current)) {
+    if (current === argument) return true;
+  }
+  return false;
+}
+function isFunction(node) {
+  return node.type === "ArrowFunctionExpression" || node.type === "FunctionExpression" || node.type === "FunctionDeclaration";
+}
+var react_prefer_updater_function_default = defineRule({
+  meta: {
+    type: "problem",
+    docs: { description: "Use the updater form when the next React state is built from the current state.", url: docsUrl(NAME131) },
+    schema: [],
+    messages: {
+      preferUpdaterFunction: "`{{setter}}` builds the next state from the `{{state}}` snapshot, which is stale after an await, in timers and effects, or when two updates run before a re-render. Pass an updater: `{{setter}}((previous) => ...)`, reading `previous` wherever the argument read `{{state}}` or a value derived from it. Compute impure values such as IDs or timestamps before the call so the updater stays pure."
+    }
+  },
+  createOnce(context) {
+    return {
+      before() {
+        const source = context.sourceCode;
+        return !isGeneratedFile({ filename: context.filename, cwd: context.cwd }, source.text) && importsReactValue(source);
+      },
+      CallExpression(node) {
+        const [argument] = node.arguments;
+        const callee = unwrapExpression(node.callee);
+        if (node.arguments.length !== 1 || argument === void 0 || argument.type === "SpreadElement" || callee.type !== "Identifier") return;
+        const value = unwrapExpression(argument);
+        if (value.type === "Identifier" || isFunction(value)) return;
+        const source = context.sourceCode;
+        const setter = findVariable(source.getScope(callee), callee.name);
+        const binding3 = setter === null ? null : stateForSetter(source, setter);
+        if (setter === null || binding3 === null || enclosingFunction(node) === binding3.owner || !readsStateEagerly(binding3.state, argument)) return;
+        context.report({ node, messageId: "preferUpdaterFunction", data: { setter: setter.name, state: binding3.state.name } });
+      }
+    };
+  }
+});
+
 // src/rules/repeated-static-call-cases.ts
-var NAME131 = "repeated-static-call-cases";
+var NAME132 = "repeated-static-call-cases";
 var TEST_MODULES2 = /* @__PURE__ */ new Set(["@jest/globals", "@playwright/test", "bun:test", "node:test", "vitest"]);
 var TEST_NAMES3 = /* @__PURE__ */ new Set(["it", "test"]);
 var TEST_MODIFIERS5 = /* @__PURE__ */ new Set(["concurrent", "fails", "only", "sequential", "skip"]);
@@ -32042,7 +32099,7 @@ var repeated_static_call_cases_default = defineRule({
     type: "suggestion",
     docs: {
       description: "Review three or more consecutive static-input call assertions as potential named cases.",
-      url: docsUrl(NAME131)
+      url: docsUrl(NAME132)
     },
     schema: [],
     messages: { repeatedStaticCallCases: "These {{count}} consecutive assertions repeat a call with static inputs. If independent, use named parameterized cases or subtests; preserve ordered scenarios as one test." }
@@ -32185,7 +32242,7 @@ function isPrimitiveReturn(node, annotation) {
 }
 
 // src/rules/require-assert-never.ts
-var NAME132 = "require-assert-never";
+var NAME133 = "require-assert-never";
 function isRuntimeHandlingStatement(statement) {
   if (statement.type === "EmptyStatement") return false;
   if (statement.type === "BreakStatement") return statement.label !== null;
@@ -32255,7 +32312,7 @@ var require_assert_never_default = defineRule({
     fixable: "code",
     docs: {
       description: "Require a never witness in empty switch defaults so discriminated unions remain exhaustive at compile time.",
-      url: docsUrl(NAME132)
+      url: docsUrl(NAME133)
     },
     schema: [],
     messages: {
@@ -32299,7 +32356,7 @@ var require_assert_never_default = defineRule({
 });
 
 // src/rules/require-button-accessible-name.ts
-var NAME133 = "require-button-accessible-name";
+var NAME134 = "require-button-accessible-name";
 var DELEGATING_PROPS = /* @__PURE__ */ new Set(["render", "asChild"]);
 function delegatesRendering(opening) {
   return opening.attributes.some(
@@ -32312,7 +32369,7 @@ var require_button_accessible_name_default = defineRule({
     type: "problem",
     docs: {
       description: "Require an accessible name on statically unnamed native or configured JSX buttons.",
-      url: docsUrl(NAME133)
+      url: docsUrl(NAME134)
     },
     schema: [
       {
@@ -32366,7 +32423,7 @@ var require_button_accessible_name_default = defineRule({
 });
 
 // src/rules/require-camelcase-properties.ts
-var NAME134 = "require-camelcase-properties";
+var NAME135 = "require-camelcase-properties";
 var SNAKE_CASE_RE2 = /^[a-z][a-z0-9]*(?:_[a-z0-9]+)+$/u;
 function blocksRename(member3, name) {
   if (member3.type === "TSIndexSignature" || "computed" in member3 && member3.computed) return true;
@@ -32380,7 +32437,7 @@ var require_camelcase_properties_default = defineRule({
     fixable: "code",
     docs: {
       description: "Require unquoted lower snake_case TypeScript properties and dot access to use camelCase.",
-      url: docsUrl(NAME134)
+      url: docsUrl(NAME135)
     },
     schema: [],
     defaultOptions: [],
@@ -32444,7 +32501,7 @@ var require_camelcase_properties_default = defineRule({
 });
 
 // src/rules/require-constrained-schema-integer.ts
-var NAME135 = "require-constrained-schema-integer";
+var NAME136 = "require-constrained-schema-integer";
 var FIELD_MINIMUMS = /* @__PURE__ */ new Map([["limit", 1], ["version", 0]]);
 var NUMBER_FACTORIES = /* @__PURE__ */ new Set(["number", "int", "int32", "uint32"]);
 var OBJECT_FACTORIES = /* @__PURE__ */ new Set(["object", "strictObject", "looseObject"]);
@@ -32486,7 +32543,7 @@ var require_constrained_schema_integer_default = defineRule({
     type: "suggestion",
     docs: {
       description: "Declare integer and lower-bound validation for numeric limit and version schema fields.",
-      url: docsUrl(NAME135)
+      url: docsUrl(NAME136)
     },
     schema: [],
     messages: { constrainInteger: "Declare `{{field}}` as an integer with a lower bound of {{minimum}} or greater, or reuse its named domain schema." }
@@ -32965,14 +33022,14 @@ function closedNominalInheritanceFix(sourceCode2, file, call, consumer, implemen
 }
 
 // src/rules/require-explicit-contract-implementation.ts
-var NAME136 = "require-explicit-contract-implementation";
+var NAME137 = "require-explicit-contract-implementation";
 var require_explicit_contract_implementation_default = defineRule({
   meta: {
     type: "problem",
     fixable: "code",
     docs: {
       description: "Require nominal inheritance when an injected abstract class is rejected by an instanceof guard.",
-      url: docsUrl(NAME136)
+      url: docsUrl(NAME137)
     },
     schema: [],
     messages: {
@@ -33663,7 +33720,7 @@ function abstractContractOperations(node, functionAliases) {
 }
 
 // src/rules/require-explicit-service-contract.ts
-var NAME137 = "require-explicit-service-contract";
+var NAME138 = "require-explicit-service-contract";
 var JAVASCRIPT_FILE_RE2 = /\.(?:js|jsx|mjs|cjs)$/iu;
 var BOUNDARY_SUFFIX = /(?:Service|ServiceImpl|Store|Repository|Provider|Executor|Enqueuer|Runner|Platform|Manager|ManagerImpl|Detector|Control|Refresher|Reader|Settlement|Retry|Stop|Catalog|DAO)$/u;
 var NON_SERVICE_SUFFIX = /(?:Router|Middleware|Builder)$/u;
@@ -33679,7 +33736,7 @@ var require_explicit_service_contract_default = defineRule({
     type: "problem",
     docs: {
       description: "Require explicit contracts for classes that invoke retained collaborators.",
-      url: docsUrl(NAME137)
+      url: docsUrl(NAME138)
     },
     schema: [],
     fixable: "code",
@@ -33715,7 +33772,7 @@ var require_explicit_service_contract_default = defineRule({
 });
 
 // src/rules/require-fetch-timeout.ts
-var NAME138 = "require-fetch-timeout";
+var NAME139 = "require-fetch-timeout";
 function emptySignal(expression, resolvesToGlobal) {
   const value = unwrapExpression(expression);
   if (value.type === "Literal" && value.value === null)
@@ -33777,7 +33834,7 @@ var require_fetch_timeout_default = defineRule({
     type: "problem",
     docs: {
       description: "Require an explicit abort signal on locally analyzable global fetch calls.",
-      url: docsUrl(NAME138)
+      url: docsUrl(NAME139)
     },
     schema: [
       {
@@ -33869,13 +33926,13 @@ var require_fetch_timeout_default = defineRule({
 });
 
 // src/rules/require-ime-safe-enter-handler.ts
-var NAME139 = "require-ime-safe-enter-handler";
+var NAME140 = "require-ime-safe-enter-handler";
 var require_ime_safe_enter_handler_default = defineRule({
   meta: {
     type: "problem",
     docs: {
       description: "Preserve IME composition before Enter submits or blurs a native text control.",
-      url: docsUrl(NAME139)
+      url: docsUrl(NAME140)
     },
     schema: [],
     messages: {
@@ -33951,7 +34008,7 @@ function nativeAction(node, event) {
 }
 
 // src/rules/require-interface-for-exported-class.ts
-var NAME140 = "require-interface-for-exported-class";
+var NAME141 = "require-interface-for-exported-class";
 var JAVASCRIPT_FILE_RE3 = /\.(?:js|jsx|mjs|cjs)$/iu;
 function isPublicInstanceMember2(member3) {
   return !member3.static && member3.accessibility !== "private" && member3.accessibility !== "protected" && member3.key.type !== "PrivateIdentifier";
@@ -34012,7 +34069,7 @@ var require_interface_for_exported_class_default = defineRule({
     type: "suggestion",
     docs: {
       description: "Require exported concrete classes with public behavior to declare a contract.",
-      url: docsUrl(NAME140)
+      url: docsUrl(NAME141)
     },
     schema: [],
     fixable: "code",
@@ -34054,7 +34111,7 @@ var require_interface_for_exported_class_default = defineRule({
 });
 
 // src/rules/require-jwt-lifetime.ts
-var NAME141 = "require-jwt-lifetime";
+var NAME142 = "require-jwt-lifetime";
 function isJoseValueImport(declaration) {
   return declaration.type === "ImportDeclaration" && declaration.importKind !== "type" && declaration.source.value === "jose";
 }
@@ -34063,7 +34120,7 @@ var require_jwt_lifetime_default = defineRule({
     type: "problem",
     docs: {
       description: "Require an expiration claim or a maximum token age when returning a verified JWT.",
-      url: docsUrl(NAME141)
+      url: docsUrl(NAME142)
     },
     schema: [],
     messages: { lifetime: "Require the JWT exp claim or set maxTokenAge before returning this verification result." }
@@ -34122,7 +34179,7 @@ var require_jwt_lifetime_default = defineRule({
 });
 
 // src/rules/require-pascal-case-zod-schema-name.ts
-var NAME142 = "require-pascal-case-zod-schema-name";
+var NAME143 = "require-pascal-case-zod-schema-name";
 var PASCAL_SCHEMA_NAME_RE = /^[A-Z][A-Za-z0-9]*Schema$/;
 var BENCHMARK_PATH_RE2 = /(^|[\\/])(?:benchmarks?|bench)[\\/]/;
 var NON_SCHEMA_TERMINALS2 = /* @__PURE__ */ new Set([
@@ -34293,7 +34350,7 @@ var require_pascal_case_zod_schema_name_default = defineRule({
     fixable: "code",
     docs: {
       description: "Require confirmed module-level Zod schema contracts to use PascalCase with a `Schema` suffix.",
-      url: docsUrl(NAME142)
+      url: docsUrl(NAME143)
     },
     schema: [],
     defaultOptions: [],
@@ -34403,7 +34460,7 @@ var require_pascal_case_zod_schema_name_default = defineRule({
 });
 
 // src/rules/require-port-for-service.ts
-var NAME143 = "require-port-for-service";
+var NAME144 = "require-port-for-service";
 var HTTP_TRANSPORT_TYPE_RE = /^(?:KyInstance|AxiosInstance|Session)$/;
 var TRANSPORT_WRAPPER_NAME_RE = /(?:Client$|^Http[A-Z])/;
 var ERROR_VALUE_NAME_RE = /Error$/;
@@ -34537,7 +34594,7 @@ var require_port_for_service_default = defineRule({
     type: "suggestion",
     docs: {
       description: "Advise when an exported service with injected collaborators has public methods not covered by its declared ports.",
-      url: docsUrl(NAME143)
+      url: docsUrl(NAME144)
     },
     schema: [],
     fixable: "code",
@@ -34577,7 +34634,7 @@ var require_port_for_service_default = defineRule({
 });
 
 // src/rules/require-sql-access-class.ts
-var NAME144 = "require-sql-access-class";
+var NAME145 = "require-sql-access-class";
 var DIRECT_EXECUTION_METHODS = /* @__PURE__ */ new Set([
   "all",
   "batch",
@@ -34775,7 +34832,7 @@ var require_sql_access_class_default = defineRule({
     type: "suggestion",
     docs: {
       description: "Keep SQL reads and writes inside a class that receives its database dependency.",
-      url: docsUrl(NAME144)
+      url: docsUrl(NAME145)
     },
     schema: [],
     messages: {
@@ -34847,7 +34904,7 @@ function staticMatcherFix(sourceCode2, property) {
 }
 
 // src/rules/require-static-next-matcher.ts
-var NAME145 = "require-static-next-matcher";
+var NAME146 = "require-static-next-matcher";
 var NEXT_ENTRY_FILE = /(?:^|[/\\])(?:middleware|proxy)\.[cm]?[jt]sx?$/u;
 function isStaticValue(node) {
   const value = unwrapExpression(node);
@@ -34879,7 +34936,7 @@ var require_static_next_matcher_default = defineRule({
     fixable: "code",
     docs: {
       description: "Require Next.js middleware and proxy matcher configuration to contain only build-time literals.",
-      url: docsUrl(NAME145)
+      url: docsUrl(NAME146)
     },
     schema: [],
     messages: {
@@ -34923,7 +34980,7 @@ var require_static_next_matcher_default = defineRule({
 });
 
 // src/rules/require-svg-accessible-name.ts
-var NAME146 = "require-svg-accessible-name";
+var NAME147 = "require-svg-accessible-name";
 var DECORATIVE_ROLES = /* @__PURE__ */ new Set(["none", "presentation"]);
 var NAME_ATTRIBUTES = ["aria-label", "aria-labelledby"];
 var NO_ICON_MODULES = [];
@@ -34932,7 +34989,7 @@ var require_svg_accessible_name_default = defineRule({
     type: "problem",
     docs: {
       description: "Require a name or explicit decorative semantics on inline SVG elements.",
-      url: docsUrl(NAME146)
+      url: docsUrl(NAME147)
     },
     schema: [],
     messages: {
@@ -34964,7 +35021,7 @@ var require_svg_accessible_name_default = defineRule({
 });
 
 // src/rules/require-use-form-default-values.ts
-var NAME147 = "require-use-form-default-values";
+var NAME148 = "require-use-form-default-values";
 var UNSAFE_FORM_MEMBERS = /* @__PURE__ */ new Set(["__defineGetter__", "__defineSetter__", "__proto__", "valueOf"]);
 function staticPropertyName2(property) {
   return property.computed ? null : getPropertyName(property);
@@ -35007,7 +35064,7 @@ var require_use_form_default_values_default = defineRule({
     type: "problem",
     docs: {
       description: "Require explicit form-level initialization or a field default for directly bound Controller fields.",
-      url: docsUrl(NAME147)
+      url: docsUrl(NAME148)
     },
     schema: [],
     messages: { requireUseFormDefaultValues: "Add non-undefined defaultValues/values to the associated useForm call or non-undefined defaultValue to this field." }
@@ -35146,7 +35203,7 @@ var require_use_form_default_values_default = defineRule({
 });
 
 // src/rules/require-use-server-in-actions-file.ts
-var NAME148 = "require-use-server-in-actions-file";
+var NAME149 = "require-use-server-in-actions-file";
 var ACTION_MODULE_RE = /(?:^|\/)app\/.*\/(?:actions|[^/]+-actions)\.[cm]?[jt]s$/u;
 function isUseServerDirective2(statement) {
   return statement.type === "ExpressionStatement" && statement.directive === "use server";
@@ -35167,7 +35224,7 @@ var require_use_server_in_actions_file_default = defineRule({
     type: "problem",
     docs: {
       description: "route action module missing the use server directive",
-      url: docsUrl(NAME148)
+      url: docsUrl(NAME149)
     },
     schema: [],
     messages: {
@@ -35193,7 +35250,7 @@ var require_use_server_in_actions_file_default = defineRule({
 });
 
 // src/rules/require-zod-form-validation.ts
-var NAME149 = "require-zod-form-validation";
+var NAME150 = "require-zod-form-validation";
 var ZOD_PARSE_METHODS = /* @__PURE__ */ new Set([
   "parse",
   "safeParse",
@@ -35252,7 +35309,7 @@ var require_zod_form_validation_default = defineRule({
     type: "problem",
     docs: {
       description: "Require Zod validation (`Schema.parse(...)` / `Schema.safeParse(...)`) when reading values out of a `FormData` object.",
-      url: docsUrl(NAME149)
+      url: docsUrl(NAME150)
     },
     schema: [],
     messages: {
@@ -35540,7 +35597,7 @@ function declaresFramework(metadata, name) {
 }
 
 // src/rules/sole-export-matches-filename.ts
-var NAME150 = "sole-export-matches-filename";
+var NAME151 = "sole-export-matches-filename";
 var EXCLUDED_STEMS = /* @__PURE__ */ new Set(["global", "index", "types"]);
 function stem3(filename) {
   const base = filename.replaceAll("\\", "/").split("/").at(-1) ?? "";
@@ -35554,7 +35611,7 @@ var sole_export_matches_filename_default = defineRule({
     type: "suggestion",
     docs: {
       description: "Make a module filename reflect its sole named public runtime export.",
-      url: docsUrl(NAME150)
+      url: docsUrl(NAME151)
     },
     schema: [],
     messages: {
@@ -35608,7 +35665,7 @@ var source_coupled_test_default = createSourceCoupledRule(
 );
 
 // src/rules/stepdown.ts
-function isFunction(node) {
+function isFunction2(node) {
   return node.type === "ArrowFunctionExpression" || node.type === "FunctionDeclaration" || node.type === "FunctionExpression";
 }
 function isErasedExpression2(node) {
@@ -35631,7 +35688,7 @@ function wrappedExpression(node) {
 function enclosingFunction2(node) {
   let current = parentOf(node);
   while (current !== null) {
-    if (isFunction(current)) return current;
+    if (isFunction2(current)) return current;
     current = parentOf(current);
   }
   return void 0;
@@ -35890,7 +35947,7 @@ function isDeferredModuleStatement(statement) {
   if (node === null || node.type === "FunctionDeclaration" || node.type === "TSDeclareFunction" || node.type === "TSInterfaceDeclaration" || node.type === "TSTypeAliasDeclaration" || node.type === "ImportDeclaration" || node.type === "ExportAllDeclaration" || node.type === "EmptyStatement") {
     return true;
   }
-  return node.type === "VariableDeclaration" && node.declarations.every((declaration) => declaration.id.type === "Identifier" && declaration.init !== null && isFunction(unwrapExpression4(declaration.init)));
+  return node.type === "VariableDeclaration" && node.declarations.every((declaration) => declaration.id.type === "Identifier" && declaration.init !== null && isFunction2(unwrapExpression4(declaration.init)));
 }
 function exportedNames(program) {
   const names = /* @__PURE__ */ new Set();
@@ -35931,7 +35988,7 @@ function moduleDefinitions(program) {
 }
 function constFunctionDefinition(declarator) {
   const initializer = declarator.init === null ? null : unwrapExpression4(declarator.init);
-  if (declarator.id.type !== "Identifier" || initializer === null || !isFunction(initializer)) return [];
+  if (declarator.id.type !== "Identifier" || initializer === null || !isFunction2(initializer)) return [];
   const { name } = declarator.id;
   return [{ key: name, name, node: declarator, functionNode: initializer, bindingNode: declarator }];
 }
@@ -36010,7 +36067,7 @@ function referencedPropertyName(node) {
 }
 function walk3(node, visitorKeys, visit2, nestedFunction = false) {
   visit2(node, nestedFunction);
-  const nested = nestedFunction || isFunction(node) || node.type === "ClassDeclaration" || node.type === "ClassExpression";
+  const nested = nestedFunction || isFunction2(node) || node.type === "ClassDeclaration" || node.type === "ClassExpression";
   forEachAstChild(node, visitorKeys, (child) => {
     walk3(child, visitorKeys, visit2, nested);
   });
@@ -36031,7 +36088,7 @@ function classScope(state, node, computedReferences, privateBandOnly) {
   const visitorKeys = state.sourceCode.visitorKeys;
   const methods = node.body.body.flatMap((member3) => member3.type === "MethodDefinition" ? [member3] : []);
   const counts = classMethodCounts(node.body, methods);
-  const implementations = methods.filter((method) => isFunction(method.value));
+  const implementations = methods.filter((method) => isFunction2(method.value));
   const implementationCounts = classMethodCounts(node.body, implementations);
   const scopeDefinitions = implementations.flatMap((method) => {
     const name = methodName(method);
@@ -36069,7 +36126,7 @@ function classScope(state, node, computedReferences, privateBandOnly) {
       return;
     }
     const methodFunction = method.value;
-    if (!isFunction(methodFunction) || methodFunction.body === null) return;
+    if (!isFunction2(methodFunction) || methodFunction.body === null) return;
     const body3 = methodFunction.body;
     const caller = methodKey(name, method.static);
     const receivers = new Map(classReceivers);
@@ -36297,7 +36354,7 @@ var stepdown_default = defineRule({
 });
 
 // src/rules/store-insert-requires-on-conflict.ts
-var NAME151 = "store-insert-requires-on-conflict";
+var NAME152 = "store-insert-requires-on-conflict";
 var INSERT_WRITE = /\bINSERT\s+(?:OR\s+\w+\s+)?INTO\s+[\w."'`?$:@-]+\s*(?:\([^)]*\)\s*)?(?:VALUES|SELECT|DEFAULT\s+VALUES)\b/i;
 var CONFLICT_HANDLED = /\bON\s+CONFLICT\b|\bON\s+DUPLICATE\s+KEY\b|\bINSERT\s+OR\s+(?:IGNORE|REPLACE)\b|\bINSERT\b[\s\S]*?\bSELECT\b[\s\S]*?\bWHERE\s+NOT\s+EXISTS\b/i;
 var REPLAY_CONTRACT_NAME = /(?:enqueue|ensure|migrate|recordOnce|schedule|seed|upsert|getOrCreate|createIfAbsent|insertIfAbsent)/i;
@@ -36326,7 +36383,7 @@ var store_insert_requires_on_conflict_default = defineRule({
     type: "problem",
     docs: {
       description: "Review conflict handling for embedded inserts in replay-named callables.",
-      url: docsUrl(NAME151)
+      url: docsUrl(NAME152)
     },
     schema: [],
     messages: {
@@ -36351,7 +36408,7 @@ var store_insert_requires_on_conflict_default = defineRule({
 });
 
 // src/rules/test-loops-over-literal-cases.ts
-var NAME152 = "test-loops-over-literal-cases";
+var NAME153 = "test-loops-over-literal-cases";
 var TEST_CALLERS3 = /* @__PURE__ */ new Set(["it", "test"]);
 var TEST_MODIFIERS6 = /* @__PURE__ */ new Set(["concurrent", "fails", "only", "sequential", "skip"]);
 var ASSERTION_ROOTS = /* @__PURE__ */ new Set([
@@ -36447,7 +36504,7 @@ var test_loops_over_literal_cases_default = defineRule({
     type: "suggestion",
     docs: {
       description: "Disallow assertions over an inline literal case loop in a test; parameterization reports and names every case independently.",
-      url: docsUrl(NAME152)
+      url: docsUrl(NAME153)
     },
     schema: [],
     messages: {
@@ -36511,7 +36568,7 @@ var test_loops_over_literal_cases_default = defineRule({
 });
 
 // src/rules/test-phase-label-comment.ts
-var NAME153 = "test-phase-label-comment";
+var NAME154 = "test-phase-label-comment";
 var PHASE_WORD = String.raw`arrange|act|assert(?:ion)?s?|given|when|then|exercise|execute|verif(?:y|ication)|cleanup|prepare|sanity(?:\s+check)?`;
 var PHASE_RE = new RegExp(
   String.raw`^[-=~*_#.\s]{0,40}(?:${PHASE_WORD})(?:\s*(?:[/&+,|]|->|and)\s*(?:${PHASE_WORD}))*[-=~*_#.\s:;!–—]{0,40}$`,
@@ -36539,7 +36596,7 @@ var test_phase_label_comment_default = defineRule({
     fixable: "code",
     docs: {
       description: "Tests must not use bare Arrange, Act, Assert, Given, When, or Then phase comments.",
-      url: docsUrl(NAME153)
+      url: docsUrl(NAME154)
     },
     schema: [],
     messages: { removeLabel: "Bare test phase label \u2014 delete it and let the test names and helpers carry the structure." }
@@ -36722,6 +36779,7 @@ var RULES = {
   "prefer-zod-infer": prefer_zod_infer_default,
   "prefer-zod-parse-output-type": prefer_zod_parse_output_type_default,
   "production-derived-test-cases": production_derived_test_cases_default,
+  "react-prefer-updater-function": react_prefer_updater_function_default,
   "repeated-static-call-cases": repeated_static_call_cases_default,
   "require-assert-never": require_assert_never_default,
   "require-button-accessible-name": require_button_accessible_name_default,

package/dist/index.js

@@ -1,6 +1,6 @@
 import {
   src_default
-} from "./chunk-E2O4LN7G.js";
+} from "./chunk-UI4ADSAA.js";
 export {
   src_default as default
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@sarj/eslint-plugin",
-  "version": "19.18.0",
+  "version": "19.19.0",
   "packageManager": "npm@12.2.1",
   "description": "Custom Oxlint rules for hypermodern TypeScript / React / Next.js projects",
   "type": "module",