Skip to content

Gut leftover C# repository runtime - #33

Merged
rmcfadden merged 1 commit into
mainfrom
gut-old-repository-runtime
Oct 13, 2026
Merged

Gut leftover C# repository runtime#33
rmcfadden merged 1 commit into
mainfrom
gut-old-repository-runtime

Conversation

@rmcfadden

@rmcfadden rmcfadden commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Remove Deterministic.Backend Repositories/ and Errors/; Datasources owns persistence.
  • Keep SQL identifier quoting via a standalone InvalidIdentifierException next to SqlIdentifier.
  • Drop unused dialect package versions and retarget the C# README.

LOC (before / after)

Before (main):

path                                                          files   lines
csharp/src/Deterministic.Backend/Repositories                    31    1232
csharp/src/Deterministic.Backend/Errors                           6      93
csharp/tests/Deterministic.Backend.Tests                         11     606
TOTAL                                                            48    1931

After (this branch):

path                                                          files   lines
csharp/src/Deterministic.Backend/Repositories                     0       0
csharp/src/Deterministic.Backend/Errors                           0       0
csharp/tests/Deterministic.Backend.Tests                          2      60
TOTAL                                                             2      60

git diff --stat

53 files changed, 7 insertions(+), 1887 deletions(-)

Test plan

  • dotnet test on csharp/Deterministic.Backend.sln — 13 passed
  • Confirm consumer apps still use Datasources-generated repositories

Made with Cursor

Ryan @cursoragent Ryan
Remove leftover C# repository and error runtime.
55b6282 
Datasources owns persistence; Deterministic.Backend keeps SQL identifier helpers and shared types.

Co-authored-by: Cursor <cursoragent@cursor.com>
@rmcfadden
rmcfadden merged commit 2a3ccbb into main Oct 13, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@rmcfadden