Fix segments thinking they have changed when switching steps

[sarahentzel]

No description provided.

[Copilot]

Pull request overview

Addresses unintended “segments changed” behavior when switching steps by suppressing region callbacks during WaveSurfer teardown.

Changes:

• Add a destroy/teardown guard (destroyingRef + prepareForDestroy) to prevent region-removed side effects during destroy/unmount.
• Hook WaveSurfer destroy/unmount paths to set the destroy guard and adjust region clearing behavior.
• Reduce controlsRef effect dependencies and clean up minor lint/comment noise.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
src/renderer/src/crud/useWavesurferRegions.tsx	Introduces destroyingRef + prepareForDestroy and guards region-removed follow-up work during teardown.
src/renderer/src/crud/useWaveSurfer.tsx	Calls prepareForDestroy() on WaveSurfer destroy/unmount; changes how clearRegions is invoked from wsClear.
src/renderer/src/components/WSAudioPlayer.tsx	Changes controlsRef setup effect dependency handling (now lint-disabled and reduced deps).
src/renderer/src/components/App/HeadStatus.tsx	Removes an unnecessary exhaustive-deps disable comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.