Review of "llama-dbusd Manager" version 1

Details Page Preview

Manage and switch local llama.cpp models directly from the GNOME top bar. Requires the companion llama-dbusd daemon running on the host system.

Extension Homepage
https://github.com/token419/llama-dbusd-manager

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) error 1 warning 2

Shexli found 3 issues that may need reviewer attention.

EGO-A-004 warning

extension files should not contain excessive ungated console logging

File contains 7 ungated console.log/warn/error calls (threshold: 5).

No excessive logging

  • extension.js:176
    console.error(`[llama-dbusd-manager] D-Bus proxy init failed: ${error.message}`)
  • extension.js:203
    console.error('[llama-dbusd-manager] Failed to initialize D-Bus proxy:', error)
  • extension.js:251
    console.error(`[llama-dbusd-manager] Property synchronization exception: ${error}`)
  • extension.js:345
    console.error(`[llama-dbusd-manager] Failed to unload model ${m}:`, error)
  • extension.js:455
    console.error(`[llama-dbusd-manager] Failed to load model ${m}:`, error)
  • extension.js:504
    console.error('[llama-dbusd-manager] Failed to launch chat URL:', err)
  • extension.js:508
    console.error('[llama-dbusd-manager] Failed to initiate chat URL launch:', err)

EGO-L-002 warning

objects created by extension should be destroyed in disable()

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

Destroy all objects

  • extension.js:90
    this._availableSection = new PopupMenu.PopupMenuSection()
  • extension.js:87
    this._loadedSection = new PopupMenu.PopupMenuSection()
  • extension.js:48
            this._statusHeaderItem = new PopupMenu.PopupBaseMenuItem({
                reactive: false,
                can_focus: false,
                style_class: 'llama-model-item llama-status-header',
            })
  • extension.js:54
            this._statusIcon = new St.Icon({
                style_class: 'llama-small-icon llama-status-icon',
                y_align: Clutter.ActorAlign.CENTER,
            })
  • extension.js:60
            this._statusLabel = new St.Label({
                style_class: 'llama-menu-title',
                x_expand: true,
                y_align: Clutter.ActorAlign.CENTER,
            })

EGO-M-004 error

metadata shell-version must only include plausible stable releases and at most one development release

Field `shell-version` contains invalid values, more than one development release, or implausible future releases.

metadata.json must be well-formed

  • metadata.json
    shell-version=['45', '46', '47', '48', '49', '50', '51']

All Versions

Version Status
2 Unreviewed
1 Rejected

Previous Reviews on this Version

token419 auto- rejected
Auto-rejected because of new version 2 was uploaded