Repository navigation
deps(sqlite-vec): move the pin from 0.1.7-alpha.2 to the stable 0.1.9 - #181
Merged
Merged
Conversation
`docs/dependency-audit.md` recorded this as a follow-up: the manifest pinned an alpha while a stable release existed upstream. The pin stays exact, because upstream is pre-v1 and `^0.1.9` would resolve `0.1.10-alpha.*`, which is an ANN change we have no reason to take implicitly. Verified on Linux x64: - `npm run typecheck` - clean - `npm test` - 469/469 pass (includes the real-`vec0` scope-filter tests) - `npm run build:unpack && npm run smoke:packaged` - 29 checks pass; the packaged app loads `sqlite-vec-linux-x64/vec0.so` from `app.asar.unpacked` and reports `vec_version()` -> `v0.1.9`; the `vec0` upsert/KNN/delete round trip still works The audit document is updated to the new version, marks the follow-up done, and corrects the recorded smoke-pass count to the current 29.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What
Move the
sqlite-vecpin from0.1.7-alpha.2to the stable0.1.9.This is the follow-up that
docs/dependency-audit.mdalready recorded under thesqlite-vecverdict: the manifest sat on an alpha while a stable release existed upstream.Why an exact pin, not
^0.1.9Upstream is still pre-v1 and explicitly reserves the right to make breaking changes.
^0.1.9resolves to0.1.10-alpha.*, which introduces ANN / DiskANN / IVF work we have no reason to adopt implicitly. The pin is therefore exact, and0.1.9(not0.1.10-alpha.4) is the target.0.1.9also fixes avec0metadata-text-column DELETE bug;vec0brute-force search is unchanged and is still the right shape for a personal knowledge base.Verification (Linux x64, WSL2)
npm run typechecknpm testvec0scope-filter tests (test/vectorFilter.test.tsruns the bundled extension undernode:sqlite)npm run build:unpack && npm run smoke:packagedFrom the packaged run:
So the extension still loads from the unpacked path in the packaged app, and the
vec0lifecycle the store depends on is intact.Docs
docs/dependency-audit.mdis updated: the installed/ships-as/upstream rows, the "alpha question" paragraph, the follow-up marked done, and the recorded smoke-pass count corrected from a stale20to the current29.Not covered here
Only Linux x64 was exercised locally. The macOS/Windows packaged smoke is left to CI (
verify.yml), which is exactly what thesqlite-vec-darwin-*/sqlite-vec-windows-x64platform packages in the lockfile are for.Rollback
Revert the single manifest line and regenerate the lockfile; the extension ships per-platform binaries and the SQLite schema is a
vec0table whose on-disk format is not changed by this bump.sqlite-vec0.1.9 still reports the samevec0capability set, so no re-index is needed.Closes #62 follow-up.