Revert "🐞 fix #13260: notify all matching field-array roots on nested setValue updates (#13420)"

This reverts commit c6c3d87.

Author: bluebill1049

src/__tests__/logic/getFieldArrayParentNames.test.ts

@@ -0,0 +1,39 @@
+import isNameInFieldArray from '../../logic/isNameInFieldArray';
+
+describe('isNameInFieldArray', () => {
+  it('should find match array field', () => {
+    expect(isNameInFieldArray(new Set(['test']), 'test.0')).toBeTruthy();
+    expect(isNameInFieldArray(new Set(['te']), 'test.0')).toBeFalsy();
+    expect(isNameInFieldArray(new Set(['te']), 'test.0')).toBeFalsy();
+    expect(isNameInFieldArray(new Set(['test1']), 'test[0]')).toBeFalsy();
+    expect(isNameInFieldArray(new Set(['test1']), 'test.0')).toBeFalsy();
+    expect(
+      isNameInFieldArray(new Set(['test']), 'test.0.data[0]'),
+    ).toBeTruthy();
+    expect(isNameInFieldArray(new Set(['test']), 'test.0.data.0')).toBeTruthy();
+    expect(isNameInFieldArray(new Set(['test']), 'test1.0.data.0')).toBeFalsy();
+    expect(isNameInFieldArray(new Set(['test']), 'data.0.data.0')).toBeFalsy();
+  });
+
+  it('should find match when field array is nested under a numeric path segment', () => {
+    // Field array registered at `steps.0.items`, Controller name is `steps.0.items.2.name`
+    // getNodeParentName returns `steps` (first numeric segment), but the field array
+    // is `steps.0.items` — so isNameInFieldArray must check deeper parents too.
+    expect(
+      isNameInFieldArray(new Set(['steps.0.items']), 'steps.0.items.2.name'),
+    ).toBeTruthy();
+    expect(
+      isNameInFieldArray(new Set(['steps.0.items']), 'steps.0.items.0.value'),
+    ).toBeTruthy();
+    // Deeper nesting: `a.0.b.1.items`
+    expect(
+      isNameInFieldArray(new Set(['a.0.b.1.items']), 'a.0.b.1.items.3.name'),
+    ).toBeTruthy();
+    // Should still return false when there's no actual match
+    expect(
+      isNameInFieldArray(new Set(['steps.0.other']), 'steps.0.items.2.name'),
+    ).toBeFalsy();
+    // Original top-level behavior should still work
+    expect(isNameInFieldArray(new Set(['items']), 'items.2.name')).toBeTruthy();
+  });
+});

src/__tests__/logic/isNameInFieldArray.test.ts

@@ -1561,95 +1561,6 @@ describe('setValue', () => {
     expect(valueNotifications).toHaveLength(0);
   });
 
-  it('should re-render useFieldArray fields when setValue targets a nested array path - issue #13260', () => {
-    type FormValues = {
-      items: { value: number }[];
-    };
-
-    const App = () => {
-      const { control, setValue } = useForm<FormValues>({
-        defaultValues: {
-          items: [{ value: 0 }, { value: 0 }, { value: 0 }],
-        },
-      });
-      const { fields } = useFieldArray({ control, name: 'items' });
-
-      return (
-        <div>
-          {fields.map((field, index) => (
-            <span key={field.id} data-testid={`item-${index}`}>
-              {field.value}
-            </span>
-          ))}
-          <button
-            type="button"
-            onClick={() => setValue('items.1.value', 42, { shouldDirty: true })}
-          >
-            update
-          </button>
-        </div>
-      );
-    };
-
-    render(<App />);
-
-    expect(screen.getByTestId('item-1').textContent).toBe('0');
-
-    act(() => {
-      fireEvent.click(screen.getByRole('button', { name: /update/i }));
-    });
-
-    expect(screen.getByTestId('item-1').textContent).toBe('42');
-  });
-
-  it('should re-render all matching field arrays when setValue targets nested field-array path', () => {
-    type FormValues = {
-      items: { subitems: { value: number }[] }[];
-    };
-
-    const App = () => {
-      const { control, setValue } = useForm<FormValues>({
-        defaultValues: {
-          items: [{ subitems: [{ value: 0 }, { value: 0 }] }],
-        },
-      });
-      const { fields: itemFields } = useFieldArray({ control, name: 'items' });
-      const { fields: subItemFields } = useFieldArray({
-        control,
-        name: 'items.0.subitems',
-      });
-
-      return (
-        <div>
-          <span data-testid="item-nested-value">
-            {itemFields[0]?.subitems?.[1]?.value}
-          </span>
-          <span data-testid="subitem-value">{subItemFields[1]?.value}</span>
-          <button
-            type="button"
-            onClick={() =>
-              setValue('items.0.subitems.1.value', 7, { shouldDirty: true })
-            }
-          >
-            update nested
-          </button>
-        </div>
-      );
-    };
-
-    render(<App />);
-
-    expect(screen.getByTestId('item-nested-value').textContent).toBe('0');
-    expect(screen.getByTestId('subitem-value').textContent).toBe('0');
-
-    act(() => {
-      fireEvent.click(screen.getByRole('button', { name: /update nested/i }));
-    });
-
-    expect(screen.getByTestId('item-nested-value').textContent).toBe('7');
-    expect(screen.getByTestId('subitem-value').textContent).toBe('7');
-  });
-
   it('should mark field as dirty when updating array of objects with shouldDirty true', () => {
     type IData = {
       data: { id: number; name: string }[];

src/__tests__/useForm/setValue.test.tsx

@@ -83,14 +83,14 @@ import unset from '../utils/unset';
 import generateWatchOutput from './generateWatchOutput';
 import getDirtyFields from './getDirtyFields';
 import getEventValue from './getEventValue';
-import getFieldArrayParentNames from './getFieldArrayParentNames';
 import getFieldValue from './getFieldValue';
 import getFieldValueAs from './getFieldValueAs';
 import getResolverOptions from './getResolverOptions';
 import getRuleValue from './getRuleValue';
 import getValidationModes from './getValidationModes';
 import hasPromiseValidation from './hasPromiseValidation';
 import hasValidation from './hasValidation';
+import isNameInFieldArray from './isNameInFieldArray';
 import isWatched from './isWatched';
 import iterateFieldsByAction from './iterateFieldsByAction';
 import schemaErrorLookup from './schemaErrorLookup';
@@ -916,12 +916,6 @@ export function createFormControl<
       const watched = isWatched(name, _names);
       const values = skipClone ? _formValues : cloneObject(_formValues);
 
-      if (!isFieldArray) {
-        for (const arrayName of getFieldArrayParentNames(_names.array, name)) {
-          _subjects.array.next({ name: arrayName, values });
-        }
-      }
-
       _subjects.state.next({
         ...(watched && _formState),
         name: _state.mount || watched ? name : undefined,
@@ -1477,10 +1471,7 @@ export function createFormControl<
           }
 
           (_options.shouldUnregister || options.shouldUnregister) &&
-            !(
-              getFieldArrayParentNames(_names.array, name).length &&
-              _state.action
-            ) &&
+            !(isNameInFieldArray(_names.array, name) && _state.action) &&
             _names.unMount.add(name);
         }
       },

src/logic/createFormControl.ts

@@ -0,0 +1,9 @@
+import type { InternalFieldName } from '../types';
+
+export default (names: Set<InternalFieldName>, name: InternalFieldName) =>
+  name
+    .split('.')
+    .some(
+      (part, index, arr) =>
+        !isNaN(Number(part)) && names.has(arr.slice(0, index).join('.')),
+    );

src/logic/getFieldArrayParentNames.ts

@@ -1,7 +1,7 @@
 import React from 'react';
 
 import getEventValue from './logic/getEventValue';
-import getFieldArrayParentNames from './logic/getFieldArrayParentNames';
+import isNameInFieldArray from './logic/isNameInFieldArray';
 import cloneObject from './utils/cloneObject';
 import get from './utils/get';
 import isBoolean from './utils/isBoolean';
@@ -67,8 +67,7 @@ export function useController<
     defaultValue,
     exact = true,
   } = props;
-  const isArrayField = !!getFieldArrayParentNames(control._names.array, name)
-    .length;
+  const isArrayField = isNameInFieldArray(control._names.array, name);
 
   const defaultValueMemo = React.useMemo(
     () =>