@cosmicdrift/kumiko-framework 0.14.0 → 0.15.0

package/src/db/__tests__/table-builder-indexes.test.ts

@@ -3,11 +3,10 @@
 // Vorher mussten Apps für unique-indices über mehrere Spalten daneben eine
 // hand-written pgTable-Definition halten — Single-Source-of-Truth gebrochen,
 // Schema-Drift programmiert. Mit entity.indexes pflegen Author die
-// Constraint deklarativ in der EntityDefinition; buildDrizzleTable rendert
+// Constraint deklarativ in der EntityDefinition; buildEntityTable rendert
 // sie via uniqueIndex/index.
 
-import { getTableConfig } from "drizzle-orm/pg-core";
-import { describe, expect, test } from "vitest";
+import { describe, expect, test } from "bun:test";
 import {
   createBooleanField,
   createEntity,
@@ -15,9 +14,29 @@ import {
   defineFeature,
   validateBoot,
 } from "../../engine";
-import { buildDrizzleTable } from "../table-builder";
+import { buildEntityTable } from "../table-builder";
 
-describe("buildDrizzleTable — entity.indexes", () => {
+// Native dialect equivalent of drizzle's getTableConfig: reads the
+// EntityTableMeta-shape exposed on every SchemaTable.
+// biome-ignore lint/suspicious/noExplicitAny: DDL index test helper
+function getTableConfig(table: any): {
+  indexes: Array<{ config: { name: string; unique: boolean; columns: Array<{ name: string }> } }>;
+} {
+  const meta = table as unknown as {
+    indexes: ReadonlyArray<{ name: string; columns: readonly string[]; unique?: boolean }>;
+  };
+  return {
+    indexes: meta.indexes.map((idx) => ({
+      config: {
+        name: idx.name,
+        unique: idx.unique === true,
+        columns: idx.columns.map((c) => ({ name: c })),
+      },
+    })),
+  };
+}
+
+describe("buildEntityTable — entity.indexes", () => {
   test("composite unique-index landet als unique=true in Drizzle table-config", () => {
     const entity = createEntity({
       fields: {
@@ -26,7 +45,7 @@ describe("buildDrizzleTable — entity.indexes", () => {
       },
       indexes: [{ unique: true, columns: ["key", "tenantId", "userId"] }],
     });
-    const tbl = buildDrizzleTable("config-value", entity);
+    const tbl = buildEntityTable("config-value", entity);
     const { indexes } = getTableConfig(tbl);
     const composite = indexes.find(
       (i) => i.config.name === "read_config_values_key_tenant_id_user_id_unique",
@@ -48,7 +67,7 @@ describe("buildDrizzleTable — entity.indexes", () => {
       },
       indexes: [{ columns: ["startedAt", "endedAt"] }],
     });
-    const tbl = buildDrizzleTable("session", entity);
+    const tbl = buildEntityTable("session", entity);
     const { indexes } = getTableConfig(tbl);
     const composite = indexes.find(
       (i) => i.config.name === "read_sessions_started_at_ended_at_idx",
@@ -62,7 +81,7 @@ describe("buildDrizzleTable — entity.indexes", () => {
       fields: { slug: createTextField({ required: true }) },
       indexes: [{ unique: true, columns: ["slug"], name: "my_custom_idx" }],
     });
-    const tbl = buildDrizzleTable("page", entity);
+    const tbl = buildEntityTable("page", entity);
     const { indexes } = getTableConfig(tbl);
     const idx = indexes.find((i) => i.config.name === "my_custom_idx");
     expect(idx).toBeDefined();
@@ -75,11 +94,11 @@ describe("buildDrizzleTable — entity.indexes", () => {
         title: createTextField({}),
       },
     });
-    expect(() => buildDrizzleTable("widget", entity)).not.toThrow();
+    expect(() => buildEntityTable("widget", entity)).not.toThrow();
   });
 
   test("Spalten die keine DB-Spalte haben (multi-files) werden via Boot-Validator gecatched", () => {
-    // buildDrizzleTable selbst überspringt fehlende Columns silently —
+    // buildEntityTable selbst überspringt fehlende Columns silently —
     // der Boot-Validator wirft.
     const entity = createEntity({
       fields: {
@@ -88,7 +107,7 @@ describe("buildDrizzleTable — entity.indexes", () => {
       indexes: [{ columns: ["attachments"] }],
     });
     // No throw at build time.
-    expect(() => buildDrizzleTable("widget", entity)).not.toThrow();
+    expect(() => buildEntityTable("widget", entity)).not.toThrow();
     // But validateBoot does.
     const feature = defineFeature("widgetFeature", (r) => {
       r.entity("widget", entity);

package/src/db/__tests__/table-builder-required.test.ts

@@ -7,7 +7,7 @@
 // generierte Variante gedriftet sind. Hier: required muss durchschlagen,
 // für jeden Field-Type.
 
-import { describe, expect, test } from "vitest";
+import { describe, expect, test } from "bun:test";
 import {
   createBooleanField,
   createDateField,
@@ -24,25 +24,23 @@ import {
   createTzField,
 } from "../../engine";
 import type { ReferenceFieldDef } from "../../engine/types";
-import { buildDrizzleTable } from "../table-builder";
+import { buildEntityTable } from "../table-builder";
 
 // Reference-fields haben keinen Factory-Helper — direkt-typed inline.
 function refField(args: Omit<ReferenceFieldDef, "type">): ReferenceFieldDef {
   return { type: "reference", ...args };
 }
 
-function colByName(table: ReturnType<typeof buildDrizzleTable>, dbName: string) {
-  // Drizzle's PgTable proxies columns through both JS-prop-name and the
-  // serialized "name" attribute. We need the underlying column.config to
-  // read .notNull, which lives at runtime under the column instance. Simplest
-  // robust path: iterate Object.values, match by name.
-  for (const col of Object.values(table) as Array<{ name?: string; notNull?: boolean }>) {
-    if (col && typeof col === "object" && col.name === dbName) return col;
+function colByName(table: unknown, dbName: string) {
+  const cols = (table as { columns?: ReadonlyArray<{ name: string; notNull?: boolean }> }).columns;
+  if (!cols) throw new Error("Table has no columns metadata");
+  for (const c of cols) {
+    if (c.name === dbName) return { name: c.name, notNull: c.notNull };
   }
   throw new Error(`Column ${dbName} not found in table`);
 }
 
-describe("buildDrizzleTable — required: true → NOT NULL", () => {
+describe("buildEntityTable — required: true → NOT NULL", () => {
   test("text field — required true makes column NOT NULL", () => {
     const entity = createEntity({
       fields: {
@@ -50,7 +48,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         subtitle: createTextField({}),
       },
     });
-    const tbl = buildDrizzleTable("widget", entity);
+    const tbl = buildEntityTable("widget", entity);
     expect(colByName(tbl, "title").notNull).toBe(true);
     expect(colByName(tbl, "subtitle").notNull).toBe(false);
   });
@@ -62,7 +60,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         tag: createSelectField({ options: ["x"] as const }),
       },
     });
-    const tbl = buildDrizzleTable("widget", entity);
+    const tbl = buildEntityTable("widget", entity);
     expect(colByName(tbl, "status").notNull).toBe(true);
     expect(colByName(tbl, "tag").notNull).toBe(false);
   });
@@ -74,7 +72,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         optional: createNumberField({}),
       },
     });
-    const tbl = buildDrizzleTable("widget", entity);
+    const tbl = buildEntityTable("widget", entity);
     expect(colByName(tbl, "count").notNull).toBe(true);
     expect(colByName(tbl, "optional").notNull).toBe(false);
   });
@@ -86,7 +84,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         archived: createBooleanField({}),
       },
     });
-    const tbl = buildDrizzleTable("widget", entity);
+    const tbl = buildEntityTable("widget", entity);
     expect(colByName(tbl, "active").notNull).toBe(true);
     // Default ohne required ergibt notNull (default macht Spalte never-null)
     expect(colByName(tbl, "archived").notNull).toBe(true);
@@ -99,7 +97,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         assignee: refField({ entity: "user" }),
       },
     });
-    const tbl = buildDrizzleTable("task", entity);
+    const tbl = buildEntityTable("task", entity);
     expect(colByName(tbl, "owner").notNull).toBe(true);
     expect(colByName(tbl, "assignee").notNull).toBe(false);
   });
@@ -115,7 +113,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         zoneOpt: createTzField({}),
       },
     });
-    const tbl = buildDrizzleTable("event", entity);
+    const tbl = buildEntityTable("event", entity);
     expect(colByName(tbl, "born_on").notNull).toBe(true);
     expect(colByName(tbl, "born_on_opt").notNull).toBe(false);
     expect(colByName(tbl, "observed_at").notNull).toBe(true);
@@ -131,7 +129,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         dropoff: createLocatedTimestampField({}),
       },
     });
-    const tbl = buildDrizzleTable("transport", entity);
+    const tbl = buildEntityTable("transport", entity);
     expect(colByName(tbl, "pickup_utc").notNull).toBe(true);
     expect(colByName(tbl, "pickup_tz").notNull).toBe(true);
     expect(colByName(tbl, "dropoff_utc").notNull).toBe(false);
@@ -156,7 +154,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         }),
       },
     });
-    const tbl = buildDrizzleTable("acl", entity);
+    const tbl = buildEntityTable("acl", entity);
     expect(colByName(tbl, "roles_default").notNull).toBe(true);
     expect(colByName(tbl, "roles_optional").notNull).toBe(true);
     expect(colByName(tbl, "roles_required").notNull).toBe(true);
@@ -170,7 +168,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         metaOptional: createEmbeddedField({ note: { type: "text" } }, { required: false }),
       },
     });
-    const tbl = buildDrizzleTable("widget", entity);
+    const tbl = buildEntityTable("widget", entity);
     expect(colByName(tbl, "meta").notNull).toBe(true);
     expect(colByName(tbl, "meta_optional").notNull).toBe(true);
   });
@@ -183,7 +181,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         tagsOptional: refField({ entity: "tag", multiple: true, required: false }),
       },
     });
-    const tbl = buildDrizzleTable("post", entity);
+    const tbl = buildEntityTable("post", entity);
     expect(colByName(tbl, "tags").notNull).toBe(true);
     expect(colByName(tbl, "tags_optional").notNull).toBe(true);
   });
@@ -195,7 +193,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         discount: createMoneyField({}),
       },
     });
-    const tbl = buildDrizzleTable("listing", entity);
+    const tbl = buildEntityTable("listing", entity);
     expect(colByName(tbl, "price").notNull).toBe(true);
     expect(colByName(tbl, "price_currency").notNull).toBe(true);
     expect(colByName(tbl, "discount").notNull).toBe(false);
@@ -209,7 +207,7 @@ describe("buildDrizzleTable — required: true → NOT NULL", () => {
         cover: createImageField({}),
       },
     });
-    const tbl = buildDrizzleTable("profile", entity);
+    const tbl = buildEntityTable("profile", entity);
     expect(colByName(tbl, "avatar").notNull).toBe(true);
     expect(colByName(tbl, "cover").notNull).toBe(false);
   });