VectorsDB support for console#2923
Conversation
- 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
Console (appwrite/console)Project ID: Tip Schedule functions to run as often as every minute with cron expressions |
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Greptile SummaryThis PR delivers full VectorsDB support in the console, completing the polymorph database abstraction that previously had Key changes:
Confidence Score: 5/5Safe to merge — all remaining findings are P2 style suggestions that do not affect runtime correctness. Prior P0/P1 concerns (hardcoded endpoint, missing dimension cap, incorrect DocumentsDBIndexType cast) are all resolved. The SDK abstraction layer is consistently and correctly implemented across all vectorsdb operations. The only remaining finding is a P2 style violation in the new embeddingModal.svelte (mixed Svelte 4/5 syntax), which does not affect functionality. embeddingModal.svelte — mixed Svelte 4/5 syntax in a new file. Important Files Changed
Reviews (2): Last reviewed commit: "merge: resolve conflicts with remote fea..." | Re-trigger Greptile |
...abases/database-[database]/collection-[collection]/(components)/editor/embeddingModal.svelte
Outdated
Show resolved
Hide resolved
...nsole)/project-[region]-[project]/databases/database-[database]/(entity)/views/create.svelte
Show resolved
Hide resolved
...region]-[project]/databases/database-[database]/collection-[collection]/indexes/+page.svelte
Outdated
Show resolved
Hide resolved
…preview, metadata defaults
...oject]/databases/database-[database]/collection-[collection]/(components)/editor/view.svelte
Outdated
Show resolved
Hide resolved
...oject]/databases/database-[database]/collection-[collection]/(components)/editor/view.svelte
Outdated
Show resolved
Hide resolved
...)/project-[region]-[project]/databases/database-[database]/table-[table]/sheetOptions.svelte
Outdated
Show resolved
Hide resolved
…embeddings, Object/HNSW send empty orders

Summary
Test plan