Skip to content

chore: Drop unsupported EF Core provider fallback#19

Merged
mnbuhl merged 2 commits intomainfrom
codex/drop-unsupported-ef-providers
May 4, 2026
Merged

chore: Drop unsupported EF Core provider fallback#19
mnbuhl merged 2 commits intomainfrom
codex/drop-unsupported-ef-providers

Conversation

@mnbuhl
Copy link
Copy Markdown
Owner

@mnbuhl mnbuhl commented May 4, 2026

Summary

Drops the unsupported EF Core provider fallback path so Atomizer.EntityFrameworkCore only supports SQL Server, PostgreSQL, and MySQL.

What changed

  • Removed the public AllowUnsafeProviderFallback option.
  • Added construction-time fail-fast behavior for unsupported EF providers.
  • Removed unsupported fallback execution branches, including the non-option-gated release fallback.
  • Removed public unsupported provider enum values (Oracle, Sqlite, Unknown).
  • Deleted SQLite fallback contract fixtures/tests.
  • Added one narrow SQLite-backed regression test that proves unsupported providers fail at storage construction.

Impact

This is an intentional breaking change for consumers relying on unsupported providers or the fallback option. Supported SQL Server, PostgreSQL, and MySQL paths remain covered.

Remove the unsafe EF Core fallback path so the storage backend has a single explicit provider contract: SQL Server, PostgreSQL, and MySQL. Unsupported providers now fail during storage construction instead of partially working through LINQ fallback paths.

Constraint: Public API breakage was explicitly approved in the PRD

Constraint: SQLite is retained only as a narrow fail-fast regression test dependency

Rejected: Keep internal SQLite fallback for tests | would preserve unsupported provider behavior

Rejected: Method-level unsupported checks | provider-neutral methods could remain accidentally usable

Confidence: high

Scope-risk: moderate

Directive: Do not reintroduce unsupported-provider fallback paths without adding a supported SQL dialect and contract tests

Tested: csharpier check .

Tested: dotnet build

Tested: dotnet test tests/Atomizer.EntityFrameworkCore.Tests/Atomizer.EntityFrameworkCore.Tests.csproj -f net8.0

Tested: dotnet test tests/Atomizer.EntityFrameworkCore.Tests/Atomizer.EntityFrameworkCore.Tests.csproj -f net10.0

Not-tested: net6.0 test execution; Microsoft.NETCore.App 6.0 runtime is not installed locally
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 4, 2026

Code Coverage

Package Line Rate Health
Atomizer 12%
Atomizer.EntityFrameworkCore 89%
Atomizer 72%
Atomizer 72%
Atomizer 12%
Atomizer.EntityFrameworkCore 89%
Atomizer 12%
Atomizer.EntityFrameworkCore 89%
Atomizer 72%
Summary 59% (7179 / 13002)

@mnbuhl mnbuhl marked this pull request as ready for review May 4, 2026 21:32
@mnbuhl mnbuhl changed the title [codex] Drop unsupported EF Core provider fallback chore: Drop unsupported EF Core provider fallback May 4, 2026
@mnbuhl mnbuhl merged commit 9a39f25 into main May 4, 2026
1 check failed
@mnbuhl mnbuhl deleted the codex/drop-unsupported-ef-providers branch May 5, 2026 06:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant