@cosmicdrift/kumiko-renderer 0.289.0 → 0.290.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer",
-  "version": "0.289.0",
+  "version": "0.290.0",
   "description": "Platform-agnostic React renderer for Kumiko screens. Contains the shared logic — primitives-contract, hooks, KumikoScreen, navigation & SSE abstractions — that any platform-specific renderer (web, native) composes. No DOM, no EventSource, no react-dom.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -15,9 +15,9 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-framework": "0.289.0",
-    "@cosmicdrift/kumiko-headless": "0.289.0",
-    "@cosmicdrift/kumiko-types": "0.289.0",
+    "@cosmicdrift/kumiko-framework": "0.290.0",
+    "@cosmicdrift/kumiko-headless": "0.290.0",
+    "@cosmicdrift/kumiko-types": "0.290.0",
     "react": "^19.2.6",
     "temporal-polyfill": "^0.3.2",
     "zod": "^4.4.3"
@@ -28,7 +28,7 @@
     "@types/react-dom": "^19.2.3",
     "jsdom": "^29.1.1",
     "react-dom": "^19.2.6",
-    "@cosmicdrift/kumiko-locale-de": "0.289.0"
+    "@cosmicdrift/kumiko-locale-de": "0.290.0"
   },
   "repository": {
     "type": "git",

package/src/__tests__/merge-search-params-into-initial.test.ts

@@ -1,5 +1,7 @@
 import { describe, expect, spyOn, test } from "bun:test";
+import type { EntityEditScreenDefinition } from "@cosmicdrift/kumiko-framework/ui-types";
 import { mergeSearchParamsIntoInitial } from "../app/kumiko-screen";
+import { layoutFieldNames } from "../app/layout-fields";
 
 type FieldDef = {
   type?: string;
@@ -188,6 +190,22 @@ describe("mergeSearchParamsIntoInitial — coercion (every field URL-prefillable
     expect(result["ownerId"]).toBe("");
   });
 
+  test("renderableFields from a groups-only layout: searchParam on a group field is kept", () => {
+    const screen: EntityEditScreenDefinition = {
+      id: "s",
+      type: "entityEdit",
+      entity: "e",
+      layout: { sections: [{ fields: [], groups: [{ title: "g", fields: ["status"] }] }] },
+    };
+    const fields: Record<string, FieldDef> = { status: { type: "text", default: "draft" } };
+    const result = mergeWithAllFieldsUrlPrefillable(
+      fields,
+      { status: "approved" },
+      layoutFieldNames(screen),
+    );
+    expect(result["status"]).toBe("approved");
+  });
+
   test("no renderableFields set given (undefined): behaves as before, all fields eligible", () => {
     const fields: Record<string, FieldDef> = { ownerId: { type: "text" } };
     const result = mergeWithAllFieldsUrlPrefillable(fields, { ownerId: "user-123" });

package/src/app/__tests__/form-schema.test.ts

@@ -41,6 +41,21 @@ describe("buildFormSchema", () => {
     }
   });
 
+  test("required field inside a groups-only section → presence issue", () => {
+    const entity = entityWith({ name: { type: "text", required: true } });
+    const screen: EntityEditScreenDefinition = {
+      id: "s",
+      type: "entityEdit",
+      entity: "e",
+      layout: { sections: [{ fields: [], groups: [{ title: "g", fields: ["name"] }] }] },
+    };
+
+    const result = buildFormSchema(entity, screen).safeParse({ name: "" });
+    expect(result.success).toBe(false);
+    if (result.success) return;
+    expect(result.error.issues[0]?.path).toEqual(["name"]);
+  });
+
   test("required field missing → issue carries the required-field i18nKey override", () => {
     const entity = entityWith({ name: { type: "text", required: true } });
     const screen = screenWith(["name"]);

package/src/app/layout-fields.ts

@@ -2,7 +2,11 @@ import type {
   EditFieldSpec,
   EntityEditScreenDefinition,
 } from "@cosmicdrift/kumiko-framework/ui-types";
-import { isFieldsEditSection, normalizeEditField } from "@cosmicdrift/kumiko-framework/ui-types";
+import {
+  isFieldsEditSection,
+  normalizeEditField,
+  sectionFieldSpecs,
+} from "@cosmicdrift/kumiko-framework/ui-types";
 
 // Normalized field specs actually rendered by the screen's layout, extension
 // (and relatedList) sections skipped. Both this and `layoutFieldNames` key
@@ -15,7 +19,7 @@ export function layoutEditFields(
   const specs: Exclude<EditFieldSpec, string>[] = [];
   for (const section of screen.layout.sections) {
     if (!isFieldsEditSection(section)) continue;
-    for (const spec of section.fields) {
+    for (const spec of sectionFieldSpecs(section)) {
       specs.push(normalizeEditField(spec));
     }
   }

package/src/app/projection-detail-shim.ts

@@ -22,7 +22,6 @@
 
 import type {
   EditFieldSpec,
-  EditFieldsSection,
   EditLayout,
   EntityDefinition,
   EntityEditScreenDefinition,
@@ -32,18 +31,9 @@ import {
   isFieldsEditSection,
   normalizeEditField,
   PROJECTION_DETAIL_ENTITY as PROJECTION_DETAIL_PSEUDO_ENTITY,
+  sectionFieldSpecs,
 } from "@cosmicdrift/kumiko-framework/ui-types";
 
-// The boot-validator treats `fields`/`groups` as mutually exclusive per
-// section, but this reads both anyway: an extra name the current section
-// shape can never carry is harmless, while a name computeEditViewModel does
-// reference and this shim omitted throws downstream ("references unknown
-// field ... on entity"). Union is strictly safer than picking one branch.
-function allFieldSpecs(section: EditFieldsSection): readonly EditFieldSpec[] {
-  const grouped = section.groups?.flatMap((group) => group.fields) ?? [];
-  return [...section.fields, ...grouped];
-}
-
 /** Minimale EntityDefinition aus den Layout-Feldern: jedes Feld ein Text-
  *  Feld — computeEditViewModel liest nur `fields[<f>].type`, Text reicht für
  *  eine reine Anzeige (kein Select/Number-spezifisches Rendering nötig). */
@@ -52,7 +42,7 @@ export function synthesizeProjectionDetailEntity(layout: EditLayout): EntityDefi
   for (const section of layout.sections) {
     // relatedList and extension sections carry no `fields` to synthesize.
     if (!isFieldsEditSection(section)) continue;
-    for (const spec of allFieldSpecs(section)) {
+    for (const spec of sectionFieldSpecs(section)) {
       fields[normalizeEditField(spec).field] = { type: "text" };
     }
   }

package/src/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.290.0",
+    "type": "fix",
+    "title": "Layout field readers see groups[].fields",
+    "detail": "layoutEditFields flattens section.groups, so required fields inside a groups-only section are presence-validated by buildFormSchema and survive the renderableFields gate for search-param prefills."
+  },
   {
     "version": "0.283.0",
     "type": "fix",