@cosmicdrift/kumiko-bundled-features 0.243.2 → 0.243.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-bundled-features",
-  "version": "0.243.2",
+  "version": "0.243.4",
   "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>",
@@ -130,12 +130,12 @@
     "./workflow-runner": "./src/workflow-runner/index.ts"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.243.2",
-    "@cosmicdrift/kumiko-framework": "0.243.2",
-    "@cosmicdrift/kumiko-headless": "0.243.2",
-    "@cosmicdrift/kumiko-renderer": "0.243.2",
-    "@cosmicdrift/kumiko-renderer-web": "0.243.2",
-    "@cosmicdrift/kumiko-types": "0.243.2",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.243.4",
+    "@cosmicdrift/kumiko-framework": "0.243.4",
+    "@cosmicdrift/kumiko-headless": "0.243.4",
+    "@cosmicdrift/kumiko-renderer": "0.243.4",
+    "@cosmicdrift/kumiko-renderer-web": "0.243.4",
+    "@cosmicdrift/kumiko-types": "0.243.4",
     "@mollie/api-client": "^4.5.0",
     "@node-rs/argon2": "^2.0.2",
     "@types/mailparser": "^3.4.6",
@@ -164,7 +164,7 @@
   "devDependencies": {
     "@testing-library/user-event": "^14.6.1",
     "@types/qrcode": "^1.5.5",
-    "@cosmicdrift/kumiko-locale-de": "0.243.2",
-    "@cosmicdrift/kumiko-locale-es": "0.243.2"
+    "@cosmicdrift/kumiko-locale-de": "0.243.4",
+    "@cosmicdrift/kumiko-locale-es": "0.243.4"
   }
 }

package/src/channel-in-app/__tests__/store-table.test.ts

@@ -0,0 +1,25 @@
+// Regression test for #missing r.storeTable() registration: inAppMessagesTable
+// was declared as a plain pgTable but never registered via r.storeTable(), so
+// collectTableMetas() (the source migrations are generated from) never saw
+// "in_app_messages" — no migration was ever emitted and every app using
+// channel-in-app crashed at runtime with `relation "in_app_messages" does not
+// exist`. Mirrors framework/src/engine/__tests__/store-table.test.ts.
+
+import { describe, expect, test } from "bun:test";
+import { collectTableMetas } from "@cosmicdrift/kumiko-framework/db";
+import { createChannelInAppFeature } from "../feature";
+
+describe("channel-in-app — in_app_messages store table", () => {
+  test("registers in_app_messages via r.storeTable", () => {
+    const feature = createChannelInAppFeature();
+    expect(feature.storeTables).toHaveProperty("in_app_messages");
+    expect(feature.storeTables["in_app_messages"]?.meta.tableName).toBe("in_app_messages");
+  });
+
+  test("in_app_messages is part of collectTableMetas() so a migration is generated for it", () => {
+    const feature = createChannelInAppFeature();
+    const metas = collectTableMetas([feature]);
+    const tableNames = metas.map((m) => m.tableName);
+    expect(tableNames).toContain("in_app_messages");
+  });
+});

package/src/channel-in-app/feature.ts

@@ -4,6 +4,7 @@ import { markAllReadWrite } from "./handlers/mark-all-read.write";
 import { markReadWrite } from "./handlers/mark-read.write";
 import { unreadCountQuery } from "./handlers/unread-count.query";
 import { inAppChannel } from "./in-app-channel";
+import { inAppMessagesTableMeta } from "./tables";
 
 export function createChannelInAppFeature(): FeatureDefinition {
   return defineFeature("channel-in-app", (r) => {
@@ -17,6 +18,10 @@ export function createChannelInAppFeature(): FeatureDefinition {
     });
     r.requires("delivery");
 
+    r.storeTable(inAppMessagesTableMeta, {
+      reason: "read_side.in_app_messages",
+    });
+
     // Register as delivery channel via extension system
     r.useExtension("deliveryChannel", "inApp", {
       mode: inAppChannel.mode,

package/src/channel-in-app/tables.ts

@@ -1,5 +1,7 @@
+import { asEntityTableMeta } from "@cosmicdrift/kumiko-framework/bun-db";
 import {
   boolean,
+  type EntityTableMeta,
   instant,
   table as pgTable,
   serial,
@@ -20,3 +22,11 @@ export const inAppMessagesTable = pgTable("in_app_messages", {
   readAt: instant("read_at"),
   createdAt: instant("created_at").default(sql`now()`).notNull(),
 });
+
+// Derived from inAppMessagesTable (table() attaches the meta as a Symbol) so
+// the two can't drift, instead of hand-duplicating the column list.
+const derivedInAppMessagesTableMeta = asEntityTableMeta(inAppMessagesTable);
+if (!derivedInAppMessagesTableMeta) {
+  throw new Error("inAppMessagesTable: table carries no EntityTableMeta — built via table()?");
+}
+export const inAppMessagesTableMeta: EntityTableMeta = derivedInAppMessagesTableMeta;

package/src/user-data-rights/__tests__/inspector-screens.boot.test.ts

@@ -147,9 +147,12 @@ describe("privacy-center screen (fw#2312 projectionDetail conversion)", () => {
       handler: "user-data-rights:write:restrict-account",
       visible: { field: "status", ne: "restricted" },
     });
-    expect(byId["restrict"]?.kind === "navigate" ? undefined : byId["restrict"]?.confirm).toBe(
-      "userDataRights.privacyCenter.restriction.dialogDescription",
-    );
+    const restrictAction = byId["restrict"];
+    expect(
+      restrictAction?.kind === "navigate" || restrictAction?.kind === "drawer"
+        ? undefined
+        : restrictAction?.confirm,
+    ).toBe("userDataRights.privacyCenter.restriction.dialogDescription");
 
     expect(byId["request-deletion"]).toMatchObject({
       handler: "user-data-rights:write:request-deletion",