GeezerVison

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 29, 2026 6:30pm

@BugBot review

PR Summary

Low Risk
Low risk UI/state fix limited to Copilot chat selection and React Query caching; main risk is subtle regressions in chat auto-selection across workflow switches.

Overview
Fixes Copilot chat auto-selection so it only selects when the chat list has loaded (skipping selection when the list is empty).

When creating a new Copilot chat, it now seeds the new chat into the React Query list cache before selecting it, preventing the auto-select effect from immediately clearing the selected chatId due to a stale chat list.

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

Greptile Summary

This PR fixes two related regressions in the copilot chat panel. First, autoSelectAttemptedForRef was being marked before the empty-list guard, so a still-loading list would permanently suppress auto-selection. Second, creating a new chat would immediately be deselected because the auto-select effect saw an ID not yet present in the (stale) React Query list; the fix optimistically seeds the new item into the cache before calling setCopilotChatId, closing the race window.

Confidence Score: 5/5

Safe to merge — targeted, well-reasoned fix with no new logic risks introduced.

Both changes are minimal and directly address the described regressions. The optimistic cache seed correctly matches the CopilotChatListItem shape and is replaced by the server response after loadCopilotChats invalidates. The auto-select tracking order change is a simple guard reordering with no side-effects. No P1 or P0 issues found.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant U as User
    participant C as handleCopilotNewChat
    participant QC as QueryClient cache
    participant S as setCopilotChatId
    participant E as autoSelect useEffect
    participant API as /api/copilot/chats

    U->>C: Click "New Chat"
    C->>API: POST /api/copilot/chats
    API-->>C: { id: "new-id" }
    C->>QC: setQueryData — prepend new chat to list
    C->>S: setCopilotChatId("new-id")
    C->>QC: invalidateQueries (loadCopilotChats)
    Note over E: Effect fires with updated list + copilotChatId set
    E->>E: copilotChatId found in list → no deselect
    QC-->>E: server refetch arrives, list still contains new chat
    E->>E: copilotChatId still set → return early ✓

_{Reviews (1): Last reviewed commit: "Keep recent copilot chat open on refresh" | Re-trigger Greptile}