@mikro-orm/sql 7.0.0-dev.299 → 7.0.0-dev.301

package/schema/DatabaseTable.js

@@ -80,10 +80,13 @@ export class DatabaseTable {
             this.columns[field] = {
                 name: prop.fieldNames[idx],
                 type: prop.columnTypes[idx],
-                generated: prop.generated instanceof RawQueryFragment ? this.platform.formatQuery(prop.generated.sql, prop.generated.params) : prop.generated,
+                generated: prop.generated instanceof RawQueryFragment
+                    ? this.platform.formatQuery(prop.generated.sql, prop.generated.params)
+                    : prop.generated,
                 mappedType,
                 unsigned: prop.unsigned && this.platform.isNumericColumn(mappedType),
-                autoincrement: prop.autoincrement ?? (primary && prop.kind === ReferenceKind.SCALAR && this.platform.isNumericColumn(mappedType)),
+                autoincrement: prop.autoincrement ??
+                    (primary && prop.kind === ReferenceKind.SCALAR && this.platform.isNumericColumn(mappedType)),
                 primary,
                 nullable: this.columns[field]?.nullable ?? !!prop.nullable,
                 nativeEnumName: prop.nativeEnumName,
@@ -105,7 +108,9 @@ export class DatabaseTable {
         });
         if ([ReferenceKind.MANY_TO_ONE, ReferenceKind.ONE_TO_ONE].includes(prop.kind) && !prop.polymorphic) {
             const constraintName = this.getIndexName(prop.foreignKeyName ?? true, prop.fieldNames, 'foreign');
-            let schema = prop.targetMeta.root.schema === '*' ? this.schema : (prop.targetMeta.root.schema ?? config.get('schema', this.platform.getDefaultSchemaName()));
+            let schema = prop.targetMeta.root.schema === '*'
+                ? this.schema
+                : (prop.targetMeta.root.schema ?? config.get('schema', this.platform.getDefaultSchemaName()));
             if (prop.referencedTableName.includes('.')) {
                 schema = undefined;
             }
@@ -154,19 +159,20 @@ export class DatabaseTable {
         return this.platform.getIndexName(this.name, columnNames, type);
     }
     getEntityDeclaration(namingStrategy, schemaHelper, scalarPropertiesForRelations) {
-        const { fksOnColumnProps, fksOnStandaloneProps, columnFks, fkIndexes, nullableForeignKeys, skippedColumnNames, } = this.foreignKeysToProps(namingStrategy, scalarPropertiesForRelations);
+        const { fksOnColumnProps, fksOnStandaloneProps, columnFks, fkIndexes, nullableForeignKeys, skippedColumnNames } = this.foreignKeysToProps(namingStrategy, scalarPropertiesForRelations);
         const name = namingStrategy.getEntityName(this.name, this.schema);
         const schema = new EntitySchema({ name, collection: this.name, schema: this.schema, comment: this.comment });
         const compositeFkIndexes = {};
         const compositeFkUniques = {};
-        const potentiallyUnmappedIndexes = this.indexes.filter(index => !index.primary // Skip primary index. Whether it's in use by scalar column or FK, it's already mapped.
-            && (( // Non-trivial non-composite indexes will be declared at the entity's metadata, though later outputted in the property
-            index.columnNames.length > 1 // All composite indexes are to be mapped to entity decorators or FK props.
-                || skippedColumnNames.includes(index.columnNames[0]) // Non-composite indexes for skipped columns are to be mapped as entity decorators.
-                || index.deferMode || index.expression || !(index.columnNames[0] in columnFks)) // Trivial non-composite indexes for scalar props are to be mapped to the column.
-            )
+        const potentiallyUnmappedIndexes = this.indexes.filter(index => !index.primary && // Skip primary index. Whether it's in use by scalar column or FK, it's already mapped.
+            // Non-trivial non-composite indexes will be declared at the entity's metadata, though later outputted in the property
+            (index.columnNames.length > 1 || // All composite indexes are to be mapped to entity decorators or FK props.
+                skippedColumnNames.includes(index.columnNames[0]) || // Non-composite indexes for skipped columns are to be mapped as entity decorators.
+                index.deferMode ||
+                index.expression ||
+                !(index.columnNames[0] in columnFks)) && // Trivial non-composite indexes for scalar props are to be mapped to the column.
             // ignore indexes that don't have all column names (this can happen in sqlite where there is no way to infer this for expressions)
-            && !(index.columnNames.some(col => !col) && !index.expression));
+            !(index.columnNames.some(col => !col) && !index.expression));
         // Helper to map column name to property name
         const columnToPropertyName = (colName) => this.getPropertyName(namingStrategy, colName);
         for (const index of potentiallyUnmappedIndexes) {
@@ -198,8 +204,13 @@ export class DatabaseTable {
                 }
             }
             // An index is trivial if it has no special options that require entity-level declaration
-            const hasAdvancedOptions = index.columns?.length || index.include?.length || index.fillFactor ||
-                index.type || index.invisible || index.disabled || index.clustered;
+            const hasAdvancedOptions = index.columns?.length ||
+                index.include?.length ||
+                index.fillFactor ||
+                index.type ||
+                index.invisible ||
+                index.disabled ||
+                index.clustered;
             const isTrivial = !index.deferMode && !index.expression && !hasAdvancedOptions;
             if (isTrivial) {
                 // Index is for FK. Map to the FK prop and move on.
@@ -213,7 +224,8 @@ export class DatabaseTable {
                     }
                 }
             }
-            const properties = ret.properties ?? this.getIndexProperties(index, columnFks, fksOnColumnProps, fksOnStandaloneProps, namingStrategy);
+            const properties = ret.properties ??
+                this.getIndexProperties(index, columnFks, fksOnColumnProps, fksOnStandaloneProps, namingStrategy);
             // If there is a column that cannot be unambiguously mapped to a prop, render an expression.
             if (typeof properties === 'undefined') {
                 ret.expression ??= schemaHelper.getCreateIndexSQL(this.name, index);
@@ -237,7 +249,7 @@ export class DatabaseTable {
             }
             schema.addIndex(ret);
         }
-        const addedStandaloneFkPropsBasedOnColumn = new Set;
+        const addedStandaloneFkPropsBasedOnColumn = new Set();
         const nonSkippedColumns = this.getColumns().filter(column => !skippedColumnNames.includes(column.name));
         for (const column of nonSkippedColumns) {
             const columnName = column.name;
@@ -259,10 +271,10 @@ export class DatabaseTable {
             schema.addProperty(prop.name, prop.type, prop);
         }
         const meta = schema.init().meta;
-        const oneToOneCandidateProperties = meta.relations
-            .filter(prop => prop.primary && prop.kind === ReferenceKind.MANY_TO_ONE);
-        if (oneToOneCandidateProperties.length === 1
-            && oneToOneCandidateProperties[0].fieldNames.length === (new Set(meta.getPrimaryProps().flatMap(prop => prop.fieldNames))).size) {
+        const oneToOneCandidateProperties = meta.relations.filter(prop => prop.primary && prop.kind === ReferenceKind.MANY_TO_ONE);
+        if (oneToOneCandidateProperties.length === 1 &&
+            oneToOneCandidateProperties[0].fieldNames.length ===
+                new Set(meta.getPrimaryProps().flatMap(prop => prop.fieldNames)).size) {
             oneToOneCandidateProperties[0].kind = ReferenceKind.ONE_TO_ONE;
         }
         return meta;
@@ -277,7 +289,8 @@ export class DatabaseTable {
         const standaloneFksBasedOnColumnNames = new Map();
         for (const currentFk of fks) {
             const fkIndex = this.findFkIndex(currentFk);
-            if (currentFk.columnNames.length === 1 && !fks.some(fk => fk !== currentFk && fk.columnNames.length === 1 && currentFk.columnNames[0] === fk.columnNames[0])) {
+            if (currentFk.columnNames.length === 1 &&
+                !fks.some(fk => fk !== currentFk && fk.columnNames.length === 1 && currentFk.columnNames[0] === fk.columnNames[0])) {
                 // Non-composite FK is the only possible one for a column. Render the column with it.
                 const columnName = currentFk.columnNames[0];
                 columnFks[columnName] ??= [];
@@ -316,7 +329,10 @@ export class DatabaseTable {
             if (nullableColumnsInFk.length > 0) {
                 nullableForeignKeys.add(currentFk);
             }
-            if (specificColumnNames.length === 1 && ((nullableColumnsInFk.length === currentFk.columnNames.length || nullableColumnsInFk.length === 0) || (nullableColumnsInFk.length === 1 && nullableColumnsInFk[0] === specificColumnNames[0]))) {
+            if (specificColumnNames.length === 1 &&
+                (nullableColumnsInFk.length === currentFk.columnNames.length ||
+                    nullableColumnsInFk.length === 0 ||
+                    (nullableColumnsInFk.length === 1 && nullableColumnsInFk[0] === specificColumnNames[0]))) {
                 // Composite FK has exactly one column which is not used in any other FK.
                 // The FK also doesn't have a mix of nullable and non-nullable columns,
                 // or its only nullable column is this very one.
@@ -391,14 +407,16 @@ export class DatabaseTable {
             // But also does not skip if the column is not nullable, and yet all involved FKs are nullable,
             // or if one or more FKs involved has multiple nullable columns.
             smart: (column) => {
-                return columnsInFks.includes(column.name)
-                    && !fksOnColumnProps.has(column.name)
-                    && (column.nullable
+                return (columnsInFks.includes(column.name) &&
+                    !fksOnColumnProps.has(column.name) &&
+                    (column.nullable
                         ? columnFks[column.name].some(fk => !fk.columnNames.some(fkColumnName => fkColumnName !== column.name && this.getColumn(fkColumnName)?.nullable))
-                        : columnFks[column.name].some(fk => !nullableForeignKeys.has(fk)));
+                        : columnFks[column.name].some(fk => !nullableForeignKeys.has(fk))));
             },
         };
-        const skippedColumnNames = this.getColumns().filter(skippingHandlers[scalarPropertiesForRelations]).map(column => column.name);
+        const skippedColumnNames = this.getColumns()
+            .filter(skippingHandlers[scalarPropertiesForRelations])
+            .map(column => column.name);
         // Check standalone FKs named after columns for potential conflicts among themselves.
         // This typically happens when two standalone FKs named after a column resolve to the same prop name
         // because the respective columns include the referenced table in the name.
@@ -439,7 +457,8 @@ export class DatabaseTable {
     findFkIndex(currentFk) {
         const fkColumnsLength = currentFk.columnNames.length;
         const possibleIndexes = this.indexes.filter(index => {
-            return index.columnNames.length === fkColumnsLength && !currentFk.columnNames.some((columnName, i) => index.columnNames[i] !== columnName);
+            return (index.columnNames.length === fkColumnsLength &&
+                !currentFk.columnNames.some((columnName, i) => index.columnNames[i] !== columnName));
         });
         possibleIndexes.sort((a, b) => {
             if (a.primary !== b.primary) {
@@ -502,7 +521,8 @@ export class DatabaseTable {
         return Array.from(propBaseNames).map(baseName => this.getPropertyName(namingStrategy, baseName, fksOnColumnProps.get(baseName)));
     }
     getSafeBaseNameForFkProp(namingStrategy, currentFk, fks, columnName) {
-        if (columnName && this.getPropertyName(namingStrategy, columnName, currentFk) !== this.getPropertyName(namingStrategy, columnName)) {
+        if (columnName &&
+            this.getPropertyName(namingStrategy, columnName, currentFk) !== this.getPropertyName(namingStrategy, columnName)) {
             // The eligible scalar column name is different from the name of the FK prop of the same column.
             // Both can be safely rendered.
             // Use the column name as a base for the FK prop.
@@ -538,7 +558,9 @@ export class DatabaseTable {
      */
     getShortestName(skipDefaultSchema = true) {
         const defaultSchema = this.platform.getDefaultSchemaName();
-        if (!this.schema || this.name.startsWith(defaultSchema + '.') || (this.schema === defaultSchema && skipDefaultSchema)) {
+        if (!this.schema ||
+            this.name.startsWith(defaultSchema + '.') ||
+            (this.schema === defaultSchema && skipDefaultSchema)) {
             return this.name;
         }
         return `${this.schema}.${this.name}`;
@@ -569,7 +591,7 @@ export class DatabaseTable {
     }
     getForeignKeyDeclaration(fk, namingStrategy, schemaHelper, fkIndex, nullable, propNameBase, fksOnColumnProps) {
         const prop = this.getPropertyName(namingStrategy, propNameBase, fk);
-        const kind = (fkIndex?.unique && !fkIndex.primary) ? this.getReferenceKind(fk, fkIndex) : this.getReferenceKind(fk);
+        const kind = fkIndex?.unique && !fkIndex.primary ? this.getReferenceKind(fk, fkIndex) : this.getReferenceKind(fk);
         const runtimeType = this.getPropertyTypeForForeignKey(namingStrategy, fk);
         const fkOptions = {};
         fkOptions.fieldNames = fk.columnNames;
@@ -584,8 +606,8 @@ export class DatabaseTable {
             const column = this.getColumn(fk.columnNames[0]);
             const defaultRaw = this.getPropertyDefaultValue(schemaHelper, column, column.type, true);
             const defaultTs = this.getPropertyDefaultValue(schemaHelper, column, column.type);
-            columnOptions.default = (defaultRaw !== defaultTs || defaultRaw === '') ? defaultTs : undefined;
-            columnOptions.defaultRaw = (column.nullable && defaultRaw === 'null') ? undefined : defaultRaw;
+            columnOptions.default = defaultRaw !== defaultTs || defaultRaw === '' ? defaultTs : undefined;
+            columnOptions.defaultRaw = column.nullable && defaultRaw === 'null' ? undefined : defaultRaw;
             columnOptions.optional = typeof column.generated !== 'undefined' || defaultRaw !== 'null';
             columnOptions.generated = column.generated;
             columnOptions.nullable = column.nullable;
@@ -607,25 +629,29 @@ export class DatabaseTable {
             nullable,
             primary: fkIndex?.primary || !fk.columnNames.some(columnName => !this.getPrimaryKey()?.columnNames.includes(columnName)),
             index: !fkIndex?.unique ? fkIndex?.keyName : undefined,
-            unique: (fkIndex?.unique && !fkIndex.primary) ? fkIndex.keyName : undefined,
+            unique: fkIndex?.unique && !fkIndex.primary ? fkIndex.keyName : undefined,
             ...fkOptions,
         };
     }
     getPropertyDeclaration(column, namingStrategy, schemaHelper, compositeFkIndexes, compositeFkUniques, columnFks, fk) {
         const prop = this.getPropertyName(namingStrategy, column.name, fk);
         const persist = !(column.name in columnFks && typeof fk === 'undefined');
-        const index = compositeFkIndexes[prop] || this.indexes.find(idx => idx.columnNames[0] === column.name && !idx.composite && !idx.unique && !idx.primary);
-        const unique = compositeFkUniques[prop] || this.indexes.find(idx => idx.columnNames[0] === column.name && !idx.composite && idx.unique && !idx.primary);
+        const index = compositeFkIndexes[prop] ||
+            this.indexes.find(idx => idx.columnNames[0] === column.name && !idx.composite && !idx.unique && !idx.primary);
+        const unique = compositeFkUniques[prop] ||
+            this.indexes.find(idx => idx.columnNames[0] === column.name && !idx.composite && idx.unique && !idx.primary);
         const kind = this.getReferenceKind(fk, unique);
         const runtimeType = this.getPropertyTypeForColumn(namingStrategy, column, fk);
-        const type = fk ? runtimeType : (Utils.keys(t).find(k => {
-            const typeInCoreMap = this.platform.getMappedType(k);
-            return (typeInCoreMap !== Type.getType(UnknownType) || k === 'unknown') && typeInCoreMap === column.mappedType;
-        }) ?? runtimeType);
-        const ignoreSchemaChanges = (type === 'unknown' && column.length) ? (column.extra ? ['type', 'extra'] : ['type']) : undefined;
+        const type = fk
+            ? runtimeType
+            : (Utils.keys(t).find(k => {
+                const typeInCoreMap = this.platform.getMappedType(k);
+                return ((typeInCoreMap !== Type.getType(UnknownType) || k === 'unknown') && typeInCoreMap === column.mappedType);
+            }) ?? runtimeType);
+        const ignoreSchemaChanges = type === 'unknown' && column.length ? (column.extra ? ['type', 'extra'] : ['type']) : undefined;
         const defaultRaw = this.getPropertyDefaultValue(schemaHelper, column, runtimeType, true);
         const defaultParsed = this.getPropertyDefaultValue(schemaHelper, column, runtimeType);
-        const defaultTs = (defaultRaw !== defaultParsed || defaultParsed === '') ? defaultParsed : undefined;
+        const defaultTs = defaultRaw !== defaultParsed || defaultParsed === '' ? defaultParsed : undefined;
         const fkOptions = {};
         if (fk) {
             fkOptions.fieldNames = fk.columnNames;
@@ -646,7 +672,7 @@ export class DatabaseTable {
             optional: defaultRaw !== 'null' || defaultTs != null || typeof column.generated !== 'undefined',
             columnType: column.type,
             default: defaultTs,
-            defaultRaw: (column.nullable && defaultRaw === 'null') ? undefined : defaultRaw,
+            defaultRaw: column.nullable && defaultRaw === 'null' ? undefined : defaultRaw,
             nullable: column.nullable,
             primary: column.primary && persist,
             autoincrement: column.autoincrement,
@@ -720,7 +746,7 @@ export class DatabaseTable {
         const defaultValue = column.default ?? 'null';
         const val = schemaHelper.normalizeDefaultValue(defaultValue, column.length);
         if (val === 'null') {
-            return raw ? 'null' : (column.nullable ? null : undefined);
+            return raw ? 'null' : column.nullable ? null : undefined;
         }
         if (propType === 'boolean' && !raw) {
             return !['0', 'false', 'f', 'n', 'no', 'off'].includes('' + column.default);
@@ -752,7 +778,9 @@ export class DatabaseTable {
     }
     addIndex(meta, index, type) {
         // If columns are specified but properties are not, derive properties from column names
-        if (index.columns?.length && !index.expression && (!index.properties || Utils.asArray(index.properties).length === 0)) {
+        if (index.columns?.length &&
+            !index.expression &&
+            (!index.properties || Utils.asArray(index.properties).length === 0)) {
             index = { ...index, properties: index.columns.map(c => c.name) };
         }
         const properties = Utils.unique(Utils.flatten(Utils.asArray(index.properties).map(prop => {
@@ -787,13 +815,15 @@ export class DatabaseTable {
         }
         const name = this.getIndexName(index.name, properties, type);
         // Process include columns (map property names to field names)
-        const includeColumns = index.include ? Utils.unique(Utils.flatten(Utils.asArray(index.include).map(prop => {
-            if (meta.properties[prop]) {
-                return meta.properties[prop].fieldNames;
-            }
-            /* v8 ignore next */
-            return [prop];
-        }))) : undefined;
+        const includeColumns = index.include
+            ? Utils.unique(Utils.flatten(Utils.asArray(index.include).map(prop => {
+                if (meta.properties[prop]) {
+                    return meta.properties[prop].fieldNames;
+                }
+                /* v8 ignore next */
+                return [prop];
+            })))
+            : undefined;
         // Process columns with advanced options (map property names to field names)
         const columns = index.columns?.map(col => {
             const fieldName = meta.properties[col.name]?.fieldNames[0] ?? col.name;

package/schema/SchemaComparator.js

@@ -59,7 +59,9 @@ export class SchemaComparator {
             if (toSchema.hasNativeEnum(key)) {
                 continue;
             }
-            if (key.startsWith(`${fromSchema.name}.`) && (fromSchema.name !== toSchema.name || toSchema.getNativeEnum(key.substring(fromSchema.name.length + 1))?.schema === '*')) {
+            if (key.startsWith(`${fromSchema.name}.`) &&
+                (fromSchema.name !== toSchema.name ||
+                    toSchema.getNativeEnum(key.substring(fromSchema.name.length + 1))?.schema === '*')) {
                 continue;
             }
             diff.removedNativeEnums.push(nativeEnum);
@@ -164,7 +166,10 @@ export class SchemaComparator {
         };
         if (this.diffComment(fromTable.comment, toTable.comment)) {
             tableDifferences.changedComment = toTable.comment;
-            this.log(`table comment changed for ${tableDifferences.name}`, { fromTableComment: fromTable.comment, toTableComment: toTable.comment });
+            this.log(`table comment changed for ${tableDifferences.name}`, {
+                fromTableComment: fromTable.comment,
+                toTableComment: toTable.comment,
+            });
             changes++;
         }
         const fromTableColumns = fromTable.getColumns();
@@ -222,7 +227,7 @@ export class SchemaComparator {
         // See if there are any removed indexes in "to" table
         for (const index of fromTableIndexes) {
             // See if index is removed in "to" table.
-            if ((index.primary && !toTable.hasPrimaryKey()) || !index.primary && !toTable.hasIndex(index.keyName)) {
+            if ((index.primary && !toTable.hasPrimaryKey()) || (!index.primary && !toTable.hasIndex(index.keyName))) {
                 tableDifferences.removedIndexes[index.keyName] = index;
                 this.log(`index ${index.keyName} removed from table ${tableDifferences.name}`);
                 changes++;
@@ -234,7 +239,10 @@ export class SchemaComparator {
                 continue;
             }
             tableDifferences.changedIndexes[index.keyName] = toTableIndex;
-            this.log(`index ${index.keyName} changed in table ${tableDifferences.name}`, { fromTableIndex: index, toTableIndex });
+            this.log(`index ${index.keyName} changed in table ${tableDifferences.name}`, {
+                fromTableIndex: index,
+                toTableIndex,
+            });
             changes++;
         }
         this.detectIndexRenamings(tableDifferences);
@@ -264,10 +272,15 @@ export class SchemaComparator {
             if (!this.diffExpression(check.expression, toTableCheck.expression)) {
                 continue;
             }
-            if (fromColumn?.enumItems && toColumn?.enumItems && !this.diffEnumItems(fromColumn.enumItems, toColumn.enumItems)) {
+            if (fromColumn?.enumItems &&
+                toColumn?.enumItems &&
+                !this.diffEnumItems(fromColumn.enumItems, toColumn.enumItems)) {
                 continue;
             }
-            this.log(`check constraint ${check.name} changed in table ${tableDifferences.name}`, { fromTableCheck: check, toTableCheck });
+            this.log(`check constraint ${check.name} changed in table ${tableDifferences.name}`, {
+                fromTableCheck: check,
+                toTableCheck,
+            });
             tableDifferences.changedChecks[check.name] = toTableCheck;
             changes++;
         }
@@ -280,7 +293,10 @@ export class SchemaComparator {
                     delete toForeignKeys[toConstraint.constraintName];
                 }
                 else if (fromConstraint.constraintName.toLowerCase() === toConstraint.constraintName.toLowerCase()) {
-                    this.log(`FK constraint ${fromConstraint.constraintName} changed in table ${tableDifferences.name}`, { fromConstraint, toConstraint });
+                    this.log(`FK constraint ${fromConstraint.constraintName} changed in table ${tableDifferences.name}`, {
+                        fromConstraint,
+                        toConstraint,
+                    });
                     tableDifferences.changedForeignKeys[toConstraint.constraintName] = toConstraint;
                     changes++;
                     delete fromForeignKeys[fromConstraint.constraintName];
@@ -295,7 +311,9 @@ export class SchemaComparator {
         }
         for (const toConstraint of Object.values(toForeignKeys)) {
             tableDifferences.addedForeignKeys[toConstraint.constraintName] = toConstraint;
-            this.log(`FK constraint ${toConstraint.constraintName} added to table ${tableDifferences.name}`, { constraint: toConstraint });
+            this.log(`FK constraint ${toConstraint.constraintName} added to table ${tableDifferences.name}`, {
+                constraint: toConstraint,
+            });
             changes++;
         }
         return changes ? tableDifferences : false;
@@ -341,7 +359,10 @@ export class SchemaComparator {
             tableDifferences.renamedColumns[removedColumnName] = addedColumn;
             delete tableDifferences.addedColumns[addedColumnName];
             delete tableDifferences.removedColumns[removedColumnName];
-            this.log(`renamed column detected in table ${tableDifferences.name}`, { old: removedColumnName, new: addedColumnName });
+            this.log(`renamed column detected in table ${tableDifferences.name}`, {
+                old: removedColumnName,
+                new: addedColumnName,
+            });
         }
     }
     /**
@@ -375,7 +396,10 @@ export class SchemaComparator {
             tableDifferences.renamedIndexes[removedIndexName] = addedIndex;
             delete tableDifferences.addedIndexes[addedIndexName];
             delete tableDifferences.removedIndexes[removedIndexName];
-            this.log(`renamed index detected in table ${tableDifferences.name}`, { old: removedIndexName, new: addedIndexName });
+            this.log(`renamed index detected in table ${tableDifferences.name}`, {
+                old: removedIndexName,
+                new: addedIndexName,
+            });
         }
     }
     diffForeignKey(key1, key2, tableDifferences) {
@@ -402,10 +426,7 @@ export class SchemaComparator {
         }
         const defaultRule = ['restrict', 'no action'];
         const rule = (key, method) => {
-            return (key[method] ?? defaultRule[0])
-                .toLowerCase()
-                .replace(defaultRule[1], defaultRule[0])
-                .replace(/"/g, '');
+            return (key[method] ?? defaultRule[0]).toLowerCase().replace(defaultRule[1], defaultRule[0]).replace(/"/g, '');
         };
         const compare = (method) => rule(key1, method) === rule(key2, method);
         return !compare('updateRule') || !compare('deleteRule');
@@ -418,7 +439,8 @@ export class SchemaComparator {
         const fromProp = this.mapColumnToProperty({ ...fromColumn, autoincrement: false });
         const toProp = this.mapColumnToProperty({ ...toColumn, autoincrement: false });
         const fromColumnType = this.platform.normalizeColumnType(fromColumn.mappedType.getColumnType(fromProp, this.platform).toLowerCase(), fromProp);
-        const fromNativeEnum = fromTable.nativeEnums[fromColumnType] ?? Object.values(fromTable.nativeEnums).find(e => e.name === fromColumnType && e.schema !== '*');
+        const fromNativeEnum = fromTable.nativeEnums[fromColumnType] ??
+            Object.values(fromTable.nativeEnums).find(e => e.name === fromColumnType && e.schema !== '*');
         let toColumnType = this.platform.normalizeColumnType(toColumn.mappedType.getColumnType(toProp, this.platform).toLowerCase(), toProp);
         const log = (msg, params) => {
             if (logging) {
@@ -430,8 +452,11 @@ export class SchemaComparator {
         if (fromColumnType !== toColumnType &&
             (!fromNativeEnum || `${fromNativeEnum.schema}.${fromNativeEnum.name}` !== toColumnType) &&
             !(fromColumn.ignoreSchemaChanges?.includes('type') || toColumn.ignoreSchemaChanges?.includes('type')) &&
-            !fromColumn.generated && !toColumn.generated) {
-            if (!toColumnType.includes('.') && fromTable.schema && fromTable.schema !== this.platform.getDefaultSchemaName()) {
+            !fromColumn.generated &&
+            !toColumn.generated) {
+            if (!toColumnType.includes('.') &&
+                fromTable.schema &&
+                fromTable.schema !== this.platform.getDefaultSchemaName()) {
                 toColumnType = `${fromTable.schema}.${toColumnType}`;
             }
             if (fromColumnType !== toColumnType) {
@@ -455,8 +480,8 @@ export class SchemaComparator {
             log(`'unsigned' changed for column ${fromTable.name}.${fromColumn.name}`, { fromColumn, toColumn });
             changedProperties.add('unsigned');
         }
-        if (!(fromColumn.ignoreSchemaChanges?.includes('default') ||
-            toColumn.ignoreSchemaChanges?.includes('default')) && !this.hasSameDefaultValue(fromColumn, toColumn)) {
+        if (!(fromColumn.ignoreSchemaChanges?.includes('default') || toColumn.ignoreSchemaChanges?.includes('default')) &&
+            !this.hasSameDefaultValue(fromColumn, toColumn)) {
             log(`'default' changed for column ${fromTable.name}.${fromColumn.name}`, { fromColumn, toColumn });
             changedProperties.add('default');
         }
@@ -471,8 +496,7 @@ export class SchemaComparator {
             changedProperties.add('enumItems');
         }
         if ((fromColumn.extra || '').toLowerCase() !== (toColumn.extra || '').toLowerCase() &&
-            !(fromColumn.ignoreSchemaChanges?.includes('extra') ||
-                toColumn.ignoreSchemaChanges?.includes('extra'))) {
+            !(fromColumn.ignoreSchemaChanges?.includes('extra') || toColumn.ignoreSchemaChanges?.includes('extra'))) {
             log(`'extra' changed for column ${fromTable.name}.${fromColumn.name}`, { fromColumn, toColumn });
             changedProperties.add('extra');
         }
@@ -579,7 +603,7 @@ export class SchemaComparator {
             if (sort1 !== sort2) {
                 return false;
             }
-            const defaultNulls = (s) => s === 'DESC' ? 'FIRST' : 'LAST';
+            const defaultNulls = (s) => (s === 'DESC' ? 'FIRST' : 'LAST');
             const nulls1 = c1.nulls?.toUpperCase() ?? defaultNulls(sort1);
             const nulls2 = c2.nulls?.toUpperCase() ?? defaultNulls(sort2);
             if (nulls1 !== nulls2) {
@@ -610,9 +634,9 @@ export class SchemaComparator {
         // expressions like check constraints might be normalized by the driver,
         // e.g. quotes might be added (https://github.com/mikro-orm/mikro-orm/issues/3827)
         const simplify = (str) => {
-            return str
+            return (str
                 ?.replace(/_\w+'(.*?)'/g, '$1')
-                .replace(/in\s*\((.*?)\)/ig, '= any (array[$1])')
+                .replace(/in\s*\((.*?)\)/gi, '= any (array[$1])')
                 // MySQL normalizes count(*) to count(0)
                 .replace(/\bcount\s*\(\s*0\s*\)/gi, 'count(*)')
                 // Remove quotes first so we can process identifiers
@@ -628,7 +652,7 @@ export class SchemaComparator {
                 .replace(/\bas\b/gi, '')
                 // Remove remaining special chars, parentheses, type casts, asterisks, and normalize whitespace
                 .replace(/[()\n[\]*]|::\w+| +/g, '')
-                .replace(/anyarray\[(.*)]/ig, '$1')
+                .replace(/anyarray\[(.*)]/gi, '$1')
                 .toLowerCase()
                 // PostgreSQL adds default aliases to aggregate functions (e.g., count(*) AS count)
                 // After removing AS and whitespace, this results in duplicate adjacent words
@@ -636,7 +660,7 @@ export class SchemaComparator {
                 // Use lookahead to match repeated patterns of 3+ chars (avoid false positives on short sequences)
                 .replace(/(\w{3,})\1/g, '$1')
                 // Remove trailing semicolon (PostgreSQL adds it to view definitions)
-                .replace(/;$/, '');
+                .replace(/;$/, ''));
         };
         return simplify(expr1) !== simplify(expr2);
     }
@@ -645,13 +669,13 @@ export class SchemaComparator {
         if (!defaultValue) {
             return null;
         }
-        const val = defaultValue
-            .replace(/^(_\w+\\)?'(.*?)\\?'$/, '$2')
-            .replace(/^\(?'(.*?)'\)?$/, '$1');
+        const val = defaultValue.replace(/^(_\w+\\)?'(.*?)\\?'$/, '$2').replace(/^\(?'(.*?)'\)?$/, '$1');
         return parseJsonSafe(val);
     }
     hasSameDefaultValue(from, to) {
-        if (from.default == null || from.default.toString().toLowerCase() === 'null' || from.default.toString().startsWith('nextval(')) {
+        if (from.default == null ||
+            from.default.toString().toLowerCase() === 'null' ||
+            from.default.toString().startsWith('nextval(')) {
             return to.default == null || to.default.toLowerCase() === 'null';
         }
         if (to.mappedType instanceof BooleanType) {

package/schema/SchemaHelper.js

@@ -78,7 +78,7 @@ export class SchemaHelper {
         tableName = this.quote(tableName);
         oldColumnName = this.quote(oldColumnName);
         const columnName = this.quote(to.name);
-        const schemaReference = (schemaName !== undefined && schemaName !== 'public') ? ('"' + schemaName + '".') : '';
+        const schemaReference = schemaName !== undefined && schemaName !== 'public' ? '"' + schemaName + '".' : '';
         const tableReference = schemaReference + tableName;
         return `alter table ${tableReference} rename column ${oldColumnName} to ${columnName}`;
     }
@@ -124,7 +124,8 @@ export class SchemaHelper {
      */
     getIndexColumns(index) {
         if (index.columns?.length) {
-            return index.columns.map(col => {
+            return index.columns
+                .map(col => {
                 let colDef = this.quote(col.name);
                 // Collation comes after column name (SQLite syntax: column COLLATE name)
                 if (col.collation) {
@@ -139,7 +140,8 @@ export class SchemaHelper {
                     colDef += ` nulls ${col.nulls}`;
                 }
                 return colDef;
-            }).join(', ');
+            })
+                .join(', ');
         }
         return index.columnNames.map(c => this.quote(c)).join(', ');
     }
@@ -256,9 +258,11 @@ export class SchemaHelper {
         return ret;
     }
     getAddColumnsSQL(table, columns) {
-        const adds = columns.map(column => {
+        const adds = columns
+            .map(column => {
             return `add ${this.createTableColumn(column, table)}`;
-        }).join(', ');
+        })
+            .join(', ');
         return [`alter table ${table.getQuotedName()} ${adds}`];
     }
     getDropColumnsSQL(tableName, columns, schemaName) {
@@ -310,12 +314,26 @@ export class SchemaHelper {
         Utils.runIfNotEmpty(() => col.push('not null'), !column.nullable && !column.generated);
         Utils.runIfNotEmpty(() => col.push('auto_increment'), column.autoincrement);
         Utils.runIfNotEmpty(() => col.push('unique'), column.autoincrement && !column.primary);
-        if (column.autoincrement && !column.generated && !compositePK && (!changedProperties || changedProperties.has('autoincrement') || changedProperties.has('type'))) {
+        if (column.autoincrement &&
+            !column.generated &&
+            !compositePK &&
+            (!changedProperties || changedProperties.has('autoincrement') || changedProperties.has('type'))) {
             Utils.runIfNotEmpty(() => col.push('primary key'), primaryKey && column.primary);
         }
         if (useDefault) {
             // https://dev.mysql.com/doc/refman/9.0/en/data-type-defaults.html
-            const needsExpression = ['blob', 'text', 'json', 'point', 'linestring', 'polygon', 'multipoint', 'multilinestring', 'multipolygon', 'geometrycollection'].some(type => column.type.toLowerCase().startsWith(type));
+            const needsExpression = [
+                'blob',
+                'text',
+                'json',
+                'point',
+                'linestring',
+                'polygon',
+                'multipoint',
+                'multilinestring',
+                'multipolygon',
+                'geometrycollection',
+            ].some(type => column.type.toLowerCase().startsWith(type));
             const defaultSql = needsExpression && !column.default.startsWith('(') ? `(${column.default})` : column.default;
             col.push(`default ${defaultSql}`);
         }
@@ -371,7 +389,9 @@ export class SchemaHelper {
                     columnNames: [fk.column_name],
                     constraintName: fk.constraint_name,
                     localTableName: schemaName ? `${schemaName}.${tableName}` : tableName,
-                    referencedTableName: fk.referenced_schema_name ? `${fk.referenced_schema_name}.${fk.referenced_table_name}` : fk.referenced_table_name,
+                    referencedTableName: fk.referenced_schema_name
+                        ? `${fk.referenced_schema_name}.${fk.referenced_table_name}`
+                        : fk.referenced_table_name,
                     referencedColumnNames: [fk.referenced_column_name],
                     updateRule: fk.update_rule.toLowerCase(),
                     deleteRule: fk.delete_rule.toLowerCase(),

package/schema/SqlSchemaGenerator.js

@@ -128,7 +128,8 @@ export class SqlSchemaGenerator extends AbstractSchemaGenerator {
         }
         const schema = options?.schema ?? this.config.get('schema', this.platform.getDefaultSchemaName());
         for (const meta of this.getOrderedMetadata(schema).reverse()) {
-            await this.driver.createQueryBuilder(meta.class, this.em?.getTransactionContext(), 'write', false)
+            await this.driver
+                .createQueryBuilder(meta.class, this.em?.getTransactionContext(), 'write', false)
                 .withSchema(schema)
                 .truncate()
                 .execute();
@@ -224,8 +225,11 @@ export class SqlSchemaGenerator extends AbstractSchemaGenerator {
         options.dropTables ??= true;
         const toSchema = this.getTargetSchema(options.schema);
         const schemas = toSchema.getNamespaces();
-        const fromSchema = options.fromSchema ?? (await DatabaseSchema.create(this.connection, this.platform, this.config, options.schema, schemas, undefined, this.options.skipTables, this.options.skipViews));
-        const wildcardSchemaTables = [...this.metadata.getAll().values()].filter(meta => meta.schema === '*').map(meta => meta.tableName);
+        const fromSchema = options.fromSchema ??
+            (await DatabaseSchema.create(this.connection, this.platform, this.config, options.schema, schemas, undefined, this.options.skipTables, this.options.skipViews));
+        const wildcardSchemaTables = [...this.metadata.getAll().values()]
+            .filter(meta => meta.schema === '*')
+            .map(meta => meta.tableName);
         fromSchema.prune(options.schema, wildcardSchemaTables);
         toSchema.prune(options.schema, wildcardSchemaTables);
         return { fromSchema, toSchema };
@@ -408,7 +412,11 @@ export class SqlSchemaGenerator extends AbstractSchemaGenerator {
             return;
         }
         const statements = groups.flatMap(group => {
-            return group.join('\n').split(';\n').map(s => s.trim()).filter(s => s);
+            return group
+                .join('\n')
+                .split(';\n')
+                .map(s => s.trim())
+                .filter(s => s);
         });
         await Utils.runSerial(statements, stmt => this.driver.execute(stmt));
     }
@@ -476,9 +484,7 @@ export class SqlSchemaGenerator extends AbstractSchemaGenerator {
                 }
                 // Check if the definition references the other view's name
                 // Use word boundary matching to avoid false positives
-                const patterns = [
-                    new RegExp(`\\b${this.escapeRegExp(otherView.name.toLowerCase())}\\b`),
-                ];
+                const patterns = [new RegExp(`\\b${this.escapeRegExp(otherView.name.toLowerCase())}\\b`)];
                 if (otherView.schema) {
                     patterns.push(new RegExp(`\\b${this.escapeRegExp(`${otherView.schema}.${otherView.name}`.toLowerCase())}\\b`));
                 }