@cosmicdrift/kumiko-renderer 0.31.1 → 0.32.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer",
-  "version": "0.31.1",
+  "version": "0.32.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>",

package/src/app/kumiko-screen.tsx

@@ -32,6 +32,7 @@ import { useCustomScreenComponent } from "./custom-screens";
 import type { FeatureSchema } from "./feature-schema";
 import { useNav } from "./nav";
 import { lastSegment } from "./qn";
+import { dispatcherErrorText, WriteFailedError } from "./write-failed-error";
 
 // KumikoScreen picks up a ScreenDefinition from the schema by qn and
 // routes it to the right renderer based on `screen.type`. Command
@@ -658,11 +659,26 @@ function EntityListBody({
         // navigate-Variante braucht keinen Dispatcher; nav ist
         // immer da (Provider von createKumikoApp).
         if (action.kind === "navigate") {
+          // Deklarativer entityId-Default: zielt die Action auf einen
+          // entityEdit-Screen, ist row.id die entityId. Nötig weil die
+          // Function-Form (action.entityId) JSON-injizierte Schemas
+          // (window.__KUMIKO_SCHEMA__) nicht überlebt — silent gedroppt,
+          // siehe RowAction-Type-Header.
+          const targetIsEntityEdit = schema.screens.some(
+            (s) => s.type === "entityEdit" && lastSegment(s.id) === action.screen,
+          );
           return {
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
             onTrigger: (row: ListRowViewModel) => {
+              const explicit = action.entityId?.(row.values);
+              const fallback = targetIsEntityEdit ? String(row.values["id"] ?? "") : undefined;
+              const entityId = explicit ?? fallback;
+              nav.navigate({
+                screenId: action.screen,
+                ...(entityId !== undefined && entityId !== "" && { entityId }),
+              });
               const params = action.params?.(row.values);
               if (params !== undefined) {
                 // setSearchParams nimmt string|null. Komplexe Werte
@@ -673,9 +689,11 @@ function EntityListBody({
                 for (const [k, v] of Object.entries(params)) {
                   stringified[k] = v === null || v === undefined ? null : String(v);
                 }
+                // NACH navigate: pushState trägt keine Query — Params die
+                // vor dem Push gesetzt werden, kleben an der ALTEN URL und
+                // sind auf dem Ziel-Screen weg (actionForm-Prefill leer).
                 nav.setSearchParams(stringified);
               }
-              nav.navigate({ screenId: action.screen });
             },
             ...(action.visible !== undefined && {
               isVisible: (row: ListRowViewModel) => action.visible?.(row.values, undefined) ?? true,
@@ -699,7 +717,16 @@ function EntityListBody({
             const buildPayload = writeAction.payload;
             const payload =
               buildPayload !== undefined ? buildPayload(row.values) : { id: row.values["id"] };
-            await dispatcher.write(writeAction.handler, payload);
+            const result = await dispatcher.write(writeAction.handler, payload);
+            // write() wirft nicht — Failure-Result MUSS hier zum Error
+            // werden, sonst schließt der Confirm-Dialog kommentarlos und
+            // der User sieht "nichts passiert" (Prod-Bug 2026-06-07).
+            if (!result.isSuccess) {
+              throw new WriteFailedError(
+                result.error,
+                dispatcherErrorText(result.error, effectiveTranslate),
+              );
+            }
           },
           isVisible:
             writeAction.visible !== undefined
@@ -708,7 +735,7 @@ function EntityListBody({
         };
       })
       .filter((a: DataTableRowAction | null): a is DataTableRowAction => a !== null);
-  }, [screen.rowActions, effectiveTranslate, dispatcher, nav]);
+  }, [screen.rowActions, effectiveTranslate, dispatcher, nav, schema.screens]);
 
   // ToolbarActions: Schema → Resolved-Form (analog rowActions).
   // navigate-kind → useNav().navigate({ screenId }), writeHandler-kind
@@ -880,15 +907,15 @@ function ActionFormBody({
     },
     [nav, screen.redirect],
   );
-  // Cancel ist nur sinnvoll wenn ein Redirect-Target gesetzt ist —
-  // sonst hätte der Button nirgendwo hin zu navigieren. Bei Forms
-  // ohne redirect bleibt der User per Sidebar/Browser-Back im Flow,
-  // analog zu Settings-Pages.
+  // Cancel ist nur sinnvoll wenn ein Navigations-Ziel existiert —
+  // sonst hätte der Button nirgendwo hin zu navigieren. cancelTarget
+  // gewinnt über redirect; `false` schaltet den Button explizit ab
+  // (Single-Action-Screens, wo Cancel nur Submit-ohne-Senden wäre).
   const handleCancel = useMemo<(() => void) | undefined>(() => {
-    if (screen.redirect === undefined) return undefined;
-    const target = screen.redirect;
+    const target = screen.cancelTarget ?? screen.redirect;
+    if (target === undefined || target === false) return undefined;
     return () => nav.navigate({ screenId: target });
-  }, [nav, screen.redirect]);
+  }, [nav, screen.redirect, screen.cancelTarget]);
   return (
     <RenderEdit
       screen={synthScreen}

package/src/app/write-failed-error.ts

@@ -0,0 +1,27 @@
+import type { DispatcherError } from "@cosmicdrift/kumiko-headless";
+
+// dispatcher.write wirft bei Server-Fehlern NICHT — es returnt
+// { isSuccess: false, error }. Action-Wiring das das Result verwirft,
+// macht Fehler unsichtbar ("Klick tut nichts"-Prod-Bug 2026-06-07).
+// Diese Klasse trägt den strukturierten DispatcherError dahin, wo eine
+// UI ihn anzeigen kann (Toast mit docsUrl).
+export class WriteFailedError extends Error {
+  readonly dispatcherError: DispatcherError;
+
+  constructor(error: DispatcherError, message: string) {
+    super(message);
+    this.name = "WriteFailedError";
+    this.dispatcherError = error;
+  }
+}
+
+// i18nKey gewinnt wenn das Bundle ihn kennt (translate returnt den Key
+// unverändert wenn nicht — Renderer-Convention), sonst message, zuletzt code.
+export function dispatcherErrorText(
+  error: DispatcherError,
+  translate: (key: string) => string,
+): string {
+  const translated = translate(error.i18nKey);
+  if (translated !== error.i18nKey) return translated;
+  return error.message !== "" ? error.message : error.code;
+}

package/src/i18n-defaults.ts

@@ -41,6 +41,22 @@ export const kumikoDefaultTranslations: TranslationsByLocale = {
     "kumiko.dialog.cancel": "Abbrechen",
     "kumiko.dialog.close": "Schließen",
 
+    // Row-Actions — Fehler-Toast wenn ein Action-Write fehlschlägt.
+    "kumiko.rowAction.failed": "Aktion fehlgeschlagen",
+
+    // Config-Cascade — Source-Badges + Cascade-Panel (ConfigCascadeView).
+    "kumiko.config.source.user": "Mein Wert",
+    "kumiko.config.source.tenant": "Tenant",
+    "kumiko.config.source.system": "System",
+    "kumiko.config.source.appOverride": "App-Override",
+    "kumiko.config.source.computed": "Berechnet",
+    "kumiko.config.source.default": "Standard",
+    "kumiko.config.source.missing": "Fehlt",
+    "kumiko.config.cascade.preset": "Vorgabe",
+    "kumiko.config.cascade.noValue": "Kein Wert gesetzt",
+    "kumiko.config.cascade.activeMarker": "aktiv",
+    "kumiko.config.cascade.resetTo": "Überschreibung zurücksetzen ({scope})",
+
     // Form — Standard-Errors (App-Code kann eigene zod-Reasons nutzen,
     // diese sind die letzte Sicherheitsschicht).
     "kumiko.form.error.generic": "Etwas ist schiefgegangen.",
@@ -81,6 +97,20 @@ export const kumikoDefaultTranslations: TranslationsByLocale = {
     "kumiko.dialog.cancel": "Cancel",
     "kumiko.dialog.close": "Close",
 
+    "kumiko.rowAction.failed": "Action failed",
+
+    "kumiko.config.source.user": "My value",
+    "kumiko.config.source.tenant": "Tenant",
+    "kumiko.config.source.system": "System",
+    "kumiko.config.source.appOverride": "App override",
+    "kumiko.config.source.computed": "Computed",
+    "kumiko.config.source.default": "Default",
+    "kumiko.config.source.missing": "Missing",
+    "kumiko.config.cascade.preset": "Preset",
+    "kumiko.config.cascade.noValue": "No value set",
+    "kumiko.config.cascade.activeMarker": "active",
+    "kumiko.config.cascade.resetTo": "Reset override ({scope})",
+
     "kumiko.form.error.generic": "Something went wrong.",
     "kumiko.form.error.version-conflict":
       "Record was modified in the meantime. Reload and try again.",

package/src/index.ts

@@ -36,6 +36,7 @@ export { KumikoScreen, qualifyNavId, qualifyScreenId } from "./app/kumiko-screen
 export type { NavApi, NavProviderProps, NavRoute, NavTarget } from "./app/nav";
 export { formatPath, NavProvider, parsePath, useNav } from "./app/nav";
 export { lastSegment } from "./app/qn";
+export { dispatcherErrorText, WriteFailedError } from "./app/write-failed-error";
 export type { RenderEditProps } from "./components/render-edit";
 export { RenderEdit } from "./components/render-edit";
 export type { RenderFieldProps } from "./components/render-field";