bug: langchain don't return id with documents

[mohamed-em2m]

hi i notice that langchain document don't return id with documents

[RyanCodrai]

Thanks for the report! Fixed in #83.

What was wrong: _search_vector in turbovec/langchain.py was constructing the returned Document objects without passing id=sid, so similarity_search, similarity_search_with_score, and similarity_search_by_vector all returned Document.id = None. The sid was already in scope on the line above (sid = self._u64_to_str[int(handle)]) — just not being threaded into the constructor. The same bug also affected the predicate Document handed to user-supplied filter callables, so id-based filters couldn't dispatch on doc.id.

Fix: one kwarg (id=sid) added in two places in _search_vector.

Regression tests pinned in #83:

• test_similarity_search_results_carry_document_id — all three search variants must return Documents with .id set, for both caller-supplied ids and store-generated UUIDs.
• test_similarity_search_callable_filter_receives_document_id — predicate Document must also carry .id.

Closing — will land when #83 merges.