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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jun 20, 2026 10:19pm

PR Summary

Medium Risk
Touches shared editor state reconciliation and workspace chrome hydration; incorrect baseline logic could show wrong content or unlock editing during agent writes, but changes are covered by new unit tests.

Overview
Improves agent file streaming and rich markdown editing, and fixes sidebar collapse painting at the wrong width on first paint.

Streaming & reconciliation: Text editor state gains hasBaseline so when a stream starts before the initial fetch (opening a file mid-agent-edit), pre-edit server content is adopted as the baseline instead of finalizing to stale text. Display uses paced useSmoothText while the reducer keeps true content for saves; isAgentEditing plus a mothership grace lock keeps editors read-only across agent edit gaps. Loading no longer unmounts the editor if the stream settles before fetch completes.

Rich markdown: Large streams throttle full re-parses (~40k+ chars). Bubble menu is Linear-style (fixed anchor, reveal on drag-release, portal to body); new link hover card and shared toolbar buttons. Mod+K is yielded from global commands via data-owned-shortcuts. Task-list checkboxes match design-system styling.

Sidebar: WorkspaceChrome is the single collapse source (data-collapsed + CSS --sidebar-width: 51px); Sidebar takes isCollapsed as a prop.

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

@greptile

Greptile Summary

This PR adds smooth streaming support for the rich-markdown editor, performance improvements for large files, and several UX fixes including a new link-hover card, a drag-select–reveal bubble menu, and a sidebar collapse refactor that eliminates FOUC on SSR.

• Streaming UX: useSmoothText gains a streaming-start snap to avoid replaying a pre-existing document word-by-word; text-editor-state adds hasBaseline to distinguish a real baseline from the initial placeholder, preventing premature finalization to stale content when streaming begins before the initial fetch.
• Rich-editor improvements: New LinkHoverCard component with edit/copy/remove actions; bubble menu refactored with fixed-position anchoring and drag-select reveal; ToolbarButton extracted into a shared component; large-file stream throttle (>40 KB → 120 ms) prevents main-thread saturation.
• Sidebar & chrome: Collapse derivation lifted to WorkspaceChrome, propagated as a prop; data-collapsed CSS variable set from the server-rendered attribute prevents the expand→collapse flash on first paint.

Confidence Score: 4/5

Safe to merge with two targeted fixes: the reconciling-path race in text-editor-state.ts, and the draftHref reset in the new link-hover card.

The streaming state machine correctly fixes the streaming-active path with the hasBaseline guard, but the parallel reconciling path still uses the bare fetchedContent !== savedContent check — leaving the same stale-content race for a fast stream that ends before the fetch. The new LinkHoverCard does not reset draftHref when activeLink changes while editing, which can silently apply the wrong URL to the wrong link. The remaining changes are well-reasoned and correct.

apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/text-editor-state.ts (reconciling path, line 151) and apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/menus/link-hover-card.tsx (onOver handler).

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[streamingContent arrives] --> B{isNextStreaming?}
    B -- yes --> C{hasBaseline?}
    C -- no, fetch arrived --> D[adoptBaseline]
    C -- yes --> E{hasFetchedAdvanced?}
    E -- yes --> F[finalize: phase=ready]
    E -- no --> G[phase=streaming]
    D --> G
    B -- no --> H{phase=streaming/reconciling?}
    H -- yes --> I{fetchedContent defined?}
    I -- yes --> J{hasFetchedAdvanced? missing hasBaseline guard}
    J -- true --> K[finalize]
    J -- false --> L[phase=reconciling]
    I -- no --> L
    H -- no --> M[ready/uninitialized]
    style J fill:#ffcccc,stroke:#cc0000

%%{init: {'theme': 'base', 'themeVariables': {"darkMode": true, "background": "#0d1117", "primaryColor": "#21262d", "primaryTextColor": "#e6edf3", "primaryBorderColor": "#8b949e", "lineColor": "#8b949e", "textColor": "#e6edf3", "edgeLabelBackground": "#161b22", "actorBkg": "#21262d", "actorBorder": "#8b949e", "actorTextColor": "#e6edf3", "actorLineColor": "#8b949e", "signalColor": "#8b949e", "signalTextColor": "#e6edf3", "noteBkgColor": "#373320", "noteBorderColor": "#d4a72c", "noteTextColor": "#f0e6c0", "labelBoxBkgColor": "#21262d", "labelBoxBorderColor": "#8b949e", "labelTextColor": "#e6edf3", "loopTextColor": "#e6edf3", "activationBkgColor": "#30363d", "activationBorderColor": "#8b949e"}}}%%
flowchart TD
    A[streamingContent arrives] --> B{isNextStreaming?}
    B -- yes --> C{hasBaseline?}
    C -- no, fetch arrived --> D[adoptBaseline]
    C -- yes --> E{hasFetchedAdvanced?}
    E -- yes --> F[finalize: phase=ready]
    E -- no --> G[phase=streaming]
    D --> G
    B -- no --> H{phase=streaming/reconciling?}
    H -- yes --> I{fetchedContent defined?}
    I -- yes --> J{hasFetchedAdvanced? missing hasBaseline guard}
    J -- true --> K[finalize]
    J -- false --> L[phase=reconciling]
    I -- no --> L
    H -- no --> M[ready/uninitialized]
    style J fill:#ffcccc,stroke:#cc0000

_{Reviews (5): Last reviewed commit: "fix(rich-md-editor): lock the editor imm..." | Re-trigger Greptile}

@cursor review

@greptile

@greptile

@cursor review

@greptile

@cursor review