@mikro-orm/mssql 7.0.17 → 7.0.18-dev.0

package/MsSqlConnection.d.ts

@@ -1,10 +1,12 @@
 import { AbstractSqlConnection, type TransactionEventBroadcaster } from '@mikro-orm/sql';
 import { type ControlledTransaction, MssqlDialect } from 'kysely';
+import { type Routine, type Transaction } from '@mikro-orm/core';
 import type { ConnectionConfiguration } from 'tedious';
 /** Microsoft SQL Server database connection using the `tedious` driver. */
 export declare class MsSqlConnection extends AbstractSqlConnection {
     createKyselyDialect(overrides: ConnectionConfiguration): MssqlDialect;
     private mapOptions;
+    callRoutine<T>(routine: Routine, args?: Record<string, unknown>, ctx?: Transaction): Promise<T>;
     commit(ctx: ControlledTransaction<any, any>, eventBroadcaster?: TransactionEventBroadcaster): Promise<void>;
     protected transformRawResult<T>(res: any, method: 'all' | 'get' | 'run'): T;
 }

package/MsSqlConnection.js

@@ -1,4 +1,4 @@
-import { AbstractSqlConnection, Utils } from '@mikro-orm/sql';
+import { AbstractSqlConnection, DatabaseSchema, Utils, } from '@mikro-orm/sql';
 import { MssqlDialect } from 'kysely';
 import * as Tedious from 'tedious';
 import * as Tarn from 'tarn';
@@ -55,6 +55,53 @@ export class MsSqlConnection extends AbstractSqlConnection {
         }
         return Utils.mergeConfig(ret, overrides);
     }
+    async callRoutine(routine, args = {}, ctx) {
+        if (routine.type === 'function') {
+            return this.callRoutineFunction(routine, args, ctx);
+        }
+        // MSSQL scalar UDF calls must be schema-qualified — `select sql_hash(...)` fails to parse.
+        const schema = routine.schema ?? this.platform.getDefaultSchemaName() ?? 'dbo';
+        const qualified = `${this.platform.quoteIdentifier(schema)}.${this.platform.quoteIdentifier(routine.name)}`;
+        // T-SQL session variables don't persist across execute() calls (different pool connections),
+        // so DECLARE/SET/EXEC/SELECT must go through as a single batch.
+        const declareLines = [];
+        const setLines = [];
+        const setValues = [];
+        const callArgs = [];
+        const inValues = [];
+        const outVars = [];
+        routine.params.forEach((p, i) => {
+            if (p.direction === 'in') {
+                callArgs.push('?');
+                inValues.push(this.convertRoutineInbound(args[p.name], p));
+                return;
+            }
+            const varName = `@_mikro_orm_routine_${i}`;
+            // Logical aliases like `'string'`/`'number'` aren't valid T-SQL types — translate them
+            // through the platform's type system, matching the DDL side in `DatabaseSchema`.
+            const declType = DatabaseSchema.resolveRoutineColumnType(p.type, this.platform);
+            declareLines.push(`declare ${varName} ${declType}`);
+            outVars.push({ name: p.name, varName, param: p });
+            if (p.direction === 'inout') {
+                setLines.push(`set ${varName} = ?`);
+                setValues.push(this.convertRoutineInbound(args[p.name], p));
+            }
+            callArgs.push(`${varName} output`);
+        });
+        const allValues = [...setValues, ...inValues];
+        const batch = [...declareLines, ...setLines, `exec ${qualified} ${callArgs.join(', ')}`];
+        if (outVars.length > 0) {
+            const selectClause = outVars.map(o => `${o.varName} as ${this.platform.quoteIdentifier(o.name)}`).join(', ');
+            batch.push(`select ${selectClause}`);
+        }
+        const result = await this.execute(batch.join('; '), allValues, 'all', ctx);
+        if (outVars.length === 0) {
+            return undefined;
+        }
+        const rows = result;
+        this.applyRoutineOutParams(rows[0] ?? {}, outVars.map(o => o.param), args);
+        return undefined;
+    }
     async commit(ctx, eventBroadcaster) {
         if ('savepointName' in ctx) {
             return;

package/MsSqlMikroORM.d.ts

@@ -2,17 +2,17 @@ import { type AnyEntity, type EntityClass, type EntitySchema, type MikroORM, typ
 import { SqlMikroORM, type SqlEntityManager } from '@mikro-orm/sql';
 import { MsSqlDriver } from './MsSqlDriver.js';
 /** Configuration options for the MSSQL driver. */
-export type MsSqlOptions<EM extends SqlEntityManager<MsSqlDriver> = SqlEntityManager<MsSqlDriver>, Entities extends (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]> = Partial<Options<MsSqlDriver, EM, Entities>>;
+export type MsSqlOptions<EM extends SqlEntityManager<MsSqlDriver> = SqlEntityManager<MsSqlDriver>, Entities extends readonly (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]> = Partial<Options<MsSqlDriver, EM, Entities>>;
 /** Creates a type-safe configuration object for the MSSQL driver. */
-export declare function defineMsSqlConfig<EM extends SqlEntityManager<MsSqlDriver> = SqlEntityManager<MsSqlDriver>, Entities extends (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]>(options: Partial<Options<MsSqlDriver, EM, Entities>>): Partial<Options<MsSqlDriver, EM, Entities>>;
+export declare function defineMsSqlConfig<EM extends SqlEntityManager<MsSqlDriver> = SqlEntityManager<MsSqlDriver>, Entities extends readonly (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]>(options: Partial<Options<MsSqlDriver, EM, Entities>>): Partial<Options<MsSqlDriver, EM, Entities>>;
 /**
  * @inheritDoc
  */
-export declare class MsSqlMikroORM<EM extends SqlEntityManager<MsSqlDriver> = SqlEntityManager<MsSqlDriver>, Entities extends (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]> extends SqlMikroORM<MsSqlDriver, EM, Entities> {
+export declare class MsSqlMikroORM<EM extends SqlEntityManager<MsSqlDriver> = SqlEntityManager<MsSqlDriver>, Entities extends readonly (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]> extends SqlMikroORM<MsSqlDriver, EM, Entities> {
     /**
      * @inheritDoc
      */
-    static init<D extends IDatabaseDriver = MsSqlDriver, EM extends EntityManager<D> = D[typeof EntityManagerType] & EntityManager<D>, Entities extends (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]>(options: Partial<Options<D, EM, Entities>>): Promise<MikroORM<D, EM, Entities>>;
+    static init<D extends IDatabaseDriver = MsSqlDriver, EM extends EntityManager<D> = D[typeof EntityManagerType] & EntityManager<D>, Entities extends readonly (string | EntityClass<AnyEntity> | EntitySchema)[] = (string | EntityClass<AnyEntity> | EntitySchema)[]>(options: Partial<Options<D, EM, Entities>>): Promise<MikroORM<D, EM, Entities>>;
     /**
      * @inheritDoc
      */

package/MsSqlPlatform.d.ts

@@ -8,6 +8,7 @@ export declare class MsSqlPlatform extends AbstractSqlPlatform {
     #private;
     protected readonly schemaHelper: MsSqlSchemaHelper;
     protected readonly exceptionConverter: MsSqlExceptionConverter;
+    formatIndexHint(indexNames: string[]): string;
     /** @inheritDoc */
     lookupExtensions(orm: MikroORM<MsSqlDriver>): void;
     /** @inheritDoc */

package/MsSqlPlatform.js

@@ -16,6 +16,9 @@ export class MsSqlPlatform extends AbstractSqlPlatform {
         bigint: 'bigint',
         boolean: 'bit',
     };
+    formatIndexHint(indexNames) {
+        return `with (index(${indexNames.join(', ')}))`;
+    }
     /** @inheritDoc */
     lookupExtensions(orm) {
         MsSqlSchemaGenerator.register(orm);
@@ -140,6 +143,7 @@ export class MsSqlPlatform extends AbstractSqlPlatform {
         return 'uniqueidentifier';
     }
     validateMetadata(meta) {
+        super.validateMetadata(meta);
         for (const prop of meta.props) {
             if ((prop.runtimeType === 'string' || ['string', 'nvarchar'].includes(prop.type)) &&
                 !['uuid'].includes(prop.type) &&

package/MsSqlSchemaHelper.d.ts

@@ -1,4 +1,4 @@
-import { type AbstractSqlConnection, type CheckDef, type Column, type DatabaseSchema, type DatabaseTable, type Dictionary, type ForeignKey, type IndexDef, SchemaHelper, type Table, type TableDifference, type Transaction, type Type } from '@mikro-orm/sql';
+import { type AbstractSqlConnection, type CheckDef, type Column, type DatabaseSchema, type DatabaseTable, type Dictionary, type ForeignKey, type IndexDef, SchemaHelper, type Table, type TableDifference, type SqlTriggerDef, type SqlRoutineDef, type Transaction, type Type } from '@mikro-orm/sql';
 /** Schema introspection helper for Microsoft SQL Server. */
 export declare class MsSqlSchemaHelper extends SchemaHelper {
     static readonly DEFAULT_VALUES: {
@@ -6,6 +6,8 @@ export declare class MsSqlSchemaHelper extends SchemaHelper {
         false: string[];
         'getdate()': string[];
     };
+    private static readonly AUTO_NOT_NULL_RE;
+    protected get bracketQuotedIdentifiers(): boolean;
     getManagementDbName(): string;
     getDropDatabaseSQL(name: string): string;
     disableForeignKeysSQL(): string;
@@ -23,6 +25,21 @@ export declare class MsSqlSchemaHelper extends SchemaHelper {
     private getEnumDefinitions;
     private getChecksSQL;
     getAllChecks(connection: AbstractSqlConnection, tablesBySchemas: Map<string | undefined, Table[]>, ctx?: Transaction): Promise<Dictionary<CheckDef[]>>;
+    /** Generates SQL to create an MSSQL trigger. MSSQL supports AFTER and INSTEAD OF only. */
+    createTrigger(table: DatabaseTable, trigger: SqlTriggerDef): string;
+    /** Generates SQL to drop an MSSQL trigger. */
+    dropTrigger(table: DatabaseTable, trigger: SqlTriggerDef): string;
+    routineParamReference(name: string): string;
+    /** T-SQL's `OUTPUT` covers both OUT and INOUT; `sys.parameters.is_output` is true for both. */
+    normaliseRoutineParamDirection(direction: 'in' | 'out' | 'inout'): 'in' | 'out' | 'inout';
+    createRoutine(routine: SqlRoutineDef): string;
+    dropRoutine(routine: SqlRoutineDef): string;
+    getAllRoutines(connection: AbstractSqlConnection): Promise<SqlRoutineDef[]>;
+    private getAllRoutineParams;
+    private unwrapMsSqlBody;
+    private getSchemaQualifiedName;
+    getDatabaseCollation(connection: AbstractSqlConnection, ctx?: Transaction): Promise<string | undefined>;
+    getAllTriggers(connection: AbstractSqlConnection, tablesBySchemas: Map<string | undefined, Table[]>): Promise<Dictionary<SqlTriggerDef[]>>;
     loadInformationSchema(schema: DatabaseSchema, connection: AbstractSqlConnection, tables: Table[], schemas?: string[], ctx?: Transaction): Promise<void>;
     getPreAlterTable(tableDiff: TableDifference, safe: boolean): string[];
     getPostAlterTable(tableDiff: TableDifference, safe: boolean): string[];

package/MsSqlSchemaHelper.js

@@ -7,6 +7,16 @@ export class MsSqlSchemaHelper extends SchemaHelper {
         false: ['0'],
         'getdate()': ['current_timestamp'],
     };
+    // `stripAutoNotNullFilter` unwraps balanced per-clause parens before calling `.exec`, so we
+    // only need to match the bare form here — previously the pattern allowed independently
+    // optional leading/trailing parens, which accepted unbalanced strings like `([col] IS NOT NULL`.
+    static AUTO_NOT_NULL_RE = /^\[([^\]]+)\]\s+IS\s+NOT\s+NULL$/i;
+    // MSSQL `filter_definition` and `where` predicates use `[…]` bracket-quoting for identifiers,
+    // so `splitTopLevelAnd` must treat `[` as opening a quoted span (otherwise `[some and col]`
+    // would split mid-identifier).
+    get bracketQuotedIdentifiers() {
+        return true;
+    }
     getManagementDbName() {
         return 'master';
     }
@@ -104,7 +114,8 @@ export class MsSqlSchemaHelper extends SchemaHelper {
       numeric_scale as numeric_scale,
       datetime_precision as datetime_precision,
       character_maximum_length as character_maximum_length,
-      columnproperty(sc.object_id, column_name, 'IsIdentity') is_identity
+      columnproperty(sc.object_id, column_name, 'IsIdentity') is_identity,
+      nullif(ic.collation_name, convert(nvarchar(128), databasepropertyex(db_name(), 'Collation'))) as collation_name
       from information_schema.columns ic
       inner join sys.columns sc on sc.name = ic.column_name and sc.object_id = object_id(ic.table_schema + '.' + ic.table_name)
       left join sys.computed_columns cmp on cmp.name = ic.column_name and cmp.object_id = object_id(ic.table_schema + '.' + ic.table_name)
@@ -156,6 +167,7 @@ export class MsSqlSchemaHelper extends SchemaHelper {
                 precision: col.numeric_precision,
                 scale: col.numeric_scale,
                 comment: col.column_comment,
+                collation: col.collation_name ?? undefined,
                 generated,
             });
         }
@@ -169,6 +181,7 @@ export class MsSqlSchemaHelper extends SchemaHelper {
       col.name as column_name,
       schema_name(t.schema_id) as schema_name,
       (case when filter_definition is not null then concat('where ', filter_definition) else null end) as expression,
+      filter_definition as filter_definition,
       ind.is_disabled as is_disabled,
       ind.type as index_type,
       ind.fill_factor as fill_factor,
@@ -213,15 +226,31 @@ export class MsSqlSchemaHelper extends SchemaHelper {
             if (index.fill_factor > 0) {
                 indexDef.fillFactor = index.fill_factor;
             }
-            if (index.column_name?.match(/[(): ,"'`]/) || index.expression?.match(/where /i)) {
-                indexDef.expression = index.expression; // required for the `getCreateIndexSQL()` call
+            /* v8 ignore next: function-based / computed-column introspection path, same as pre-PR */
+            if (index.column_name?.match(/[(): ,"'`]/)) {
+                indexDef.expression = index.expression;
                 indexDef.expression = this.getCreateIndexSQL(index.table_name, indexDef, !!index.expression);
             }
+            else if (index.filter_definition) {
+                // Auto-NOT-NULL stripping runs post-mapIndexes (needs the consolidated column list).
+                indexDef.where = index.filter_definition;
+            }
             ret[key] ??= [];
             ret[key].push(indexDef);
         }
         for (const key of Object.keys(ret)) {
             ret[key] = await this.mapIndexes(ret[key]);
+            for (const idx of ret[key]) {
+                if (idx.where) {
+                    const stripped = this.stripAutoNotNullFilter(idx.where, idx.columnNames, MsSqlSchemaHelper.AUTO_NOT_NULL_RE);
+                    if (stripped === '') {
+                        delete idx.where;
+                    }
+                    else {
+                        idx.where = stripped;
+                    }
+                }
+            }
         }
         return ret;
     }
@@ -306,6 +335,193 @@ export class MsSqlSchemaHelper extends SchemaHelper {
         }
         return ret;
     }
+    /** Generates SQL to create an MSSQL trigger. MSSQL supports AFTER and INSTEAD OF only. */
+    createTrigger(table, trigger) {
+        if (trigger.expression) {
+            return trigger.expression;
+        }
+        /* v8 ignore next 3 */
+        if (trigger.timing === 'before') {
+            throw new Error(`MSSQL does not support BEFORE triggers. Use AFTER or INSTEAD OF for trigger "${trigger.name}".`);
+        }
+        const timing = trigger.timing.toUpperCase();
+        const events = trigger.events.map(e => e.toUpperCase()).join(', ');
+        const qualifiedName = this.getSchemaQualifiedName(table, trigger.name);
+        return `create trigger ${qualifiedName} on ${table.getQuotedName()} ${timing} ${events} as begin ${trigger.body}; end`;
+    }
+    /** Generates SQL to drop an MSSQL trigger. */
+    dropTrigger(table, trigger) {
+        return `drop trigger if exists ${this.getSchemaQualifiedName(table, trigger.name)}`;
+    }
+    routineParamReference(name) {
+        return `@${name}`;
+    }
+    /** T-SQL's `OUTPUT` covers both OUT and INOUT; `sys.parameters.is_output` is true for both. */
+    normaliseRoutineParamDirection(direction) {
+        return direction === 'out' ? 'inout' : direction;
+    }
+    createRoutine(routine) {
+        if (routine.expression) {
+            return routine.expression;
+        }
+        const qualifiedName = this.qualifiedRoutineName(routine);
+        const params = routine.params
+            .map(p => {
+            const dir = p.direction === 'out' || p.direction === 'inout' ? ' OUTPUT' : '';
+            return `@${p.name} ${p.type}${dir}`;
+        })
+            .join(', ');
+        const body = this.wrapRoutineBody(routine.body ?? '');
+        if (routine.type === 'procedure') {
+            return `create or alter procedure ${qualifiedName} ${params} as ${body}`;
+        }
+        const returnType = routine.returns?.type ?? 'nvarchar(max)';
+        return `create or alter function ${qualifiedName}(${params}) returns ${returnType} as ${body}`;
+    }
+    dropRoutine(routine) {
+        const kind = routine.type === 'procedure' ? 'procedure' : 'function';
+        return `drop ${kind} if exists ${this.qualifiedRoutineName(routine)}`;
+    }
+    async getAllRoutines(connection) {
+        const sql = `
+      select
+        s.name as schema_name,
+        o.name as name,
+        case
+          when o.type = 'P' then 'procedure'
+          when o.type in ('FN', 'IF', 'TF') then 'function'
+        end as kind,
+        m.definition as definition,
+        ep.value as comment
+      from sys.objects o
+      join sys.schemas s on s.schema_id = o.schema_id
+      join sys.sql_modules m on m.object_id = o.object_id
+      left join sys.extended_properties ep
+        on ep.major_id = o.object_id and ep.minor_id = 0 and ep.name = 'MS_Description'
+      where o.type in ('P', 'FN', 'IF', 'TF')
+        and o.is_ms_shipped = 0
+    `;
+        const [rows, paramsAndReturns] = await Promise.all([
+            connection.execute(sql),
+            this.getAllRoutineParams(connection),
+        ]);
+        const { params, returns } = paramsAndReturns;
+        return rows.map(row => ({
+            name: row.name,
+            schema: row.schema_name,
+            type: row.kind,
+            body: this.unwrapMsSqlBody(row.definition),
+            comment: row.comment ?? undefined,
+            params: params.get(`${row.schema_name}.${row.name}`) ?? [],
+            returns: row.kind === 'function'
+                ? (returns.get(`${row.schema_name}.${row.name}`) ?? { type: 'nvarchar(max)', nullable: true })
+                : undefined,
+        }));
+    }
+    async getAllRoutineParams(connection) {
+        // `parameter_id = 0` is the function's return type; positive IDs are formal parameters.
+        const sql = `
+      select
+        s.name as schema_name,
+        o.name as routine_name,
+        p.name as param_name,
+        type_name(p.user_type_id) as type,
+        p.is_output as is_output,
+        p.parameter_id as position
+      from sys.parameters p
+      join sys.objects o on o.object_id = p.object_id
+      join sys.schemas s on s.schema_id = o.schema_id
+      where o.type in ('P', 'FN', 'IF', 'TF')
+        and o.is_ms_shipped = 0
+      order by o.object_id, p.parameter_id
+    `;
+        const rows = await connection.execute(sql);
+        const params = new Map();
+        const returns = new Map();
+        for (const row of rows) {
+            const key = `${row.schema_name}.${row.routine_name}`;
+            if (row.position === 0) {
+                returns.set(key, { type: row.type, nullable: true });
+                continue;
+            }
+            if (!params.has(key)) {
+                params.set(key, []);
+            }
+            // is_output is true for both OUT and INOUT; we always report `inout`. See normaliseRoutineParamDirection.
+            params.get(key).push({
+                name: row.param_name.replace(/^@/, ''),
+                type: row.type,
+                direction: row.is_output ? 'inout' : 'in',
+            });
+        }
+        return { params, returns };
+    }
+    unwrapMsSqlBody(definition) {
+        const asMatch = /\bas\s+([\s\S]*)$/i.exec(definition);
+        return this.stripRoutineBody(asMatch ? asMatch[1] : definition);
+    }
+    getSchemaQualifiedName(table, name) {
+        const defaultSchema = this.platform.getDefaultSchemaName();
+        if (table.schema && table.schema !== defaultSchema) {
+            return `${this.quote(table.schema)}.${this.quote(name)}`;
+        }
+        return this.quote(name);
+    }
+    async getDatabaseCollation(connection, ctx) {
+        const [row] = await connection.execute(`select convert(nvarchar(128), databasepropertyex(db_name(), 'Collation')) as collation`, [], 'all', ctx);
+        return row?.collation;
+    }
+    async getAllTriggers(connection, tablesBySchemas) {
+        const conditions = [];
+        for (const [schema, tables] of tablesBySchemas) {
+            const names = tables.map(t => this.platform.quoteValue(t.table_name)).join(', ');
+            const schemaName = this.platform.quoteValue(schema ?? this.platform.getDefaultSchemaName());
+            conditions.push(`(schema_name(p.schema_id) = ${schemaName} and p.name in (${names}))`);
+        }
+        const sql = `select t.name as trigger_name, schema_name(p.schema_id) as schema_name,
+      p.name as table_name, te.type_desc as event,
+      case when t.is_instead_of_trigger = 1 then 'INSTEAD OF' else 'AFTER' end as timing,
+      object_definition(t.object_id) as definition
+    from sys.triggers t
+    join sys.trigger_events te on t.object_id = te.object_id
+    join sys.objects p on t.parent_id = p.object_id
+    where (${conditions.join(' or ')})
+    order by t.name, te.type_desc`;
+        const allTriggers = await connection.execute(sql);
+        const ret = {};
+        const triggerMap = new Map();
+        for (const row of allTriggers) {
+            const key = this.getTableKey(row);
+            const dedupeKey = `${key}:${row.trigger_name}`;
+            const event = row.event.toLowerCase();
+            if (triggerMap.has(dedupeKey)) {
+                const existing = triggerMap.get(dedupeKey);
+                if (!existing.events.includes(event)) {
+                    existing.events.push(event);
+                }
+                continue;
+            }
+            // Parse body from full trigger definition
+            let body = '';
+            if (row.definition) {
+                const bodyMatch = /\bas\s+begin\s+([\s\S]*)\s*end\s*;?\s*$/i.exec(row.definition);
+                if (bodyMatch) {
+                    body = bodyMatch[1].trim().replace(/;\s*$/, '');
+                }
+            }
+            ret[key] ??= [];
+            const trigger = {
+                name: row.trigger_name,
+                timing: row.timing.toLowerCase(),
+                events: [event],
+                forEach: 'row', // MSSQL has no FOR EACH ROW/STATEMENT syntax; match the metadata default to avoid false diffs
+                body,
+            };
+            ret[key].push(trigger);
+            triggerMap.set(dedupeKey, trigger);
+        }
+        return ret;
+    }
     async loadInformationSchema(schema, connection, tables, schemas, ctx) {
         if (tables.length === 0) {
             return;
@@ -315,12 +531,18 @@ export class MsSqlSchemaHelper extends SchemaHelper {
         const indexes = await this.getAllIndexes(connection, tablesBySchema, ctx);
         const checks = await this.getAllChecks(connection, tablesBySchema, ctx);
         const fks = await this.getAllForeignKeys(connection, tablesBySchema, ctx);
+        const triggers = await this.getAllTriggers(connection, tablesBySchema);
+        const dbCollation = await this.getDatabaseCollation(connection, ctx);
         for (const t of tables) {
             const key = this.getTableKey(t);
             const table = schema.addTable(t.table_name, t.schema_name, t.table_comment);
+            table.collation = dbCollation;
             const pks = await this.getPrimaryKeys(connection, indexes[key], table.name, table.schema);
             const enums = this.getEnumDefinitions(checks[key] ?? []);
             table.init(columns[key], indexes[key], checks[key], pks, fks[key], enums);
+            if (triggers[key]) {
+                table.setTriggers(triggers[key]);
+            }
         }
     }
     getPreAlterTable(tableDiff, safe) {
@@ -436,7 +658,11 @@ export class MsSqlSchemaHelper extends SchemaHelper {
         else {
             col.push(columnType);
         }
-        Utils.runIfNotEmpty(() => col.push('identity(1,1)'), column.autoincrement);
+        Utils.runIfNotEmpty(() => col.push(this.getCollateSQL(column.collation)), column.collation);
+        // `IDENTITY(1,1)` is rejected inside `ALTER COLUMN`, so it must only be emitted when the
+        // change actually involves the identity attribute or is a fresh column (no `changedProperties`).
+        Utils.runIfNotEmpty(() => col.push('identity(1,1)'), column.autoincrement &&
+            (!changedProperties || changedProperties.has('autoincrement') || changedProperties.has('type')));
         Utils.runIfNotEmpty(() => col.push('null'), column.nullable);
         Utils.runIfNotEmpty(() => col.push('not null'), !column.nullable && !column.generated);
         if (column.autoincrement &&
@@ -458,7 +684,7 @@ export class MsSqlSchemaHelper extends SchemaHelper {
             const [constraint] = this.getDropDefaultsSQL(table.name, [column], table.schema);
             parts.push(constraint);
         }
-        if (changedProperties.has('type') || changedProperties.has('nullable')) {
+        if (changedProperties.has('type') || changedProperties.has('nullable') || changedProperties.has('collation')) {
             const col = this.createTableColumn(column, table, changedProperties);
             parts.push(`alter table ${table.getQuotedName()} alter column ${col}`);
         }
@@ -481,7 +707,7 @@ export class MsSqlSchemaHelper extends SchemaHelper {
         const clustered = index.clustered ? 'clustered ' : '';
         let sql = `create ${index.unique ? 'unique ' : ''}${clustered}index ${keyName} on ${this.quote(tableName)} `;
         if (index.expression && partialExpression) {
-            return sql + `(${index.expression})` + this.getMsSqlIndexSuffix(index);
+            return sql + `(${index.expression})` + this.getMsSqlIndexSuffix(index) + this.getIndexWhereClause(index);
         }
         // Build column list with advanced options
         const columns = this.getIndexColumns(index);
@@ -490,7 +716,7 @@ export class MsSqlSchemaHelper extends SchemaHelper {
         if (index.include?.length) {
             sql += ` include (${index.include.map(c => this.quote(c)).join(', ')})`;
         }
-        sql += this.getMsSqlIndexSuffix(index);
+        sql += this.getMsSqlIndexSuffix(index) + this.getIndexWhereClause(index);
         // Disabled indexes need to be created first, then disabled
         if (index.disabled) {
             sql += `;\nalter index ${keyName} on ${this.quote(tableName)} disable`;
@@ -533,13 +759,16 @@ export class MsSqlSchemaHelper extends SchemaHelper {
         if (index.expression) {
             return index.expression;
         }
-        const needsWhereClause = index.unique && index.columnNames.some(column => table.getColumn(column)?.nullable);
-        if (!needsWhereClause) {
+        const needsAutoNotNull = index.unique && index.columnNames.some(column => table.getColumn(column)?.nullable);
+        if (!needsAutoNotNull) {
             return this.getCreateIndexSQL(table.getShortestName(), index);
         }
-        // Generate without disabled suffix, insert WHERE clause, then re-add disabled
-        let sql = this.getCreateIndexSQL(table.getShortestName(), { ...index, disabled: false });
-        sql += ' where ' + index.columnNames.map(c => `${this.quote(c)} is not null`).join(' and ');
+        // Strip `index.where` from the base SQL so we can combine it with the auto NOT-NULL guard
+        // ourselves, wrapping the user predicate in parens to defuse operator precedence
+        // (a bare `a = 1 or b = 2 and [col] is not null` would bind as `a = 1 or (b = 2 and …)`).
+        let sql = this.getCreateIndexSQL(table.getShortestName(), { ...index, where: undefined, disabled: false });
+        const autoNotNull = index.columnNames.map(c => `${this.quote(c)} is not null`).join(' and ');
+        sql += index.where ? ` where (${index.where}) and ${autoNotNull}` : ` where ${autoNotNull}`;
         if (index.disabled) {
             sql += `;\nalter index ${this.quote(index.keyName)} on ${table.getQuotedName()} disable`;
         }

package/README.md

@@ -2,7 +2,7 @@
   <a href="https://mikro-orm.io"><img src="https://raw.githubusercontent.com/mikro-orm/mikro-orm/master/docs/static/img/logo-readme.svg?sanitize=true" alt="MikroORM" /></a>
 </h1>
 
-TypeScript ORM for Node.js based on Data Mapper, [Unit of Work](https://mikro-orm.io/docs/unit-of-work/) and [Identity Map](https://mikro-orm.io/docs/identity-map/) patterns. Supports MongoDB, MySQL, MariaDB, PostgreSQL, SQLite (including libSQL), MSSQL and Oracle databases.
+TypeScript ORM for Node.js based on Data Mapper, [Unit of Work](https://mikro-orm.io/docs/unit-of-work/) and [Identity Map](https://mikro-orm.io/docs/identity-map/) patterns. Supports MongoDB, MySQL, MariaDB, PostgreSQL (including CockroachDB and PGlite), SQLite (including libSQL), MSSQL and Oracle databases.
 
 > Heavily inspired by [Doctrine](https://www.doctrine-project.org/) and [Hibernate](https://hibernate.org/).
 
@@ -19,6 +19,7 @@ Install a driver package for your database:
 
 ```sh
 npm install @mikro-orm/postgresql   # PostgreSQL
+npm install @mikro-orm/pglite       # PGlite (embedded PostgreSQL in WASM)
 npm install @mikro-orm/mysql        # MySQL
 npm install @mikro-orm/mariadb      # MariaDB
 npm install @mikro-orm/sqlite       # SQLite

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/mssql",
-  "version": "7.0.17",
+  "version": "7.0.18-dev.0",
   "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",
@@ -47,7 +47,7 @@
     "copy": "node ../../scripts/copy.mjs"
   },
   "dependencies": {
-    "@mikro-orm/sql": "7.0.17",
+    "@mikro-orm/sql": "7.0.18-dev.0",
     "kysely": "0.29.2",
     "tarn": "3.0.2",
     "tedious": "19.2.1",
@@ -57,7 +57,7 @@
     "@mikro-orm/core": "^7.0.17"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.0.17"
+    "@mikro-orm/core": "7.0.18-dev.0"
   },
   "engines": {
     "node": ">= 22.17.0"