Add memory drawer

[ved015]

Revamp memory model in phone to memory-drawer comes up with smooth animation

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Updated (UTC)
✅ Deployment successful! View logs	supermemory-mcp	87723cb	May 29 2026, 03:16 PM

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status	Name	Latest Commit	Preview URL	Updated (UTC)
✅ Deployment successful! View logs	supermemory-app	87723cb	Commit Preview URL Branch Preview URL	May 29 2026, 03:16 PM

[claude]

Claude finished @ved015's task in 1m 35s —— View job

---

Review Complete

• Gathering context from PR changes
• Analyzing changed files for bugs and security issues
• Checking boundary code and surrounding context
• Submitting review

---

Overview: Refactors the document modal to use a Drawer component on mobile devices, adding tab-based navigation between Content and Insights views with responsive legend/graph adjustments.

Issues found: None — this looks good to ship.

The implementation is clean and well-structured:

• Shared content extracted into modalContent variable, eliminating code duplication between Dialog and Drawer
• Mobile detection with isMobile correctly drives the conditional rendering at the component level
• Proper imports for Drawer, DrawerContent, DrawerTitle from @repo/ui/components/drawer and Tabs components
• DialogPrimitive import already present (line 16) for the close button functionality
• All optional chaining (_document?.title, _document?.memoryEntries, etc.) properly guards against undefined access
• The hasDocumentInsights boolean correctly determines whether to show the tabbed interface on mobile
• Legend component properly handles compact mode with scrollable content and sensible height constraints (maxHeight - 56 with 112px minimum)
• Graph positioning adjusts correctly for compact viewports (12px vs 16px spacing)

Score: 10/10

[claude]

Overview: Refactors the document modal to use a Drawer component on mobile devices, improving the mobile UX with better gesture support and responsive legend/graph components.

Issues found: None — this looks good to ship.

The implementation is clean:

• Shared content extracted into modalContent variable, avoiding code duplication between Dialog and Drawer
• Mobile/desktop conditional rendering is straightforward with the isMobile check at render time
• Legend component properly handles compact mode with scrollable content and sensible height constraints
• Graph positioning adjusts correctly for compact viewports

Score: 10/10

[MaheshtheDev]

LGTM