Repository navigation
Fix NativeAOT vector mapping and metadata serialization - #634
Conversation
HavenDV
left a comment
There was a problem hiding this comment.
Reviewed the complete vector mapping, metadata serialization, sample, and regression-test changes. The contribution preserves the typed API and its POCO serialization behavior while adding an explicit dynamic-record path for NativeAOT.
Validated the combined changes locally on macOS arm64: full solution Release build (zero errors; existing warnings remain), 42 core tests, 8 splitter abstraction tests, and 2 C# splitter tests passed. Published a real NativeAOT Chroma sample with reflection disabled and executed its synthetic-embedding upsert/read/metadata/search smoke against loopback-only Chroma 1.5.9; it passed. No provider endpoint or paid API was invoked.
The reviewed source files are unchanged by the subsequent merges from main. Current-main SQLite follow-up 68e3613 also covers LocalRAG. Awaiting the refreshed required CI check before merging. Existing unrelated LangChain reflection/AOT warnings and advisories are not claims of full-library NativeAOT compatibility.
Fixes #631
Issue: #631
Summary
Validation