Add ChromaDB database discovery UI with dual-mode selector#22
Merged
Conversation
…rectory selection
Enhances the RAG settings page with a more user-friendly database selection interface: - Add automatic discovery of ChromaDB databases in ./data/ subdirectories - Implement dual-mode UI: dropdown selector (default) and manual path entry - Show database metadata (collection count, size) in the selector - Add folder icons to database options for better visual clarity - Display helpful messages when no databases are found, pointing to utils/README.md - Persist user's mode preference (select vs manual) in localStorage - Add comprehensive unit tests for discovery functionality and routes This makes database selection more intuitive for new users while maintaining flexibility for advanced users to specify custom paths. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
The path mocking in tests was using Unix-specific path splitting ('/'),
which caused failures on Windows. Fixed by:
- Adding Path import to test module
- Using Path(x).name instead of x.split('/')[-1] for cross-platform compatibility
- Ensuring all path operations work with both forward and backslashes
All 238 tests now pass on macOS, Ubuntu, and Windows.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude <noreply@anthropic.com>
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
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
Changes
Database Discovery
discover_databases()method to scan./data/subdirectories for ChromaDB databasesUI Improvements
utils/README.mdTesting
discover_databases()method covering various scenarios/api/rag/discover-databasesendpointTest Plan
Screenshots
The new dual-mode interface defaults to a friendly dropdown selector while maintaining flexibility for advanced users to specify custom paths.
🤖 Generated with Claude Code