1. Please remove `schemas/gschemas.compiled`. Not needed for 45+ packages. 2. Please remove 42-44. Not compatible with those versions. 3. You don't need to use any parameters for `this.getSettings()` since you already specified that in `metadata.json`. 4. Already have `window` in the entry point, I recommend to not use `get_root()` (line 37 `prefs.js`). 5. What's the reason for that if condition (line 18-19 `common.js`)? Why not simply 6. Please use aysnc for reading file content since you are in shell process (line 107 `common.js`): [Gio.File.load_contents_async](https://gjs-docs.gnome.org/gio20~2.0/gio.file#method-load_contents_async) 7. Don't add anything to the `extension` instance outside of the default class. That makes the cleanup review process super hard (line 55-89 `extension.js`). 8. Also create `uninit` that you call on disable and window close request in prefs (line 11 `common.js`): ```js window.connect('close-request', () => { // clean up here }); ```