@cosmicdrift/kumiko-framework 0.162.0 → 0.163.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cosmicdrift/kumiko-framework",
-  "version": "0.162.0",
+  "version": "0.163.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>",
@@ -182,7 +182,7 @@
     "./package.json": "./package.json"
   },
   "dependencies": {
-    "@cosmicdrift/kumiko-types": "0.162.0",
+    "@cosmicdrift/kumiko-types": "0.163.0",
     "bullmq": "^5.76.7",
     "bun-types": "^1.3.13",
     "hono": "^4.12.18",
@@ -198,7 +198,7 @@
     "zod": "^4.4.3"
   },
   "devDependencies": {
-    "@cosmicdrift/kumiko-dispatcher-live": "0.162.0",
+    "@cosmicdrift/kumiko-dispatcher-live": "0.163.0",
     "bun-types": "^1.3.13",
     "pino-pretty": "^13.1.3"
   },

package/src/db/__tests__/replay-migration-sql.test.ts

@@ -56,6 +56,38 @@ CREATE INDEX IF NOT EXISTS "read_accounts_tenant_id_idx" ON "read_accounts" ("te
     }
   });
 
+  // Hand-edited migrations legitimately add "IF NOT EXISTS"/"IF EXISTS" to
+  // ADD/DROP COLUMN (the generator itself never emits it, but app authors
+  // are explicitly allowed to hand-edit before committing — see the header
+  // comment every generated migration carries).
+  test("ADD COLUMN IF NOT EXISTS (hand-edited) still extends the table", () => {
+    const dir = tmpMigrationsDir();
+    try {
+      write(dir, "0001_init.sql", `CREATE TABLE IF NOT EXISTS "read_a" ("id" uuid PRIMARY KEY);`);
+      write(dir, "0002_add-col.sql", `ALTER TABLE "read_a" ADD COLUMN IF NOT EXISTS "title" text;`);
+      const replayed = replayMigrationsDir(dir);
+      expect([...(replayed.get("read_a")?.columns ?? [])].sort()).toEqual(["id", "title"]);
+    } finally {
+      rmSync(dir, { recursive: true, force: true });
+    }
+  });
+
+  test("DROP COLUMN IF EXISTS (hand-edited) still removes the column", () => {
+    const dir = tmpMigrationsDir();
+    try {
+      write(
+        dir,
+        "0001_init.sql",
+        `CREATE TABLE IF NOT EXISTS "read_a" ("id" uuid PRIMARY KEY, "title" text);`,
+      );
+      write(dir, "0002_drop-col.sql", `ALTER TABLE "read_a" DROP COLUMN IF EXISTS "title";`);
+      const replayed = replayMigrationsDir(dir);
+      expect([...(replayed.get("read_a")?.columns ?? [])]).toEqual(["id"]);
+    } finally {
+      rmSync(dir, { recursive: true, force: true });
+    }
+  });
+
   test("commented-out destructive DROP TABLE is not replayed", () => {
     const dir = tmpMigrationsDir();
     try {

package/src/db/replay-migration-sql.ts

@@ -57,14 +57,18 @@ function applyStatement(schema: Map<string, { columns: Set<string> }>, statement
   const createTableName = create?.[1];
   const createBody = create?.[2];
 
-  const addColumn = statement.match(/^ALTER TABLE\s+"([^"]+)"\s+ADD COLUMN\s+"([^"]+)"/i);
+  const addColumn = statement.match(
+    /^ALTER TABLE\s+"([^"]+)"\s+ADD COLUMN\s+(?:IF NOT EXISTS\s+)?"([^"]+)"/i,
+  );
   const addColumnTable = addColumn?.[1];
   const addColumnName = addColumn?.[2];
 
   const dropTable = statement.match(/^DROP TABLE\s+(?:IF EXISTS\s+)?"([^"]+)"/i);
   const dropTableName = dropTable?.[1];
 
-  const dropColumn = statement.match(/^ALTER TABLE\s+"([^"]+)"\s+DROP COLUMN\s+"([^"]+)"/i);
+  const dropColumn = statement.match(
+    /^ALTER TABLE\s+"([^"]+)"\s+DROP COLUMN\s+(?:IF EXISTS\s+)?"([^"]+)"/i,
+  );
   const dropColumnTable = dropColumn?.[1];
   const dropColumnName = dropColumn?.[2];