@mikro-orm/sql 7.1.3-dev.0 → 7.1.3-dev.2

package/dialects/sqlite/SqliteSchemaHelper.js

@@ -536,8 +536,10 @@ export class SqliteSchemaHelper extends SchemaHelper {
                 this.append(ret, this.getRenameIndexSQL(diff.name, index, oldIndexName));
             }
         }
-        for (const trigger of Object.values(diff.removedTriggers)) {
-            this.append(ret, this.dropTrigger(diff.toTable, trigger));
+        if (!safe) {
+            for (const trigger of Object.values(diff.removedTriggers)) {
+                this.append(ret, this.dropTrigger(diff.toTable, trigger));
+            }
         }
         for (const trigger of Object.values(diff.changedTriggers)) {
             this.append(ret, this.dropTrigger(diff.toTable, trigger));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/sql",
-  "version": "7.1.3-dev.0",
+  "version": "7.1.3-dev.2",
   "description": "TypeScript ORM for Node.js based on Data Mapper, Unit of Work and Identity Map patterns. Supports MongoDB, MySQL, PostgreSQL and SQLite databases as well as usage with vanilla JavaScript.",
   "keywords": [
     "data-mapper",
@@ -53,7 +53,7 @@
     "@mikro-orm/core": "^7.1.2"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.1.3-dev.0"
+    "@mikro-orm/core": "7.1.3-dev.2"
   },
   "engines": {
     "node": ">= 22.17.0"

package/schema/SchemaComparator.js

@@ -170,6 +170,8 @@ export class SchemaComparator {
         return diff;
     }
     compareRoutines(fromSchema, toSchema, diff) {
+        // `ignoreRoutines` makes routines create-only: new ones are still added, but existing routines are never diffed for drop/alter.
+        const ignoreRoutines = this.#platform.getConfig().get('schemaGenerator').ignoreRoutines;
         // Case-fold so user-written `'sql_hash'` matches Oracle's introspected `'SQL_HASH'`.
         const routineKey = (r) => ((r.schema ? `${r.schema}.` : '') + r.name).toLowerCase();
         const fromByKey = new Map(fromSchema.getRoutines().map(r => [routineKey(r), r]));
@@ -181,15 +183,17 @@ export class SchemaComparator {
                 this.log(`routine ${key} added`);
                 continue;
             }
-            if (this.diffRoutine(fromRoutine, toRoutine)) {
+            if (!ignoreRoutines && this.diffRoutine(fromRoutine, toRoutine)) {
                 diff.changedRoutines[key] = { from: fromRoutine, to: toRoutine };
                 this.log(`routine ${key} changed`, { fromRoutine, toRoutine });
             }
         }
-        for (const [key, fromRoutine] of fromByKey) {
-            if (!toByKey.has(key)) {
-                diff.removedRoutines[key] = fromRoutine;
-                this.log(`routine ${key} removed`);
+        if (!ignoreRoutines) {
+            for (const [key, fromRoutine] of fromByKey) {
+                if (!toByKey.has(key)) {
+                    diff.removedRoutines[key] = fromRoutine;
+                    this.log(`routine ${key} removed`);
+                }
             }
         }
     }
@@ -493,21 +497,24 @@ export class SchemaComparator {
             this.log(`trigger ${trigger.name} added to table ${tableDifferences.name}`, { trigger });
             changes++;
         }
-        for (const trigger of fromTableTriggers) {
-            if (!toTable.hasTrigger(trigger.name)) {
-                tableDifferences.removedTriggers[trigger.name] = trigger;
-                this.log(`trigger ${trigger.name} removed from table ${tableDifferences.name}`);
-                changes++;
-                continue;
-            }
-            const toTableTrigger = toTable.getTrigger(trigger.name);
-            if (this.diffTrigger(trigger, toTableTrigger)) {
-                this.log(`trigger ${trigger.name} changed in table ${tableDifferences.name}`, {
-                    fromTableTrigger: trigger,
-                    toTableTrigger,
-                });
-                tableDifferences.changedTriggers[trigger.name] = toTableTrigger;
-                changes++;
+        // `ignoreTriggers` makes triggers create-only: declared ones are still added above, but existing triggers are never diffed for drop/alter.
+        if (!this.#platform.getConfig().get('schemaGenerator').ignoreTriggers) {
+            for (const trigger of fromTableTriggers) {
+                if (!toTable.hasTrigger(trigger.name)) {
+                    tableDifferences.removedTriggers[trigger.name] = trigger;
+                    this.log(`trigger ${trigger.name} removed from table ${tableDifferences.name}`);
+                    changes++;
+                    continue;
+                }
+                const toTableTrigger = toTable.getTrigger(trigger.name);
+                if (this.diffTrigger(trigger, toTableTrigger)) {
+                    this.log(`trigger ${trigger.name} changed in table ${tableDifferences.name}`, {
+                        fromTableTrigger: trigger,
+                        toTableTrigger,
+                    });
+                    tableDifferences.changedTriggers[trigger.name] = toTableTrigger;
+                    changes++;
+                }
             }
         }
         const fromForeignKeys = { ...fromTable.getForeignKeys() };

package/schema/SchemaHelper.js

@@ -386,8 +386,10 @@ export class SchemaHelper {
         for (const check of Object.values(diff.changedChecks)) {
             ret.push(this.dropConstraint(diff.name, check.name));
         }
-        for (const trigger of Object.values(diff.removedTriggers)) {
-            ret.push(this.dropTrigger(diff.toTable, trigger));
+        if (!safe) {
+            for (const trigger of Object.values(diff.removedTriggers)) {
+                ret.push(this.dropTrigger(diff.toTable, trigger));
+            }
         }
         for (const trigger of Object.values(diff.changedTriggers)) {
             ret.push(this.dropTrigger(diff.toTable, trigger));