Upgrade yorkie-js-sdk to 0.7.6#150
Conversation
Fixes a bug where inline styles (bold, italic, etc.) were lost on the peer side after a block split. In 0.7.5, cloneElement passed undefined for attributes, so the split-created inline node had no styles. The local user saw correct rendering because the cache masked the issue, but peers re-parsing from the Yorkie Tree saw unstyled text. Key changes in 0.7.6: - cloneElement now copies attributes via deepcopy (PR #1224) - Prevent race conditions between sync loop and detach (PR #1228) - Unify presence event emission with reconcilePresence (PR #1229) Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
The yorkie-js-sdk 0.7.6 upgrade increased the vendor-yorkie chunk from ~697 KB to ~701 KB, exceeding the previous 700 KB limit. Co-Authored-By: Claude Opus 4.6 (1M context) <[email protected]>
|
Caution Review failedPull request was closed or merged during review 📝 WalkthroughWalkthroughThis PR bumps Yorkie dependencies ( Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~15 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Verification: verify:selfResult: ✅ PASS in 117.8s
Verification: verify:integrationResult: ✅ PASS |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Summary
@yorkie-js/sdkand@yorkie-js/reactfrom 0.7.5 to 0.7.6Bug fixed
In 0.7.5,
cloneElementpassedundefinedfor attributes when splitting a Tree element node. This caused inline styles (bold, italic, fontSize, etc.) to be lost on the peer side after a block split:remote-change, saw unstyled textReproduced on wafflebase.io (0.7.5) — after bold "abcd" is split at "ab|cd", the peer's Tree shows
bold: "true"on "ab" but no attributes on "cd".In 0.7.6,
cloneElementcopies attributes viathis.attrs?.deepcopy().Other improvements from 0.7.6
Test plan
pnpm verify:fast— 101 frontend tests pass (5 new split-style tests added)pnpm verify:self— all builds + chunk gate pass🤖 Generated with Claude Code
Summary by CodeRabbit