@cosmicdrift/kumiko-renderer 0.249.0 → 0.250.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-renderer",
-  "version": "0.249.0",
+  "version": "0.250.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,8 +15,8 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-framework": "0.249.0",
-    "@cosmicdrift/kumiko-headless": "0.249.0",
+    "@cosmicdrift/kumiko-framework": "0.250.0",
+    "@cosmicdrift/kumiko-headless": "0.250.0",
     "react": "^19.2.6",
     "temporal-polyfill": "^0.3.2",
     "zod": "^4.4.3"
@@ -27,7 +27,7 @@
     "@types/react-dom": "^19.2.3",
     "jsdom": "^29.1.1",
     "react-dom": "^19.2.6",
-    "@cosmicdrift/kumiko-locale-de": "0.249.0"
+    "@cosmicdrift/kumiko-locale-de": "0.250.0"
   },
   "repository": {
     "type": "git",

package/src/app/kumiko-screen.tsx

@@ -763,6 +763,7 @@ function EntityEditUpdateForm({
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
+            confirmRequired: false,
             ...(actionIcon !== undefined && { icon: actionIcon }),
             onPress: () => {
               if (id === "") return;
@@ -783,6 +784,7 @@ function EntityEditUpdateForm({
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
+            confirmRequired: false,
             ...(actionIcon !== undefined && { icon: actionIcon }),
             onPress: () => {
               nav.navigate({
@@ -802,6 +804,7 @@ function EntityEditUpdateForm({
           id: action.id,
           label: effectiveTranslate(action.label),
           ...(action.style !== undefined && { style: action.style }),
+          confirmRequired: false,
           ...(actionIcon !== undefined && { icon: actionIcon }),
           onPress: () => {
             const initialValues =
@@ -1612,6 +1615,7 @@ function EntityListBody({
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
+            confirmRequired: false,
             ...(actionIcon !== undefined && { icon: actionIcon }),
             onTrigger: (row: ListRowViewModel) => runNavigate(navigateAction, row),
             ...(actionVisible !== undefined && {
@@ -1627,6 +1631,7 @@ function EntityListBody({
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
+            confirmRequired: false,
             ...(actionIcon !== undefined && { icon: actionIcon }),
             onTrigger: (row: ListRowViewModel) => {
               const initialValues =
@@ -1709,6 +1714,7 @@ function EntityListBody({
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
+            confirmRequired: false,
             ...(actionIcon !== undefined && { icon: actionIcon }),
             onTrigger: () => nav.navigate({ screenId: action.screen }),
           };
@@ -1718,6 +1724,7 @@ function EntityListBody({
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
+            confirmRequired: false,
             ...(actionIcon !== undefined && { icon: actionIcon }),
             onTrigger: () => openDrawer(action),
           };
@@ -2017,6 +2024,7 @@ function ProjectionListBody({
           id: action.id,
           label: effectiveTranslate(action.label),
           ...(action.style !== undefined && { style: action.style }),
+          confirmRequired: false,
           ...(actionIcon !== undefined && { icon: actionIcon }),
           onTrigger: () => nav.navigate({ screenId: target }),
         });
@@ -2027,6 +2035,7 @@ function ProjectionListBody({
           id: action.id,
           label: effectiveTranslate(action.label),
           ...(action.style !== undefined && { style: action.style }),
+          confirmRequired: false,
           ...(actionIcon !== undefined && { icon: actionIcon }),
           onTrigger: () => openDrawer(action),
         });
@@ -2315,6 +2324,7 @@ function ProjectionDetailBody({
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
+            confirmRequired: false,
             ...(actionIcon !== undefined && { icon: actionIcon }),
             onPress: () => {
               if (id === "") return;
@@ -2348,6 +2358,7 @@ function ProjectionDetailBody({
             id: action.id,
             label: effectiveTranslate(action.label),
             ...(action.style !== undefined && { style: action.style }),
+            confirmRequired: false,
             ...(actionIcon !== undefined && { icon: actionIcon }),
             onPress: () => {
               nav.navigate({
@@ -2367,6 +2378,7 @@ function ProjectionDetailBody({
           id: action.id,
           label: effectiveTranslate(action.label),
           ...(action.style !== undefined && { style: action.style }),
+          confirmRequired: false,
           ...(actionIcon !== undefined && { icon: actionIcon }),
           onPress: () => {
             const initialValues =
@@ -2728,6 +2740,7 @@ function ActionFormBody({
       onSubmit={handleSubmitted}
       {...(handleCancel !== undefined && { onCancel: handleCancel })}
       {...(screen.submitLabel !== undefined && { submitLabel: screen.submitLabel })}
+      {...(screen.submitStyle !== undefined && { submitVariant: screen.submitStyle })}
       {...(translate !== undefined && { translate })}
     />
   );

package/src/app/row-actions.ts

@@ -156,6 +156,7 @@ function buildNavigateRowAction(
     id: action.id,
     label: translate(action.label),
     ...(action.style !== undefined && { style: action.style }),
+    confirmRequired: false,
     ...(actionIcon !== undefined && { icon: actionIcon }),
     onTrigger: (row: ListRowViewModel) => runProjectionRowNavigate(nav, action, row),
     ...(visible !== undefined && {
@@ -195,6 +196,7 @@ function buildDrawerRowAction(
     id: action.id,
     label: translate(action.label),
     ...(action.style !== undefined && { style: action.style }),
+    confirmRequired: false,
     ...(actionIcon !== undefined && { icon: actionIcon }),
     onTrigger: (row: ListRowViewModel) => {
       openDrawer(action, params !== undefined ? evalRowExtractor(params, row.values) : undefined);

package/src/components/__tests__/render-edit-action-button.test.tsx

@@ -137,6 +137,26 @@ describe("RenderEditActionButton", () => {
     await waitFor(() => expect(pressed).toBe(1));
   });
 
+  // fw#2752: schema-driven navigate/drawer actions set confirmRequired: false
+  // to opt a danger-styled action out of the forced dialog — the colour still
+  // marks it destructive, but the target form is itself the confirmation.
+  test("danger style with confirmRequired=false fires onPress directly, no dialog", async () => {
+    let pressed = 0;
+    renderAction({
+      id: "open-terminate-form",
+      label: "Terminate",
+      style: "danger",
+      confirmRequired: false,
+      onPress: async () => {
+        pressed += 1;
+      },
+    });
+
+    fireEvent.click(rtlScreen.getByTestId("render-edit-action-open-terminate-form"));
+    expect(rtlScreen.queryByTestId("render-edit-action-open-terminate-form-dialog")).toBeNull();
+    await waitFor(() => expect(pressed).toBe(1));
+  });
+
   test("onPress failure reports via onError", async () => {
     const errors: Array<string | null> = [];
     renderAction(

package/src/components/render-edit-action-button.tsx

@@ -39,8 +39,11 @@ export function RenderEditActionButton({
 
   const variant = action.style ?? "secondary";
   // Same rule as RowActionWriteHandler: "danger" forces a confirm even
-  // without an explicit confirm key.
-  const needsConfirm = action.confirm !== undefined || action.style === "danger";
+  // without an explicit confirm key — unless `confirmRequired` overrides it
+  // (schema-driven navigate/drawer actions, where the target form is itself
+  // the confirmation).
+  const needsConfirm =
+    action.confirm !== undefined || (action.confirmRequired ?? action.style === "danger");
   const showIconOnly = iconOnly && action.icon !== undefined;
 
   return (

package/src/components/render-edit-types.ts

@@ -76,6 +76,8 @@ export type RenderEditProps<TValues extends FormValues, TCtx = unknown> = {
    *  "Save" can be replaced by domain-specific strings ("Approve" /
    *  "Dispatch" / etc.). */
   readonly submitLabel?: string;
+  /** Visual style of the submit button (actionForm `submitStyle`). Default "primary". */
+  readonly submitVariant?: "primary" | "danger";
   /** Per-field extra content inline after the label (e.g.
    *  ConfigSourceBadge). Called with the field name, returns a ReactNode or
    *  undefined. */
@@ -168,6 +170,10 @@ export type RenderEditAction = {
   readonly style?: "primary" | "secondary" | "danger";
   readonly confirm?: string;
   readonly confirmLabel?: string;
+  /** Overrides the default "danger implies a confirm dialog" rule. Schema-driven
+   *  navigate/drawer actions set it to false: the colour marks the action as
+   *  destructive, but the target form is the confirmation. */
+  readonly confirmRequired?: boolean;
   /** Resolved icon (author `RowAction.icon` or the id-derived default) —
    *  drives both the icon-left-of-text render and the icon-only collapse
    *  rule (see `shouldRenderActionsIconOnly`). */

package/src/components/render-edit.tsx

@@ -212,6 +212,7 @@ export function RenderEdit<TValues extends FormValues, TCtx = unknown>(
     actions,
     onRelatedListDrawerAction,
     submitLabel,
+    submitVariant,
     labelAppendix,
     fieldAppendix,
     entityId: entityIdProp,
@@ -1032,7 +1033,7 @@ export function RenderEdit<TValues extends FormValues, TCtx = unknown>(
           type="submit"
           disabled={(snapshot.isUnchanged && !extensionDirty) || isSubmitting || disabled}
           loading={isSubmitting}
-          variant="primary"
+          variant={submitVariant ?? "primary"}
           icon="check"
           testId="render-edit-submit"
         >

package/src/components/render-list.tsx

@@ -130,6 +130,10 @@ export type ToolbarActionButton = {
   readonly style?: "primary" | "secondary" | "danger";
   readonly confirm?: string;
   readonly confirmLabel?: string;
+  /** Overrides the default "danger implies a confirm dialog" rule. Schema-driven
+   *  navigate/drawer actions set it to false: the colour marks the action as
+   *  destructive, but the target form is the confirmation. */
+  readonly confirmRequired?: boolean;
   readonly onTrigger: () => Promise<void> | void;
   /** Id-derived default icon (ACTION_ICON_BY_ID in kumiko-screen.tsx) —
    *  ToolbarAction has no author-declared icon field, unlike RowAction. */
@@ -492,7 +496,8 @@ function ToolbarActionView({
   };
 
   const variant: "primary" | "secondary" | "danger" = action.style ?? "secondary";
-  const needsConfirm = action.confirm !== undefined || action.style === "danger";
+  const needsConfirm =
+    action.confirm !== undefined || (action.confirmRequired ?? action.style === "danger");
   const showIconOnly = iconOnly && action.icon !== undefined;
 
   return (

package/src/primitives.tsx

@@ -488,8 +488,9 @@ export type DataTableRowAction = {
   readonly id: string;
   /** Translated Label. */
   readonly label: string;
-  /** Visual-Style — danger triggert in der Default-Primitive eine rote
-   *  Variante UND erzwingt einen Confirm-Dialog wenn keiner gesetzt ist. */
+  /** Visual style — "danger" renders the red variant in the default
+   *  primitive. Whether it also forces a confirm dialog is governed by
+   *  `confirmRequired` (falls back to `style === "danger"` when unset). */
   readonly style?: "primary" | "secondary" | "danger";
   /** Translated Confirm-Prompt (Description im Dialog) — wenn gesetzt,
    *  öffnet ein Modal vor der Ausführung. Bei style=danger ohne expliziten
@@ -498,6 +499,10 @@ export type DataTableRowAction = {
   /** Translated Confirm-Button-Label im Dialog. Default = `label`
    *  (Action-Label wird wiederverwendet). */
   readonly confirmLabel?: string;
+  /** Overrides the default "danger implies a confirm dialog" rule. Schema-driven
+   *  navigate/drawer actions set it to false: the colour marks the action as
+   *  destructive, but the target form is the confirmation. */
+  readonly confirmRequired?: boolean;
   /** Wird mit der ListRowViewModel der geklickten Row aufgerufen. Async
    *  erlaubt — der Renderer kann während der Promise-Resolution einen
    *  Loading-State auf dem Button zeigen. */