feat: add support for unnamed vectors in Qdrant collections #78
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add USE_UNNAMED_VECTORS environment variable to enable unnamed vector support - Create UnnamedVectorProvider wrapper class for embedding providers - Modify QdrantConnector to handle both named and unnamed vector operations - Update README with documentation and examples for unnamed vectors - Support existing Qdrant collections that use default vector fields This allows the MCP server to work with Qdrant databases that were created without specifying vector names, which is common when using embeddings like Google Gemini directly. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
…compatibility
- Add model_config = {'extra': 'ignore'} to all BaseSettings classes
- This ensures environment variables are properly loaded without validation errors
- Add test script to verify unnamed vectors functionality
🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>
- Add __main__.py entry point for module execution - This allows the module to be run with 'python -m mcp_server_qdrant' - Fixes 'No module named mcp_server_qdrant.__main__' error 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Support both 'text' and 'document' fields for content - Use entire payload as metadata if 'metadata' field is not present - Store content in both 'text' and 'document' fields for compatibility - Fixes KeyError when searching collections created by other applications This ensures compatibility with various Qdrant collection schemas, especially those created by applications like Diolog that use 'text' instead of 'document' for content storage. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
- Add COLLECTION_NAMES environment variable for comma-separated list of collections - Update QdrantSettings to parse multiple collection names - Modify tool registration to require collection_name parameter when multiple collections are configured - Update tool descriptions to list available collections - Add documentation for multiple collections usage
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds support for unnamed vectors in Qdrant collections, allowing the MCP server to work with existing Qdrant databases that use the default vector field instead of named vector fields.
Problem
When using Qdrant collections created with unnamed vectors (common when using embeddings like Google Gemini), the MCP server would fail with the error:
Solution
USE_UNNAMED_VECTORSenvironment variable that enables unnamed vector supportUnnamedVectorProviderwrapper class that signals to use unnamed vectorsQdrantConnectorto handle both named and unnamed vector operationsUsage
To use with unnamed vectors, simply set the environment variable:
Testing
The changes maintain backward compatibility - existing users with named vectors will see no change. Only users who explicitly enable
USE_UNNAMED_VECTORS=truewill use the new functionality.Related Issues
This addresses user feedback about incompatibility with existing Qdrant collections that use unnamed vectors.
🤖 Generated with Claude Code