@cosmicdrift/kumiko-renderer-web 0.209.0 → 0.210.0

package/package.json

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

package/src/__tests__/projection-detail.test.tsx

@@ -176,12 +176,12 @@ describe("KumikoScreen / projectionDetail", () => {
     await waitFor(() => screen.getByTestId("kumiko-screen-record-missing"));
   });
 
-  // fw#2245: a projectionDetail has no write path — its footer is Cancel-only
-  // and defaults to shown (pre-fw#2245 behavior) so existing screens without
-  // an explicit opt-in keep working; `hideActions: true` turns it off without
-  // losing back-navigation (shell-breadcrumb.ts also resolves listScreenId
-  // for this screen type, independent of the footer — see shell-breadcrumb.test.ts).
-  test("shows Cancel by default when listScreenId is set", async () => {
+  // A projectionDetail has no write path, so there's nothing for a "Cancel"
+  // button to discard — RenderEdit's Cancel is never wired up for this
+  // screen type, regardless of `listScreenId`. Back-navigation goes through
+  // the breadcrumb instead (shell-breadcrumb.ts resolves `listScreenId`
+  // independently of the footer — see shell-breadcrumb.test.ts).
+  test("never shows Cancel, even when listScreenId is set", async () => {
     const withListScreen: ProjectionDetailScreenDefinition = {
       ...detailScreen,
       listScreenId: "session-list",
@@ -208,19 +208,30 @@ describe("KumikoScreen / projectionDetail", () => {
       </DispatcherProvider>,
     );
 
-    await waitFor(() => screen.getByTestId("render-edit-cancel"));
+    await waitFor(() => screen.getByTestId("render-edit-form"));
+    expect(screen.queryByTestId("render-edit-cancel")).toBeNull();
   });
 
-  test("hideActions:true hides the Cancel button", async () => {
-    const hiddenActionsScreen: ProjectionDetailScreenDefinition = {
+  // Regression: declared `actions` used to disappear along with Cancel
+  // whenever a screen hid its RenderEdit footer to get rid of the latter —
+  // both now render independently of each other.
+  test("declared actions render alongside no Cancel button", async () => {
+    const withActionsScreen: ProjectionDetailScreenDefinition = {
       ...detailScreen,
       listScreenId: "session-list",
-      hideActions: true,
+      actions: [
+        {
+          kind: "navigate",
+          id: "open-user",
+          label: "sessions.detail.action.openUser",
+          screen: "user-detail",
+        },
+      ],
     };
-    const hiddenActionsSchema: FeatureSchema = {
+    const withActionsSchema: FeatureSchema = {
       featureName: "sessions",
       entities: {},
-      screens: [hiddenActionsScreen],
+      screens: [withActionsScreen],
     };
     const dispatcher: Dispatcher = createMockDispatcher({
       query: (async () => ({
@@ -232,14 +243,14 @@ describe("KumikoScreen / projectionDetail", () => {
     render(
       <DispatcherProvider dispatcher={dispatcher}>
         <KumikoScreen
-          schema={hiddenActionsSchema}
+          schema={withActionsSchema}
           qn="sessions:screen:session-detail"
           entityId="sess-1"
         />
       </DispatcherProvider>,
     );
 
-    await waitFor(() => screen.getByTestId("render-edit-form"));
+    await waitFor(() => screen.getByTestId("render-edit-action-open-user"));
     expect(screen.queryByTestId("render-edit-cancel")).toBeNull();
   });
 

package/src/layout/__tests__/shell-breadcrumb.test.ts

@@ -76,10 +76,10 @@ describe("resolveDetailBreadcrumb", () => {
     ]);
   });
 
-  // fw#2245: projectionDetail has no entity for `listFromEntity` to pair
-  // against — listScreenId is its only back-navigation source, same role it
-  // plays for `custom` above. Matters once a screen hides its RenderEdit
-  // action-bar footer (hideActions) and has no entityList rowAction either.
+  // projectionDetail has no entity for `listFromEntity` to pair against —
+  // listScreenId is its only back-navigation source, same role it plays for
+  // `custom` above. Matters because RenderEdit's Cancel button is never
+  // wired up for this screen type and there's no entityList rowAction either.
   test("projectionDetail uses listScreenId", () => {
     const screens: ScreenDefinition[] = [
       {

package/src/layout/shell-breadcrumb.ts

@@ -35,9 +35,8 @@ export function resolveDetailBreadcrumb(
 
   // projectionDetail has no entity to pair with an entityList (unlike
   // entityEdit above) — `listScreenId` is its only back-navigation source.
-  // Needed so the breadcrumb still offers "back" when a screen hides its
-  // RenderEdit action-bar footer (fw#2245's `hideActions`) and isn't
-  // reachable via a listFromRowAction either.
+  // Needed because RenderEdit's Cancel button is never wired up for this
+  // screen type and isn't reachable via a listFromRowAction either.
   const listFromProjectionDetailParent =
     detail.type === "projectionDetail" && detail.listScreenId !== undefined
       ? screens.find((s) => lastSegment(s.id) === detail.listScreenId)