feat: add chat memory imports to extension#1115
Conversation
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-mcp | b93d6f8 | Jun 13 2026, 07:37 PM |
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
supermemory-app | b93d6f8 | Commit Preview URL Branch Preview URL |
Jun 13 2026, 07:38 PM |
|
Claude finished @ved015's task in 1m 21s —— View job Review Complete
Summary: Re-reviewed the PR after the latest sync. The implementation remains solid with no bugs, security issues, or logic errors. The new Grok memory import follows the same safe patterns as the existing Claude implementation, and the popup UI changes for manual Gemini import are well-structured. Score: 10/10 — Ready to ship. |
There was a problem hiding this comment.
Overview: Adds memory import functionality for Claude, Gemini, and Grok to the browser extension, including a "Save to supermemory" button injected into Claude's memory dialog and a manual paste flow for Gemini/Grok.
Issues found: None — this looks good to ship.
The implementation is solid:
- DOM injection in
claude.tsproperly guards against duplicate buttons with ID checks - The
innerHTMLusage is safe sincebrowser.runtime.getURL()returns a trusted extension URL - Error handling is comprehensive throughout both content script and popup
- The message passing correctly uses
data.htmlanddata.contentwhich are both handled by the existingsaveMemoryToSupermemoryfunction in background.ts - Race conditions are prevented by the observer throttling and duplicate checks
Score: 10/10
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
There was a problem hiding this comment.
Overview: Adds memory import functionality for Claude, Gemini, and Grok to the browser extension, including a "Save to supermemory" button injected into Claude's memory dialog and a manual paste flow for Gemini/Grok.
Issues found: None — this looks good to ship.
The implementation is solid:
- DOM injection in
claude.tsproperly guards against duplicate buttons with ID checks - The
innerHTMLusage is safe sincebrowser.runtime.getURL()returns a trusted extension URL - Error handling is comprehensive throughout both content script and popup
- The message passing correctly uses
data.htmlanddata.contentwhich are both handled by the existingsaveMemoryToSupermemoryfunction in background.ts - Race conditions are prevented by the observer throttling and duplicate checks
Score: 10/10
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
demo shared on work-done