1. Please don't put git urls into the description. 2. Also destroy and null out in disable: ```js this._configManager?.destroy(); this._configManager = null; ``` [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 3. Always better to create destroy function instead of registering the signal (line 46 `plugins/datetime/widget.js`): ```js destroy() { if (this._timeoutId) { GLib.source_remove(this._timeoutId); this._timeoutId = null; } super.destroy(); } ``` 4. Use the import at top of the class (line 143 `shared/services/servicePluginRegistry.js`). 5. I recommend to move these file to a more obvious folder so it obvious that they are only used in `prefs.js` process: - `shared/components/componentProfileRow.js` - `shared/utilities/utilityIconGtk.js`