fix: simplify hotel page copy without render loop - #18
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars. ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: 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 |
🌍 Web previewPreview is pinned to commit |
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
Summary
Root cause
The previous copy layer observed the entire document with
characterData: truewhile also assigning text inside the observer callback.discoveryMeta.textContentwas rewritten on every callback, which could continually retrigger MutationObserver and cause the hotel page to lag, freeze, or render incorrectly.Fix
Copy cleanup is now idempotent and scoped. There is no global DOM observer.