Skip to content

Round 7: unify model settings and detect embedding dimensions - #130

Merged
Yasou13 merged 7 commits into
mainfrom
mvp/round-7
Oct 26, 2026
Merged

Yasou13 merged 7 commits into
mainfrom
mvp/round-7

Conversation

@Yasou13

@Yasou13 Yasou13 commented Oct 26, 2026 •

Copy link
Copy Markdown
Owner

Round 7 configures extraction and embeddings with MESA_LLM_MODEL and MESA_EMBEDDING_MODEL provider:model values. Removed settings fail startup with migration guidance; Ollama embedding supports automatic dimension detection and preserves the egress fence. Vector-free stores adopt the configured embedding identity, including pending projection lanes, while stores with vectors still require a rebuild. Documentation now describes model-optional writes and BM25 search.

Validation at 9a155e1: final Round 7 checker verdict READY; fresh R7-premerge full gate PASS (lane, lock, sync, lint, mypy, layer imports, 1623 local tests and 7 MCP tests). No additional code changes were required during the pre-merge review.

Yasou13 and others added 7 commits October 25, 2026 21:01
…dentity

P3-03 lets a store with no vector artifact adopt a new embedding model at
startup, but the backfill only moved SKIPPED lanes to the new identity. A
record admitted under the old model whose VECTOR lane was still pending,
retrying or dead-lettered failed every attempt with "embedding identity
mismatch", went DEAD_LETTER and never became searchable (before P3-03 the
restart was refused instead). The startup backfill now aligns every
unfinished VECTOR lane of a mutation without vector artifacts; mutations
with vectors are still never rewritten.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings October 26, 2026 19:22

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota.

@Yasou13
Yasou13 merged commit 64f72f0 into main Oct 26, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants