@cosmicdrift/kumiko-renderer 0.364.0 → 0.365.1

package/dist/app/form-schema.js

@@ -1,4 +1,4 @@
-import { evalFieldCondition, findUnavailableSelectOptions, NO_WIDGET_FIELD_TYPES, } from "@cosmicdrift/kumiko-framework/ui-types";
+import { evalFieldCondition, findUnavailableSelectOptions, isUnchangedValue, NO_WIDGET_FIELD_TYPES, } from "@cosmicdrift/kumiko-framework/ui-types";
 import { I18N_KEY_PARAM } from "@cosmicdrift/kumiko-headless";
 import * as z from "zod";
 import { layoutEditFields } from "./layout-fields.js";
@@ -34,8 +34,7 @@ function isRequiredValueMissing(spec, field, record, unchangedFrom) {
         return false;
     if (isPresent(record[spec.field]))
         return false;
-    return !(unchangedFrom !== undefined &&
-        JSON.stringify(record[spec.field]) === JSON.stringify(unchangedFrom[spec.field]));
+    return !(unchangedFrom !== undefined && isUnchangedValue(record[spec.field], unchangedFrom[spec.field]));
 }
 // Client-side presence validation for the auto-wired entityEdit path —
 // checks that every rendered required field HAS a value, not that the
@@ -79,17 +78,24 @@ export function buildFormSchema(entity, screen, verb, unchangedFrom) {
         // @cast-boundary form-values
         const record = values;
         const selectFieldsToCheck = {};
+        // The server validates the merged row, so a locked select still fails there; the
+        // client reports it on the condition field the user can actually change.
+        const lockedSelectFields = {};
         for (const spec of fields) {
             const field = entity.fields[spec.field];
             if (!field)
                 continue;
             // Hidden on create, read-only on update: the user cannot resolve a presence error.
-            if (excludedFromWrite.has(spec.field))
+            if (excludedFromWrite.has(spec.field)) {
+                lockedSelectFields[spec.field] = field;
                 continue;
+            }
             // Not operable by the user — a presence error would be unresolvable,
             // same reason as the NO_WIDGET_FIELD_TYPES check below.
-            if (spec.readOnly !== undefined && evalFieldCondition(spec.readOnly, record))
+            if (spec.readOnly !== undefined && evalFieldCondition(spec.readOnly, record)) {
+                lockedSelectFields[spec.field] = field;
                 continue;
+            }
             selectFieldsToCheck[spec.field] = field;
             if (!isRequiredValueMissing(spec, field, record, unchangedFrom))
                 continue;
@@ -103,11 +109,15 @@ export function buildFormSchema(entity, screen, verb, unchangedFrom) {
         }
         // Against `unchangedFrom` the helper still checks a stored value whose
         // discriminator changed — the server validates the merged row.
-        for (const issue of findUnavailableSelectOptions(selectFieldsToCheck, record, unchangedFrom)) {
+        for (const issue of findUnavailableSelectOptions({ ...lockedSelectFields, ...selectFieldsToCheck }, record, unchangedFrom)) {
+            const lockedDef = lockedSelectFields[issue.field];
+            const blamedField = lockedDef !== undefined && lockedDef.type === "select"
+                ? (lockedDef.conditionalOptions?.find((rule) => rule.options.includes(issue.value) && !evalFieldCondition(rule.when, record))?.when.field ?? issue.field)
+                : issue.field;
             ctx.addIssue({
                 code: "custom",
-                path: [issue.field],
-                message: `"${issue.field}" is not available for the current selection.`,
+                path: [blamedField],
+                message: `"${blamedField}" is not available for the current selection.`,
                 params: { [I18N_KEY_PARAM]: OPTION_NOT_AVAILABLE_I18N_KEY, allowed: issue.allowed },
             });
         }

package/dist/app/list-facets.js

@@ -217,16 +217,20 @@ export function clampDateRange(current, bound, value) {
 // starts at that day's first instant, "to the 14th" includes everything up to
 // its last. Going through the next day's start keeps both ends right across a
 // DST boundary, where the day is not 24h long.
-function dayBoundInstant(date, bound, timeZone) {
-    let plainDate;
+function parsePlainDate(date) {
     try {
-        plainDate = Temporal.PlainDate.from(date);
+        return Temporal.PlainDate.from(date);
     }
     catch {
-        // Half-typed or hand-crafted URL value — drop the bound rather than send
-        // something the handler's ISO-datetime schema would reject.
         return null;
     }
+}
+function dayBoundInstant(date, bound, timeZone) {
+    // Half-typed or hand-crafted URL value — drop the bound rather than send
+    // something the handler's ISO-datetime schema would reject.
+    const plainDate = parsePlainDate(date);
+    if (plainDate === null)
+        return null;
     const start = plainDate.toZonedDateTime(timeZone).toInstant();
     return bound === "from"
         ? start.toString()
@@ -242,7 +246,10 @@ function dayBoundInstant(date, bound, timeZone) {
 // A shared or hand-built URL can carry from > to; the handler's `from <= to`
 // refine would answer 422, so the bounds are put in order before conversion.
 function swapInvertedRange(range) {
-    const inverted = range.from !== "" && range.to !== "" && range.from > range.to;
+    // Comparing raw strings would swap a valid bound against garbage and flip the filter's meaning.
+    const from = parsePlainDate(range.from);
+    const to = parsePlainDate(range.to);
+    const inverted = from !== null && to !== null && Temporal.PlainDate.compare(from, to) > 0;
     return inverted ? { from: range.to, to: range.from } : range;
 }
 export function buildDateRangePayload(specs, urlFilters, timeZone) {

package/dist/app/nav.d.ts

@@ -18,7 +18,9 @@ export type ObjectTarget = {
 export type NavTarget = ScreenTarget | ObjectTarget;
 export type NavigateOptions = {
     /** Search params of the target route, set in the same history entry as
-     *  the path. Without them the target starts with an empty query. */
+     *  the path. Without them a different target starts with an empty query;
+     *  navigating to the current path keeps the current query (pass
+     *  `searchParams: {}` to clear it). */
     readonly searchParams?: Readonly<Record<string, string>>;
 };
 export type NavApi = {

package/dist/components/file-list-field.js

@@ -13,6 +13,8 @@ export function FileListField({ field, id, kind, onChange, hasError, }) {
     const { Input, Button, Grid, GridCell, Text } = usePrimitives();
     const t = useTranslation();
     const fileIds = readFileIds(field.value);
+    // The server accepts repeated ids, so the id alone is not a unique key.
+    const keys = fileIds.map((fileId, index) => `${fileId}#${fileIds.slice(0, index).filter((x) => x === fileId).length}`);
     const disabled = field.readOnly;
     const canAdd = !disabled && (field.maxCount === undefined || fileIds.length < field.maxCount);
     const inputProps = {
@@ -42,7 +44,7 @@ export function FileListField({ field, id, kind, onChange, hasError, }) {
                                 onChange(fileIds.filter((_, i) => i !== index));
                             else
                                 onChange(fileIds.map((existing, i) => (i === index ? next : existing)));
-                        } }), !disabled && (_jsxs(Grid, { columns: "auto", children: [index > 0 && (_jsx(Button, { variant: "ghost", size: "icon", icon: "chevron-up", ariaLabel: t("kumiko.field.file-list.move-up"), testId: `${id}-${index}-move-up`, onClick: () => move(index, index - 1) })), index < fileIds.length - 1 && (_jsx(Button, { variant: "ghost", size: "icon", icon: "chevron-down", ariaLabel: t("kumiko.field.file-list.move-down"), testId: `${id}-${index}-move-down`, onClick: () => move(index, index + 1) }))] }))] }, fileId))), canAdd && (_jsxs(GridCell, { children: [_jsx(Text, { variant: "muted", children: t("kumiko.field.file-list.add") }), _jsx(Input, { ...inputProps, id: `${id}-add`, value: null, onChange: (next) => {
+                        } }), !disabled && (_jsxs(Grid, { columns: "auto", children: [index > 0 && (_jsx(Button, { variant: "ghost", size: "icon", icon: "chevron-up", ariaLabel: t("kumiko.field.file-list.move-up"), testId: `${id}-${index}-move-up`, onClick: () => move(index, index - 1) })), index < fileIds.length - 1 && (_jsx(Button, { variant: "ghost", size: "icon", icon: "chevron-down", ariaLabel: t("kumiko.field.file-list.move-down"), testId: `${id}-${index}-move-down`, onClick: () => move(index, index + 1) }))] }))] }, keys[index]))), canAdd && (_jsxs(GridCell, { children: [_jsx(Text, { variant: "muted", children: t("kumiko.field.file-list.add") }), _jsx(Input, { ...inputProps, id: `${id}-add`, value: null, onChange: (next) => {
                             if (next !== null)
                                 onChange([...fileIds, next]);
                         } })] }, "add"))] }));

package/dist/components/render-list.js

@@ -31,14 +31,20 @@ export function RenderList(props) {
     // mount and can flush after a keystroke landing in the same commit,
     // resetting localQ and swallowing the input.
     const [syncedSearchValue, setSyncedSearchValue] = useState(searchValue);
-    // The parent may echo our own debounced emit back late (router/URL round
-    // trip); adopting that stale echo would erase keys typed in between.
-    const lastEmittedRef = useRef(undefined);
+    // The parent may echo our own debounced emits back late (router/URL round
+    // trip), possibly after a newer emit already went out; adopting such a stale
+    // echo would erase keys typed in between. All unechoed emits are kept, and a
+    // matching echo drops itself and every older one.
+    const pendingEmitsRef = useRef([]);
     if (searchValue !== syncedSearchValue) {
         setSyncedSearchValue(searchValue);
-        if (searchValue !== lastEmittedRef.current) {
+        const echoIndex = pendingEmitsRef.current.indexOf(searchValue ?? "");
+        if (echoIndex >= 0) {
+            pendingEmitsRef.current.splice(0, echoIndex + 1);
+        }
+        else {
             setLocalQ(searchValue ?? "");
-            lastEmittedRef.current = undefined;
+            pendingEmitsRef.current = [];
         }
     }
     useEffect(() => {
@@ -47,7 +53,7 @@ export function RenderList(props) {
         if (localQ === (searchValue ?? ""))
             return;
         const timer = setTimeout(() => {
-            lastEmittedRef.current = localQ;
+            pendingEmitsRef.current.push(localQ);
             onSearchChange(localQ);
         }, SEARCH_DEBOUNCE_MS);
         return () => clearTimeout(timer);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer",
-  "version": "0.364.0",
+  "version": "0.365.1",
   "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.364.0",
-    "@cosmicdrift/kumiko-headless": "0.364.0",
-    "@cosmicdrift/kumiko-types": "0.364.0",
+    "@cosmicdrift/kumiko-framework": "0.365.1",
+    "@cosmicdrift/kumiko-headless": "0.365.1",
+    "@cosmicdrift/kumiko-types": "0.365.1",
     "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.364.0"
+    "@cosmicdrift/kumiko-locale-de": "0.365.1"
   },
   "repository": {
     "type": "git",

package/src/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.365.0",
+    "type": "fix",
+    "title": "List search keeps the newest typed text; file lists tolerate duplicate ids; date ranges and locked selects validate correctly",
+    "detail": "A late echo of an older debounced search emit no longer overwrites newer typing. Duplicate file ids no longer collide as React keys. An inverted date range is swapped only when both bounds parse. Select rules on writeExcluded or readOnly fields are now checked by the form schema."
+  },
   {
     "version": "0.364.0",
     "type": "fix",