Fix the stale scoped-store test literal while aligning the package comment around concrete struct-literal usage.
Co-Authored-By: Virgil <virgil@lethean.io>
Replace the remaining shorthand variable names in the implementation, examples, and supporting docs with explicit names.
Co-Authored-By: Virgil <virgil@lethean.io>
Replace fmt, errors, strings, path/filepath with Core primitives
across 8 files. Keep strings for SplitSeq/FieldsSeq/Builder/Repeat.
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Replace all fmt.Errorf and errors.New calls in production code
(store.go, scope.go) with structured coreerr.E() from go-log.
Sentinel errors (ErrNotFound, ErrQuotaExceeded) are now *log.Err
values; errors.Is() chains continue to work via Unwrap(). Update
four coverage_test.go assertions to match the new op-prefixed
error format from the All() iterator.
Co-Authored-By: Virgil <virgil@lethean.io>
Cover previously unreachable error paths in New(), GetAll(), and Render():
- New: schema conflict when index named "kv" pre-exists
- GetAll: scan error via NULL-key injection after table restructure
- GetAll: rows iteration error via database page corruption
- Render: scan error via same NULL-key technique
- Render: rows iteration error via same corruption technique
GetAll and Render now at 100%. Only 3 stmts remain uncovered in New()
(sql.Open lazy-error and PRAGMA busy_timeout — both genuinely
unreachable without driver mocking).
Co-Authored-By: Charon <charon@lethean.io>
