feat(sidebar): pin chats within projects by 12ya · Pull Request #3913 · pingdotgg/t3code

12ya · 2026-07-12T14:43:53Z

What Changed

Persist pinned chat state per logical project.
Keep pinned chats above the normal thread sort order for each project.
Add hover pin and unpin controls without consuming title space.
Support pinning and unpinning by dragging a chat across the pinned divider.
Show a compact divider with a centered pin icon and distinct pin/unpin feedback colors.
Animate chats moving between pinned and regular sections.

Why

Important chats should stay easy to reach without being pinned globally across unrelated projects. Project-scoped pinning keeps each sidebar group predictable while preserving the existing thread sort order for unpinned chats.

The hover control provides a quick direct action, while divider-crossing drag behavior makes bulk sidebar organization feel natural.

UI Changes

Interaction demo:

Screen.Recording.2026-07-12.at.11.51.13.PM.mp4

Checklist

This PR is small and focused
I explained what changed and why
I included before/after screenshots for any UI changes
I included a video for animation/interaction changes

Note

Low Risk
Client-only UI and localStorage preferences; no server or auth changes. Drag-and-drop and preview logic are the main regression surfaces.

Overview
Adds project-scoped thread pinning in the sidebar: pinned chats stay at the top of each project’s list (after the existing sort), with state in pinnedThreadKeysByProject persisted via uiStateStore.

Users can pin/unpin from hover actions on each row and by dragging a thread across a pin divider (dnd-kit); divider visibility and pin vs unpin intent use new helpers in Sidebar.logic.ts. Rows split into pinned and regular sections with a SidebarPinnedDivider and safer drag activation (blocked on buttons/inputs).

getVisibleThreadsForProject now accepts an optional getThreadId so folded previews work with scoped thread keys in grouped projects. Thread row UX shifts to explicit hover/focus row actions (pin + archive) and shouldHideThreadMeta instead of pure CSS group-hover for metadata.

^{Reviewed by Cursor Bugbot for commit 02c19ff. Bugbot is set up for automated code reviews on this repo. Configure here.}

Note

Add pin-to-top support for chat threads within sidebar projects

Introduces per-project thread pinning stored in uiStateStore as pinnedThreadKeysByProject, persisted across sessions.
Adds a pin/unpin button on thread rows and drag-to-pin UX: dragging a thread across a divider pins or unpins it via resolveThreadPinCrossingAction.
Pinned threads are sorted to the top of each project's thread list using prioritizeThreadsByPinnedKeys, preserving relative order within each group.
New sidebar logic utilities (shouldBlockThreadDragActivation, shouldHideThreadMeta, shouldShowPinnedThreadDivider) manage drag safety, metadata visibility, and divider rendering.
Behavioral Change: thread row action visibility now depends on hover/focus/mobile state and suppresses during active drags.

^{Macroscope summarized 02c19ff.}

Persist pinned thread keys per logical project, keep pinned chats above the project's normal sort order, and separate pinned chats with a compact divider.

coderabbitai · 2026-07-12T14:44:06Z

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 2121fdbe-e6fc-4830-87f5-76089d202041

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

Track row hover and focus directly instead of relying on nested group selectors, and hide row actions while a thread drag is active.

Render the pinned divider during active drags when all threads are in one group, allowing the first pin and last unpin gestures to cross a measurable boundary.

macroscopeapp · 2026-07-13T00:41:10Z

Approvability

Verdict: Needs human review

New feature introducing thread pinning with drag-and-drop, new UI components, and state management. Additionally, 3 medium-severity bugs have been flagged in review comments identifying potential correctness issues.

^{You can customize Macroscope's approvability policy. Learn more.}

Prevent drag activation from interactive controls, preserve mobile row metadata beside actions, and keep scoped active threads visible in collapsed previews.

Overlay the temporary divider when only pinned or unpinned threads exist so mounting it during a drag does not shift the active row.

cursor

Cursor Bugbot has reviewed your changes using high effort and found 3 potential issues.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit 02c19ff. Configure here.}

cursor · 2026-07-13T01:01:18Z

+              scopedThreadKey(scopeThreadRef(thread.environmentId, thread.id)),
+            ),
+        )
+      : threadPreview.hiddenThreads;


Stale preview hides active thread

Medium Severity

The useMemo for renderedThreads omits activeRouteThreadKey from its dependencies. When a project is expanded and its thread list is folded, this causes the sidebar to display a stale thread preview, potentially hiding the active thread or showing a previously active one.

^{Reviewed by Cursor Bugbot for commit 02c19ff. Configure here.}

cursor · 2026-07-13T01:01:18Z

+    <SidebarMenuSubItem
+      ref={nodeRef}
+      aria-hidden="true"
+      className={`flex h-3 w-full items-center gap-1.5 px-2 transition-colors before:h-px before:flex-1 after:h-px after:flex-1 ${overlaysThreadList ? "absolute inset-x-0 z-10" : ""} ${colorClass}`}


Overlay divider clips on last unpin

Medium Severity

The SidebarPinnedDivider is absolutely positioned with inset-x-0 but no top or bottom when either the pinned or regular thread list is empty. This misplaces its hit target at the top of the list and can hide the divider, leading to incorrect unpinning behavior during drag-and-drop.

^{Reviewed by Cursor Bugbot for commit 02c19ff. Configure here.}

cursor · 2026-07-13T01:01:18Z

-    hiddenThreads: threads.filter((thread) => !visibleThreadIds.has(thread.id)),
-    visibleThreads: threads.filter((thread) => visibleThreadIds.has(thread.id)),
+    hiddenThreads: threads.filter((thread) => !visibleThreadIds.has(getThreadId(thread))),
+    visibleThreads: threads.filter((thread) => visibleThreadIds.has(getThreadId(thread))),


Spurious Show more with empty hidden

Medium Severity

When getVisibleThreadsForProject pulls the active thread into the folded preview, it still returns hasHiddenThreads: true even if hiddenThreads is empty. Wiring that into the sidebar makes hasOverflowingThreads stay true whenever total threads exceed the preview limit, so “Show more” can appear after every below-fold chat is already visible—common once pinned rows fill the preview and the active unpinned chat is force-included.

Additional Locations (1)

apps/web/src/components/Sidebar.tsx#L1557-L1565

^{Reviewed by Cursor Bugbot for commit 02c19ff. Configure here.}

feat(sidebar): pin chats within projects

b127970

Persist pinned thread keys per logical project, keep pinned chats above the project's normal sort order, and separate pinned chats with a compact divider.

github-actions Bot added size:L 100-499 changed lines (additions + deletions). vouch:unvouched PR author is not yet trusted in the VOUCHED list. labels Jul 12, 2026

macroscopeapp Bot reviewed Jul 12, 2026

View reviewed changes

Comment thread apps/web/src/components/Sidebar.tsx

12ya added 4 commits July 13, 2026 09:19

feat(sidebar): pin threads with drag and drop

53a27cb

refactor(sidebar): simplify thread pin controls

5833599

refactor(sidebar): pin threads by divider crossing

a06830f

fix(sidebar): animate pinned thread groups

6656e1c

12ya force-pushed the main branch from 89b4109 to 6656e1c Compare July 13, 2026 00:19

12ya added 2 commits July 13, 2026 09:32

fix(sidebar): ignore controls when starting thread drags

fd614e5

fix(sidebar): scope thread actions to active rows

d2c46a7

Track row hover and focus directly instead of relying on nested group selectors, and hide row actions while a thread drag is active.

12ya marked this pull request as ready for review July 13, 2026 00:36

macroscopeapp Bot reviewed Jul 13, 2026

View reviewed changes

Comment thread apps/web/src/components/Sidebar.tsx

fix(sidebar): enable drag pinning at group edges

b8c812d

Render the pinned divider during active drags when all threads are in one group, allowing the first pin and last unpin gestures to cross a measurable boundary.

cursor Bot reviewed Jul 13, 2026

View reviewed changes

Comment thread apps/web/src/components/Sidebar.tsx

Comment thread apps/web/src/components/Sidebar.tsx

Comment thread apps/web/src/components/Sidebar.tsx

cursor Bot reviewed Jul 13, 2026

View reviewed changes

Comment thread apps/web/src/components/Sidebar.tsx Outdated

12ya added 2 commits July 13, 2026 09:49

fix(sidebar): harden thread row interactions

9ceec82

Prevent drag activation from interactive controls, preserve mobile row metadata beside actions, and keep scoped active threads visible in collapsed previews.

fix(sidebar): prevent edge divider drag jumps

02c19ff

Overlay the temporary divider when only pinned or unpinned threads exist so mounting it during a drag does not shift the active row.

cursor Bot reviewed Jul 13, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

feat(sidebar): pin chats within projects#3913

feat(sidebar): pin chats within projects#3913
12ya wants to merge 10 commits into
pingdotgg:mainfrom
12ya:main

12ya commented Jul 12, 2026 •

edited by macroscopeapp Bot

Loading

Uh oh!

coderabbitai Bot commented Jul 12, 2026 •

edited

Loading

Review skipped

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

macroscopeapp Bot commented Jul 13, 2026 •

edited

Loading

Uh oh!

Uh oh!

cursor Bot left a comment

Uh oh!

cursor Bot Jul 13, 2026

Uh oh!

cursor Bot Jul 13, 2026

Uh oh!

cursor Bot Jul 13, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Conversation

12ya commented Jul 12, 2026 • edited by macroscopeapp Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What Changed

Why

UI Changes

Checklist

Add pin-to-top support for chat threads within sidebar projects

Uh oh!

coderabbitai Bot commented Jul 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Review skipped

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

macroscopeapp Bot commented Jul 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Approvability

Uh oh!

Uh oh!

cursor Bot left a comment

Choose a reason for hiding this comment

Uh oh!

cursor Bot Jul 13, 2026

Choose a reason for hiding this comment

Stale preview hides active thread

Uh oh!

cursor Bot Jul 13, 2026

Choose a reason for hiding this comment

Overlay divider clips on last unpin

Uh oh!

cursor Bot Jul 13, 2026

Choose a reason for hiding this comment

Spurious Show more with empty hidden

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

12ya commented Jul 12, 2026 •

edited by macroscopeapp Bot

Loading

coderabbitai Bot commented Jul 12, 2026 •

edited

Loading

macroscopeapp Bot commented Jul 13, 2026 •

edited

Loading