fix(sqlite): normalize column names in ALTER TABLE ADD/DROP COLUMN

[stakeswky]

Fixes #4307

Problem

When or statements use mixed-case column names (e.g., barBaz), the column name was stored in the catalog with its original case. However, queries normalize all unquoted identifiers to lowercase via the identifier() function, causing a mismatch and "column does not exist" errors.

Solution

This fix ensures also normalize column names using identifier(), matching the behavior of and queries. This aligns with SQLite's case-insensitive identifier handling.

Changes

• internal/engine/sqlite/convert.go: Apply identifier() to column names in ALTER TABLE ADD COLUMN and DROP COLUMN handlers (lines 70 and 91)
• internal/engine/sqlite/catalog_test.go: Add regression test for mixed-case column names in ALTER TABLE ADD COLUMN

Testing

All existing tests pass, including the new regression test case that reproduces the issue from #4307.

$ go test ./internal/engine/sqlite/ -run TestUpdate -v
--- PASS: TestUpdate (0.08s)
    --- PASS: TestUpdate/3 (0.01s)  # New mixed-case test

[dosubot]

Related Documentation

Checked 0 published document(s) in 1 knowledge base(s). No updates required.

^{How did I do? Any feedback?}