GeezerVison

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Apr 15, 2026 11:45pm

PR Summary

Medium Risk
Touches core chat streaming and cancellation paths plus client-side caching, which can affect message ordering/duplication and queued follow-up delivery. Bright Data changes add polling and output-shaping that may impact runtime behavior and timeouts.

Overview
Improves mothership/copilot chat streaming by introducing buildEffectiveChatTranscript to merge Redis stream snapshots into the returned chat messages, and updating both copilot and mothership chat GET APIs to normalize persisted messages and append a consistent “live assistant” tail while a stream is active.

Refactors the home useChat hook to drive streaming updates through the React Query TaskChatHistory cache (stable live-assistant:<streamId> IDs), improve stop/cancel handling (mark executing tools as cancelled), and add a sessionStorage-backed queued-send handoff to safely resume queued follow-ups across navigation/stop boundaries. The /api/copilot/chat/stop route now always persists a cancelled completion block (and uses generateId), with a new route test.

Fixes Bright Data tool behavior by polling the async Discover API until completion (platform timeout-aligned), echoing key output fields from request params across several tools, and adding taskId to Discover outputs; also expands resource attachment types, tweaks handleTaskStatusEvent to invalidate only the workspace task list, and includes minor doc/registry/lockfile updates.

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

Greptile Summary

This PR delivers two independent fixes: (1) BrightData Discover now correctly polls for async task results via postProcess and echoes input params back through output fields across all BrightData tools; (2) Mothership streaming is refactored so the live assistant transcript is materialized from Redis-backed stream events (effective-transcript.ts) and pushed directly into the React Query cache, replacing the previous local useState approach and eliminating the need for server refetches during an active stream.

Confidence Score: 5/5

Safe to merge; all findings are P2 quality-of-life improvements that do not affect the primary streaming or tool-execution paths.

Three P2 findings: polling elapsed-time precision, no retry on transient HTTP errors in BrightData Discover, and reduced cache invalidation for passive observers. None break core functionality — the streaming experience is correctly handled, and the BrightData issues only affect edge-case reliability in the polling loop.

apps/sim/tools/brightdata/discover.ts — timing and error-handling in the polling loop

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant Client as useChat browser
    participant RQ as React Query Cache
    participant Server as Next.js API
    participant Redis as Redis stream events

    Client->>Server: POST copilot/chat send message
    Server-->>Client: SSE stream text/tool/complete events
    Client->>RQ: upsertTaskChatHistory on each flush

    alt Stream completes normally
        Client->>Client: finalize
        Client->>RQ: invalidateChatQueries includeDetail true
        RQ->>Server: GET copilot/chat
        Server->>Redis: fetch streamSnapshot
        Server->>Server: buildEffectiveChatTranscript
        Server-->>RQ: persisted messages plus live snapshot
    end

    alt SSE task_status event any tab
        Server-->>Client: task_status event
        Client->>RQ: invalidateQueries taskKeys.list workspaceId only
    end

_{Reviews (1): Last reviewed commit: "fix(mothership): chat stream structuring..." | Re-trigger Greptile}