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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 10, 2026 7:07pm

@BugBot review

PR Summary

Medium Risk
Touches interactive tab selection/removal/drag-and-drop behavior and adds serialized bulk mutations, which could cause subtle UI state or persistence edge cases (selection reset, active tab fallback, partial failures). Scope is limited to client-side mothership resource tab/dropdown components.

Overview
Adds multi-select to resource-tabs (shift for ranges; cmd/ctrl to toggle) with visual highlighting, selection reset on chat switch, and smarter active-tab fallback when deselecting.

Enables multi-drag of selected tabs by emitting SIM_RESOURCES_DRAG_TYPE payloads (with a custom combined drag image) and bulk close of selected tabs via the close button or middle-click, persisting removals sequentially.

Updates AddResourceDropdown to support an excludeTypes blacklist and uses it to hide folder/task entries from the add-resource menu in mothership tabs.

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

@BugBot review

Greptile Summary

This PR adds multi-select to resource tabs — Shift+click for contiguous range selection and Cmd/Ctrl+click for individual toggle — along with multi-resource drag using a composite ghost image and bulk close. It also fixes folder/task entries appearing in the add-resource dropdown by introducing an excludeTypes filter prop.

Confidence Score: 5/5

Safe to merge; the one finding is a minor UX edge case where shift+click silently degrades to a plain click after closing a singly-selected tab.

All findings are P2. The stale selectedIds/anchorIdRef issue has no visible rendering impact and only mildly affects the next shift+click after a single-tab close. Core multi-select, drag, and bulk-close logic is sound.

No files require special attention beyond the cleanup suggestion in resource-tabs.tsx.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[Tab Click] --> B{Modifier key?}
    B -->|Shift| C{Anchor exists?}
    C -->|Yes| D[Select contiguous range\nanchorIdRef → anchorIdx]
    C -->|No| E[Plain click\nSet anchor]
    B -->|Cmd / Ctrl| F{Tab already selected?}
    F -->|Yes| G[Deselect tab\nFallback onSelect if active]
    F -->|No| H[Add tab to selection\nUpdate anchor if none]
    B -->|None| E

    A2[Close X or Middle-click] --> I{isMulti?\nselectedIds has tab AND size > 1}
    I -->|Yes| J[Remove all selected tabs\nClear selectedIds + anchor]
    I -->|No| K[Remove single tab\n⚠ selectedIds/anchor NOT cleared]

    A3[Drag start] --> L{Multi-drag?\nselected > 1 AND tab in selection}
    L -->|Yes| M[Set SIM_RESOURCES_DRAG_TYPE\nBuild composite ghost image\nDisable internal reorder]
    L -->|No| N[Set SIM_RESOURCE_DRAG_TYPE\nEnable internal reorder]

_{Reviews (1): Last reviewed commit: "Try catch resource tab deletion independ..." | Re-trigger Greptile}