@cosmicdrift/kumiko-framework 0.232.0 → 0.234.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-framework",
-  "version": "0.232.0",
+  "version": "0.234.0",
   "description": "Framework core — engine, pipeline, API, DB, and every other bit that makes Kumiko go.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -198,7 +198,7 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-types": "0.232.0",
+    "@cosmicdrift/kumiko-types": "0.234.0",
     "bullmq": "^5.76.7",
     "bun-types": "^1.3.13",
     "hono": "^4.13.1",
@@ -214,7 +214,7 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.232.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.234.0",
     "bun-types": "^1.3.13",
     "pino-pretty": "^13.1.3"
   },

package/src/engine/__tests__/factories-number-unit.test.ts

@@ -0,0 +1,23 @@
+// createNumberField({ unit }) passthrough — pins that NumberFieldDef.unit
+// survives the factory's Partial<Omit<...>> spread, static string and
+// sibling-field-reference form alike.
+
+import { describe, expect, test } from "bun:test";
+import { createNumberField } from "../factories";
+
+describe("createNumberField — unit passthrough", () => {
+  test("static unit string is preserved", () => {
+    const f = createNumberField({ unit: "km" });
+    expect(f.unit).toBe("km");
+  });
+
+  test("sibling-field unit reference is preserved", () => {
+    const f = createNumberField({ unit: { field: "mileageUnit" } });
+    expect(f.unit).toEqual({ field: "mileageUnit" });
+  });
+
+  test("omitted unit stays undefined", () => {
+    const f = createNumberField();
+    expect(f.unit).toBeUndefined();
+  });
+});

package/src/engine/boot-validator/screens.ts

@@ -94,7 +94,7 @@ function validateNoWidgetRequiredField(
 function validateRowActionNavigateParams(
   featureName: string,
   screenId: string,
-  screenType: "entityList" | "projectionList" | "projectionDetail",
+  screenType: "entityList" | "projectionList" | "projectionDetail" | "entityEdit",
   screenEntity: string | undefined,
   action: RowAction,
   target: { readonly featureName: string; readonly screen: ScreenDefinition } | undefined,
@@ -396,7 +396,7 @@ type RowActionNavigateRuntime = RowActionNavigateBase & {
 function resolveRowActionNavigateTarget(
   featureName: string,
   screenId: string,
-  screenType: "entityList" | "projectionList" | "projectionDetail",
+  screenType: "entityList" | "projectionList" | "projectionDetail" | "entityEdit",
   actionLabel: "rowAction" | "action",
   action: RowActionNavigateRuntime,
   allScreenQns: ReadonlySet<string>,
@@ -1307,6 +1307,50 @@ export function validateScreens(
         }
       }
       validateWizardLayout(feature.name, screenId, "entityEdit", screen.layout, featureMap);
+      // Header actions reuse RowAction — same shape and same checks as
+      // projectionDetail's `actions` above (the loaded record stands in
+      // for the row). `rowClick` is rejected outright — entityEdit has no
+      // row to click either.
+      if (screen.actions !== undefined) {
+        for (const action of screen.actions) {
+          if (action.kind === "navigate" && action.rowClick === true) {
+            throw new Error(
+              `[Feature ${feature.name}] Screen "${qualifyEntityName(feature.name, "screen", screenId)}" ` +
+                `(entityEdit) action "${action.id}" sets rowClick: true — there is no row to click ` +
+                `on an edit screen. Remove rowClick.`,
+            );
+          }
+          if (action.kind === "navigate") {
+            const target = resolveRowActionNavigateTarget(
+              feature.name,
+              screenId,
+              "entityEdit",
+              "action",
+              action,
+              allScreenQns,
+              navTargetShortIds,
+              screensByShortId,
+              detailForScreens,
+            );
+            validateRowActionNavigateParams(
+              feature.name,
+              screenId,
+              "entityEdit",
+              screen.entity,
+              action,
+              target,
+            );
+          } else {
+            if (!allWriteHandlerQns.has(action.handler)) {
+              throw new Error(
+                `[Feature ${feature.name}] Screen "${screenId}" (entityEdit) action "${action.id}" ` +
+                  `handler "${action.handler}" is not a registered write-handler. Check the QN spelling ` +
+                  `(expected "<feature>:write:<short>") and that the handler is declared via r.writeHandler(...).`,
+              );
+            }
+          }
+        }
+      }
       if (screen.redirect !== undefined) {
         // Same rule as actionForm's redirect: short screen-ID (same-feature)
         // or a fully-qualified cross-feature QN (#1946).

package/src/ui-types/index.ts

@@ -52,7 +52,7 @@ export type {
   TextFieldDef,
 } from "../engine/types/fields";
 export type { AccessRule } from "../engine/types/handlers";
-export type { NavDefinition, NavIconKey } from "../engine/types/nav";
+export type { IconKey, NavDefinition, NavIconKey } from "../engine/types/nav";
 export type {
   ActionFormScreenDefinition,
   ConfigEditScreenDefinition,