refactor(ef): split EntityFramework into a provider-neutral core + Postgres package

[Mercurial]

What

Splits Argus.Sync.EntityFramework into the EF Core ecosystem's shape — a provider-neutral core plus a per-provider package — and refreshes dependencies along the way. Two commits:

1. chore(deps) — refresh NuGet packages to latest within the net10 line (Microsoft.Extensions.* + EF Core trio → 10.0.9, Npgsql.EFCore → 10.0.2, AspNetCore.OpenApi → 10.0.9, Swashbuckle → 10.2.2, Spectre.Console → 0.57.0, Test.Sdk → 18.6.0, coverlet → 10.0.1). xunit stays on 2.x (no v3 migration); Chrysalis/Utxorpc alpha pins untouched.
2. refactor(ef) — the package split (below).

The split

Package	Role	Depends on
Argus.Sync.EntityFramework	Provider-neutral EF Core backend: CardanoDbContext, the EF unit-of-work, and the new public AddCardanoEntityFrameworkIndexer<T>(configureProvider) seam	Argus.Sync + Microsoft.EntityFrameworkCore.Relational
Argus.Sync.EntityFramework.Postgres (new)	PostgreSQL / Npgsql provider: AddCardanoPostgresIndexer<T> (a thin UseNpgsql wrapper) + the advisory single-instance lock	Argus.Sync.EntityFramework + Npgsql.EntityFrameworkCore.PostgreSQL

AddCardanoPostgresIndexer is now a small wrapper that (1) builds the UseNpgsql delegate, (2) registers the Postgres lock, (3) calls the core seam. Adding SQLite/SQL Server later is a sibling wrapper over the same seam — this lands the structural half of #178.

The advisory-lock worker is renamed PostgresSingleInstanceLockWorker → PostgresSingleInstanceLock to match the sibling MongoSingleInstanceLock (git tracked it as a rename).

⚠️ Breaking

Consumers of AddCardanoPostgresIndexer must now:

• add the Argus.Sync.EntityFramework.Postgres package (it pulls in the core transitively), and
• add using Argus.Sync.EntityFramework.Postgres;.

Acceptable in alpha; suggests a notable bump (e.g. v1.3.0-alpha) at release. The pm2 stability consumers will need the new package on their next bump (not touched here).

Also

• Fix the commandTimout → commandTimeout typo on the public registration.
• Make the core docs provider-neutral ("Postgres schema" → "database schema", etc.).
• Wire the new package into the Example, Tests (IndexerRegistrationTest, SingleInstanceLockTest), the release.yml + ci.yml pack lists, and the README.

Verification

• ✅ dotnet build -c Release — 0 warnings, 0 errors.
• ✅ dotnet format --verify-no-changes — clean.
• ✅ dotnet test --filter "Category!=Integration&Category!=DataGeneration" — 22 passed; IndexerRegistrationTest proves AddCardanoPostgresIndexer still wires the EF seam + renamed lock + worker.
• ✅ dotnet pack both EF packages — nuspecs confirm core → Microsoft.EntityFrameworkCore.Relational (no Npgsql); Postgres → Argus.Sync.EntityFramework + Npgsql.

Out of scope

No new provider (SQLite/etc.) and no generic lease-lock — the remaining half of #178 (each non-Postgres provider needs its own ISingleInstanceLock). The CardanoNodeConnection:RollbackBuffer key is unchanged (consistent with Mongo + the live consumers).

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Splits the Entity Framework backend into a provider-neutral core (Argus.Sync.EntityFramework) plus a new Postgres provider package (Argus.Sync.EntityFramework.Postgres), and updates dependency versions and wiring across examples, tests, CI, and release packaging to match the new package layout.

Changes:

• Introduces AddCardanoEntityFrameworkIndexer<T>(configureProvider) in the core EF package and moves UseNpgsql + advisory-lock wiring into a new Postgres provider package.
• Renames and relocates the Postgres single-instance lock implementation to the new provider package and updates tests/example accordingly.
• Updates docs, workflows, and packing lists to include the new package; refreshes shared NuGet package versions.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 1 comment.

Show a summary per file

File	Description
src/Argus.Sync.Tests/EndToEnd/SingleInstanceLockTest.cs	Updates test to use the new Postgres provider namespace and renamed lock type.
src/Argus.Sync.Tests/EndToEnd/IndexerRegistrationTest.cs	Updates DI assertions for the renamed lock type and provider namespace.
src/Argus.Sync.Tests/Argus.Sync.Tests.csproj	Adds project reference to the new Postgres provider project.
src/Argus.Sync.Example/Program.cs	Switches example registration to Argus.Sync.EntityFramework.Postgres.
src/Argus.Sync.Example/Argus.Sync.Example.csproj	Adds project reference to the new Postgres provider project.
src/Argus.Sync.EntityFramework/EfServiceCollectionExtensions.cs	Replaces Postgres-specific registration with provider-neutral AddCardanoEntityFrameworkIndexer.
src/Argus.Sync.EntityFramework/EfBlockUnitOfWorkFactory.cs	Minor refactor: uses IDbContextTransaction via a namespace import.
src/Argus.Sync.EntityFramework/CardanoDbContext.cs	Documentation wording updated to be provider-neutral.
src/Argus.Sync.EntityFramework/Argus.Sync.EntityFramework.csproj	Removes Npgsql dependency; references Microsoft.EntityFrameworkCore.Relational for provider-neutral core.
src/Argus.Sync.EntityFramework.Postgres/PostgresSingleInstanceLock.cs	Moves/renames the Postgres lock into the provider package namespace.
src/Argus.Sync.EntityFramework.Postgres/PostgresServiceCollectionExtensions.cs	Adds new Postgres-specific DI entry point wrapping the provider-neutral seam.
src/Argus.Sync.EntityFramework.Postgres/Argus.Sync.EntityFramework.Postgres.csproj	Adds new provider package project referencing core + Npgsql EF provider.
README.md	Updates installation and usage instructions to reference the new provider package.
Directory.Packages.props	Refreshes centralized package versions (EF Core, Npgsql EF provider, tooling, etc.).
.github/workflows/release.yml	Adds packing of the new Postgres provider package.
.github/workflows/ci.yml	Includes the new Postgres provider package in CI packing loop.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.