dal-go / dalgo2sql / 29031139528

89%

Build Type

push

github

Committed by trakhimenok

Commit Message

feat: PlaceholderDialect option (Postgres $N) and bracket-ident stripping

- DbOptions.Placeholder selects positional-parameter style: PlaceholderQuestion
  (default '?', SQLite/MySQL — zero value, fully backward-compatible) or
  PlaceholderDollar ('$1','$2',… for PostgreSQL). getter/setter/deleter/
  updater emit via placeholder(n); buildSingleRecordQuery rewrites the final
  SQL for the dialect.
- emitSQL strips MSSQL-style [table] bracket quoting that
  dal.structuredQuery.String() emits, which both SQLite and PostgreSQL reject.

Both operate on parameterized SQL (values are placeholders, never inline), so
they cannot affect data. Needed by the new dalgo2postgres driver; dalgo2sqlite
suite stays green (backward-compatible).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AhDJhFNWHBf2jhPKEqfL8T

Coverage Stats

49 of 52 new or added lines in 7 files covered. (94.23%)

42 existing lines in 2 files now uncovered.

1190 of 1338 relevant lines covered (88.94%)

1.03 hits per line