Skip to content

Add Github Copilot support to the ai-usagebar plugin - #672

Merged
ItsLemmy merged 2 commits into
noctalia-dev:mainfrom
dirkvdb:main
Sep 9, 2026
Merged

Add Github Copilot support to the ai-usagebar plugin#672
ItsLemmy merged 2 commits into
noctalia-dev:mainfrom
dirkvdb:main

Conversation

@dirkvdb

@dirkvdb dirkvdb commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Plugin

  • Id: felipeartur/ai-usagebar
  • New plugin
  • Update to an existing plugin (version bumped in plugin.toml)

What it does

Adds GitHub Copilot as a selectable provider in the ai-usagebar plugin. The provider is displayed as Github Copilot and uses the existing GitHub Copilot icon mapping.

ai-usagebar >= 1.12.0 added Github Copilot support

External dependencies

  • ai-usagebar — provides usage data for configured AI providers.

Testing

  • Tested on Niri
  • Tested on Hyprland
  • Tested on Sway
  • Tested on another compositor:
  • Noctalia version tested against: 5.0.1
  • Plugin API level: 22

Validation performed:

  • Confirmed the plugin shows my github copilot usage

Screenshots / Videos

image

Checklist

  • The directory name matches the part of id after the / in plugin.toml exactly.
  • It ships plugin.toml, README.md, thumbnail.webp, and translations/en.json.
  • README.md follows the README template, documents every entry id and dependency, and includes exact panel IPC commands and launcher prefixes where applicable.
  • thumbnail.webp is present and relevant; for a new plugin I created it with the thumbnail generator, and for an update I regenerated it with the generator if the visual identity or user-facing appearance changed.
  • version follows semver and is bumped in this PR; plugin_api is the oldest API level this plugin requires.
  • Every non-English translation in this PR uses a locale supported by Noctalia core, and I can read, write, and understand that language well enough to review and maintain it (no unreviewed machine/LLM translations).
  • I did not edit catalog.toml; CI generates it.
  • This PR touches exactly one plugin directory.

Code review attestation

Plugins run as trusted, unsandboxed Luau in the user's session. Confirm:

  • The code is readable and not obfuscated, minified, or generated.
  • It does not download and execute remote code.
  • Every network call, filesystem write, and spawned process is something the description above accounts for.
  • I have the right to publish this code under the license declared in plugin.toml.

@Noctalia-CI
Noctalia-CI marked this pull request as draft September 8, 2026 19:12
@Noctalia-CI

Copy link
Copy Markdown
Contributor

This pull request was converted to a draft because its description is missing required
parts of the pull request template.

Missing:

  • the ## Checklist heading
  • the ## Code review attestation heading
  • the - **Noctalia version tested against:** field
  • the checklist entry: Update to an existing plugin (version bumped in plugin.toml)
  • the checklist entry: Tested on Hyprland
  • the checklist entry: The directory name matches the part of id after the / in plugin.toml exactly.
  • the checklist entry: It ships plugin.toml, README.md, thumbnail.webp, and translations/en.json.
  • the checklist entry: README.md follows the README template, documents every entry id and dependency, and includes exact panel IPC commands and launcher prefixes where applicable.
  • the checklist entry: thumbnail.webp is present and relevant; for a new plugin I created it with the thumbnail generator, and for an update I regenerated it with the generator if the visual identity or user-facing appearance changed.
  • the checklist entry: version follows semver and is bumped in this PR; plugin_api is the oldest API level this plugin requires.
  • the checklist entry: Every non-English translation in this PR uses a locale supported by Noctalia core, and I can read, write, and understand that language well enough to review and maintain it (no unreviewed machine/LLM translations).
  • the checklist entry: I did not edit catalog.toml; CI generates it.
  • the checklist entry: This PR touches exactly one plugin directory.
  • the checklist entry: The code is readable and not obfuscated, minified, or generated.
  • the checklist entry: It does not download and execute remote code.
  • the checklist entry: Every network call, filesystem write, and spawned process is something the description above accounts for.
  • the checklist entry: I have the right to publish this code under the license declared in plugin.toml.
  • exactly one checked plugin type: New plugin or Update to an existing plugin
  • at least one checked compositor testing entry

Add the items above to the description, keeping their exact wording, then mark the pull
request ready for review. Draft pull requests may leave boxes unchecked. Before a pull
request is ready for review, exactly one plugin type, at least one tested compositor, and
every item under Checklist and Code review attestation must be checked.

Sections that only offer context may be deleted; nothing else about this pull request was
changed.

@Noctalia-CI

Copy link
Copy Markdown
Contributor

CC @FelipeArtur: this pull request was automatically moved to draft until you have had a chance to look at it. It will be marked ready for review automatically once you reply here.

@FelipeArtur

Copy link
Copy Markdown
Contributor

Hi @dirkvdb, thanks for adding this and testing it with your Copilot account! Adding it to the provider selector makes sense, and the existing icon mapping keeps the change small.

Could you make a few adjustments before we move forward?

  • Bump the plugin version in plugin.toml.
  • Keep the new translation key in en.json, but remove the changes to pt-BR.json and tr.json. The repository handles those through Noctalia Translate.
  • Restore the full PR template, including the Checklist and Code review attestation sections. The Hyprland checkbox needs [x] instead of [ x], and the Noctalia version field needs its closing formatting. The bot’s comment lists the missing items.
  • Use “GitHub Copilot” consistently in the README and labels.

The implementation looks straightforward from the diff. Thanks for including a screenshot of it working!

@Noctalia-CI
Noctalia-CI marked this pull request as ready for review September 8, 2026 19:25
@Noctalia-CI
Noctalia-CI marked this pull request as draft September 8, 2026 19:26
@Noctalia-CI

Noctalia-CI commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

The description now contains the required template structure.

@dirkvdb

dirkvdb commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

@FelipeArtur

I made the requested changes

@dirkvdb
dirkvdb marked this pull request as ready for review September 8, 2026 19:41

@FelipeArtur FelipeArtur left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the updates, @dirkvdb! Nice work adding Copilot support 🤙

  • Version bumped to 2.2.1.
  • Translation changes kept in en.json.
  • PR template and checklist fixed.
  • “GitHub Copilot” naming corrected.
  • Copilot selector uses the existing provider handling.

That covers my review comments. Approved!

@dirkvdb

dirkvdb commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

Thanks for the quick responses!

@ItsLemmy
ItsLemmy merged commit 0027c6b into noctalia-dev:main Sep 9, 2026
3 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants