Skip to content

feat(web): Current/All space filter + redesigned chat history sheet#1050

Merged
graphite-app[bot] merged 1 commit into
mainfrom
worktree-chat-history-all-spaces
Jun 5, 2026
Merged

feat(web): Current/All space filter + redesigned chat history sheet#1050
graphite-app[bot] merged 1 commit into
mainfrom
worktree-chat-history-all-spaces

Conversation

@MaheshtheDev
Copy link
Copy Markdown
Member

Add a Current/All scope toggle (default Current) to Nova chat history; All fetches threads across every space via the new scope=all API, with each row showing its space.

Redesign the sheet to fit the design system: cap width at 380px (was ~92vw), add search, date-grouped rows (Today/Yesterday/Previous 7/30 days/Older), hover-reveal delete, header space chip, solid primary New Conversation button.

Part of ENG-728: https://linear.app/supermemory/issue/ENG-728

@linear-code
Copy link
Copy Markdown

linear-code Bot commented Jun 4, 2026

ENG-728

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jun 4, 2026

Deploying with  Cloudflare Workers  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 ad5734c Commit Preview URL

Branch Preview URL
Jun 05 2026, 03:03 PM

@graphite-app graphite-app Bot requested a review from Dhravya June 4, 2026 22:56
Copy link
Copy Markdown
Member Author


How to use the Graphite Merge Queue

Add 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.

@claude
Copy link
Copy Markdown

claude Bot commented Jun 4, 2026

Claude finished @MaheshtheDev's task in 1m 44s —— View job


Code Review Complete

  • Loading review tools
  • Fetching PR diff
  • Analyzing code changes for bugs and security issues
  • Checking existing review comments
  • Submitting review

Overview: Adds a Current/All space scope toggle to the chat history sheet with search functionality and date-grouped thread display.

Issues found: None — this looks good to ship.

The implementation is solid:

  • URL construction using URLSearchParams is safe and handles the scope parameter correctly
  • Date grouping logic properly handles timezone-relative bucketing (Today/Yesterday/etc.)
  • Search filtering correctly handles null/undefined titles with (t.title || "")
  • State cleanup on sheet close properly resets confirmingDeleteId and historySearch
  • The useMemo for threadGroups has correct dependencies
  • Keyboard accessibility is properly implemented with role="button", tabIndex={0}, and Enter/Space handling
  • The useEffect correctly re-fetches when historyScope changes
  • No XSS vulnerabilities — React's built-in escaping handles user content

Score: 10/10

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages Bot commented Jun 4, 2026

Deploying with  Cloudflare Workers  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 ad5734c Jun 05 2026, 03:01 PM

Copy link
Copy Markdown

@claude claude Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Overview: Adds a Current/All space scope toggle to the chat history sheet with search functionality and date-grouped thread display.

Issues found: None — this looks good to ship.

The implementation is solid:

  • URL construction using URLSearchParams is safe and handles the scope parameter correctly
  • Date grouping logic properly handles timezone-relative bucketing (Today/Yesterday/etc.)
  • Search filtering correctly handles null/undefined titles with (t.title || "")
  • State cleanup on sheet close properly resets confirmingDeleteId and historySearch
  • The useMemo for threadGroups has correct dependencies (threads, historySearch)
  • Keyboard accessibility is properly implemented with role="button", tabIndex={0}, and Enter/Space handling
  • The useEffect correctly re-fetches when historyScope changes (via fetchThreads dependency)
  • No XSS vulnerabilities — React's built-in escaping handles user content

Score: 10/10

@graphite-app
Copy link
Copy Markdown

graphite-app Bot commented Jun 5, 2026

Merge activity

…1050)

Add a Current/All scope toggle (default Current) to Nova chat history; All fetches threads across every space via the new scope=all API, with each row showing its space.

Redesign the sheet to fit the design system: cap width at 380px (was ~92vw), add search, date-grouped rows (Today/Yesterday/Previous 7/30 days/Older), hover-reveal delete, header space chip, solid primary New Conversation button.

Part of ENG-728: https://linear.app/supermemory/issue/ENG-728
@graphite-app graphite-app Bot force-pushed the worktree-chat-history-all-spaces branch from 26a8333 to ad5734c Compare June 5, 2026 15:00
@graphite-app graphite-app Bot merged commit ad5734c into main Jun 5, 2026
5 of 7 checks passed
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.

2 participants