Add RAF cleanup on unmount to prevent state updates after unmount

[Copilot]

• Review feedback from PR #1331 Enhance scripting performance when dragging canvas #1332 comment #2797797038
• Add cleanup useEffect to cancel RAF on component unmount
• Move cleanup useEffect after all event handlers (comment #3889831649)
• Verify lint passes
• Verify tests pass
• Address memory leak risk when component unmounts with pending RAF

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[langonginc]

@copilot Move it after all event handlers

[Copilot]

@copilot Move it after all event handlers

Moved cleanup useEffect to after all event handlers for better code organization. (commit 0927479)

[Copilot]

Pull request overview

Adds a component-unmount cleanup to prevent a pending requestAnimationFrame callback from firing after SvgWrapper has unmounted, avoiding post-unmount setState calls and related leak/warning risk.

Changes:

• Add a useEffect cleanup that cancels any in-flight RAF on unmount.
• Reset rafRef.current to null during cleanup to keep state consistent.