The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 14, 2026 10:48pm

PR Summary

Medium Risk
Touches chat resource persistence and React Query optimistic mutation behavior, plus several UI interaction paths (multi-delete, active selection, keyboard navigation), which could cause subtle state/ordering regressions if edge cases are missed.

Overview
Improves chat resource tab UX and consistency: the active tab now auto-scrolls into view when changed, multi-select removal fires concurrent remove mutations, and chat hydration no longer forces the active resource to the last tab if the current selection still exists.

Makes resource removal more atomic and robust: the DELETE /api/mothership/chat/resources path now removes an entry via an in-DB JSONB filter and returns the updated resources, and useRemoveChatResource switches to a granular optimistic update that tracks removed items and restores only those on error.

Enhances the workspace switcher dropdown for large accounts with search + keyboard navigation (autofocus, arrow-key highlight, enter-to-switch), and adds cmd/ctrl-click or middle-click to open a workspace in a new tab.

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

Greptile Summary

This PR adds a workspace search bar with keyboard navigation to the workspace switcher dropdown, fixes resource tab auto-switching behavior on send/abort, implements multi-select delete for resource tabs with optimistic client-side removal, and adds auto-scroll to bring the active resource tab into view. The changes are well-structured and follow the existing patterns. All findings are minor P2 suggestions.

Confidence Score: 5/5

Safe to merge — all findings are P2 style/UX suggestions with no correctness or data-integrity issues.

The core logic for multi-select delete, workspace search, keyboard navigation, and tab auto-scroll is sound. PostgreSQL row locking makes concurrent resource deletions safe server-side, and TanStack Query's onSettled invalidation ensures cache consistency even on partial errors. All remaining comments are minor style improvements.

workspace-header.tsx has two minor UX polish items (magic number threshold, highlight state leaking).

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant U as User
    participant RT as ResourceTabs
    participant UC as useChat
    participant RQ as React Query Cache
    participant API as /api/mothership/chat/resources

    Note over U,API: Multi-select tab delete
    U->>RT: Click × on selected tab (N tabs selected)
    RT->>UC: onRemoveResource(type, id) × N (loop)
    UC->>UC: setResources batched → single render
    RT->>RT: setSelectedIds(remove deleted)
    loop for each non-ephemeral target
        RT->>RQ: removeResource.mutate({chatId, type, id})
        RQ->>RQ: onMutate → optimistic remove from cache
        RQ->>API: DELETE /resources {chatId, type, id}
        API->>API: SQL UPDATE JSONB (row-locked, atomic)
        API-->>RQ: {success, resources}
        RQ->>RQ: onSettled → invalidateQueries(detail)
    end

    Note over U,API: Workspace search + keyboard nav
    U->>RT: Opens workspace dropdown
    RT->>RT: setHighlightedIndex(0) + rAF focus
    U->>RT: Types search query
    RT->>RT: filter workspaces by name
    U->>RT: ArrowDown / ArrowUp
    RT->>RT: setHighlightedIndex(±1 mod len)
    U->>RT: Enter key
    RT->>RT: onWorkspaceSwitch(filteredWorkspaces[highlightedIndex])

_{Reviews (1): Last reviewed commit: "Add search bar to workspace modal" | Re-trigger Greptile}