@cosmicdrift/kumiko-renderer-web 0.220.0 → 0.220.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer-web",
-  "version": "0.220.0",
+  "version": "0.220.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.220.0",
-    "@cosmicdrift/kumiko-headless": "0.220.0",
-    "@cosmicdrift/kumiko-renderer": "0.220.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.220.1",
+    "@cosmicdrift/kumiko-headless": "0.220.1",
+    "@cosmicdrift/kumiko-renderer": "0.220.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.220.0"
+    "@cosmicdrift/kumiko-locale-de": "0.220.1"
   },
   "repository": {
     "type": "git",

package/src/primitives/index.tsx

@@ -673,8 +673,8 @@ function DefaultDataTable({
   // scrollen typischerweise mit dem Page-Container, nicht intern.
   // Sticky würde mit der Topbar konkurrieren.
   const hasTableActions = rowActions !== undefined && rowActions.length > 0;
-  const tableContent =
-    rows.length === 0 ? (
+  function tableInner(): ReactNode {
+    return rows.length === 0 ? (
       <div
         data-testid={testId !== undefined ? `${testId}-empty` : "render-list-empty"}
         className="flex flex-col items-center justify-center rounded-md border border-dashed p-12 text-sm text-muted-foreground gap-3"
@@ -777,6 +777,8 @@ function DefaultDataTable({
         </Table>
       </div>
     );
+  }
+  const tableContent = tableInner();
 
   // Pager wird IMMER unter der Tabelle gerendert (auch bei rows=[]),
   // damit der User bei einem Filter-Hit-of-Zero zurückblättern kann
@@ -1701,7 +1703,6 @@ function DefaultForm({
 export type FormScreenShellWidth = FormWidth;
 
 const formScreenShellWidth: Record<FormScreenShellWidth, string> = {
-  sm: "max-w-sm mx-auto",
   "3xl": "max-w-3xl mx-auto",
   "4xl": "max-w-4xl mx-auto",
   full: "max-w-full",

package/src/widgets/infinity-list.tsx

@@ -20,12 +20,6 @@ export type InfinityListProps<TData = unknown, TRow = Readonly<Record<string, un
   /** Must derive from row content (e.g. `row.id`), not from `index` — a
    *  live refresh reorders rows (new/changed rows move to the front). */
   readonly rowId: (row: TRow, index: number) => string;
-  /** Pass this when `rowId` depends on list position — the live-merge path
-   *  needs a position-independent identity. Without it, an index-based
-   *  `rowId` (legitimate when live-merge isn't used) can collide between
-   *  the freshly-fetched first page's local indices and the accumulated
-   *  list's indices, silently dropping an unrelated mid-list row on merge. */
-  readonly rowKey?: (row: TRow) => string;
   readonly renderRow: (row: TRow) => ReactNode;
   readonly emptyState?: ReactNode;
   readonly className?: string;
@@ -55,7 +49,6 @@ export function InfinityList<TData = unknown, TRow = Readonly<Record<string, unk
   rows,
   nextCursor,
   rowId,
-  rowKey,
   renderRow,
   emptyState,
   className,
@@ -80,16 +73,6 @@ export function InfinityList<TData = unknown, TRow = Readonly<Record<string, unk
   nextCursorRef.current = nextCursor;
   const rowIdRef = useRef(rowId);
   rowIdRef.current = rowId;
-  const rowKeyRef = useRef(rowKey);
-  rowKeyRef.current = rowKey;
-  // Position-independent when rowKey is supplied, otherwise falls back to
-  // the (possibly position-dependent) rowId — same identity function used
-  // for both sides of the live-merge diff below so a position-based rowId
-  // can no longer collide across the fresh page's local indices and the
-  // accumulated list's indices (fw#1829).
-  const identifyRef = useRef((row: TRow, index: number): string =>
-    rowKeyRef.current !== undefined ? rowKeyRef.current(row) : rowIdRef.current(row, index),
-  );
 
   // Discards a response whose request was superseded by a newer one before
   // it resolved (e.g. two searches fired in quick succession) — without
@@ -121,9 +104,8 @@ export function InfinityList<TData = unknown, TRow = Readonly<Record<string, unk
       if (cursor === null) {
         // Same identity as refreshFirstPage's freshIds/staleRows diff below
         // — this ref is what that diff's previousFirstPageIds reads, so it
-        // must agree on rowKey vs rowId or the two id spaces never overlap.
         firstPageIdsRef.current = new Set(
-          nextRows.map((row, index) => identifyRef.current(row, index)),
+          nextRows.map((row, index) => rowIdRef.current(row, index)),
         );
       }
       setState((prev) => {
@@ -174,7 +156,7 @@ export function InfinityList<TData = unknown, TRow = Readonly<Record<string, unk
     // skip: background live refresh failed, keep showing the current rows
     if (!res.isSuccess) return;
     const freshRows = rowsRef.current(res.data);
-    const freshIds = new Set(freshRows.map((row, index) => identifyRef.current(row, index)));
+    const freshIds = new Set(freshRows.map((row, index) => rowIdRef.current(row, index)));
     // A row absent from freshIds is stale for one of two reasons: it was
     // dropped from page 1 (deleted, or filtered out elsewhere) and must be
     // pruned, or it belongs to an already-accumulated later page and must
@@ -194,7 +176,7 @@ export function InfinityList<TData = unknown, TRow = Readonly<Record<string, unk
         return { kind: "ready", rows: freshRows, cursor: nextCursorRef.current(res.data) };
       }
       const staleRows = prev.rows.filter((row, index) => {
-        const id = identifyRef.current(row, index);
+        const id = rowIdRef.current(row, index);
         return !freshIds.has(id) && !previousFirstPageIds.has(id);
       });
       return { kind: "ready", rows: [...freshRows, ...staleRows], cursor: prev.cursor };