Skip to content

VectorsDB support for console#2923

Open
premtsd-code wants to merge 35 commits intofeat-documentsdbfrom
feat-vectordb
Open

VectorsDB support for console#2923
premtsd-code wants to merge 35 commits intofeat-documentsdbfrom
feat-vectordb

Conversation

@premtsd-code
Copy link
Copy Markdown
Contributor

Summary

  • Full VectorsDB SDK integration (collections, documents, indexes, CRUD operations)
  • VectorsDB-specific UI: embedding editor with auto-fold, dimension field, HNSW index types
  • Route all collection pages through database-type-aware SDK helpers
  • Fix naming consistency (vectordb → vectorsdb)
  • Update SDK to 5fe49af with VectorsDB support
  • Add VectorsDB assets (SVGs, empty states)

Test plan

  • Create a vectorsdb database and collection
  • CRUD documents with embeddings
  • Create/delete HNSW indexes
  • Verify activity, usage, settings pages work for vectorsdb collections
  • Verify documentsdb/tablesdb collections still work unchanged

ItzNotABug and others added 7 commits February 11, 2026 17:27
- Route delete/update/index operations through database-type-aware SDK
  (settings, indexes, activity, usage pages were hardcoding documentsDB)
- Add updateEntity, deleteIndex methods to sdk.ts helper
- Fix pagination and document loading to use correct SDK for vectorsdb
- Fix realtime events to listen for both documentsdb and vectorsdb
- Fix embeddings: pass number[] directly instead of JSON.stringify roundtrip
- Auto-fold embeddings array in editor on document load
- Add !isVectorsDb guard to useMockSuggestions derivation
- Use single sliceString for bracket matching instead of per-char rope traversal
@appwrite
Copy link
Copy Markdown

appwrite bot commented Mar 20, 2026

Console (appwrite/console)

Project ID: 688b7bf400350cbd60e9

Sites (1)
Site Status Logs Preview QR
 console-stage
688b7cf6003b1842c9dc
Ready Ready View Logs Preview URL QR Code

Tip

Custom domains work with both CNAME for subdomains and NS records for apex domains

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Mar 20, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 42b459b2-e413-425b-b30d-307241946751

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat-vectordb

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 20, 2026

Greptile Summary

This PR adds full VectorsDB support to the console: SDK integration for collections, documents, and HNSW indexes; a dimension field in the collection-create flow; an embedding editor with auto-fold; and VectorsDB assets/empty states. All five issues flagged in prior rounds (hardcoded embedding URL, missing dimension max, misleading index-type cast in +page.svelte, stale HNSW order: null, and duplicate Import/Export buttons) have been addressed. The remaining findings are non-blocking style and UX polish items.

Confidence Score: 5/5

Safe to merge — all prior blocking issues resolved; remaining findings are P2 style and UX polish.

All five previously flagged P1 issues have been fixed. The three remaining findings are P2: a misleading type cast that has no runtime effect, command-centre labels that use the wrong terminology for VectorsDB/DocumentsDB but remain functional, and a suggestColumns production-path concern that only surfaces when mock AI suggestions are disabled. None block correctness or data integrity.

database-[database]/+layout.svelte (hardcoded labels) and collection-[collection]/+layout.svelte (suggestColumns tableId) warrant a follow-up pass before the feature goes to general availability.

Important Files Changed

Filename Overview
src/routes/(console)/project-[region]-[project]/databases/database-[database]/(entity)/helpers/sdk.ts Comprehensive VectorsDB SDK integration — all CRUD operations, index creation/deletion, and entity management correctly dispatch to baseSdk.vectorsDB with proper parameter mapping.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/(entity)/helpers/terminology.ts VectorsDB added to terminology map and DatabaseType union; toSupportiveIndex still carries a misleading as TablesDBIndexType cast that should be dropped.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/collection-[collection]/indexes/createIndex.svelte HNSW index types added for VectorsDB; stale order: null after switching away from HNSW is correctly reset via $effect; auto-fill of embeddings field works as intended.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/collection-[collection]/(components)/editor/embeddingModal.svelte Now uses sdk.vectorsDB.createTextEmbeddings SDK method instead of a hardcoded URL; error handling and state cleanup look correct.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/+layout.svelte Command-centre entries use hardcoded "Create table" / "Go to tables" labels even though terminology is imported — wrong for VectorsDB and DocumentsDB databases.
src/routes/(console)/project-[region]-[project]/databases/database-[database]/collection-[collection]/+layout.svelte VectorsDB sample-data generation correctly wraps fields in metadata and zero-fills embeddings; suggestColumns production path passes tableId for collection IDs which may fail for VectorsDB in production.

Reviews (7): Last reviewed commit: "fix: use collection-specific CreateIndex..." | Re-trigger Greptile

…ordb

# Conflicts:
#	bun.lock
#	package.json
#	src/routes/(console)/project-[region]-[project]/databases/database-[database]/(entity)/views/field/activity.svelte
#	src/routes/(console)/project-[region]-[project]/databases/database-[database]/collection-[collection]/activity/+page.ts
…ordb

# Conflicts:
#	src/routes/(console)/project-[region]-[project]/databases/database-[database]/collection-[collection]/usage/[[period]]/+page.ts
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.

3 participants