1. Checking with `this._isDisposed` or `this._isDestroyed` is a bad practice. Your instance should be either destroyed or disposed after calling the destroy or dispose function. and then null the instance, so nothing is going to access it after. That way you will avoid racy situations. 2. Yes, the it's better to use async for catching query info (line 204 `src/Manager/Database.js`). I'm asking that because you should use async when it's possible to avoid freezing the shell process. 3. Timeout should be removed on destroy and before creating a new one (line 1010, 1018 `src/UI/Popup.js`). 4. As mentioned before, you should remove all of the unnecessary try-catch wrappers. If you are using AI, please note we have a rule about AI and that can be easily avoided by not including the unnecessary code: [EGO Review Guidelines: AI](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#extensions-must-not-be-ai-generated)