@mikro-orm/mssql 7.1.9-dev.8 → 7.1.9

package/MsSqlSchemaGenerator.d.ts

@@ -8,4 +8,5 @@ export declare class MsSqlSchemaGenerator extends SchemaGenerator {
     }): Promise<void>;
     clear(options?: ClearDatabaseOptions): Promise<void>;
     getDropSchemaSQL(options?: Omit<DropSchemaOptions, 'dropDb'>): Promise<string>;
+    protected startsBatch(statement: string): boolean;
 }

package/MsSqlSchemaGenerator.js

@@ -1,4 +1,6 @@
 import { SchemaGenerator } from '@mikro-orm/sql';
+/** MSSQL rejects these unless they are the first statement in a query batch. */
+const BATCH_FIRST_STATEMENT = /^create\s+(or\s+alter\s+)?(trigger|view|proc(edure)?|function|schema)\b/i;
 /** Schema generator with MSSQL-specific behavior for clearing and dropping schemas. */
 export class MsSqlSchemaGenerator extends SchemaGenerator {
     static register(orm) {
@@ -40,4 +42,7 @@ export class MsSqlSchemaGenerator extends SchemaGenerator {
     async getDropSchemaSQL(options = {}) {
         return super.getDropSchemaSQL({ dropForeignKeys: true, ...options });
     }
+    startsBatch(statement) {
+        return BATCH_FIRST_STATEMENT.test(statement);
+    }
 }

package/MsSqlSchemaHelper.d.ts

@@ -67,6 +67,10 @@ export declare class MsSqlSchemaHelper extends SchemaHelper {
     dropViewIfExists(name: string, schema?: string): string;
     getAddColumnsSQL(table: DatabaseTable, columns: Column[]): string[];
     appendComments(table: DatabaseTable): string[];
+    alterTableComment(table: DatabaseTable, comment?: string): string;
+    getChangeColumnCommentSQL(tableName: string, to: Column, schemaName?: string): string;
+    /** Comments are stored as `MS_Description` extended properties, which have separate add/update/drop procedures. */
+    private getCommentSQL;
     inferLengthFromColumnType(type: string): number | undefined;
     protected wrap(val: string | undefined, type: Type<unknown>): string | undefined;
 }

package/MsSqlSchemaHelper.js

@@ -347,7 +347,7 @@ export class MsSqlSchemaHelper extends SchemaHelper {
         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`;
+        return `create trigger ${qualifiedName} on ${table.getQuotedName()} ${timing} ${events} as begin ${this.normalizeTriggerBody(trigger.body)} end`;
     }
     /** Generates SQL to drop an MSSQL trigger. */
     dropTrigger(table, trigger) {
@@ -794,31 +794,49 @@ export class MsSqlSchemaHelper extends SchemaHelper {
             return this.createTableColumn(column, table);
         })
             .join(', ');
-        return [`alter table ${table.getQuotedName()} add ${adds}`];
+        const sql = [`alter table ${table.getQuotedName()} add ${adds}`];
+        for (const column of columns) {
+            if (column.comment) {
+                sql.push(this.getCommentSQL(table.schema, table.name, column.comment, column.name));
+            }
+        }
+        return sql;
     }
     appendComments(table) {
         const sql = [];
-        const schema = this.platform.quoteValue(table.schema);
-        const tableName = this.platform.quoteValue(table.name);
         if (table.comment) {
-            const comment = this.platform.quoteValue(table.comment);
-            sql.push(`if exists(select * from sys.fn_listextendedproperty(N'MS_Description', N'Schema', N${schema}, N'Table', N${tableName}, null, null))
-  exec sys.sp_updateextendedproperty N'MS_Description', N${comment}, N'Schema', N${schema}, N'Table', N${tableName}
-else
-  exec sys.sp_addextendedproperty N'MS_Description', N${comment}, N'Schema', N${schema}, N'Table', N${tableName}`);
+            sql.push(this.getCommentSQL(table.schema, table.name, table.comment));
         }
         for (const column of table.getColumns()) {
             if (column.comment) {
-                const comment = this.platform.quoteValue(column.comment);
-                const columnName = this.platform.quoteValue(column.name);
-                sql.push(`if exists(select * from sys.fn_listextendedproperty(N'MS_Description', N'Schema', N${schema}, N'Table', N${tableName}, N'Column', N${columnName}))
-  exec sys.sp_updateextendedproperty N'MS_Description', N${comment}, N'Schema', N${schema}, N'Table', N${tableName}, N'Column', N${columnName}
-else
-  exec sys.sp_addextendedproperty N'MS_Description', N${comment}, N'Schema', N${schema}, N'Table', N${tableName}, N'Column', N${columnName}`);
+                sql.push(this.getCommentSQL(table.schema, table.name, column.comment, column.name));
             }
         }
         return sql;
     }
+    alterTableComment(table, comment) {
+        return this.getCommentSQL(table.schema, table.name, comment);
+    }
+    getChangeColumnCommentSQL(tableName, to, schemaName) {
+        return this.getCommentSQL(schemaName, tableName, to.comment, to.name);
+    }
+    /** Comments are stored as `MS_Description` extended properties, which have separate add/update/drop procedures. */
+    getCommentSQL(schemaName, tableName, comment, columnName) {
+        const schema = this.platform.quoteValue(schemaName ?? this.platform.getDefaultSchemaName());
+        const table = this.platform.quoteValue(tableName);
+        const level1 = `N'Schema', N${schema}, N'Table', N${table}`;
+        const level2 = columnName ? `N'Column', N${this.platform.quoteValue(columnName)}` : '';
+        const exists = `if exists(select * from sys.fn_listextendedproperty(N'MS_Description', ${level1}, ${level2 || 'null, null'}))`;
+        const target = level2 ? `${level1}, ${level2}` : level1;
+        if (!comment) {
+            return `${exists}\n  exec sys.sp_dropextendedproperty N'MS_Description', ${target}`;
+        }
+        const value = this.platform.quoteValue(comment);
+        return `${exists}
+  exec sys.sp_updateextendedproperty N'MS_Description', N${value}, ${target}
+else
+  exec sys.sp_addextendedproperty N'MS_Description', N${value}, ${target}`;
+    }
     inferLengthFromColumnType(type) {
         const match = /^(\w+)\s*\(\s*(-?\d+|max)\s*\)/.exec(type);
         if (!match) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikro-orm/mssql",
-  "version": "7.1.9-dev.8",
+  "version": "7.1.9",
   "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,17 +47,17 @@
     "copy": "node ../../scripts/copy.mjs"
   },
   "dependencies": {
-    "@mikro-orm/sql": "7.1.9-dev.8",
+    "@mikro-orm/sql": "7.1.9",
     "kysely": "0.29.4",
     "tarn": "3.1.2",
     "tedious": "20.0.0",
     "tsqlstring": "1.0.1"
   },
   "devDependencies": {
-    "@mikro-orm/core": "^7.1.8"
+    "@mikro-orm/core": "^7.1.9"
   },
   "peerDependencies": {
-    "@mikro-orm/core": "7.1.9-dev.8"
+    "@mikro-orm/core": "7.1.9"
   },
   "engines": {
     "node": ">= 22.17.0"