1. Use `console.*` instead of `log()`. Logs should be only available on debug mode. For debug use `console.debug()` instead of `console.log()`: [Port Guide 45: Logging](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#logging) 2. Please remove: - `.eslintrc.cjs` - `.gitignore` - `global.d.ts` - `package-lock.json` - `package.json` - `tool-server/README.md` - `tool-server/requirements.txt` - `tool-server/server.py` [EGO Review Guidelines: unnecessary files](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#don-t-include-unnecessary-files) 3. You cannot expose user's info without user's permission (line 7-61, 83-106 `lib/tools.js`). 4. You should call `abort()` on destroy and don't forget to call destroy on the `ToolExecutor` class instance (line 69 `lib/tools.js`): [`Soup.Session.abort()`](https://gjs-docs.gnome.org/soup30~3.0/soup.session#method-abort)

I don't get what you mean in point 3: 3. You cannot expose user's info without user's permission (line 7-61, 83-106 `lib/tools.js`). Do you say this because of the get_system_info tool?

Aren't the user info sent to the URL?

ah, you mean the longitude latitude?

Yes. If giving the user an option for that is too much work, a simple mention in the description (`metadata.json`) would be enough.

ok. Thanks. Will fix