@gobing-ai/ts-db 0.3.18 → 0.3.20

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;AA8ED;;;;;;;;;;;;;;;GAeG;AACH,wBAAsB,eAAe,CAAC,OAAO,EAAE,SAAS,EAAE,OAAO,CAAC,EAAE,gBAAgB,GAAG,OAAO,CAAC,IAAI,CAAC,CAiDnG"}
+{"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"}

package/dist/migrate.js

@@ -27,10 +27,11 @@ function validateMigrationTableName(table) {
  * Each migration is executed with adapter.exec() for file-based or adapter.run() for journal tracking.
  */
 async function applyEmbeddedMigrations(adapter, journalTable, logger) {
-    // Validate journal table name — this is an internal constant, never user input.
-    if (!/^__[a-z_]+$/.test(journalTable)) {
-        throw new Error(`Invalid migration journal table name: ${journalTable}`);
-    }
+    // The caller (`applyMigrations`) already ran the journal name through
+    // `validateMigrationTableName`; re-validate with the same rule so a custom but
+    // legal name (e.g. `my_migrations`) isn't rejected here while the file-based path
+    // accepted it — one validator, no divergence between the two migration paths.
+    validateMigrationTableName(journalTable);
     // Get already-applied hashes
     const appliedHashes = new Set();
     try {
@@ -101,7 +102,9 @@ export async function applyMigrations(adapter, options) {
             logger.info(`Applying database migrations from ${folder}`);
             await drizzleMigrate(adapter.getDrizzleDb(), {
                 migrationsFolder: folder,
-                ...(options?.migrationsTable !== undefined ? { migrationsTable: options.migrationsTable } : {}),
+                // Pass the validated `table`, not the raw option — keep one validated
+                // name flowing to both the file-based migrator and the embedded fallback.
+                ...(options?.migrationsTable !== undefined ? { migrationsTable: table } : {}),
             });
             logger.info('Database migrations complete');
             return;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@gobing-ai/ts-db",
-    "version": "0.3.18",
+    "version": "0.3.20",
     "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.3.18",
-        "@gobing-ai/ts-utils": "^0.3.18"
+        "@gobing-ai/ts-runtime": "^0.3.20",
+        "@gobing-ai/ts-utils": "^0.3.20"
     },
     "peerDependencies": {
         "drizzle-orm": ">=0.38.0",

package/src/migrate.ts

@@ -66,10 +66,11 @@ async function applyEmbeddedMigrations(
     journalTable: string,
     logger: MigrationLogger,
 ): Promise<void> {
-    // Validate journal table name — this is an internal constant, never user input.
-    if (!/^__[a-z_]+$/.test(journalTable)) {
-        throw new Error(`Invalid migration journal table name: ${journalTable}`);
-    }
+    // The caller (`applyMigrations`) already ran the journal name through
+    // `validateMigrationTableName`; re-validate with the same rule so a custom but
+    // legal name (e.g. `my_migrations`) isn't rejected here while the file-based path
+    // accepted it — one validator, no divergence between the two migration paths.
+    validateMigrationTableName(journalTable);
     // Get already-applied hashes
     const appliedHashes = new Set<string>();
     try {
@@ -152,7 +153,9 @@ export async function applyMigrations(adapter: DbAdapter, options?: MigrationOpt
 
             await drizzleMigrate(adapter.getDrizzleDb(), {
                 migrationsFolder: folder,
-                ...(options?.migrationsTable !== undefined ? { migrationsTable: options.migrationsTable } : {}),
+                // Pass the validated `table`, not the raw option — keep one validated
+                // name flowing to both the file-based migrator and the embedded fallback.
+                ...(options?.migrationsTable !== undefined ? { migrationsTable: table } : {}),
             });
             logger.info('Database migrations complete');
             return;