1. Please remove `init()` method (line 1238 prefs.js). Not needed. 2. You are doing too much in the constructor of the default class you are exporting (line 3279 extension.js): That is breaking three ego rules: - https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-initialization-for-static-resources - https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects - https://gjs.guide/extensions/review-guidelines/review-guidelines.html#disconnect-all-signals Please move them to enable and null those out/ destroy / disconnect signals in disable. 3. `isEnabled` is a bad practice (line 3348 extension.js). You really don't need it to check something like line 3315, 3300 and 3429. 4. Please remove the import versions. 5. Don't use `var`. Use `let` and `const` instead. 6. Extensions cannot use `run_dispose()` (line 252 and 405 extension.js): > This function should only be called from object system implementations. https://gjs-docs.gnome.org/gobject20~2.0/gobject.object#method-run_dispose 7. Please use `TextDecoder` instead of `byteArray` (line 222 extension.js): https://developer.mozilla.org/en-US/docs/Web/API/TextDecoder

Thank you for the review, I will update the code