Skip to content

fix(wasm): share CSV and Markdown document sessions - #1109

Merged
andiwand merged 2 commits into
mainfrom
review/91-wasm-document-sessions
Oct 5, 2026
Merged

andiwand merged 2 commits into
mainfrom
review/91-wasm-document-sessions

Conversation

@andiwand

@andiwand andiwand commented Oct 5, 2026 •

Copy link
Copy Markdown
Member

🤖 Generated with Claude Code

CSV advertised edit/save support but WebAssembly document queries and edits attempted a DocumentFile cast and failed. Cache CSV and Markdown through their own document interfaces, and render from that same cached document. Commit service initialization only after its views are available.

The CSV integration test and added Markdown assertions fail on the parent. All 68 Node/WebAssembly tests pass, including CSV edits reflected in both rendered HTML and saved bytes.

@andiwand
andiwand force-pushed the review/90-wasm-reentrancy branch from ebf00b3 to 4baecb7 Compare October 5, 2026 21:32
Base automatically changed from review/90-wasm-reentrancy to main October 5, 2026 21:32
andiwand and others added 2 commits October 5, 2026 23:33
warm and document_of each listed the document, CSV and Markdown files; a
new kind added to one would have been missed in the other.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Gb1fLafqqzehpqPuU6uBfn
@andiwand
andiwand force-pushed the review/91-wasm-document-sessions branch from 960b746 to 960e8fa Compare October 5, 2026 21:33
@andiwand
andiwand merged commit 82c8c30 into main Oct 5, 2026
23 checks passed
@andiwand
andiwand deleted the review/91-wasm-document-sessions branch October 5, 2026 21:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant