I did notice i was loading pushoverAPI object immediately whenever generalPage.js is loaded when prefs.js: fillPreferencesWindow is invoked, which is indeed not per guidelines. Have adjusted.

Still creating it as global, not local. The thing is, when the window is closed, that global variable won't get garbage collected. To fix that: - Just make it local (if you are not accessing `pushoverAPI` from out side of the module) or; - Use dependency injection for `GeneralPage`. btw, there are some unnecessary comments in the code that needs to be removed too. Log should be only available on debug mode: https://gjs.guide/extensions/review-guidelines/review-guidelines.html#no-excessive-logging