Filter out full-text catalog operations from history table creation

When the migration history table is created, the model differ generates
AlterDatabaseOperation with full-text catalog annotations. The SQL generator
then emits CREATE FULLTEXT CATALOG too early, causing the later actual
migration to fail.

Filter out SqlServerAnnotationNames.FullTextCatalogs from
AlterDatabaseOperation in SqlServerHistoryRepository.GetCreateCommands(),
following the same approach as npgsql/efcore.pg#3713.

This is a workaround for #34991.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: roji

src/EFCore.Relational/Migrations/HistoryRepository.cs

@@ -85,7 +85,14 @@ protected virtual string MigrationIdColumnName
             .FindProperty(nameof(HistoryRow.MigrationId))!
             .GetColumnName();
 
-    private IModel EnsureModel()
+    /// <summary>
+    ///     This is an internal API that supports the Entity Framework Core infrastructure and not subject to
+    ///     the same compatibility standards as public APIs. It may be changed or removed without notice in
+    ///     any release. You should only use it directly in your code with extreme caution and knowing that
+    ///     doing so can result in application failures when updating to a new Entity Framework Core release.
+    /// </summary>
+    [EntityFrameworkInternal]
+    protected IModel EnsureModel()
     {
         if (_model == null)
         {

src/EFCore.SqlServer/Migrations/Internal/SqlServerHistoryRepository.cs

@@ -2,6 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 using System.Text;
+using Microsoft.EntityFrameworkCore.SqlServer.Metadata.Internal;
 
 namespace Microsoft.EntityFrameworkCore.SqlServer.Migrations.Internal;
 
@@ -24,6 +25,36 @@ public SqlServerHistoryRepository(HistoryRepositoryDependencies dependencies)
     {
     }
 
+    /// <summary>
+    ///     This is an internal API that supports the Entity Framework Core infrastructure and not subject to
+    ///     the same compatibility standards as public APIs. It may be changed or removed without notice in
+    ///     any release. You should only use it directly in your code with extreme caution and knowing that
+    ///     doing so can result in application failures when updating to a new Entity Framework Core release.
+    /// </summary>
+    protected override IReadOnlyList<MigrationCommand> GetCreateCommands()
+    {
+        // TODO: This is a hack around https://github.com/dotnet/efcore/issues/34991: provider-specific conventions may add
+        // database-level annotations (e.g. full-text catalogs) to the model, and the default EF logic causes them to be created
+        // at this point, when the history table is being created. This is too early, and causes the later actual migration to fail.
+        // So we filter out full-text catalog annotations from AlterDatabaseOperation.
+        // This follows the same approach as the Npgsql provider (npgsql/efcore.pg#3713).
+#pragma warning disable EF1001 // Internal EF Core API usage.
+        var model = EnsureModel();
+#pragma warning restore EF1001 // Internal EF Core API usage.
+
+        var operations = Dependencies.ModelDiffer.GetDifferences(null, model.GetRelationalModel());
+
+        foreach (var operation in operations)
+        {
+            if (operation is AlterDatabaseOperation alterDatabaseOperation)
+            {
+                alterDatabaseOperation.RemoveAnnotation(SqlServerAnnotationNames.FullTextCatalogs);
+            }
+        }
+
+        return Dependencies.MigrationsSqlGenerator.Generate(operations, model);
+    }
+
     /// <summary>
     ///     This is an internal API that supports the Entity Framework Core infrastructure and not subject to
     ///     the same compatibility standards as public APIs. It may be changed or removed without notice in

test/EFCore.SqlServer.Tests/Migrations/SqlServerHistoryRepositoryTest.cs

@@ -21,6 +21,22 @@ [ProductVersion] nvarchar(32) NOT NULL,
     CONSTRAINT [PK___EFMigrationsHistory] PRIMARY KEY ([MigrationId])
 );
 
+""", sql, ignoreLineEndingDifferences: true);
+    }
+
+    [ConditionalFact]
+    public void GetCreateScript_works_with_full_text_catalog()
+    {
+        var sql = CreateHistoryRepository(configureModel: b => b.HasFullTextCatalog("MyCatalog")).GetCreateScript();
+
+        Assert.Equal(
+            """
+CREATE TABLE [__EFMigrationsHistory] (
+    [MigrationId] nvarchar(150) NOT NULL,
+    [ProductVersion] nvarchar(32) NOT NULL,
+    CONSTRAINT [PK___EFMigrationsHistory] PRIMARY KEY ([MigrationId])
+);
+
 """, sql, ignoreLineEndingDifferences: true);
     }
 
@@ -149,17 +165,18 @@ public void GetEndIfScript_works()
 """, sql, ignoreLineEndingDifferences: true);
     }
 
-    private static IHistoryRepository CreateHistoryRepository(string schema = null)
+    private static IHistoryRepository CreateHistoryRepository(string schema = null, Action<ModelBuilder> configureModel = null)
         => new TestDbContext(
                 new DbContextOptionsBuilder()
                     .UseInternalServiceProvider(SqlServerTestHelpers.Instance.CreateServiceProvider())
                     .UseSqlServer(
                         new SqlConnection("Database=DummyDatabase"),
                         b => b.MigrationsHistoryTable(HistoryRepository.DefaultTableName, schema))
-                    .Options)
+                    .Options,
+                configureModel)
             .GetService<IHistoryRepository>();
 
-    private class TestDbContext(DbContextOptions options) : DbContext(options)
+    private class TestDbContext(DbContextOptions options, Action<ModelBuilder> configureModel = null) : DbContext(options)
     {
         public DbSet<Blog> Blogs { get; set; }
 
@@ -169,6 +186,7 @@ public IQueryable<TableFunction> TableFunction()
 
         protected override void OnModelCreating(ModelBuilder modelBuilder)
         {
+            configureModel?.Invoke(modelBuilder);
         }
     }