Skip to content

feat: Qdrant RAG provider#1419

Open
Anush008 wants to merge 12 commits intohelixml:mainfrom
Anush008:main
Open

feat: Qdrant RAG provider#1419
Anush008 wants to merge 12 commits intohelixml:mainfrom
Anush008:main

Conversation

@Anush008
Copy link
Copy Markdown

@Anush008 Anush008 commented Dec 4, 2025

Description

This PR adds Qdrant as a RAG provider in Helix.

Testing

I've unit tested the rag_qdrant.go implementation with a local Qdrant instance.

  • You can run Qdrant with
docker run -p 6333:6333 -p 6334:6334 qdrant/qdrant
  • Set RAG_QDRANT_HOST, RAG_QDRANT_PORT to "localhost" and 6334 respectively.

You can view the vectors at http://localhost:6333/dashboard.

Signed-off-by: Anush008 <anushshetty90@gmail.com>
Signed-off-by: Anush008 <anushshetty90@gmail.com>
@Anush008 Anush008 marked this pull request as ready for review December 5, 2025 08:15
@Anush008
Copy link
Copy Markdown
Author

Hey @lukemarsden.
Just bumping this PR. Please take a look when possible.

@chocobar
Copy link
Copy Markdown
Collaborator

Could you add a rag_qdrant_test.go file? All other RAG providers have test suites (see rag_typesense_test.go for the pattern). The tests use testify/suite and should cover Index, Query, and Delete operations.

@chocobar
Copy link
Copy Markdown
Collaborator

Should the default collection name "helix-documents" and search limit 10 be configurable via RAGSettings rather than hardcoded? Other providers seem to allow collection name configuration?

Comment thread api/pkg/rag/rag_qdrant.go
@Anush008
Copy link
Copy Markdown
Author

Should the default collection name "helix-documents" and search limit 10 be configurable via RAGSettings rather than hardcoded? Other providers seem to allow collection name configuration?

They are configurable.

@Anush008
Copy link
Copy Markdown
Author

Included the test suites that run on a local Qdrant instance.

docker run -p 6333:6333 -p 6334:6334 qdrant/qdrant

You can view the vectors at http://localhost:6333/dashboard.

@lukemarsden lukemarsden force-pushed the main branch 2 times, most recently from 93e4998 to c07feb5 Compare December 28, 2025 18:08
@Anush008 Anush008 requested a review from chocobar January 1, 2026 09:28
@Anush008
Copy link
Copy Markdown
Author

Hey @chocobar @lukemarsden
Just bumping this PR. Please review when possible.

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