feat(assistant): add dexie db leveraging browser indexed db #219
Conversation
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
- Install Dexie.js for IndexedDB management - Create database schema in features/assistant/db/ - Define StoredConversation and StoredMessage types - Implement basic CRUD operations for conversations and messages - Filter out system messages when saving (only user/assistant visible)
- Create useChatHistory hook for conversation management - Auto-save messages to IndexedDB with debouncing (500ms) - Save conversation metadata (model, selectedServers) - Generate conversation title from first user message - Export useChatHistory from assistant feature index
- Load most recent conversation when ChatProvider mounts - Restore selected model from conversation metadata - Only load if no messages are already present - Export useChatHistory hook from assistant feature
- Start new conversation instead of clearing current one - Ensures clearMessages() properly resets conversation state
- Required for React hooks (useEffect) in Next.js App Router - Fixes build error for client-side hook
- More descriptive name matching project structure
- Expose isLoadingHistory from ChatContext - Show centered Loader2 spinner in ChatInterface (line 54) - Hide empty state and messages while loading
- IndexedDB loads too fast for spinner to be visible - Removed isLoadingHistory from ChatContext and ChatInterface - Keeps code simpler without unnecessary loading state
- Expose conversations and management functions from ChatContext - Create ConversationList component with search functionality - Add popover in ChatHeader with chevron indicator - Support selecting, deleting, and creating conversations - Include date-fns for relative time formatting
- Fix hydration error: use sibling buttons instead of nested buttons - Add margin to ChevronDown for better spacing from 'Assistant' - Fix 'New conversation' not clearing messages by preventing auto-reload after initial mount - Center garbage icon vertically with items-center - Improve delete button styling with hover states
- Add space-y-1 to prevent delete icon overlap on hover
- Use secondary variant for dark background by default
- Add overflow-hidden to conversation item containers - Properly constrain text to allow truncation
- Add isLoadingConversationRef to skip auto-save on load - Prevents updatedAt from being updated when viewing old conversations - Conversations now stay sorted by actual last message time
- Replace native button elements with Button component - Use ghost variant for conversation items - Use icon size for delete button
- Replace h-4 w-4 with size-4 - Replace h-6 w-6 with size-6
- Replace native button with Button component - Use ghost variant for consistent styling
- Replace custom button elements with Button from @/components/ui/button - Update image-modal.tsx close button to use Button variant='ghost' - Update reasoning.tsx toggle button to use Button component - Update tool-call.tsx toggle buttons to use Button component - Remove redundant comment in sidebar.tsx
github-actions
bot
added
size/L
peppescg
changed the title
There was a problem hiding this comment.
Pull request overview
This PR implements conversation history persistence for the assistant using Dexie.js as an IndexedDB wrapper, enabling browser-based chat history storage and management. The implementation includes auto-save with debouncing, auto-restore of the last conversation, and a full conversation management UI.
Key changes:
- Database layer with Dexie.js for storing conversations and messages in IndexedDB
- Auto-save mechanism with 500ms debounce to persist chat messages
- Conversation list UI with search, delete, and selection capabilities accessible via popover
- Assistant sidebar moved from left to right side of the application
Reviewed changes
Copilot reviewed 18 out of 19 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
src/features/assistant/db/database.ts |
Defines Dexie database schema with conversations and messages tables |
src/features/assistant/db/types.ts |
TypeScript interfaces for StoredConversation and StoredMessage |
src/features/assistant/db/storage.ts |
Database operations for CRUD on conversations and messages |
src/features/assistant/db/index.ts |
Barrel export for database module |
src/features/assistant/hooks/use-chat-history.ts |
Custom hook managing chat history state and database interactions |
src/features/assistant/contexts/chat-context.tsx |
Integration of chat history with auto-save and auto-restore logic |
src/features/assistant/components/sidebar.tsx |
Changed sidebar position from left to right |
src/features/assistant/components/sidebar-content.tsx |
Added conversation management props to ChatInterface |
src/components/chat/conversation-list.tsx |
New component for displaying and managing conversation list |
src/components/chat/chat-interface.tsx |
Added conversation management props |
src/components/chat/chat-header.tsx |
Added popover trigger for conversation list and changed button variant |
src/components/ui/popover.tsx |
New shadcn/ui popover component |
src/components/navbar.tsx |
Reorganized navbar layout moving logo and assistant trigger |
package.json |
Added dexie and @radix-ui/react-popover dependencies |
pnpm-lock.yaml |
Lock file updates for new dependencies |
src/features/assistant/index.ts |
Exported useChatHistory hook |
Files not reviewed (1)
- pnpm-lock.yaml: Language not supported
- Create useChatTransport hook for transport creation with refs pattern - Create useChatPersistence hook for auto-load and auto-save logic - Simplify ChatProvider from 238 to 97 lines - Improve separation of concerns and testability
- Use getConversation(id) instead of getAllConversations().find() - Extract extractTitleFromMessages helper function - ChatInterface now uses useChatContext directly - Simplify sidebar-content from 50 to 11 lines
github-actions
bot
added
size/XL
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Large PR Detected
This PR exceeds 1000 lines of changes and requires justification before it can be reviewed.
How to unblock this PR:
Add a section to your PR description with the following format:
## Large PR Justification
[Explain why this PR must be large, such as:]
- Generated code that cannot be split
- Large refactoring that must be atomic
- Multiple related changes that would break if separated
- Migration or data transformationAlternative:
Consider splitting this PR into smaller, focused changes (< 1000 lines each) for easier review and reduced risk.
See our Contributing Guidelines for more details on the pull request process.
This review will be automatically dismissed once you add the justification section.
- Replace delete-all + insert-all pattern with upsert - Reduces I/O operations by ~50% for large conversations - bulkPut inserts new messages and updates existing ones
- Save only when streaming completes (status: streaming -> ready) - Remove setTimeout/debounce pattern - Reduce from 129 to 97 lines - Cleaner state tracking with previousStatusRef
github-actions
bot
added
size/XL
- Remove async from clearMessages to avoid unhandled promise - Use .catch() for error handling instead of await - UI clears immediately, conversation creation happens in background
- Move 'New Conversation' button from dropdown to header - Remove 'Clear Chat' button and related confirmation dialog - Remove 'New conversation' option from conversation list dropdown - Simplify ChatHeader and ConversationList props - Fix duplicate hover text color warning
- Add deleteAllConversations function to storage layer - Add clearAll method to useChatHistory hook - Add clearAllConversations to ChatContext - Add 'Clear all messages' button to conversation list dropdown - Includes confirmation dialog before clearing
- Replace ErrorAlert component with toast notification for chat errors - Disable 'New Conversation' button when chat is empty - Remove unused ErrorAlert import and component
- Simplify ConversationItem to use native buttons - Remove conflicting hover states from nested Button components - Show delete icon only on hover with smooth transition - Fix visual overlap issues in dropdown
- Add overflow-hidden to container to prevent content overflow - Replace ScrollArea with native overflow-y-auto for simpler layout - Always show delete icon (remove hover-only visibility) - Reduce max-height to max-h-64 for better containment
github-actions
bot
added
size/XL
peppescg
dismissed
github-actions[bot]’s stale review
the refactor reduce the number of lines
Contributor
There was a problem hiding this comment.
Large PR Detected
This PR exceeds 1000 lines of changes and requires justification before it can be reviewed.
How to unblock this PR:
Add a section to your PR description with the following format:
## Large PR Justification
[Explain why this PR must be large, such as:]
- Generated code that cannot be split
- Large refactoring that must be atomic
- Multiple related changes that would break if separated
- Migration or data transformationAlternative:
Consider splitting this PR into smaller, focused changes (< 1000 lines each) for easier review and reduced risk.
See our Contributing Guidelines for more details on the pull request process.
This review will be automatically dismissed once you add the justification section.
rdimitrov
approved these changes
Dec 23, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Chat History with Dexie.js
Kapture.2025-12-23.at.17.26.01.mp4
Overview
Implemented conversation history persistence for the assistant using Dexie.js as an IndexedDB wrapper. Conversations are automatically saved in the browser and can be restored, searched, deleted and managed by the user.
Database Structure
Database name:
ToolHiveCloudUIChatDBTables:
conversations: stores conversation metadata (id, title, model, createdAt, updatedAt)messages: stores messages (id, conversationId, role, content, parts, metadata, createdAt)Features
UI
secondaryvariant (dark background by default)Moved the assistant chat to the right side of the application