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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 8, 2026 2:53am

PR Summary

Medium Risk
Adds new drag-and-drop pathways and context/resource synchronization in the chat input, which can subtly affect text insertion, selection state, and resource tab behavior. Risk is moderate because changes span multiple interactive UI components but don’t touch auth or server-side data handling.

Overview
Enables dragging items from the workspace sidebar into the chat input to insert @ mentions/resources, including multi-select drag via a new application/x-sim-resources payload and a custom drag-ghost preview.

Extends the resource model to include task and wires task/workflow/folder resources through the add-resource dropdown, resource registry invalidation, and context mapping; the chat input now also emits onContextRemove so removing a mention/context can remove the corresponding resource tab.

Refactors workflow color ring styling by centralizing the border alpha logic in workflowBorderColor() and replaces scattered ${color}60 usages across landing previews, logs, and sidebar UI. Also standardizes drag MIME types via SIM_RESOURCE_DRAG_TYPE/SIM_RESOURCES_DRAG_TYPE constants and improves keyboard UX in the + menu (Tab-to-select).

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

Greptile Summary

This PR adds drag-and-drop support so users can drag workflows, folders, and past chat items from the sidebar directly into the chat input — equivalent to tagging with @ or the + menu. Multi-selection drags (select several items, drop them all) are also supported.

Key changes:

• lib/copilot/resource-types.ts: new SIM_RESOURCE_DRAG_TYPE / SIM_RESOURCES_DRAG_TYPE constants and task resource type replace scattered raw MIME strings
• sidebar/utils.ts: new buildDragResources (O(1) map lookup) and createSidebarDragGhost (imperative DOM pill with CSS-variable theming) are shared by WorkflowItem, FolderItem, and SidebarTaskItem
• user-input.tsx: handles both single (SIM_RESOURCE_DRAG_TYPE) and batch (SIM_RESOURCES_DRAG_TYPE) drops; eagerly syncs atInsertPosRef to chain successive insertions, then resets after the batch; handleDragEnter/handleDragLeave now guard against showing the file-drop overlay during resource drags
• context-mention-icon.tsx: new shared component consolidates the duplicated icon-switch logic from user-input and user-message-content, adding a past_chat → Blimp icon case
• home.tsx: extracts resolveResourceFromContext shared between handleContextAdd and the new handleContextRemove; folder/past_chat are intentionally excluded (no resource panel UI)
• workflowBorderColor() helper centralises the ${color}60 border-alpha pattern used across several components

Confidence Score: 5/5

Safe to merge — the previously reported P0/P1 issues (stale ref after batch drop, intentional folder/past_chat omission) are resolved. Only two minor P2 findings remain.

Both stale-ref issues flagged in previous rounds are fixed. The only remaining findings are: (1) ghost element not cleaned up on component unmount — an edge-case leak that has no user-visible impact under normal usage, and (2) the removal of the #888 workflow-icon fallback in user-input.tsx — a cosmetic regression only visible when a workflow isn't in the query cache. Neither blocks shipping.

sidebar/utils.ts, workflow-item.tsx, folder-item.tsx, sidebar.tsx — all share the same ghost-element teardown gap on unmount.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A["User drags WorkflowItem / FolderItem / SidebarTaskItem"] --> B["handleDragStart"]
    B --> C["buildDragResources(selection, workspaceId)\n(reads React Query cache)"]
    C --> D["e.dataTransfer.setData\n(SIM_RESOURCES_DRAG_TYPE, JSON)"]
    B --> E["createSidebarDragGhost(label, icon)\n(appends to document.body)"]
    E --> F["e.dataTransfer.setDragImage(ghost, …)"]
    F --> G["User drags over chat input"]
    G --> H{"handleContainerDragOver\ntypes.includes SIM_RESOURCE*?"}
    H -- yes --> I["e.preventDefault → dropEffect = copy"]
    H -- no --> J["handleDragEnter shows file overlay"]
    I --> K["User drops on chat input"]
    K --> L{"handleContainerDrop\nSIM_RESOURCES_DRAG_TYPE present?"}
    L -- yes --> M["JSON.parse → for each resource:\nhandleResourceSelect(resource)\natInsertPosRef chains"]
    M --> N["atInsertPosRef.current = null (after batch)"]
    L -- no --> O["SIM_RESOURCE_DRAG_TYPE fallback\n(single resource)"]
    O --> P["handleResourceSelect(resource)\natInsertPosRef.current = null"]
    M --> Q["mapResourceToContext → ChatContext\naddContext → onContextAdd → addResource"]
    P --> Q
    G --> R["handleDragEnd\nremove ghost from DOM"]

_{Reviews (5): Last reviewed commit: "fix(chat): guard chatId before adding ta..." | Re-trigger Greptile}

@greptile

@cursor review

@greptile

@cursor review

@greptile

@cursor review

@greptile

@cursor review