@cosmicdrift/kumiko-renderer-web 0.242.0 → 0.243.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer-web",
-  "version": "0.242.0",
+  "version": "0.243.1",
   "description": "Web-platform bindings for @cosmicdrift/kumiko-renderer. HTML default-primitives, browser history-based navigation, EventSource-backed live events, and a one-call createKumikoApp that mounts the whole stack via react-dom.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -16,9 +16,9 @@
     "./styles.css": "./src/styles.css"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.242.0",
-    "@cosmicdrift/kumiko-headless": "0.242.0",
-    "@cosmicdrift/kumiko-renderer": "0.242.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.243.1",
+    "@cosmicdrift/kumiko-headless": "0.243.1",
+    "@cosmicdrift/kumiko-renderer": "0.243.1",
     "@radix-ui/react-dialog": "^1.1.15",
     "@radix-ui/react-dropdown-menu": "^2.1.16",
     "@radix-ui/react-label": "^2.1.8",
@@ -64,7 +64,7 @@
     "@types/react-dom": "^19.2.3",
     "jsdom": "^29.1.1",
     "tailwindcss": "^4.3.0",
-    "@cosmicdrift/kumiko-locale-de": "0.242.0"
+    "@cosmicdrift/kumiko-locale-de": "0.243.1"
   },
   "repository": {
     "type": "git",

package/src/__tests__/kumiko-screen.test.tsx

@@ -1889,6 +1889,114 @@ describe("KumikoScreen", () => {
     expect(navigateCalls[0]).toEqual({ screenId: "task-edit", entityId: "x" });
   });
 
+  // #2670: a handler that writes a child record reports the child's id, but
+  // the redirect target is the parent's detail screen — `idFrom` names the
+  // payload field the navigation id comes from, so the handler keeps
+  // reporting what it actually wrote.
+  test("actionForm mit redirect-Objekt + idFrom: navigiert mit der Eltern-ID statt data.id (#2670)", async () => {
+    const navigateCalls: NavTarget[] = [];
+    const dispatcher = makeDispatcher({
+      write: (async () => ({
+        isSuccess: true,
+        data: { id: "child", leaseId: "parent" },
+      })) as unknown as Dispatcher["write"],
+    });
+    const memoryNav = {
+      route: { screenId: "quick-add" },
+      navigate: (target: NavTarget) => {
+        if ("screenId" in target) navigateCalls.push(target);
+      },
+      replace: () => undefined,
+      hrefFor: (t: NavTarget) => ("screenId" in t ? `/${t.screenId}` : ""),
+      searchParams: {},
+      setSearchParams: () => undefined,
+    };
+    const editScreen: EntityEditScreenDefinition = {
+      id: "task-edit",
+      type: "entityEdit",
+      entity: "task",
+      layout: { sections: [{ title: "x", fields: ["title"] }] },
+    };
+    const actionScreen: ActionFormScreenDefinition = {
+      id: "quick-add",
+      type: "actionForm",
+      handler: "tasks:write:task:quick-add",
+      fields: { title: { type: "text", required: true } },
+      layout: { sections: [{ title: "x", fields: ["title"] }] },
+      redirect: { screen: "task-edit", idFrom: "leaseId" },
+    };
+
+    const { NavProvider } = await import("@cosmicdrift/kumiko-renderer");
+    render(
+      <NavProvider value={memoryNav}>
+        <DispatcherProvider dispatcher={dispatcher}>
+          <KumikoScreen
+            schema={{ ...schema, screens: [actionScreen, editScreen, listScreen] }}
+            qn="tasks:screen:quick-add"
+          />
+        </DispatcherProvider>
+      </NavProvider>,
+    );
+
+    const titleInput = screen.getByTestId("field-title").querySelector("input") as HTMLInputElement;
+    fireEvent.change(titleInput, { target: { value: "go" } });
+    await clickSubmitOnceEnabled();
+    await waitFor(() => expect(navigateCalls.length).toBe(1));
+    expect(navigateCalls[0]).toEqual({ screenId: "task-edit", entityId: "parent" });
+  });
+
+  test("actionForm mit blankem String-redirect: navigiert weiter mit data.id, nicht mit einem anderen Payload-Feld (#2670)", async () => {
+    const navigateCalls: NavTarget[] = [];
+    const dispatcher = makeDispatcher({
+      write: (async () => ({
+        isSuccess: true,
+        data: { id: "child", leaseId: "parent" },
+      })) as unknown as Dispatcher["write"],
+    });
+    const memoryNav = {
+      route: { screenId: "quick-add" },
+      navigate: (target: NavTarget) => {
+        if ("screenId" in target) navigateCalls.push(target);
+      },
+      replace: () => undefined,
+      hrefFor: (t: NavTarget) => ("screenId" in t ? `/${t.screenId}` : ""),
+      searchParams: {},
+      setSearchParams: () => undefined,
+    };
+    const editScreen: EntityEditScreenDefinition = {
+      id: "task-edit",
+      type: "entityEdit",
+      entity: "task",
+      layout: { sections: [{ title: "x", fields: ["title"] }] },
+    };
+    const actionScreen: ActionFormScreenDefinition = {
+      id: "quick-add",
+      type: "actionForm",
+      handler: "tasks:write:task:quick-add",
+      fields: { title: { type: "text", required: true } },
+      layout: { sections: [{ title: "x", fields: ["title"] }] },
+      redirect: "task-edit",
+    };
+
+    const { NavProvider } = await import("@cosmicdrift/kumiko-renderer");
+    render(
+      <NavProvider value={memoryNav}>
+        <DispatcherProvider dispatcher={dispatcher}>
+          <KumikoScreen
+            schema={{ ...schema, screens: [actionScreen, editScreen, listScreen] }}
+            qn="tasks:screen:quick-add"
+          />
+        </DispatcherProvider>
+      </NavProvider>,
+    );
+
+    const titleInput = screen.getByTestId("field-title").querySelector("input") as HTMLInputElement;
+    fireEvent.change(titleInput, { target: { value: "go" } });
+    await clickSubmitOnceEnabled();
+    await waitFor(() => expect(navigateCalls.length).toBe(1));
+    expect(navigateCalls[0]).toEqual({ screenId: "task-edit", entityId: "child" });
+  });
+
   // #2416: entityEdit-create passes the newly created record's id through to
   // the post-submit navigate as `entityId` (extractCreatedId); actionForm's
   // redirect did the same navigate but never extracted the id. The with-id

package/src/widgets/__tests__/infinity-list.test.tsx

@@ -54,7 +54,12 @@ globalThis.IntersectionObserver = class {
   disconnect(): void {}
 } as unknown as typeof IntersectionObserver;
 
-function fireIntersect(): void {
+// Waiting for the row alone races the observer-registering effect; each page
+// registers a fresh observer, so wait for one more than we last fired.
+let firedObservers = 0;
+async function fireIntersect(): Promise<void> {
+  await waitFor(() => expect(observers.length).toBeGreaterThan(firedObservers));
+  firedObservers = observers.length;
   observers[observers.length - 1]?.([{ isIntersecting: true }]);
 }
 
@@ -118,7 +123,7 @@ describe("InfinityList", () => {
     });
     renderWithDispatcher(list("inbox:query:message:list"), dispatcher);
     await waitFor(() => expect(screen.getByText("Erste")).toBeTruthy());
-    fireIntersect();
+    await fireIntersect();
     await waitFor(() => expect(screen.getByText("Zweite")).toBeTruthy());
     expect(screen.getByText("Erste")).toBeTruthy();
     expect(calls).toBe(2);
@@ -157,7 +162,7 @@ describe("InfinityList", () => {
     });
     renderWithDispatcher(list("inbox:query:message:list"), dispatcher);
     await waitFor(() => expect(screen.getByText("Zweite")).toBeTruthy());
-    fireIntersect();
+    await fireIntersect();
     await waitFor(() => expect(screen.getByText("Dritte")).toBeTruthy());
     expect(screen.getAllByText("Zweite")).toHaveLength(1);
   });
@@ -316,7 +321,7 @@ describe("InfinityList", () => {
       );
 
       await waitFor(() => expect(screen.getByText("Alt-1")).toBeTruthy());
-      fireIntersect();
+      await fireIntersect();
       await waitFor(() => expect(screen.getByText("Alt-2")).toBeTruthy());
       expect(calls.length).toBe(2);
 
@@ -434,7 +439,7 @@ describe("InfinityList", () => {
       );
 
       await waitFor(() => expect(screen.getByText("Erste")).toBeTruthy());
-      fireIntersect();
+      await fireIntersect();
       await waitFor(() => expect(screen.getByText("Zweite")).toBeTruthy());
       expect(calls.length).toBe(2);