1. No need to use `this.initTranslation()`: - line 36 `extension.js` - line 34 `prefs.js` > Consider this method deprecated. > Only specify gettext-domain in metadata.json. > GNOME Shell can automatically initiate the translation for you > when it sees the gettext-domain key in metadata.json. [Port Guide 45: Extension Utils](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#extensionutils) 2. Don't hold instances of objects as static if you don't want to clean that up in disable or window close request: - line 41 `extension.js` - line 39 `prefs.js` [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) btw, for `prefs.js`, window close request signal is: ```js window.connect('close-request', () => { // destroy and null up here }); ``` 3. Wrong gettext domain (line 2 `schemas/org.gnome.shell.extensions.astra-monitor.gschema.xml`). 4. Unused imports: - line 20 `src/monitor.js` - line 19 `src/menu.js` - line 26 `src/header.js` - line 22 `src/grid.js` - line 20, 24 `src/graph.js` - line 21 `src/storage/storageGraph.js` - line 18 `src/sensors/sensorsMonitor.js` - line 18, 20 `src/sensors/sensorsMenu.js` - line 18 `src/memory/memoryMonitor.js` 5. `lookupByUUID()` is a bad practice (line 110 `src/menu.js`). You can send `this` from the entry point to the class needing it when it's possible (dependency injection).