@cosmicdrift/kumiko-framework 0.31.0 → 0.31.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-framework",
-  "version": "0.31.0",
+  "version": "0.31.1",
   "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>",

package/src/db/__tests__/table-builder-meta-lockstep.test.ts

@@ -0,0 +1,62 @@
+import { describe, expect, test } from "bun:test";
+import { asEntityTableMeta } from "../../bun-db/query";
+import { createEntity } from "../../engine/factories";
+import type { ColumnMeta, IndexMeta } from "../entity-table-meta";
+import { buildEntityTableMeta } from "../entity-table-meta";
+import { buildEntityTable } from "../table-builder";
+
+// Lock-step-Guard: buildEntityTable (Runtime-/Test-Stack-Pfad, Meta am
+// KUMIKO_META_SYMBOL) und buildEntityTableMeta (Migrations-Pfad) müssen
+// für dieselbe EntityDefinition identische Spalten + Indexes produzieren.
+// Drift hier = Migration und Prod-Tabelle (bzw. collectTableMetas-Output)
+// gehen auseinander — gefunden als #255-Follow-up: select/number/bigInt
+// verloren ihre deklarierten defaults auf dem Builder-Pfad.
+
+const entityWithDefaults = createEntity({
+  table: "read_lockstep_probe",
+  fields: {
+    title: { type: "text", required: true, default: "untitled" },
+    active: { type: "boolean", default: true },
+    status: { type: "select", options: ["open", "done"], required: true, default: "open" },
+    tags: { type: "multiSelect", options: ["a", "b"] },
+    attempt: { type: "number", required: true, default: 1 },
+    bytes: { type: "bigInt", default: 0 },
+    price: { type: "money" },
+    meta: { type: "embedded", fields: {} },
+    startedAt: { type: "timestamp", required: true },
+  },
+});
+
+function byName<T extends { name: string }>(items: readonly T[]): readonly T[] {
+  return [...items].sort((a, b) => a.name.localeCompare(b.name));
+}
+
+describe("buildEntityTable ↔ buildEntityTableMeta lock-step", () => {
+  const fromBuilder = asEntityTableMeta(buildEntityTable("lockstepProbe", entityWithDefaults));
+  const fromMeta = buildEntityTableMeta("lockstepProbe", entityWithDefaults);
+
+  test("builder table carries an EntityTableMeta", () => {
+    expect(fromBuilder).toBeDefined();
+  });
+
+  test("identical columns (incl. declared defaults)", () => {
+    expect(byName<ColumnMeta>(fromBuilder?.columns ?? [])).toEqual(
+      byName<ColumnMeta>(fromMeta.columns),
+    );
+  });
+
+  test("identical indexes", () => {
+    expect(byName<IndexMeta>(fromBuilder?.indexes ?? [])).toEqual(
+      byName<IndexMeta>(fromMeta.indexes),
+    );
+  });
+
+  test("declared defaults survive the builder path", () => {
+    const cols = new Map((fromBuilder?.columns ?? []).map((c) => [c.name, c]));
+    expect(cols.get("status")?.defaultSql).toBe("'open'");
+    expect(cols.get("attempt")?.defaultSql).toBe("1");
+    expect(cols.get("bytes")?.defaultSql).toBe("0");
+    expect(cols.get("title")?.defaultSql).toBe("'untitled'");
+    expect(cols.get("active")?.defaultSql).toBe("true");
+  });
+});

package/src/db/dialect.ts

@@ -272,8 +272,11 @@ export function instant(
 }
 
 // moneyAmount kept as a customType-style API but produces a bigint column.
+// bigintJsMode "bigint" — money cents must round-trip as JS bigint (lock-step
+// with entity-table-meta's money rendering; without it bun-db reads the
+// column as number and loses precision past 2^53).
 export const moneyAmount = (name: string): ColumnBuilder<number> =>
-  buildColumn(name, "bigint") as ColumnBuilder<number>;
+  buildColumn(name, "bigint", { bigintJsMode: "bigint" }) as ColumnBuilder<number>;
 
 // ---- Index + primaryKey helpers ----
 

package/src/db/table-builder.ts

@@ -80,7 +80,12 @@ function fieldToColumns(
               : boolean(snakeName),
       };
     case "select": {
-      const col = text(snakeName);
+      // default() durchreichen — entity-table-meta rendert deklarierte
+      // defaults, dieser Builder MUSS lock-step bleiben (sonst trägt das
+      // Meta am buildEntityTable-Objekt eine andere Spalte als die
+      // Migration, #255-Follow-up-Befund).
+      const base = text(snakeName);
+      const col = field.default !== undefined ? base.default(field.default) : base;
       return { [name]: field.required ? col.notNull() : col };
     }
     case "multiSelect":
@@ -95,7 +100,8 @@ function fieldToColumns(
       // multi-select.
       return { [name]: jsonb(snakeName).default([]).notNull() };
     case "number": {
-      const col = integer(snakeName);
+      const base = integer(snakeName);
+      const col = field.default !== undefined ? base.default(field.default) : base;
       return { [name]: field.required ? col.notNull() : col };
     }
     case "bigInt": {
@@ -104,7 +110,8 @@ function fieldToColumns(
       // JS-`bigint` — JSON-serialisierbar, Frontend-tauglich. Wer >2^53
       // braucht (Astronomie-Astronomie), nutzt einen Text-Field mit
       // eigenem Codec.
-      const col = bigint(snakeName, { mode: "number" });
+      const base = bigint(snakeName, { mode: "number" });
+      const col = field.default !== undefined ? base.default(field.default) : base;
       return { [name]: field.required ? col.notNull() : col };
     }
     case "reference":