Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| 🔵 In progress View logs |
supermemory-mcp | b5add5a | Feb 17 2026, 03:12 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | e501b0d | Commit Preview URL Branch Preview URL |
Apr 14 2026, 08:04 PM |
How to use the Graphite Merge QueueAdd the label Main to this PR to add it to the merge queue. You must have a Graphite account in order to use the merge queue. Sign up using this link. An organization admin has enabled the Graphite Merge Queue in this repository. Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue. This stack of pull requests is managed by Graphite. Learn more about stacking. |
de3ca2a to
22aeaaf
Compare
d2d0d8d to
6bcf26f
Compare
6bcf26f to
58622d3
Compare
|
@sreedharsreeram Waiting for the demo! |
58622d3 to
ec07caf
Compare
ec07caf to
fb08ea5
Compare
|
Claude finished @sreedharsreeram's task in 1m 26s —— View job Reviewing PR #744: Supermemory-Cartesia SDK
Overview: Adds a Python SDK package ( Issues found: None blocking. I reviewed the implementation across all 8 files. Key observations:
The code is clean, well-documented, and follows good practices for a fire-and-forget analytics SDK. Score: 9/10 The implementation is production-ready. The only reason it's not 10/10 is the untracked nature of |
There was a problem hiding this comment.
Overview: Adds a Python SDK for integrating Supermemory with Cartesia Line voice agents, enabling persistent memory retrieval and storage during voice conversations.
Issues found: None blocking.
The Sentry-flagged bug regarding _messages_sent_count not being incremented for first messages has been fixed (see agent.py:417). The untracked background task concern is a valid note but acceptable for fire-and-forget storage operations in this context.
Code quality observations:
- Clean separation of concerns (agent wrapper, utilities, exceptions)
- Proper error handling with graceful degradation (memory failures don't break the voice flow)
- Good defensive programming with
hasattrchecks for Cartesia event compatibility - Appropriate timeout (10s) on memory retrieval to prevent blocking voice interactions
Score: 9/10
The implementation is solid and ready to ship. The only reason it's not 10/10 is the untracked asyncio.create_task() calls—while acceptable for analytics, adding task references with done callbacks would be the ideal pattern for production robustness.
fb08ea5 to
e501b0d
Compare
| config: Optional[MemoryConfig] = None, | ||
| base_url: Optional[str] = None, | ||
| ): | ||
| """Initialize the Supermemory Cartesia agent wrapper. | ||
|
|
||
| Args: | ||
| agent: The inner Cartesia Line agent to wrap. | ||
| api_key: Supermemory API key (or SUPERMEMORY_API_KEY env var). | ||
| container_tag: Primary container tag for memory scoping (e.g., user ID). | ||
| custom_id: Required. Custom ID to store all conversation messages in the same document. | ||
| Useful for grouping multi-turn conversations (e.g., call ID, conversation ID). |
There was a problem hiding this comment.
Bug: The agent's __init__ method does not accept the session_id parameter used throughout the documentation, causing a TypeError. The documentation also incorrectly marks the required custom_id as optional.
Severity: CRITICAL
Suggested Fix
Update the documentation in apps/docs/integrations/cartesia.mdx to use the correct custom_id parameter instead of the non-existent session_id parameter in all code examples. Also, update the parameter table in the documentation to mark custom_id as required.
Prompt for AI Agent
Review the code at the location below. A potential bug has been identified by an AI
agent.
Verify if this is a real issue. If it is, propose a fix; if not, explain why it's not
valid.
Location: packages/cartesia-sdk-python/src/supermemory_cartesia/agent.py#L79-L96
Potential issue: The `SupermemoryCartesiaAgent` constructor signature in `agent.py` does
not accept a `session_id` parameter. However, the primary integration documentation in
`apps/docs/integrations/cartesia.mdx` exclusively uses `session_id` in all its code
examples. Additionally, the documentation incorrectly marks the `custom_id` parameter as
optional, when the code enforces it as a required, non-empty string. Any user following
the documentation will encounter an immediate `TypeError: __init__() got an unexpected
keyword argument 'session_id'` when trying to instantiate the agent, making the
integration unusable as documented.

TL;DR
Added Python SDK for integrating Supermemory with Cartesia Line voice agents, enabling persistent memory capabilities.
What changed?
Created a new Python SDK package (
supermemory_cartesia) that provides:SupermemoryCartesiaAgentwrapper class that enhances Cartesia Line agents with memory capabilitiesThe implementation includes:
How to test?
Why make this change?
This SDK enables Cartesia Line voice agents to maintain persistent memory across conversations, enhancing user experience by:
The integration is designed to be lightweight and non-blocking, ensuring that memory operations don't impact the responsiveness of voice interactions.