fix: improve cleanup logic

[caio-pizzol]

This commit refines the cleanup process in the SuperDocTemplateBuilder component by ensuring that the instance is properly referenced before calling the destroy method. It also resets additional references to prevent memory leaks.

[linear]

SD-543 Fix memory leak: clear trigger refs on unmount

Summary

Ensure cleanup refs are cleared on unmount to prevent memory leaks when component is destroyed during active menu interaction

Tasks

• Add cleanup for triggerCleanupRef.current in main useEffect return
• Add cleanup for menuTriggerFromRef.current in main useEffect return
• Clear superdocRef.current references before setting to null on unmount
• Test unmount during active menu state to verify no console warnings

Notes

Problem: When component unmounts while menu is open or trigger is active, refs hold closures that reference editor/DOM that no longer exist

Current code (src/index.tsx ~line 400):

return () => {
  if (superdocRef.current) {
    if (typeof superdocRef.current.destroy === 'function') {
      superdocRef.current.destroy();
    }
    superdocRef.current = null;
  }
};

Missing: Clear triggerCleanupRef.current and menuTriggerFromRef.current before destroying

Fix:

return () => {
  // Clear trigger refs
  triggerCleanupRef.current = null;
  menuTriggerFromRef.current = null;
  
  // Destroy editor
  if (superdocRef.current) {
    if (typeof superdocRef.current.destroy === 'function') {
      superdocRef.current.destroy();
    }
    superdocRef.current = null;
  }
};

Verification: Mount component → type trigger → unmount → verify no memory leaks in Chrome DevTools

[superdoc-bot]

🎉 This PR is included in version 0.2.0-next.2 🎉

The release is available on:

• npm package (@next dist-tag)
• GitHub release

Your semantic-release bot 📦🚀

[superdoc-bot]

🎉 This PR is included in version 0.2.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀