dal-go / dalgo2sql / 29027462965

89%

Build Type

push

github

Committed by trakhimenok

Commit Message

feat: map[string]any record data support in Get/GetMulti; contract fixes

- Get/GetMulti with map data issue SELECT * and scan rows generically into
  the map (all columns incl. the PK column; []byte TEXT converted to string)
- not-found errors wrap the key via dal.NewErrNotFoundByKey and satisfy
  dal.IsNotFound
- records reader returns dal.ErrNoMoreRecords (wraps io.EOF) per the
  QueryExecutor contract
- existsSingle no longer leaks *sql.Rows (fixes a write-lock deadlock on
  SQLite when Set follows an existence check)
- modernc.org/sqlite added as a test-only pure-Go driver so the suite runs
  without cgo

Driven by the OpenVaultDB MVP (dalgo2sqlite consumes this for map-shaped
document records).

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

Coverage Stats

109 of 136 new or added lines in 3 files covered. (80.15%)

2 existing lines in 1 file now uncovered.

1150 of 1296 relevant lines covered (88.73%)

1.02 hits per line