Refactor code generation framework to enhance modularity and database…#9
Merged
MelbourneDeveloper merged 1 commit intoNimblesite:mainfrom Jan 5, 2026
Merged
Conversation
Contributor
Author
|
@MelbourneDeveloper, any update? |
Collaborator
|
Thanks for the logging architecture proposals! We've added you to CONTRIBUTORS.md (3853be7). The detailed implementation plan didn't fit the current direction, but we appreciate the thoughtful work you put into it. |
…-agnostic capabilities. Introduce new interfaces and templates for improved extensibility, along with comprehensive tests for validation and error handling.
MelbourneDeveloper
force-pushed
the
feature/AddLogging
branch
from
fd2ceba to
dc49e55
Compare
MelbourneDeveloper
added a commit
that referenced
this pull request
Jan 5, 2026
MelbourneDeveloper
added a commit
that referenced
this pull request
Apr 7, 2026
#8 record fields now use the column name verbatim (snake_case preserved) instead of forcing PascalCase. The old SQLite.Cli kept literal snake_case field names, so consumer code that referenced rec.user_id etc. was broken when the Postgres.Cli renamed them to rec.UserId. Replaced ToPascalCase(col.Name) with col.Name in all 5 emit sites (record decl, reader read, 4 bulk insert ops). #9 parameter type inference is now schema-aware. Previously InferParameterType only saw the parameter name and returned 'Guid' for any *id parameter, which broke text-id columns (e.g. tokens.id text). Now we pass the metadata columns list and match the parameter name (case-insensitive) to a column; if found, use the column's actual C# type (with the nullable suffix stripped). The name-based fallback now defaults *id to 'string' rather than 'Guid' so text PKs work without column metadata too. Bumps Postgres.Cli to 0.2.3-beta.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
…-agnostic capabilities. Introduce new interfaces and templates for improved extensibility, along with comprehensive tests for validation and error handling.