fix(data): fix Npgsql migrations and suppress PendingModelChangesWarning (#249)

Two bugs caught by Docker Compose sanity check of the PostgreSQL path.

- Populate BuildTargetModel in Npgsql seed migration designer files
  so Npgsql's SQL generator can resolve column types when applying
  InsertData operations (SeedStarting11, SeedSubstitutes)
- Suppress PendingModelChangesWarning for the postgres provider path
  via ConfigureWarnings; hand-crafted designer files cannot replicate
  Npgsql-injected runtime annotations (Relational:MaxIdentifierLength,
  UseIdentityByDefaultColumn), causing a false-positive that aborted
  MigrateAsync() at startup; BuildTargetModel is still populated so
  InsertData SQL generation continues to work correctly

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: nanotaboada

CHANGELOG.md

@@ -62,6 +62,12 @@ This project uses famous football stadiums (A-Z) that hosted FIFA World Cup matc
 
 ### Fixed
 
+- Populate `BuildTargetModel` in Npgsql seed migration designer files so Npgsql's SQL generator can resolve column types when applying `InsertData` operations.
+- Suppress `PendingModelChangesWarning` for the postgres provider path — hand-crafted designer files cannot replicate Npgsql-injected runtime annotations (`Relational:MaxIdentifierLength`, `UseIdentityByDefaultColumn`), causing a false-positive that aborted `MigrateAsync()` at startup.
+- Normalize `DATABASE_PROVIDER` to lowercase in `entrypoint.sh` via `tr` so `POSTGRES`, `Postgres`, etc. are handled consistently with `AddDbContextPool`.
+- Trim and normalize `DATABASE_PROVIDER` in `AddDbContextPool` before the provider switch; add explicit `sqlite`/empty case; throw `InvalidOperationException` for unrecognized values so typos no longer silently fall through to SQLite.
+- Move `Npgsql.EntityFrameworkCore.PostgreSQL` package from the "Development dependencies" `ItemGroup` to "Runtime dependencies".
+
 ### Removed
 
 ---

src/Dotnet.Samples.AspNetCore.WebApi/Extensions/ServiceCollectionExtensions.cs

@@ -9,6 +9,7 @@
 using Dotnet.Samples.AspNetCore.WebApi.Validators;
 using FluentValidation;
 using Microsoft.EntityFrameworkCore;
+using Microsoft.EntityFrameworkCore.Diagnostics;
 using Microsoft.EntityFrameworkCore.Migrations;
 using Microsoft.OpenApi;
 using Serilog;
@@ -35,9 +36,7 @@ IWebHostEnvironment environment
     {
         services.AddDbContextPool<PlayerDbContext>(options =>
         {
-            var provider = (
-                Environment.GetEnvironmentVariable("DATABASE_PROVIDER") ?? ""
-            )
+            var provider = (Environment.GetEnvironmentVariable("DATABASE_PROVIDER") ?? "")
                 .Trim()
                 .ToLowerInvariant();
 
@@ -50,6 +49,13 @@ IWebHostEnvironment environment
                             "DATABASE_URL is required when DATABASE_PROVIDER=postgres."
                         );
                     options.UseNpgsql(connectionString);
+                    // Hand-crafted designer files cannot replicate Npgsql-injected runtime
+                    // annotations (Relational:MaxIdentifierLength, UseIdentityByDefaultColumn),
+                    // causing a false-positive PendingModelChangesWarning. Suppressed here;
+                    // BuildTargetModel is still populated so InsertData SQL generation works.
+                    options.ConfigureWarnings(w =>
+                        w.Ignore(RelationalEventId.PendingModelChangesWarning)
+                    );
                     break;
 
                 case "sqlite":