Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: move @mastra/vector-qdrant to @mastra/qdrant #1782

Merged
merged 5 commits into from
Feb 7, 2025

Conversation

TylerBarnes
Copy link
Contributor

@TylerBarnes TylerBarnes commented Feb 7, 2025

Generated by windsurf:

Move @mastra/vector-qdrant to @mastra/qdrant

This PR moves the Qdrant vector store package to a new structure, following the same pattern as the Pinecone package move.

Changes

New Package (@mastra/qdrant)

  • Created in stores/qdrant
  • Moved source files to src/vector directory
  • Added new src/index.ts that re-exports everything
  • Updated package name and installation instructions
  • Added comprehensive CHANGELOG with migration guide

Old Package (@mastra/vector-qdrant)

  • Added deprecation error message
  • Created new README with deprecation notice
  • Removed test files and simplified package.json
  • Package remains for reference during transition

Documentation

  • Updated import paths in documentation and examples
  • Added migration instructions in both packages
  • Maintained old package reference in vector-stores README

CI/CD

  • Added Qdrant service to test-combined-stores.yml
  • Updated workspace dependencies

Testing

  • Verified no remaining references to old package in examples
  • Verified no remaining references in packages/integrations
  • Updated all documentation references
  • CI configuration is updated

Migration Guide for Users

  1. Remove old package:

    pnpm remove @mastra/vector-qdrant
  2. Install new package:

    pnpm add @mastra/qdrant
  3. Update imports:

    // Old import
    import { QdrantVector } from '@mastra/vector-qdrant';
    // New import
    import { QdrantVector } from '@mastra/qdrant';

No breaking changes in functionality.

Your Name added 5 commits February 7, 2025 14:03
- Copied package from vector-stores/qdrant to stores/qdrant
- Moved source files to src/vector directory
- Added new src/index.ts that re-exports from vector/index.ts
- Updated README to use pnpm for installation
- Added comprehensive CHANGELOG with migration guide and history
- Updated import paths in documentation and examples
- Added deprecation error to index.ts
- Updated package.json with deprecation notice
- Simplified scripts (removed test/dev)
- Removed test files and configs
- Added deprecation README
- Added Qdrant service to test-combined-stores.yml
- Updated package name in stores/qdrant/package.json
- Updated workspace dependencies
Copy link

changeset-bot bot commented Feb 7, 2025

⚠️ No Changeset found

Latest commit: c96a32c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Copy link

vercel bot commented Feb 7, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
assistant-ui ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 10:17pm
bird-checker ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 10:17pm
crypto-chatbot ❌ Failed (Inspect) Feb 7, 2025 10:17pm
mastra ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 10:17pm
mastra-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 10:17pm
openapi-spec-writer 🔄 Building (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 10:17pm
travel-ai ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 7, 2025 10:17pm

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