Creating settings instance many times is a bad practice. Please get it only once and use that instance (for both `prefs.js` and `extension.js`).

For `extension.js`, doing this would be enough: ```js enable() { this.settings = this.getSettings(); } disable() { this.settings = null; } ``` and for `prefs.js` as I explained before, you can cleanup on window close request.