Review of "AI Assistant" version 1

Details Page Preview

Artificial intelligence LLM chat integration and MCP server exposing the shell environment

Extension Homepage
https://github.com/2nv2u/gnome-shell-extension-ai-assistant

No comments.

FAQ

Files

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

All Versions

Version Status
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
Please don't use `gnome` in uuid and text domain.
2nv2u posted a review
Can you elaborate why? I'm using this as a subdomain of my own domain...
JustPerfection posted a review
For text domain it can get approved like this but not for using GNOME in uuid since GNOME may want to use `gnome` domain in the future.
2nv2u posted a review
I understand using gnome.org would not be ok but I'm using gnome.2nv2u.com, I don't think my domain will ever be used by gnome itself right?
JustPerfection posted a review
By domain I don't mean the web domain. Maybe one day we want to use that as a namespace. GNOME in the uuid should be preserved for GNOME only. That why I usually add this link under the rejection: [EGO Review Guidelines: Trademarks](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#copyrights-and-trademarks)
2nv2u posted a review
I'm not sure if I follow what you are referring at, are you saying I cannot use the word gnome as part of my own domain in making extensions for it? I've already been using this for almost a decade for my other extension. The link you provided feels to me referring to branding in general, in the uuid section it clearly states I must own the domain / namespace which I do: This must be of the form extension-id@user-domain. extension-id and user-domain MUST only contain numbers, letters, period (.), underscore (_) and dash (-). user-domain MUST be a domain or namespace you control and be separated from extension-id with an @ symbol.
JustPerfection posted a review
I understand. IIRC, it was like 4 years ago that the extension team had a discussion about this. Since then we are not allowing extensions using `gnome` in the uuid.
2nv2u posted a review
Ok, fair enough, I will change it into something else, is there a way to change this in my other extension without making it a new one?
JustPerfection posted a review
Don't touch the other extension (Syncthing Indicator), only change this one and I'll review that as a new extension.
JustPerfection posted a review
Please wait on changing the uuid. We have a discussion about this on [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org). Feel free to join the extension room and read our discussion.
JustPerfection rejected
Ok, you can have the uuid. but please fix these issues: 1. You should call `this._settingTimer.cancel()` in disable (line 26 `extension.js`). 2. Most `assistant/.../extension.js` files using `chatgpt.com`. Is that intended?
JustPerfection posted a review
Reviewers Note: [MR](https://gitlab.gnome.org/World/javascript/gjs-guide/-/merge_requests/303/) for updating review guidelines regarding the use of `gnome` in `uuid`.
2nv2u posted a review
Ok, great, thanks! As for the chatgpt.com, yes this is a implementation of using chatgpt (directly) instead of open webui.