[PR #4787] [MERGED] VectorDB class migration #4773

Closed
opened 2026-02-22 18:36:30 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/Mintplex-Labs/anything-llm/pull/4787
Author: @shatfield4
Created: 12/15/2025
Status: Merged
Merged: 1/13/2026
Merged by: @timothycarambat

Base: masterHead: vectordb-class-migration


📝 Commits (10+)

📊 Changes

14 files changed (+892 additions, -860 deletions)

View changed files

📝 .github/workflows/dev-build.yaml (+1 -1)
📝 server/__tests__/utils/vectorDbProviders/pgvector/index.test.js (+3 -1)
📝 server/utils/helpers/index.js (+11 -11)
📝 server/utils/vectorDbProviders/astra/index.js (+74 -52)
server/utils/vectorDbProviders/base.js (+201 -0)
📝 server/utils/vectorDbProviders/chroma/index.js (+79 -52)
📝 server/utils/vectorDbProviders/chromacloud/index.js (+27 -18)
📝 server/utils/vectorDbProviders/lance/index.js (+88 -58)
📝 server/utils/vectorDbProviders/milvus/index.js (+73 -49)
📝 server/utils/vectorDbProviders/pgvector/index.js (+109 -98)
📝 server/utils/vectorDbProviders/pinecone/index.js (+57 -36)
📝 server/utils/vectorDbProviders/qdrant/index.js (+67 -43)
📝 server/utils/vectorDbProviders/weaviate/index.js (+81 -54)
📝 server/utils/vectorDbProviders/zilliz/index.js (+21 -387)

📄 Description

Pull Request Type

  • feat
  • 🐛 fix
  • ♻️ refactor
  • 💄 style
  • 🔨 chore
  • 📝 docs

Relevant Issues

resolves #4687

What is in this change?

  • Refactors all vector databases to use classes instead of the object structure that was previously used
  • Creates VectorDatabase base class that is extended onto all vector database providers to ensure all required functions are declared
  • Refactors Zilliz Cloud provider to extend the Milvus class helping reduce code complexity since they have identical functions for interacting with the vector database

Additional Information

Developer Validations

  • I ran yarn lint from the root of the repo & committed changes
  • Relevant documentation has been updated
  • I have tested my code functionality
  • Docker build succeeds locally

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/Mintplex-Labs/anything-llm/pull/4787 **Author:** [@shatfield4](https://github.com/shatfield4) **Created:** 12/15/2025 **Status:** ✅ Merged **Merged:** 1/13/2026 **Merged by:** [@timothycarambat](https://github.com/timothycarambat) **Base:** `master` ← **Head:** `vectordb-class-migration` --- ### 📝 Commits (10+) - [`ceb415f`](https://github.com/Mintplex-Labs/anything-llm/commit/ceb415f7065cfaf5c55c69b9d04734b8370167e6) Migrate Astra to class (#4722) - [`7ee2146`](https://github.com/Mintplex-Labs/anything-llm/commit/7ee2146af4fe2f9bde671ce4a2dcdec12f786a47) Migrate LanceDB to class (#4721) - [`eebb393`](https://github.com/Mintplex-Labs/anything-llm/commit/eebb39333f5bb7b6832d5cdecd129913ce6f064e) Migrate Pinecone to class (#4726) - [`b1823e4`](https://github.com/Mintplex-Labs/anything-llm/commit/b1823e48f5af3251f13aa7bf424db6144ddaf7b4) Migrate Zilliz to class (#4729) - [`97cfcad`](https://github.com/Mintplex-Labs/anything-llm/commit/97cfcadc8194e9dee53d8f75e2b30912caeaf30f) Migrate Weaviate to class (#4728) - [`5736097`](https://github.com/Mintplex-Labs/anything-llm/commit/5736097e678249a5b8a2a0d87a7a6871e4a14ce9) Migrate Qdrant to class (#4727) - [`dfe0e1a`](https://github.com/Mintplex-Labs/anything-llm/commit/dfe0e1a04ef3fc353bf0e0cfd88509a97f1c51c6) Migrate Milvus to class (#4725) - [`1301349`](https://github.com/Mintplex-Labs/anything-llm/commit/13013491385fd0c604fcd1ece1e63870b697e156) Migrate Chroma to class (#4723) - [`4fb118e`](https://github.com/Mintplex-Labs/anything-llm/commit/4fb118ebfd487b27971eada8b0b8f787c2013e3c) Migrate Chroma Cloud to class (#4724) - [`c173ff5`](https://github.com/Mintplex-Labs/anything-llm/commit/c173ff5a129b6b5c03024dd9df92002c1827f89c) Migrate PGVector to class (#4730) ### 📊 Changes **14 files changed** (+892 additions, -860 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/dev-build.yaml` (+1 -1) 📝 `server/__tests__/utils/vectorDbProviders/pgvector/index.test.js` (+3 -1) 📝 `server/utils/helpers/index.js` (+11 -11) 📝 `server/utils/vectorDbProviders/astra/index.js` (+74 -52) ➕ `server/utils/vectorDbProviders/base.js` (+201 -0) 📝 `server/utils/vectorDbProviders/chroma/index.js` (+79 -52) 📝 `server/utils/vectorDbProviders/chromacloud/index.js` (+27 -18) 📝 `server/utils/vectorDbProviders/lance/index.js` (+88 -58) 📝 `server/utils/vectorDbProviders/milvus/index.js` (+73 -49) 📝 `server/utils/vectorDbProviders/pgvector/index.js` (+109 -98) 📝 `server/utils/vectorDbProviders/pinecone/index.js` (+57 -36) 📝 `server/utils/vectorDbProviders/qdrant/index.js` (+67 -43) 📝 `server/utils/vectorDbProviders/weaviate/index.js` (+81 -54) 📝 `server/utils/vectorDbProviders/zilliz/index.js` (+21 -387) </details> ### 📄 Description ### Pull Request Type <!-- For change type, change [ ] to [x]. --> - [ ] ✨ feat - [ ] 🐛 fix - [x] ♻️ refactor - [ ] 💄 style - [ ] 🔨 chore - [ ] 📝 docs ### Relevant Issues <!-- Use "resolves #xxx" to auto resolve on merge. Otherwise, please use "connect #xxx" --> resolves #4687 ### What is in this change? <!-- Describe the changes in this PR that are impactful to the repo. --> - Refactors all vector databases to use classes instead of the object structure that was previously used - Creates `VectorDatabase` base class that is extended onto all vector database providers to ensure all required functions are declared - Refactors Zilliz Cloud provider to extend the Milvus class helping reduce code complexity since they have identical functions for interacting with the vector database ### Additional Information <!-- Add any other context about the Pull Request here that was not captured above. --> ### Developer Validations <!-- All of the applicable items should be checked. --> - [x] I ran `yarn lint` from the root of the repo & committed changes - [x] Relevant documentation has been updated - [x] I have tested my code functionality - [x] Docker build succeeds locally --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-22 18:36:30 -05:00
yindo closed this issue 2026-02-22 18:36:30 -05:00
yindo changed title from [PR #4787] VectorDB class migration to [PR #4787] [MERGED] VectorDB class migration 2026-06-05 15:20:18 -04:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Mintplex-Labs/anything-llm#4773