@cosmicdrift/kumiko-headless 0.186.0 → 0.186.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-headless",
-  "version": "0.186.0",
+  "version": "0.186.2",
   "description": "Headless UI logic for Kumiko — Dispatcher contract, Form-Controller, View-Model, Nav-Resolver. Plattform- und React-frei; jeder Renderer (renderer, renderer-web, renderer-native, …) komponiert darauf.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -36,7 +36,7 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-framework": "0.186.0",
+    "@cosmicdrift/kumiko-framework": "0.186.2",
     "temporal-polyfill": "^0.3.2",
     "zod": "^4.4.3"
   },

package/src/form/__tests__/submit.test.ts

@@ -147,6 +147,39 @@ describe("createFormController — submit()", () => {
     expect(disp.writeSpy).toHaveBeenCalledWith("app:write:task:update", { title: "world" });
   });
 
+  test("payloadMode: 'values' — strips an untouched optional field seeded with \"\"", async () => {
+    // `dueDate` was never touched by the user; buildInitialValues seeded it
+    // with "" for the controlled input. A `.optional()` server schema
+    // rejects "" but accepts a missing key, so the key must be dropped
+    // entirely — not sent as `dueDate: undefined`.
+    const disp = makeDispatcher();
+    const form = createFormController({
+      initial: { title: "hello", dueDate: "" },
+      submit: { dispatcher: disp, type: "app:write:task:create" },
+    });
+
+    await form.submit();
+
+    const call = disp.writeSpy.mock.calls[0]?.[1] as Record<string, unknown>;
+    expect("dueDate" in call).toBe(false);
+    expect(call["title"]).toBe("hello");
+  });
+
+  test("payloadMode: 'values' — keeps an explicitly-cleared field that started non-empty", async () => {
+    // `note` had a real value initially; the user cleared it via setField.
+    // That's a genuine change, not an untouched seed — it must survive.
+    const disp = makeDispatcher();
+    const form = createFormController({
+      initial: { note: "x" },
+      submit: { dispatcher: disp, type: "app:write:task:create" },
+    });
+    form.setField("note", "");
+
+    await form.submit();
+
+    expect(disp.writeSpy).toHaveBeenCalledWith("app:write:task:create", { note: "" });
+  });
+
   test("stale-submit race: edits during the in-flight write stay dirty after success", async () => {
     // User submits "hello", the network takes 50ms. During those 50ms the
     // user types "world" into the same field. The server sees "hello"

package/src/form/form-controller.ts

@@ -74,6 +74,24 @@ function valuesDiff<TValues extends FormValues>(
   return out;
 }
 
+// buildInitialValues seeds untouched optional fields with "" for controlled
+// inputs; a `.optional()` server schema accepts undefined but not "".
+function stripUntouchedEmptyStrings<TValues extends FormValues>(
+  values: TValues,
+  initial: TValues,
+): TValues {
+  const v = values as Record<string, unknown>; // @cast-boundary form-values
+  const ini = initial as Record<string, unknown>; // @cast-boundary form-values
+  let out: Record<string, unknown> | undefined;
+  for (const key of Object.keys(v)) {
+    if (v[key] === "" && Object.is(v[key], ini[key])) {
+      out ??= { ...v };
+      delete out[key];
+    }
+  }
+  return (out ?? v) as TValues;
+}
+
 // Shallow-freeze so accidental mutations on the snapshot (e.g. a test
 // writing `snapshot.values.title = "x"`) throw in strict mode instead of
 // silently diverging from the controller's internal state. Deep-freeze
@@ -304,7 +322,7 @@ export function createFormController<TValues extends FormValues, TCtx = unknown>
         }
         payload = submittedSnapshot.changes;
       } else {
-        payload = submittedValues;
+        payload = stripUntouchedEmptyStrings(submittedValues, submittedSnapshot.initial);
       }
 
       const runWrite = async (): Promise<SubmitResult<TData>> => {