1. Also null out in disable: ```js settingsManager = null; ``` [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 2. You cannot create instance of objects in the global scope (line 109 `utils/SettingsManager.js`): - [EGO Review Guidelines: Initialization](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-initialization-for-static-resources) - [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 3. `lookupByUUID()` is a bad practice (line 31 `utils/SettingsManager.js`). You can send `this` from the entry point to the class needing it when it's possible (dependency injection). 4. Please remove line 304-306 `extension.js`. Not needed. 5. Please remove the timeout in the same class (line 298 `extension.js`). To do that, use this inside `indicator` use: ```js destroy() { if (this._refreshTimeout) { clearTimeout(this._refreshTimeout); } super.destroy(); } `` 6. Please use a less generic name for the default class you are exporting (line 40 `prefs.js`).