🐞 fix: respect parent-provided useFieldArray rules (#13082)

[Han5991]

⏺ Summary

fixes #13082

Problem

When rules were passed to useFieldArray from a parent component, they were wrapped in React.useMemo without proper dependencies, causing the rules to be registered only once during initial render. If the rules object reference changed (e.g., created inline or from useMemo in parent), the validation would not update
properly.

Solution

• Removed unnecessary React.useMemo wrapper around the control.register call in src/useFieldArray.ts:112-118

Changes

• src/useFieldArray.ts: Simplified rules registration by removing useMemo wrapper
• src/tests/useFieldArray.test.tsx: Added comprehensive test case validating that rules from parent component are properly enforced

[bluebill1049]

thanks for the PR!

[bluebill1049]

could we try to use rules with ref instead? would be good if we can still keep the memo.

const rulesRef = useRef(rules)

reulsRef.current = rules

[Han5991]

  const rulesRef = React.useRef(rules);

  rulesRef.current = rules;

  control._names.array.add(name);

  rulesRef.current &&
    (control as Control<TFieldValues, any, TTransformedValues>).register(
      name as FieldPath<TFieldValues>,
      rulesRef.current as RegisterOptions<TFieldValues>,
    );

rulesRef as suggested. This keeps the latest rules value in a ref while preserving memoization. Test passes. Does this look right?

[bluebill1049]

const rulesRef = React.useRef(rules);

  rulesRef.current = rules;

  const _actioned = React.useRef(false);

  control._names.array.add(name);

  React.useMemo(
    () =>
      rules &&
      (control as Control<TFieldValues, any, TTransformedValues>).register(
        name as FieldPath<TFieldValues>,
        rulesRef.current as RegisterOptions<TFieldValues>,
      ),
    [control, name],
  );

what about this?

[Han5991]

re-ran the test suite, but it still fails.

[Han5991]

  React.useMemo(
    () =>
      rules &&
      (control as Control<TFieldValues, any, TTransformedValues>).register(
        name as FieldPath<TFieldValues>,
        rules as RegisterOptions<TFieldValues>,
      ),
    // eslint-disable-next-line react-hooks/exhaustive-deps
    [control, name, fields.length, rules],
  );

What do you think about this code?

[bluebill1049]

ty very much for the fix