Review of "Linux Usage" version 1

Details Page Preview

Monitor Codex, Claude, and Copilot quotas from the GNOME top bar.

Extension Homepage
https://github.com/KinanLak/linux-usage

No comments.

FAQ

Files

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

Shexli (experimental) warning 4

Shexli found 4 issues that may need reviewer attention.

EGO-X-004 warning

extensions should avoid synchronous file IO in shell code

Shell code should avoid synchronous file IO APIs like `GLib.file_get_contents()` and `Gio.File.load_contents()`.

File Operations

  • providers/catalog.js:21
    file.load_contents(null)

EGO-P-006 warning

unnecessary build and translation artifacts should not be shipped

Compiled GSettings schemas should not be shipped for 45+ packages.

Don't include unnecessary files

  • schemas/gschemas.compiled
    schemas/gschemas.compiled

EGO-P-007 warning

JavaScript files should be reachable from extension.js or prefs.js

Some JavaScript files are not reachable from `extension.js` or `prefs.js` imports.

Don't include unnecessary files

  • helper/cache.js
  • helper/dbus.js
  • helper/helper.js
  • helper/http.js
  • helper/providers/claude.js
  • helper/providers/codex.js
  • helper/providers/copilot.js
  • helper/registry.js
  • helper/utils.js
  • preferences-app.js

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

  • ui/popup.js:597
    actor.connect("destroy", () => global.display.set_cursor(Meta.Cursor.DEFAULT))
  • ui/popup.js:633
    actor.connect("destroy", () => tooltip.destroy())
  • ui/popup.js:589
            actor.connect("enter-event", () => {
                global.display.set_cursor(Meta.Cursor.POINTING_HAND);
                return Clutter.EVENT_PROPAGATE;
            })
  • ui/popup.js:593
            actor.connect("leave-event", () => {
                global.display.set_cursor(Meta.Cursor.DEFAULT);
                return Clutter.EVENT_PROPAGATE;
            })
  • ui/popup.js:607
            actor.connect("notify::hover", () => {
                if (actor.hover) {
                    tooltip.set({
                        text: label,
                        visible: true,
                        opacity: 0,
                    });
                    const [stageX, stageY] = actor.get_transformed_position(

All Versions

Version Status
5 Unreviewed
4 Unreviewed
3 Rejected
2 Rejected
1 Unreviewed

Previous Reviews on this Version

Kinaneuh posted a review
Kinaneuh posted a review
Can be dismissed since new versions exist