fix: review

Author: elevensky

src/Field.tsx

@@ -627,17 +627,14 @@ class Field extends React.Component<InternalFieldProps, FieldState> implements F
 function WrapperField<Values = any>({ name, ...restProps }: FieldProps<Values>) {
   const fieldContext = React.useContext(FieldContext);
   const listContext = React.useContext(ListContext);
+  const mergedIsListField = restProps.isListField || (!restProps.isList && !!listContext);
   const namePath = name !== undefined ? getNamePath(name) : undefined;
 
   let key: string = 'keep';
   if (!restProps.isListField) {
     key = `_${(namePath || []).join('_')}`;
   }
 
-  if (restProps.isListField === undefined && !restProps.isList && listContext) {
-    restProps.isListField = listContext.inList;
-  }
-
   // Warning if it's a directly list field.
   // We can still support multiple level field preserve.
   if (
@@ -649,7 +646,15 @@ function WrapperField<Values = any>({ name, ...restProps }: FieldProps<Values>)
     warning(false, '`preserve` should not apply on Form.List fields.');
   }
 
-  return <Field key={key} name={namePath} {...restProps} fieldContext={fieldContext} />;
+  return (
+    <Field
+      key={key}
+      name={namePath}
+      isListField={mergedIsListField}
+      {...restProps}
+      fieldContext={fieldContext}
+    />
+  );
 }
 
 export default WrapperField;

src/List.tsx

@@ -60,7 +60,6 @@ const List: React.FunctionComponent<ListProps> = ({
         const pathName = namePath[len];
         return [keyManager.keys[pathName], namePath.slice(len + 1)];
       },
-      inList: true,
     }),
     [prefixName],
   );

src/ListContext.ts

@@ -3,7 +3,6 @@ import type { InternalNamePath } from './interface';
 
 export interface ListContextProps {
   getKey: (namePath: InternalNamePath) => [React.Key, InternalNamePath];
-  inList: boolean;
 }
 
 const ListContext = React.createContext<ListContextProps | null>(null);