1. Please remove `po` folder: [EGO Review Guidelines: unnecessary files](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#don-t-include-unnecessary-files) You can compile the translation files when you are using `pack`: [GJS Guide: Translation](https://gjs.guide/extensions/development/translations.html#compiling-translations) 2. Move line 9 `extension.js` to enable and null that 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) 3. Don't use sync spawn since that can freeze the shell process (line 30, 31 `extension.js`): [GJS Guide: Subprocesses](https://gjs.guide/guides/gio/subprocesses.html#complete-examples) 4. You can lookup for an app with `Shell.AppSystem`. No need for spawn command: [GJS: Shell.AppSystem](https://gjs-docs.gnome.org/shell16~16/shell.appsystem) 5. Timeout should be removed on destroy (line 288 `extension.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 6. Use `console.*` instead of `log()`: [Port Guide 45: Logging](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#logging) 7. You shouldn't force terminate all `ffmpeg` processes (line 186, 189 and 399 `extension.js`). Your extension is not the only one using it. 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