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:13pm

PR Summary

Medium Risk
Changes task update signaling from per-chat status payloads to coarse workspace-level list invalidations, which may affect UI freshness/performance and could miss detail refreshes in edge cases.

Overview
Unifies task SSE notifications around “workspace task list changed” events. Server-side publishers across chat create/rename/delete/stop and stream lifecycle now call taskPubSub.publishTaskListChanged({ workspaceId }) and stop sending chatId/type payloads.

The mothership SSE endpoint subscribes via onTaskListChanged and emits empty task_status events, and the client useTaskEvents handler is simplified to always invalidate taskKeys.lists() (no payload parsing or chat-detail invalidation). Separately, use-chat refactors invalidation into invalidateTaskHistory vs invalidateTaskList, and avoids auto-refetching active chat history on finalize (refetchType: 'none'), updating tests/mocks accordingly.

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

Greptile Summary

This PR simplifies the task event pub/sub system by replacing granular TaskStatusEvent payloads (with chatId, type) with a single workspace-scoped TaskListChangedEvent. The core behavioral fix is in finalize inside use-chat.ts: the detail query is now marked stale with refetchType: 'none' rather than triggering an immediate refetch, which prevents the client from racing the server-side DB write to completion.

Confidence Score: 5/5

Safe to merge — the simplification is consistent across all publishers and the single subscriber, and the refetchType:'none' fix correctly addresses the DB-write race condition.

All callers of the old publishStatusChanged API are updated, tests cover the new behaviour, and the intentional asymmetry between finalize (refetchType:'none') and stop (refetchType:'active') is correctly implemented. No P0/P1 findings.

No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Client as Browser (use-chat.ts)
    participant API as API Route
    participant PubSub as taskPubSub (Redis)
    participant SSE as SSE Endpoint
    participant RQ as React Query Cache

    Note over Client,RQ: Normal stream completion (finalize)
    Client->>RQ: invalidateTaskHistory('none') stale, no refetch
    Client->>RQ: invalidateTaskList()
    API->>PubSub: publishTaskListChanged({workspaceId})
    PubSub-->>SSE: task:status_changed event
    SSE-->>Client: task_status {}
    Client->>RQ: invalidateQueries(taskKeys.lists())

    Note over Client,RQ: Manual stop (stopChat)
    Client->>API: POST /api/copilot/chat/stop
    API->>PubSub: publishTaskListChanged({workspaceId})
    API-->>Client: 200 OK
    Client->>RQ: invalidateChatQueries() - invalidateTaskHistory('active') + invalidateTaskList()
    PubSub-->>SSE: task:status_changed event
    SSE-->>Client: task_status {}
    Client->>RQ: invalidateQueries(taskKeys.lists())

_{Reviews (1): Last reviewed commit: "simplify event" | Re-trigger Greptile}