Skip to content

Conversation

@sohamd22
Copy link
Collaborator

@sohamd22 sohamd22 commented Oct 19, 2025

TL;DR

Added conversationId parameter to the supermemory wrapper in the chat API.

What changed?

Added the conversation ID (id) as the conversationId parameter when initializing the language model with supermemory in the chat API route handler.

How to test?

  1. Make a POST request to the chat API endpoint
  2. Verify that conversations are properly associated with their IDs in the supermemory system
  3. Check that memory retrieval works correctly across multiple messages in the same conversation

Why make this change?

This change ensures that the supermemory system can properly track and associate memories with specific conversations by their ID. Without this parameter, the system might not correctly maintain context between different messages in the same conversation, potentially leading to degraded conversation quality and memory retrieval issues.

@vercel
Copy link

vercel bot commented Oct 19, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
chatbot Error Error Oct 19, 2025 8:38pm

💡 Enable Vercel Agent with $100 free credit for automated AI reviews

@sohamd22 sohamd22 requested a review from Dhravya October 19, 2025 20:38
Copy link
Collaborator Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@sohamd22 sohamd22 self-assigned this Oct 19, 2025
@sohamd22 sohamd22 marked this pull request as ready for review October 19, 2025 20:39
@Dhravya Dhravya merged commit 13bd7c6 into main Oct 22, 2025
1 of 4 checks passed
Dhravya pushed a commit that referenced this pull request Nov 7, 2025
Feat: NewChat now have different ConversationID. Retains user's Memory
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