fix(collab-doc): stream every external file write into open editors, not just edit_content

[waleedlatif1]

Summary

Fixes a real bug found via live testing: a copilot/mothership edit to a markdown file that another user has open did not stream into their editor.

Root-caused from the running server logs + local Redis:

• The mothership's "Prepend sentence" ran read + file + function_execute — zero apply-edit calls.
• Redis confirmed the prepended text never entered the live doc stream.

The live-doc merge bridge (mergeEditIntoLiveFileDoc) was wired into the edit_content tool only. Every other server-side write — the file tool (/api/tools/file/manage), function_execute (/api/function/execute via writeWorkspaceFileByPath), create_file overwrite, and the PUT /content route — went straight to updateWorkspaceFileContent and skipped the merge. The durable file changed (list refreshed) but the open editor never updated.

Fix — centralize at the one chokepoint every external writer shares

• updateWorkspaceFileContent gains an opt-out { syncLiveDoc } (default on); after the durable write it merges markdown writes into any open collaborative doc (best-effort, no-op when nobody has it open). Any current or future writer is covered automatically.
• persist opts out — it IS the doc→markdown projection, so merging back would be a persist→merge→persist self-loop.
• create_file opts its empty shell out (real content arrives via a later write) so an open editor never flickers to empty on overwrite.
• edit_content drops its now-redundant explicit merge call.
• Binary writers (image/video/audio/ffmpeg/download) are naturally excluded — the merge is gated to markdown.

Note

Bumps the api-validation route baseline 994 → 996 to match the true route count already on realtime-rooms (pre-existing ratchet drift from an earlier merge — not routes added by this PR; verified against a clean tree).

Type of Change

• Bug fix

Testing

• New unit tests: markdown write merges via the chokepoint; syncLiveDoc:false and non-markdown writes do not.
• Full gates green: sim + realtime tsc, lint, api-validation, boundaries, prune, 35 + 74 targeted tests.
• Root cause verified live (server logs + Redis stream). End-to-end 2-browser verify still owed on the branch.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

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

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
docs	Skipped		Jul 30, 2026 12:15am

[cursor]

PR Summary

Medium Risk
Touches the shared file-write path and live-doc merge behavior for all markdown updates; incorrect opt-in/out could cause editor flicker or persist loops, though defaults and tests target the main failure modes.

Overview
Fixes copilot and other server-side markdown writes not appearing in open collaborative editors — only edit_content used to call mergeEditIntoLiveFileDoc; durable storage updated but the live Yjs doc did not.

updateWorkspaceFileContent is now the single chokepoint: after a successful blob/DB update it best-effort merges markdown into any open file doc via mergeEditIntoLiveFileDoc (default syncLiveDoc: true). edit_content drops its duplicate merge. writeWorkspaceFileByPath forwards syncLiveDoc on overwrites.

Opt-outs: relay persistFileDoc and create_file empty shells pass syncLiveDoc: false to avoid persist→merge loops and briefly blanking an open editor.

Editor: collaborative setEditable runs in a queueMicrotask so Yjs-driven readiness updates do not trigger flushSync during React render.

Adds unit tests for merge vs opt-out vs non-markdown; bumps API route validation baseline 994 → 996 (pre-existing drift).

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

[greptile-apps]

Greptile Summary

Centralizes live collaborative-doc merge on every durable workspace file write so open markdown editors receive external updates.

• updateWorkspaceFileContent merges markdown writes into open collab docs by default (syncLiveDoc).
• persist and empty create_file shells opt out; edit_content drops its redundant explicit merge.
• Defers TipTap setEditable via microtask to avoid flushSync-during-render; bumps api-validation route baseline 994→996.

Confidence Score: 5/5

This PR appears safe to merge; no blocking failures remain from prior or current review.

No blocking failure remains.

Important Files Changed

Filename	Overview
apps/sim/lib/uploads/contexts/workspace/workspace-file-manager.ts	Adds default-on markdown live-doc merge after successful durable update; opt-out via syncLiveDoc false.
apps/sim/lib/collab-doc/persist.ts	Opts persist out of live merge to avoid persist→merge→persist loops.
apps/sim/lib/copilot/tools/server/files/edit-content.ts	Removes redundant per-tool merge; relies on shared chokepoint.
apps/sim/lib/copilot/tools/server/files/create-file.ts	Opts empty-shell create out of merge to avoid blanking open editors.
apps/sim/lib/copilot/vfs/resource-writer.ts	Forwards optional syncLiveDoc on overwrite path.
apps/sim/app/workspace/[workspaceId]/files/components/file-viewer/rich-markdown-editor/rich-markdown-editor.tsx	Defers setEditable to a cancellable microtask to avoid flushSync during React render.
apps/sim/lib/uploads/contexts/workspace/workspace-file-storage-accounting.test.ts	Covers default merge, syncLiveDoc false, and non-markdown skip.
scripts/check-api-validation-contracts.ts	Ratchets route baseline to 996 for pre-existing drift on the base branch.

Sequence Diagram

sequenceDiagram
  participant Writer as External writer<br/>(file tool / function_execute / PUT content / edit_content)
  participant UWC as updateWorkspaceFileContent
  participant Storage as Durable storage + DB
  participant Merge as mergeEditIntoLiveFileDoc
  participant Editor as Open collab markdown editor

  Writer->>UWC: content Buffer (+ optional syncLiveDoc)
  UWC->>Storage: upload + finalize metadata
  alt "syncLiveDoc !== false and markdown"
    UWC->>Merge: fileId + utf-8 markdown
    Merge->>Editor: CRDT merge (best-effort)
  else syncLiveDoc false or non-markdown
    Note over UWC: skip live merge
  end

Loading

_{Reviews (2): Last reviewed commit: "fix(collab-doc): defer setEditable out o..." | Re-trigger Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

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