Regression: serializeHtml now 10x slower

[shaungrady]

Description

Migrating from 30.5.3 to 33.0.0, I'm noticing a huge decrease in the performance of the serializeHtml call. It looks like it occurred with the change in this PR: #3149

I've attached below profiles for the serializeHtml call in the old and new versions with a document consisting of 107 root child nodes.

v30.5.3 (108ms)

v33.0.0 (1.2s)

Steps to Reproduce

Create a decently sized document of 100+ nodes, execute the serialize call.

Expected Behavior

Environment

• slate: 0.103.0
• slate-react: 0.102.0
• react: 18.2.0
• browser: Firefox

Bounty

Click here to add a bounty via Algora.

Funding

• You can sponsor this specific effort via a Polar.sh pledge below
• We receive the pledge once the issue is completed & verified

[zbeyens]

Thanks for the report. Let’s revert this for now and explore a better solution. Please open a PR at your earliest convenience.

[shaungrady]

Sounds good, @zbeyens! I'm not sure how soon I can get to this, so please don't wait on me if you need this addressed urgently.

[dimaanj]

@shaungrady Which version of react you are using btw?

[shaungrady]

@dimaanj Good question: 18.2.0