@cosmicdrift/kumiko-bundled-features 0.358.0 → 0.359.0

package/dist/custom-fields/handlers/update-tenant-field.write.js

@@ -4,13 +4,14 @@ import { fieldDefinitionAggregateId } from "../aggregate-id.js";
 import { DEFAULT_FIELD_DEFINITION_WRITE_ROLES } from "../constants.js";
 import { fieldDefinitionExecutor } from "../executor.js";
 import { buildFieldDefinitionColumns } from "../lib/field-definition-row.js";
-import { updateFieldPayloadSchema } from "../schemas.js";
+import { fieldTypeSchema, updateFieldPayloadSchema } from "../schemas.js";
 export function createUpdateTenantFieldHandler(opts = {}) {
     return {
         name: "update-tenant-field",
         schema: updateFieldPayloadSchema,
         access: { roles: opts.roles ?? DEFAULT_FIELD_DEFINITION_WRITE_ROLES },
-        description: "Replaces a tenant-owned custom-field definition with a complete new spec while refusing any change to the field's type; use it to relabel or re-constrain an existing field rather than to create or remove one.",
+        description: "Replaces a tenant-owned custom-field definition with a complete new spec; serializedField.type may be omitted, in which case the stored type applies, and a different type is refused. Use it to relabel or re-constrain an existing field rather than to create or remove one.",
+        excludedFields: ["type"],
         handler: async (event, ctx) => {
             const payload = event.payload; // @cast-boundary engine-payload
             const tenantId = event.user.tenantId;
@@ -22,16 +23,24 @@ export function createUpdateTenantFieldHandler(opts = {}) {
             if (!existing) {
                 return failNotFound("field-definition", aggregateId);
             }
-            if (existing["type"] !== payload.serializedField.type) {
+            const storedType = fieldTypeSchema.safeParse(existing["type"]);
+            if (!storedType.success) {
+                throw new Error(`update-tenant-field: stored definition ${aggregateId} has an invalid type`);
+            }
+            const requestedType = payload.serializedField.type;
+            if (requestedType !== undefined && requestedType !== storedType.data) {
                 return failUnprocessable("field_type_immutable", {
                     entityName: payload.entityName,
                     fieldKey: payload.fieldKey,
-                    currentType: existing["type"],
-                    requestedType: payload.serializedField.type,
+                    currentType: storedType.data,
+                    requestedType,
                 });
             }
             // entityName/fieldKey sind die Identität — nicht Teil der changes.
-            const { entityName: _entityName, fieldKey: _fieldKey, ...changes } = buildFieldDefinitionColumns(payload);
+            const { entityName: _entityName, fieldKey: _fieldKey, ...changes } = buildFieldDefinitionColumns({
+                ...payload,
+                serializedField: { ...payload.serializedField, type: storedType.data },
+            });
             return fieldDefinitionExecutor.update({ id: aggregateId, changes }, event.user, ctx.db, {
                 skipOptimisticLock: true,
             });

package/dist/custom-fields/schemas.d.ts

@@ -1,4 +1,13 @@
 import * as z from "zod";
+export declare const fieldTypeSchema: z.ZodEnum<{
+    boolean: "boolean";
+    date: "date";
+    embedded: "embedded";
+    enum: "enum";
+    money: "money";
+    number: "number";
+    text: "text";
+}>;
 export declare const customFieldAccessSchema: z.ZodOptional<z.ZodObject<{
     read: z.ZodOptional<z.ZodArray<z.ZodString>>;
     write: z.ZodOptional<z.ZodArray<z.ZodString>>;
@@ -8,15 +17,6 @@ export declare const defineFieldPayloadSchema: z.ZodObject<{
     entityName: z.ZodString;
     fieldKey: z.ZodString;
     serializedField: z.ZodObject<{
-        type: z.ZodEnum<{
-            boolean: "boolean";
-            date: "date";
-            embedded: "embedded";
-            enum: "enum";
-            money: "money";
-            number: "number";
-            text: "text";
-        }>;
         fieldAccess: z.ZodOptional<z.ZodObject<{
             read: z.ZodOptional<z.ZodArray<z.ZodString>>;
             write: z.ZodOptional<z.ZodArray<z.ZodString>>;
@@ -28,6 +28,15 @@ export declare const defineFieldPayloadSchema: z.ZodObject<{
                 delete: "delete";
             }>;
         }, z.core.$strip>>;
+        type: z.ZodEnum<{
+            boolean: "boolean";
+            date: "date";
+            embedded: "embedded";
+            enum: "enum";
+            money: "money";
+            number: "number";
+            text: "text";
+        }>;
     }, z.core.$loose>;
     required: z.ZodDefault<z.ZodBoolean>;
     searchable: z.ZodDefault<z.ZodBoolean>;
@@ -38,16 +47,11 @@ export type DefineFieldPayload = z.infer<typeof defineFieldPayloadSchema>;
 export declare const updateFieldPayloadSchema: z.ZodObject<{
     entityName: z.ZodString;
     fieldKey: z.ZodString;
+    required: z.ZodDefault<z.ZodBoolean>;
+    searchable: z.ZodDefault<z.ZodBoolean>;
+    displayOrder: z.ZodDefault<z.ZodNumber>;
+    label: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodString>>;
     serializedField: z.ZodObject<{
-        type: z.ZodEnum<{
-            boolean: "boolean";
-            date: "date";
-            embedded: "embedded";
-            enum: "enum";
-            money: "money";
-            number: "number";
-            text: "text";
-        }>;
         fieldAccess: z.ZodOptional<z.ZodObject<{
             read: z.ZodOptional<z.ZodArray<z.ZodString>>;
             write: z.ZodOptional<z.ZodArray<z.ZodString>>;
@@ -59,13 +63,18 @@ export declare const updateFieldPayloadSchema: z.ZodObject<{
                 delete: "delete";
             }>;
         }, z.core.$strip>>;
+        type: z.ZodOptional<z.ZodEnum<{
+            boolean: "boolean";
+            date: "date";
+            embedded: "embedded";
+            enum: "enum";
+            money: "money";
+            number: "number";
+            text: "text";
+        }>>;
     }, z.core.$loose>;
-    required: z.ZodDefault<z.ZodBoolean>;
-    searchable: z.ZodDefault<z.ZodBoolean>;
-    displayOrder: z.ZodDefault<z.ZodNumber>;
-    label: z.ZodOptional<z.ZodRecord<z.ZodString, z.ZodString>>;
 }, z.core.$strip>;
-export type UpdateFieldPayload = DefineFieldPayload;
+export type UpdateFieldPayload = z.infer<typeof updateFieldPayloadSchema>;
 export declare const deleteFieldPayloadSchema: z.ZodObject<{
     entityName: z.ZodString;
     fieldKey: z.ZodString;

package/dist/custom-fields/schemas.js

@@ -1,7 +1,7 @@
 import * as z from "zod";
 import { SUPPORTED_FIELD_TYPES } from "./constants.js";
 // Field-Type-Validator — pinnt valid type-Werte für fieldDefinition.
-const fieldTypeSchema = z.enum(SUPPORTED_FIELD_TYPES);
+export const fieldTypeSchema = z.enum(SUPPORTED_FIELD_TYPES);
 // Per-field access-control. When set, `set/clear-custom-field` handlers
 // require the calling user to hold at least one of the listed roles —
 // in addition to the handler-level RBAC. Absent or empty `write` means
@@ -33,9 +33,7 @@ export const customFieldAccessSchema = z
 // Everything else stays loose pending B2's per-type discriminated-union —
 // EXCEPT `sensitive`, which is explicitly rejected below: custom fields
 // carry supplemental business data only, PII support was removed (#972).
-const serializedFieldSchema = z
-    .looseObject({
-    type: fieldTypeSchema,
+const serializedFieldBaseShape = {
     fieldAccess: customFieldAccessSchema,
     // `retention` (T1.5d): per-field expiry. The retention-cron strips
     // values whose host-row `modified_at` is older than `keepFor`. Strategy
@@ -50,9 +48,19 @@ const serializedFieldSchema = z
         strategy: z.enum(["delete", "anonymize"]),
     })
         .optional(),
-})
+};
+const rejectSensitive = [
+    (v) => !("sensitive" in v),
+    "custom fields don't support PII — `sensitive` was removed (#972). Model personal data as a schema entity field with a pii/userOwned/tenantOwned annotation instead.",
+];
+const serializedFieldSchema = z
+    .looseObject({ type: fieldTypeSchema, ...serializedFieldBaseShape })
     .refine((v) => typeof v["type"] === "string", "serializedField must have a string `type`")
-    .refine((v) => !("sensitive" in v), "custom fields don't support PII — `sensitive` was removed (#972). Model personal data as a schema entity field with a pii/userOwned/tenantOwned annotation instead.");
+    .refine(...rejectSensitive);
+// Update may omit `type`: it is immutable, so the handler takes it from the stored definition.
+const updateSerializedFieldSchema = z
+    .looseObject({ type: fieldTypeSchema.optional(), ...serializedFieldBaseShape })
+    .refine(...rejectSensitive);
 // i18n-labels — `{ de: "...", en: "...", ... }`. Mindestens ein Eintrag.
 const labelSchema = z.record(z.string().min(2).max(8), z.string().min(1));
 // Payload für `define-tenant-field` und `define-system-field` write-handler.
@@ -69,11 +77,13 @@ export const defineFieldPayloadSchema = z.object({
     displayOrder: z.number().int().min(0).default(0),
     label: labelSchema.optional(),
 });
-// Payload für `update-tenant-field` — bewusst dieselbe Shape wie define
-// (Vollersatz-Semantik: der Edit-Screen schickt den kompletten neuen Stand;
-// entityName+fieldKey sind die Identität, type ist immutable und wird im
-// Handler gegen den Bestand geprüft).
-export const updateFieldPayloadSchema = defineFieldPayloadSchema;
+// Payload for `update-tenant-field`: full replacement like define (the edit
+// screen sends the complete new state; entityName+fieldKey are the identity),
+// except that `serializedField.type` may be omitted. type is immutable; a
+// differing value is refused in the handler.
+export const updateFieldPayloadSchema = defineFieldPayloadSchema.extend({
+    serializedField: updateSerializedFieldSchema,
+});
 // Payload für `delete-tenant-field` / `delete-system-field`.
 export const deleteFieldPayloadSchema = z.object({
     entityName: z.string().min(1).max(64),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-bundled-features",
-  "version": "0.358.0",
+  "version": "0.359.0",
   "description": "Built-in features — tenant, user, auth, delivery. The stuff you'd rewrite anyway, already typed.",
   "license": "BUSL-1.1",
   "author": "Marc Frost <marc@cosmicdriftgamestudio.com>",
@@ -516,12 +516,12 @@
     }
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.358.0",
-    "@cosmicdrift/kumiko-framework": "0.358.0",
-    "@cosmicdrift/kumiko-headless": "0.358.0",
-    "@cosmicdrift/kumiko-renderer": "0.358.0",
-    "@cosmicdrift/kumiko-renderer-web": "0.358.0",
-    "@cosmicdrift/kumiko-types": "0.358.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.359.0",
+    "@cosmicdrift/kumiko-framework": "0.359.0",
+    "@cosmicdrift/kumiko-headless": "0.359.0",
+    "@cosmicdrift/kumiko-renderer": "0.359.0",
+    "@cosmicdrift/kumiko-renderer-web": "0.359.0",
+    "@cosmicdrift/kumiko-types": "0.359.0",
     "@mollie/api-client": "^4.5.0",
     "@node-rs/argon2": "^2.0.2",
     "@types/mailparser": "^3.4.6",
@@ -1050,8 +1050,8 @@
   ],
   "devDependencies": {
     "@testing-library/user-event": "^14.6.1",
-    "@cosmicdrift/kumiko-locale-de": "0.358.0",
-    "@cosmicdrift/kumiko-locale-es": "0.358.0",
+    "@cosmicdrift/kumiko-locale-de": "0.359.0",
+    "@cosmicdrift/kumiko-locale-es": "0.359.0",
     "jsqr": "^1.4.0"
   }
 }

package/src/custom-fields/changes.json

@@ -1,4 +1,10 @@
 [
+  {
+    "version": "0.359.0",
+    "type": "improvement",
+    "title": "update-tenant-field keeps the stored type when serializedField.type is omitted",
+    "detail": "custom-fields: update-tenant-field accepts serializedField without type and keeps the stored type; a different type is still refused with 422 field_type_immutable. The handler declares excludedFields [\"type\"]."
+  },
   {
     "version": "0.338.0",
     "type": "fix",