Review of "System HUD" version 31

Details Page Preview

Clean UI system monitor for CPU, GPU, memory, storage, network, power.

Extension Homepage
https://github.com/LalaloyXyz/System-info-HUD

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) manual_review 1

Shexli found 1 issue that may need reviewer attention.

EGO-A-005 manual_review

extensions should not access the clipboard directly

Direct clipboard access via `St.Clipboard.get_default()` requires reviewer scrutiny.

Review Guidelines

  • ui/uiManager.js:1109
    St.Clipboard.get_default()

All Versions

Previous Reviews on this Version

JustPerfection rejected
1. Remove unnecessary try catch wrappers: - line 28 `prefs.js` [Extension Best Practices Guidelines: Unnecessary try-catch](https://gjs.guide/extensions/review-guidelines/best-practices.html#avoid-unnecessary-try-catch-wrappers) 2. Selective disable isn't allowed: - line 56 `clients/workerClient.js` - line 1375 `ui/uiManager.js` 3. You cannot use old imports in 45+ extensions (line 31 `companion/service.js`). 4. `companion/worker.js` and what it uses are standalone apps. Standalone gjs script files should start with `#!@GJS@ -m` and it's better to move them to `app` folder. 5. Where do you call: - `viewport.destroy()` (line 1390 `ui/uiManager.js`) - `graph.destroy()` (line 67 `ui/updateData.js`)