mirror of
https://github.com/asg017/sqlite-vec.git
synced 2026-04-25 00:36:56 +02:00
vec0Update_Delete_ClearMetadata's long-text branch runs a DELETE via sqlite3_step, which returns SQLITE_DONE (101) on success. The code checked for failure but never normalized the success case to SQLITE_OK. The function's epilogue returned SQLITE_DONE as-is, which the caller (vec0Update_Delete) treated as an error, aborting the DELETE scan and silently leaving rows behind. - Normalize rc to SQLITE_OK after successful sqlite3_step in ClearMetadata - Move sqlite3_finalize before the rc check (cleanup on all paths) - Add test_delete_by_metadata_with_long_text regression test - Update test_deletes snapshot (row 3 now correctly deleted) Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com> |
||
|---|---|---|
| .. | ||
| __snapshots__ | ||
| afbd | ||
| correctness | ||
| fuzz | ||
| leak-fixtures | ||
| minimum | ||
| .gitignore | ||
| .python-version | ||
| build.rs | ||
| Cargo.lock | ||
| Cargo.toml | ||
| conftest.py | ||
| fuzz.py | ||
| helpers.py | ||
| pyproject.toml | ||
| skip.test-correctness.py | ||
| sqlite-vec-internal.h | ||
| test-auxiliary.py | ||
| test-general.py | ||
| test-insert-delete.py | ||
| test-knn-distance-constraints.py | ||
| test-loadable.py | ||
| test-metadata.py | ||
| test-partition-keys.py | ||
| test-unit.c | ||
| test-wasm.mjs | ||
| unittest.rs | ||
| utils.py | ||
| uv.lock | ||