mirror of
https://github.com/tursodatabase/limbo.git
synced 2025-12-23 08:21:09 +00:00
c2a8a6f178
I ended up hitting #1974 today and wanted to fix it. I worked with Claude to generate a more comprehensive set of queries that could fail aside from just the insert query described in the issue. He got most of them right - lots of cases were indeed failing. The ones that were gibberish, he told me I was absolutely right for pointing out they were bad. But alas. With the test cases generated, we can work on fixing it. The place where the assertion was hit, all we need to do there is return true (but we assert that this is indeed a string literal, it shouldn't be anything else at this point). There are then just a couple of places where we need to make sure we handle double quotes correctly. We already tested for single quotes in a couple of places, but never for double quotes. There is one funny corner case where you can just select "col" from tbl, and if there is no column "col" on the table, that is treated as a string literal. We handle that too. Fixes #1974 Closes #2152 |
||
|---|---|---|
| .. | ||
| commands | ||
| config | ||
| docs | ||
| app.rs | ||
| build.rs | ||
| Cargo.toml | ||
| helper.rs | ||
| input.rs | ||
| main.rs | ||
| mcp_server.rs | ||
| opcodes_dictionary.rs | ||
| SQL.sublime-syntax | ||