memory-leak-audit

Replace raw addEventListener or .onclick assignments with addDisposableListener() and register the result via this._register(). For one-time lifecycle events, wrap the event with Event.once() so the listener removes itself after the first invocation.

Do not call this._register() inside methods that run repeatedly, since each call adds another listener to the class store. Instead use a MutableDisposable field and assign the new listener to its .value, which disposes the previous one automatically.

Use Event.once for events that should fire only once, such as disposal, close, or first-change notifications. A regular subscription stays registered forever after firing, leaking the listener and its captured references.

Call ensureNoDisposablesAreLeakedInTestSuite() at the top of the test suite so leaked disposables fail the test automatically. You can also compare listener counts before and after repeated operations to confirm counts stay stable.

A DisposableStore leaks if nothing disposes it when its model goes away. Register model.onWillDispose(() => store.dispose()) inside the store itself so the store cleans up all its subscriptions when the model lifecycle ends.