🐞 fix #5234 should render error logic for valid state (#5235)

* fix #5234 should render error logic for valid state

* Revert "fix #5234 should render error logic for valid state"

This reverts commit 082dd3d.

* isValid check for render

* code reducation

* fix cypress test

* more code reducation

* combine logic

* remove if

* improve logic

* Revert "improve logic"

This reverts commit a5f2c5f.

* Revert "Revert "improve logic""

This reverts commit b1438cc.

* fix tests

cypress/integration/isValid.ts

@@ -7,7 +7,7 @@ describe('isValid', () => {
     cy.get('#isValid').contains('false');
     cy.get('input[name="lastName"]').type('test');
     cy.get('#isValid').contains('true');
-    cy.get('#renderCount').contains('4');
+    cy.get('#renderCount').contains('3');
     cy.get('#toggle').click();
     cy.get('#isValid').contains('false');
     cy.get('#toggle').click();

src/useForm.ts

@@ -190,34 +190,24 @@ export function useForm<
     ): boolean | void => {
       const previousError = get(formStateRef.current.errors, name);
 
-      let shouldReRender =
-        shouldRender ||
-        !deepEqual(previousError, error, true) ||
-        (readFormStateRef.current.isValid &&
-          isUndefined(error) &&
-          get(fieldsWithValidationRef.current, name) &&
-          !get(validFieldsRef.current, name));
-
       if (error) {
         unset(validFieldsRef.current, name);
-        shouldReRender =
-          shouldReRender ||
-          !previousError ||
-          !deepEqual(previousError, error, true);
         set(formStateRef.current.errors, name, error);
       } else {
-        if (get(fieldsWithValidationRef.current, name) || resolverRef.current) {
+        (get(fieldsWithValidationRef.current, name) || resolverRef.current) &&
           set(validFieldsRef.current, name, true);
-          shouldReRender = shouldReRender || previousError;
-        }
-
         unset(formStateRef.current.errors, name);
       }
 
       if (
-        (shouldReRender && !isNullOrUndefined(shouldRender)) ||
-        !isEmptyObject(state) ||
-        isWatched
+        (shouldRender ||
+          isWatched ||
+          (error ? !deepEqual(previousError, error, true) : previousError) ||
+          !isEmptyObject(state) ||
+          (readFormStateRef.current.isValid &&
+            formStateRef.current.isValid !==
+              (resolverRef.current ? !!isValid : getIsValid()))) &&
+        !isNullOrUndefined(shouldRender)
       ) {
         const updatedFormState = {
           ...state,