@stacksjs/database 0.70.91 → 0.70.93

package/dist/migrations.d.ts

@@ -30,9 +30,6 @@ export type { MigrationResult as MigrationResultType };
  *     directory clean and prevents future runs from re-discovering it.
  */
 export declare function preprocessSqliteMigrations(): void;
-/**
- * Run database migrations
- */
 export declare function runDatabaseMigration(): Promise<Result<string, Error>>;
 /**
  * Reset the database (drop all tables)

package/dist/migrations.js

@@ -247,7 +247,7 @@ async function ensureDatabaseExists() {
 async function hideDisabledFeatureMigrations() {
   const hidden = [];
   try {
-    const { FEATURE_NAMES, migrationFeature } = await import("@stacksjs/buddy"), { feature: isFeatureEnabled } = await import("@stacksjs/config"), fs = await import("node:fs/promises"), migrationsDir = path.projectPath("database/migrations");
+    const { appModelClaimsTable, FEATURE_NAMES, migrationFeature, migrationTable } = await import("@stacksjs/buddy"), { feature: isFeatureEnabled } = await import("@stacksjs/config"), fs = await import("node:fs/promises"), migrationsDir = path.projectPath("database/migrations");
     if (!existsSync(migrationsDir))
       return hidden;
     const disabledFeatures = new Set(FEATURE_NAMES.filter((f) => !isFeatureEnabled(f)));
@@ -258,6 +258,9 @@ async function hideDisabledFeatureMigrations() {
       const owner = migrationFeature(file);
       if (!owner || !disabledFeatures.has(owner))
         continue;
+      const table = migrationTable(file);
+      if (table && appModelClaimsTable(table))
+        continue;
       const original = join(migrationsDir, file), hiddenPath = `${original}.disabled`;
       await fs.rename(original, hiddenPath);
       hidden.push({ original, hidden: hiddenPath, feature: owner });
@@ -301,6 +304,50 @@ async function writeMigrateMarker(appliedCount) {
     await fs.writeFile(file, body, "utf8");
   } catch {}
 }
+function idempotentSql(sql) {
+  const stmts = sql.split(";").map((s) => s.trim()).filter(Boolean);
+  if (stmts.length === 0)
+    return sql;
+  const out = [];
+  for (const raw of stmts) {
+    const stmt = raw.replace(/\bADD\s+COLUMN\s+(?!IF\s+NOT\s+EXISTS\b)/gi, "ADD COLUMN IF NOT EXISTS "), m = /^ALTER\s+TABLE\s+("?\w+"?)\s+ADD\s+CONSTRAINT\s+("?\w+"?)/i.exec(stmt);
+    if (m) {
+      const drop = `ALTER TABLE ${m[1]} DROP CONSTRAINT IF EXISTS ${m[2]}`;
+      if ((out[out.length - 1] ?? "").toUpperCase() !== drop.toUpperCase())
+        out.push(drop);
+    }
+    out.push(stmt);
+  }
+  return `${out.join(`;
+`)};
+`;
+}
+function makeMigrationsIdempotent() {
+  const migrationsDir = join(process.cwd(), "database", "migrations");
+  let files;
+  try {
+    files = readdirSync(migrationsDir).filter((f) => f.endsWith(".sql"));
+  } catch {
+    return;
+  }
+  for (const f of files) {
+    const p = join(migrationsDir, f);
+    let sql;
+    try {
+      sql = readFileSync(p, "utf8");
+    } catch {
+      continue;
+    }
+    if (!/\bADD\s+(?:COLUMN|CONSTRAINT)\b/i.test(sql))
+      continue;
+    const next = idempotentSql(sql);
+    if (next !== sql)
+      try {
+        writeFileSync(p, next);
+        log.debug(`[migration] made idempotent: ${f}`);
+      } catch {}
+  }
+}
 export async function runDatabaseMigration() {
   const startedAt = Date.now(), hidden = await hideDisabledFeatureMigrations();
   let lockHandle = null;
@@ -312,6 +359,8 @@ export async function runDatabaseMigration() {
     lockHandle = await acquireMigrationLock(dialect, lockDb);
     if (dialect === "sqlite")
       preprocessSqliteMigrations();
+    else if (dialect === "postgres")
+      makeMigrationsIdempotent();
     const modelsDir = path.userModelsPath(), appliedBefore = await countAppliedMigrations();
     log.debug(`[migration] Running migrations from: ${modelsDir}`);
     await qbExecuteMigration(modelsDir);

package/package.json

@@ -2,7 +2,7 @@
   "name": "@stacksjs/database",
   "type": "module",
   "sideEffects": false,
-  "version": "0.70.91",
+  "version": "0.70.93",
   "description": "The Stacks database integration.",
   "author": "Chris Breuer",
   "contributors": [
@@ -58,15 +58,15 @@
     "bun-query-builder": "^0.1.50"
   },
   "devDependencies": {
-    "@stacksjs/cli": "0.70.91",
-    "@stacksjs/config": "0.70.91",
-    "@stacksjs/logging": "0.70.91",
-    "@stacksjs/router": "0.70.91",
+    "@stacksjs/cli": "0.70.93",
+    "@stacksjs/config": "0.70.93",
+    "@stacksjs/logging": "0.70.93",
+    "@stacksjs/router": "0.70.93",
     "better-dx": "^0.2.16",
-    "@stacksjs/path": "0.70.91",
-    "@stacksjs/query-builder": "0.70.91",
-    "@stacksjs/storage": "0.70.91",
-    "@stacksjs/strings": "0.70.91",
-    "@stacksjs/utils": "0.70.91"
+    "@stacksjs/path": "0.70.93",
+    "@stacksjs/query-builder": "0.70.93",
+    "@stacksjs/storage": "0.70.93",
+    "@stacksjs/strings": "0.70.93",
+    "@stacksjs/utils": "0.70.93"
   }
 }