@famgia/omnify-cli 2.0.44 → 2.0.46

package/dist/index.js

@@ -1394,6 +1394,16 @@ function getCliVersion() {
 function hasPluginGenerators(plugins) {
   return plugins.some((p) => p.generators && p.generators.length > 0);
 }
+function toTableName(schemaName) {
+  const snake = schemaName.replace(/([A-Z])/g, "_$1").toLowerCase().replace(/^_/, "");
+  if (snake.endsWith("y") && !["ay", "ey", "iy", "oy", "uy"].some((v) => snake.endsWith(v))) {
+    return snake.slice(0, -1) + "ies";
+  }
+  if (snake.endsWith("s") || snake.endsWith("x") || snake.endsWith("ch") || snake.endsWith("sh")) {
+    return snake + "es";
+  }
+  return snake + "s";
+}
 function getExistingMigrationTables(migrationsDir) {
   const existingTables = /* @__PURE__ */ new Set();
   if (!existsSync9(migrationsDir)) {
@@ -2387,7 +2397,37 @@ async function runGenerate(options) {
     allMigrationRecords = counts.migrationRecords;
   }
   logger.step("Updating lock file...");
-  let newLockFile = updateLockFile(existingLock, currentSnapshots, config.database.driver);
+  const migratedSchemas = /* @__PURE__ */ new Set();
+  for (const record of allMigrationRecords) {
+    for (const schemaName of record.schemas) {
+      migratedSchemas.add(schemaName);
+    }
+  }
+  const modifiedWithoutMigration = [];
+  for (const change of comparison.changes) {
+    if (change.changeType === "modified") {
+      const hasMigration = migratedSchemas.has(change.schemaName) || // Also check if there's an ALTER migration by table name
+      allMigrationRecords.some(
+        (r) => r.type === "alter" && r.tableName === toTableName(change.schemaName)
+      );
+      if (!hasMigration) {
+        modifiedWithoutMigration.push(change.schemaName);
+        logger.warn(`\u26A0\uFE0F  Schema '${change.schemaName}' was modified but no ALTER migration was generated.`);
+      }
+    }
+  }
+  let snapshotsToUpdate = currentSnapshots;
+  if (modifiedWithoutMigration.length > 0 && existingLock && isLockFileV2(existingLock)) {
+    logger.warn("Keeping old state for schemas without migrations to ensure they are detected on next run.");
+    snapshotsToUpdate = { ...currentSnapshots };
+    for (const schemaName of modifiedWithoutMigration) {
+      if (existingLock.schemas[schemaName]) {
+        snapshotsToUpdate[schemaName] = existingLock.schemas[schemaName];
+        logger.debug(`  Keeping old state for: ${schemaName}`);
+      }
+    }
+  }
+  let newLockFile = updateLockFile(existingLock, snapshotsToUpdate, config.database.driver);
   if (allMigrationRecords.length > 0) {
     logger.debug(`Adding ${allMigrationRecords.length} migration(s) to lock file...`);
     for (const record of allMigrationRecords) {