Fixing update script for schema (#2463)

* Add DefaultValue property to SchemaDesignerColumn and update related logic

* Refactor SchemaDesigner session handling and update method calls for clarity

* Improve session disposal handling and validate connection in SchemaDesignerSession constructor

* Refactor SchemaDesigner session handling and improve report generation logic

* Update DacFx package version to 170.0.81 and comment out unused code in SqlProjectsService

* Adding ref to latest sql proj nuget to get rid of compile errors

* Fix handling of NULL default values in SchemaDesignerModelProvider

* wip sd

* Implement enhancements to improve performance in data processing

* Remove UpdateScript property from SchemaDesignerReportObject

* Rename SchemaDesignerReportObject to SchemaDesignerChangeReport and update references in GetReportResponse

* Refactor SchemaDesigner: remove EventSchemaReady, update column types, and streamline session handling

* Sort available schemas and data types for improved organization

* Remove obsolete NuGet packages: Microsoft.SqlServer.DacFx, Microsoft.SqlServer.DacFx.Projects, and Microsoft.SqlServer.DacFx.Tools

* Refactor SqlProjectsService: update CreateSqlProjectParams and TargetPlatform utility references

* Refactor SchemaDesignerSession: simplify schema designer initialization logic

Author: aasimkhan30

src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/Contracts/EventSchemaReady.cs

@@ -25,7 +25,7 @@ public class GetReportRequest
     /// </summary>
     public class GetReportResponse
     {
-        public List<SchemaDesignerReportObject>? Reports { get; set; }
+        public List<SchemaDesignerChangeReport>? Reports { get; set; }
         public string? UpdateScript { get; set; }
     }
 

src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/Contracts/RequestGetReport.cs

@@ -27,7 +27,7 @@ public class SchemaDesignerColumn
         /// <summary>
         /// Gets or sets the max length of the column
         /// </summary>
-        public int? MaxLength { get; set; }
+        public string? MaxLength { get; set; }
         /// <summary>
         /// Gets or sets the precision of the column
         /// </summary>
@@ -47,22 +47,18 @@ public class SchemaDesignerColumn
         /// <summary>
         /// Gets or sets identity seed of the column
         /// </summary>
-        public int? IdentitySeed { get; set; }
+        public decimal? IdentitySeed { get; set; }
         /// <summary>
         /// Gets or sets identity increment of the column
         /// </summary>
-        public int? IdentityIncrement { get; set; }
+        public decimal? IdentityIncrement { get; set; }
         /// <summary>
         /// Gets or sets if the column is a nullable column
         /// </summary>
         public bool IsNullable { get; set; }
         /// <summary>
-        /// Get or sets if the column is has unique constraint
-        /// </summary>
-        public bool IsUnique { get; set; }
-        /// <summary>
         /// Gets or sets the default value of the column
         /// </summary>
-        public string Collation { get; set; }
+        public string? DefaultValue { get; set; }
     }
 }

src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/Contracts/SchemaDesignerColumn.cs

@@ -8,11 +8,10 @@
 
 namespace Microsoft.SqlTools.ServiceLayer.SchemaDesigner
 {
-    public class SchemaDesignerReportObject
+    public class SchemaDesignerChangeReport
     {
         public Guid? TableId { get; set; }
         public string TableName { get; set; } = string.Empty;
-        public string UpdateScript { get; set; } = string.Empty;
         public SchemaDesignerReportTableState TableState { get; set; } = SchemaDesignerReportTableState.CREATED;
         public List<string> ActionsPerformed { get; set; } = new List<string>();
     }

src/Microsoft.SqlTools.ServiceLayer/SchemaDesigner/Contracts/SchemaDesignerReportObject.cs

@@ -105,24 +105,9 @@ public static string GenerateColumnDefinition(SchemaDesignerColumn column)
             StringBuilder sb = new StringBuilder();
             sb.Append($"[{column.Name}] {column.DataType}");
 
-            // Handle length specification for applicable data types
-            if (column.MaxLength.HasValue && column.MaxLength != -1)
+            if (column.MaxLength != null && column.MaxLength == "0")
             {
-                if (IsLengthBasedType(column.DataType))
-                {
-                    if (IsBytePairDatatype(column.DataType))
-                    {
-                        sb.Append($"({column.MaxLength / 2})");
-                    }
-                    else
-                    {
-                        sb.Append($"({column.MaxLength})");
-                    }
-                }
-            }
-            else if (column.MaxLength == -1 && IsLengthBasedType(column.DataType))
-            {
-                sb.Append("(MAX)");
+                sb.Append("(0)");
             }
 
             // Handle precision and scale only for decimal/numeric types
@@ -138,22 +123,11 @@ public static string GenerateColumnDefinition(SchemaDesignerColumn column)
                 }
             }
 
-            if (!string.IsNullOrEmpty(column.Collation) && column.Collation != "NULL")
-            {
-                sb.Append($" COLLATE {column.Collation}");
-            }
-
-
             if (!column.IsNullable)
             {
                 sb.Append(" NOT NULL");
             }
 
-            if (column.IsUnique)
-            {
-                sb.Append(" UNIQUE");
-            }
-
             if (column.IsIdentity)
             {
                 sb.Append($" IDENTITY({column.IdentitySeed},{column.IdentityIncrement})");