Add Qdrant vector database REST wrapper#487
Conversation
|
CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Before we can accept your contribution, we ask that you sign the Arakoo Contributor License Agreement. You can sign the CLA by adding a new comment to this pull request and pasting exactly the following text. I have read the Arakoo CLA Document and I hereby sign the CLA You can retrigger this bot by commenting recheck in this Pull Request |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9da691b334
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| config(); | ||
|
|
||
| type QdrantPointId = number | string; | ||
| type QdrantVector = number[] | Record<string, number[]>; |
There was a problem hiding this comment.
Allow named-vector query shape in search args
QdrantVector is reused for searchPoints, but its object branch is Record<string, number[]>, which rejects Qdrant's valid named-vector search payload shape ({ vector: number[], name: string }). In a strict TypeScript project, this makes multi-vector searches fail at compile time unless callers cast to any, effectively breaking a supported Qdrant search mode introduced by this wrapper.
Useful? React with 👍 / 👎.
Summary
Validation
Closes #273