Review of "App Manager Remover" version 2

Details Page Preview

A unified application manager that lists user-installed apps from Flatpak, Snap, and Deb sources with one-click uninstall. Hides system components using a 6-layer filtering strategy. Uninstallation is confirmed via a modal dialog and authenticated through pkexec (PolicyKit).

Extension Homepage
https://github.com/Lokoyote/App-manager-remover

No comments.

Diff Against

Files

Note: Binary files aren't shown on the web site. To see all files, please download the extension zipfile.

Shexli (experimental) warning 2

Shexli found 2 issues that may need reviewer attention.

EGO-X-002 warning

extensions should not use synchronous subprocess APIs in shell code

Shell code should avoid synchronous subprocess APIs like `GLib.spawn_command_line_sync()` and `GLib.spawn_sync()`.

Complete Examples

  • extension.js:67
    GLib.spawn_command_line_sync(cmdString)

EGO-L-003 warning

signals connected by extension should be disconnected in disable()

Signals assigned in `enable()` are missing matching disconnect calls in `disable()` or its helper methods.

Disconnect all signals

  • extension.js:987
    this._win.connect('closed', () => this._bk.hide())

All Versions

Version Status
4 Active
3 Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

fmuellner rejected
Spawning a command synchronously means that the whole desktop freezes until the command returns. Those are definitely not commands that you should run that way, so rejecting. Please change the extension to spawn those commands asynchronously.