1. Please add GitHub or GitLab url to the `metadata.json` so users can report the bugs: [EGO Review Guidelines: url](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#metadata-json-must-be-well-formed) 2. Don't use sync spawn since that can freeze the shell process (line 71 `utils.js`): [GJS Guide: Subprocesses](https://gjs.guide/guides/gio/subprocesses.html#complete-examples) 3. Use `console.*` instead of `log()`: [Port Guide 45: Logging](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#logging) 4. Please remove `main`, `type`, `stylesheet` and `prefs` from `metadata.json`. Not needed. 5. Timeout should be removed on destroy (line 56 `extension.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 6. Use `this.openPreferences()` from the entry point instead of spawn command (line 132 `extension.js`). 7. Move line 171 `extension.js` to enable and null it out in disable: - [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) 8. Move line 179 to the `PasswordManager.destroy()`. If you need any help with your extension you can ask us on: - [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions