Let AddSqlServer/AddSqlite easy to configure the options from provider

[WeihanLi]

I want to resolve the interceptors from dependency injection using the minimal api, for now, the AddSqlServer/AddSqlite does not support configuring the DbContextOptionsBuilder from the service provider, maybe we could provide an extension for it as additions to #25192.

Currently, we have to do like this

builder.Services.AddDbContext<AppDbContext>((provider, options) =>
{
    options.AddInterceptors(provider.GetRequiredService<AutoUpdateInterceptor>());
    options.UseSqlServer(builder.Configuration.GetConnectionString("Default"));
});

with the new extension, we may simplify it like follows:

builder.Services.AddSqlServer<AppDbContext>(builder.Configuration.GetConnectionString("Default"), 
    _ => { }, (provider, options) => 
    {
        options.AddInterceptors(provider.GetRequiredService<AutoUpdateInterceptor>());
    });

something likes:

public static IServiceCollection AddSqlServer<TContext>(
    this IServiceCollection serviceCollection,
    string? connectionString,
    Action<SqlServerDbContextOptionsBuilder>? sqlServerOptionsAction = null,
    Action<DbContextOptionsBuilder>? optionsAction = null)
    where TContext : DbContext
{
    return serviceCollection.AddSqlServer<TContext>(connectionString, sqlServerOptionsAction, (_, options) => optionsAction?.Invoke(options));
}

public static IServiceCollection AddSqlServer<TContext>(
    this IServiceCollection serviceCollection,
    string? connectionString,
    Action<SqlServerDbContextOptionsBuilder>? sqlServerOptionsAction = null,
    Action<IServiceProvider, DbContextOptionsBuilder>? optionsAction = null)
    where TContext : DbContext
{
    return serviceCollection.AddDbContext<TContext>((Action<IServiceProvider, DbContextOptionsBuilder>) ((provider, options) =>
    {
        optionsAction?.Invoke(provider, options);
        options.UseSqlServer(connectionString, sqlServerOptionsAction);
    }));
}

[ajcvickers]

We will discuss, but, to me, the existing API seems like a good fit:

builder.Services.AddDbContext<AppDbContext>((provider, options) => options
    .AddInterceptors(provider.GetRequiredService<AutoUpdateInterceptor>())
    .UseSqlServer(builder.Configuration.GetConnectionString("Default")));

This should also work:

services.AddDbContext<AppDbContext>((provider, options) => options
    .AddInterceptors(provider.GetRequiredService<AutoUpdateInterceptor>())
    .UseSqlServer("name:Default"));

[WeihanLi]

thanks

[ajcvickers]

Discussed and team agreed not to make this change.