1. Please disconnect on disable (line 42, 102 `extension.js`): [Review Guidelines: Disconnect all signals](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#disconnect-all-signals) 2. Line 103-105 `extension.js` should be happening on disable not on `this.connect('destroy',...`. 3. The `preferences/window_picker/` folder is running inside the shell process. I recommend to don't hold those modules in the `preferences` folder since that can misleading the reviewers.

Are you sure about number 2? It seems like this is something that should happen every time a window preview is destroyed, not just when the extension is disabled?

Oh! I though you are using destroy signal for the `RoundedWindowCornersReborn` class. The transpile is removing all of the empty lines and that can be confusing for the reviewers. Please use empty lines between functions.

What about number 1 then? It disconnects automatically when the callback is executed, does it still have to be disconnected in disable()? > Please use empty lines between functions. This isn't something that the typescript compiler supports, unfortunately. I'll look into alternative typescript transpilers and try to switch to one that preserves empty lines.

line 42 should be disconnected on disable.