@gobing-ai/ts-db 0.5.26 → 0.5.27

package/dist/migrate.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"migrate.d.ts","sourceRoot":"","sources":["../src/migrate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,uBAAuB,CAAC;AAGxD,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AAG3C;;;;;;GAMG;AACH,MAAM,WAAW,eAAe;IAC5B,IAAI,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,IAAI,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,KAAK,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;CAC5B;AAED;;GAEG;AACH,MAAM,WAAW,gBAAgB;IAC7B,oEAAoE;IACpE,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,6EAA6E;IAC7E,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,mDAAmD;IACnD,EAAE,CAAC,EAAE,UAAU,CAAC;IAChB,yDAAyD;IACzD,MAAM,CAAC,EAAE,eAAe,CAAC;CAC5B;AA+ED;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,eAAe,CAAC,OAAO,EAAE,SAAS,EAAE,OAAO,CAAC,EAAE,gBAAgB,GAAG,OAAO,CAAC,IAAI,CAAC,CAmDnG"}
+{"version":3,"file":"migrate.d.ts","sourceRoot":"","sources":["../src/migrate.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,uBAAuB,CAAC;AAGxD,OAAO,KAAK,EAAE,SAAS,EAAa,MAAM,WAAW,CAAC;AAGtD;;;;;;GAMG;AACH,MAAM,WAAW,eAAe;IAC5B,IAAI,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,IAAI,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;IACxB,KAAK,CAAC,GAAG,EAAE,MAAM,GAAG,IAAI,CAAC;CAC5B;AAED;;GAEG;AACH,MAAM,WAAW,gBAAgB;IAC7B,oEAAoE;IACpE,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,6EAA6E;IAC7E,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,mDAAmD;IACnD,EAAE,CAAC,EAAE,UAAU,CAAC;IAChB,yDAAyD;IACzD,MAAM,CAAC,EAAE,eAAe,CAAC;CAC5B;AAkFD;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,eAAe,CAAC,OAAO,EAAE,SAAS,EAAE,OAAO,CAAC,EAAE,gBAAgB,GAAG,OAAO,CAAC,IAAI,CAAC,CAmDnG"}

package/dist/migrate.js

@@ -24,7 +24,9 @@ function validateMigrationTableName(table) {
  * Apply migrations from embedded SQL strings (for compiled binaries).
  *
  * Checks the journal table and applies only migrations that haven't run yet.
- * Each migration is executed with adapter.exec() for file-based or adapter.run() for journal tracking.
+ * Each migration's SQL statements and its journal INSERT run as one
+ * `adapter.batch` call — a single SQLite transaction, atomic per migration
+ * (task 0113): a journal failure rolls the schema changes back with it.
  */
 async function applyEmbeddedMigrations(adapter, journalTable, logger) {
     // The caller (`applyMigrations`) already ran the journal name through
@@ -48,16 +50,18 @@ async function applyEmbeddedMigrations(adapter, journalTable, logger) {
         if (appliedHashes.has(migration.hash))
             continue;
         logger.info(`Applying embedded migration: ${migration.tag}`);
-        // Split on semicolons and execute each non-empty statement
+        // Split on semicolons; batch the migration's statements with its journal
+        // INSERT so they commit or roll back together (task 0113, ADR-020 seam).
         const statements = migration.sql
             .split(';')
             .map((s) => s.trim())
             .filter((s) => s.length > 0);
-        for (const stmt of statements) {
-            await adapter.exec(stmt);
-        }
-        // Record in journal
-        await adapter.run(`INSERT INTO "${journalTable}" (hash, created_at) VALUES (?, ?)`, migration.hash, Date.now());
+        const ops = statements.map((sql) => ({ sql, params: [] }));
+        ops.push({
+            sql: `INSERT INTO "${journalTable}" (hash, created_at) VALUES (?, ?)`,
+            params: [migration.hash, Date.now()],
+        });
+        await adapter.batch(ops);
         applied++;
     }
     if (applied > 0) {

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@gobing-ai/ts-db",
-    "version": "0.5.26",
+    "version": "0.5.27",
     "description": "@gobing-ai/ts-db — a drizzle-free database facade: typed DAOs over Bun SQLite / Cloudflare D1, a small predicate query spec, single-source-of-truth tables, and migrations. Drizzle stays an internal detail.",
     "keywords": [
         "typescript",
@@ -66,8 +66,8 @@
         "release": "echo 'Manual publish is disabled. Releases go through GitHub Actions via Trusted Publishing — push a tag: git tag @gobing-ai/ts-db-v<version> && git push --tags' && exit 1"
     },
     "dependencies": {
-        "@gobing-ai/ts-runtime": "^0.5.26",
-        "@gobing-ai/ts-utils": "^0.5.26"
+        "@gobing-ai/ts-runtime": "^0.5.27",
+        "@gobing-ai/ts-utils": "^0.5.27"
     },
     "peerDependencies": {
         "drizzle-orm": ">=0.38.0",

package/src/migrate.ts

@@ -1,7 +1,7 @@
 import type { FileSystem } from '@gobing-ai/ts-runtime';
 import { getProcessCwd, resolvePath } from '@gobing-ai/ts-runtime';
 
-import type { DbAdapter } from './adapter';
+import type { DbAdapter, DbBatchOp } from './adapter';
 import { embeddedMigrations } from './embedded-migrations';
 
 /**
@@ -59,7 +59,9 @@ function validateMigrationTableName(table: string): string {
  * Apply migrations from embedded SQL strings (for compiled binaries).
  *
  * Checks the journal table and applies only migrations that haven't run yet.
- * Each migration is executed with adapter.exec() for file-based or adapter.run() for journal tracking.
+ * Each migration's SQL statements and its journal INSERT run as one
+ * `adapter.batch` call — a single SQLite transaction, atomic per migration
+ * (task 0113): a journal failure rolls the schema changes back with it.
  */
 async function applyEmbeddedMigrations(
     adapter: DbAdapter,
@@ -88,18 +90,19 @@ async function applyEmbeddedMigrations(
 
         logger.info(`Applying embedded migration: ${migration.tag}`);
 
-        // Split on semicolons and execute each non-empty statement
+        // Split on semicolons; batch the migration's statements with its journal
+        // INSERT so they commit or roll back together (task 0113, ADR-020 seam).
         const statements = migration.sql
             .split(';')
             .map((s) => s.trim())
             .filter((s) => s.length > 0);
 
-        for (const stmt of statements) {
-            await adapter.exec(stmt);
-        }
-
-        // Record in journal
-        await adapter.run(`INSERT INTO "${journalTable}" (hash, created_at) VALUES (?, ?)`, migration.hash, Date.now());
+        const ops: DbBatchOp[] = statements.map((sql) => ({ sql, params: [] }));
+        ops.push({
+            sql: `INSERT INTO "${journalTable}" (hash, created_at) VALUES (?, ?)`,
+            params: [migration.hash, Date.now()],
+        });
+        await adapter.batch(ops);
         applied++;
     }