@allhailai/formfoundry-core 1.2.0 → 1.2.1

package/dist/index.cjs

@@ -117,13 +117,28 @@ function createHookDispatcher() {
       };
       return next(payload);
     },
-    // BUG-34 fix: async dispatch that awaits each middleware result
+    // BUG-34 + E1 fix: async dispatch that processes middleware sequentially.
+    // Unlike dispatch() (recursive chain with wrapping), dispatchAsync uses
+    // a sequential pipeline: each middleware receives the output of the
+    // previous one. Short-circuiting works by not calling next().
+    // This is intentionally different from dispatch() because sync middleware
+    // cannot await async next() calls, so wrapping patterns don't compose.
     async dispatchAsync(payload) {
       if (middlewares.length === 0) return payload;
       let current = payload;
       for (const middleware of middlewares) {
-        const result = middleware(current, (p) => p);
-        current = result instanceof Promise ? await result : result;
+        let called = false;
+        const next = (p) => {
+          called = true;
+          current = p;
+          return p;
+        };
+        const result = middleware(current, next);
+        if (result instanceof Promise) {
+          current = await result;
+        } else if (!called) {
+          return result;
+        } else ;
       }
       return current;
     }
@@ -545,13 +560,20 @@ function createNode(options = {}) {
       if (node.type === "group" && node.children.length > 0) {
         const obj = typeof resetValue === "object" && resetValue !== null && !Array.isArray(resetValue) ? resetValue : void 0;
         for (const child of node.children) {
+          const savedParent = child.parent;
+          child.parent = null;
           child.reset(obj?.[child.name]);
+          child.parent = savedParent;
         }
         recomputeValue(node);
       } else if (node.type === "list" && node.children.length > 0) {
         const arr = Array.isArray(resetValue) ? resetValue : void 0;
         for (let i = 0; i < node.children.length; i++) {
-          node.children[i].reset(arr?.[i]);
+          const child = node.children[i];
+          const savedParent = child.parent;
+          child.parent = null;
+          child.reset(arr?.[i]);
+          child.parent = savedParent;
         }
         recomputeValue(node);
       } else {
@@ -577,10 +599,14 @@ function createNode(options = {}) {
      * before settlement completes.
      */
     async settle() {
-      if (pendingCount <= 0) return;
-      return new Promise((resolve) => {
-        settleResolvers.push(resolve);
-      });
+      if (pendingCount > 0) {
+        await new Promise((resolve) => {
+          settleResolvers.push(resolve);
+        });
+      }
+      for (const child of node.children) {
+        await child.settle();
+      }
     },
     async submit() {
     }
@@ -1375,8 +1401,13 @@ function useFormFoundryField(options) {
   }, [validationBehavior, validate, node]);
   const depsKey = dependsOn?.join(",") ?? "";
   const depsValues = dependsOn?.map((dep) => JSON.stringify(getValue(dep))).join("|") ?? "";
+  const depsInitializedRef = React4.useRef(false);
   React4.useEffect(() => {
     if (!dependsOn || dependsOn.length === 0) return;
+    if (!depsInitializedRef.current) {
+      depsInitializedRef.current = true;
+      return;
+    }
     if (touched || hasBlurredRef.current) {
       void validate();
     }
@@ -1471,7 +1502,7 @@ function useFormNode(id) {
   return id ? getNode(id) : void 0;
 }
 function useFormNodes(ids) {
-  const [, forceRender] = React4.useState(0);
+  const [renderCount, forceRender] = React4.useState(0);
   const nodeUnsubsRef = React4.useRef(/* @__PURE__ */ new Map());
   const triggerRender = React4.useCallback(() => {
     forceRender((n) => n + 1);
@@ -1512,11 +1543,13 @@ function useFormNodes(ids) {
       nodeUnsubsRef.current.clear();
     };
   }, [idsKey, triggerRender]);
-  const result = /* @__PURE__ */ new Map();
-  for (const id of ids) {
-    result.set(id, getNode(id));
-  }
-  return result;
+  return React4.useMemo(() => {
+    const result = /* @__PURE__ */ new Map();
+    for (const id of ids) {
+      result.set(id, getNode(id));
+    }
+    return result;
+  }, [idsKey, renderCount]);
 }
 function useFormPrint(defaultOptions) {
   const { values } = useFormValues();