Skip to content

fix(di): share store singletons and preserve registrations - #622

Open
nmummau wants to merge 1 commit into
Eventuous:devfrom
nmummau:fix/issue-608-store-registrations
Open

nmummau wants to merge 1 commit into
Eventuous:devfrom
nmummau:fix/issue-608-store-registrations

Conversation

@nmummau

@nmummau nmummau commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Map reader, writer, and store interfaces to the same registered singleton regardless of tracing. Preserve existing services and avoid duplicate traced store registrations.

Host-based regressions reproduced 16 failures before the fix. All 31 DI tests now pass on net10.0; Stores.cs line coverage is 78/78.

Fixes #608
Fixes #610
Fixes #611

Map reader, writer, and store interfaces to the same registered singleton regardless of tracing. Preserve existing services and avoid duplicate traced store registrations.

Host-based regressions reproduced 16 failures before the fix. All 31 DI tests now pass on net10.0; Stores.cs line coverage is 78/78.

Fixes Eventuous#608
@github-actions

Copy link
Copy Markdown

Test Results

   48 files  ± 0     48 suites  ±0   14m 7s ⏱️ - 1m 32s
  645 tests +28    645 ✅ +28  0 💤 ±0  0 ❌ ±0 
1 318 runs  +84  1 318 ✅ +84  0 💤 ±0  0 ❌ ±0 

Results for commit f3fe99e. ± Comparison against base commit 60a7643.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant