Merged
Conversation
grillinr
commented
Mar 1, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a dedicated “Messages” tab and conversation UI to improve discoverability/usability of direct messages (Issue #78), plus supporting backend APIs for user search/autocomplete and richer thread metadata.
Changes:
- Adds a new Messages tab screen with thread list + “new conversation” modal (username autocomplete).
- Adds a new conversation screen with message bubbles, a multi-line composer, and WebSocket-driven real-time updates.
- Adds backend
/users/searchendpoint and extends DM thread payloads withpeer_picture.
Reviewed changes
Copilot reviewed 14 out of 16 changed files in this pull request and generated 8 comments.
Show a summary per file
| File | Description |
|---|---|
| frontend/services/api.ts | Extends ApiDirectMessageThread with peer_picture; adds searchUsers() API helper. |
| frontend/package-lock.json | Lockfile updates from dependency install/metadata changes. |
| frontend/index.js | Adds Expo Router entry point. |
| frontend/components/Post.tsx | Removes unused import. |
| frontend/components/MessageThreadItem.tsx | New thread list item UI (avatar/initials, unread badge) + navigation into conversation. |
| frontend/components/MessageComposer.tsx | New multi-line message input with send button + loading state. |
| frontend/components/MessageBubble.tsx | New chat bubble UI with timestamp + pending/error state. |
| frontend/app/conversation/[username].tsx | New conversation screen: loads history, optimistic send, WebSocket receive. |
| frontend/app/(tabs)/message.tsx | New “Messages” tab screen: threads list, refresh, new chat modal with autocomplete. |
| frontend/app/(tabs)/_layout.tsx | Registers the new message tab. |
| backend/api/main.go | Registers /users/search route. |
| backend/api/internal/handlers/user_routes.go | Adds SearchUsers handler. |
| backend/api/internal/database/user_queries.go | Adds DB query for username-prefix search. |
| backend/api/internal/database/direct_message_queries.go | Adds peer_picture to DM thread query/struct. |
| backend/devbits.db | Updated database artifact included in PR. |
| AGENTS.md | Adds repository guidelines (build/test/lint, conventions). |
Files not reviewed (1)
- frontend/package-lock.json: Language not supported
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Contributor
2 tasks
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
) * Initial plan * fix: use API_BASE_URL for WebSocket URL in conversation screen Co-authored-by: grillinr <169214325+grillinr@users.noreply.github.com> * fix: merge reconnection logic from base branch while keeping API_BASE_URL approach Co-authored-by: elifouts <116454864+elifouts@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: grillinr <169214325+grillinr@users.noreply.github.com> Co-authored-by: elifouts <116454864+elifouts@users.noreply.github.com> Co-authored-by: Eli Fouts <eligfouts@gmail.com>
- Added authorLabel prop to MessageBubble for displaying message authors. - Improved styling in MessageBubble for better visual consistency. - Updated MessageComposer to include layout animations for dynamic height adjustments. - Refactored input handling in MessageComposer to optimize performance. - Removed unused terminal animation code from MyHeader component. - Added terminal icon mapping in IconSymbol for better icon management.
Collaborator
|
@copilot open a new pull request to apply changes based on the comments in this thread |
Contributor
Collaborator
Author
|
@elifouts can you test this in the backend? |
Collaborator
Author
|
@copilot address the failing tests on this or |
Contributor
… expectations (#136) * Initial plan * fix: exclude deleted_user sentinel from search results and update test expectations Co-authored-by: grillinr <169214325+grillinr@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: grillinr <169214325+grillinr@users.noreply.github.com>
Collaborator
Author
|
Waiting on review from either @DerekCorniello or @elifouts |
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 23 out of 24 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Collaborator
|
once copilot changes are committed this lgtm and i trust between the two of you and copilot this is fine. |
DerekCorniello
previously approved these changes
Mar 6, 2026
Collaborator
DerekCorniello
left a comment
DerekCorniello
left a comment
There was a problem hiding this comment.
see previous comment
Collaborator
|
cant really check pr till i get environment set up but will get to this |
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.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
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.
closes #78