1. Use `console.*` instead of `log()`. Logs should be only available on debug mode. For debug use `console.debug()` instead of `console.log()`: [Port Guide 45: Logging](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#logging) 2. You already have destroy function. Move eveything from line 46-107 `mirroredIndicatorButton.js` to the `destroy()`. 3. Timeout should be removed on destroy and alos before creating a new one: - line 412, 459 `mirroredIndicatorButton.js` - line 760, 768, 811, 940 `mmoverview.js` - line 192, 474, 527, 583 `screenshotPatch.js` 4. Unnecessary checks and try-catch wrappers still happening in this versions. Please note, we have a rul for AI: [EGO Review Guidelines: AI](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#extensions-must-not-be-ai-generated) 5. Avoid sending large diff. It's hard to review.

The timeouts in mmoverview.js and screenshotPatch.js are one-shot timeouts that return GLib.SOURCE_REMOVE. These don't need pre-creation cleanup because they self-terminate and aren't stored as instance variables. Should i do it other way ?

sorry for the large diff since i do fix a lot of bugs and add two big feature in it

Yes, those timeouts should be also removed. As we mentioned in the guidelines: > Note that all sources MUST be removed in disable(), > even if the callback function will eventually return false > or GLib.SOURCE_REMOVE. [EGO Review Guidelines: Remove main loop sources](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources)