Review of "GNOME Speech2Text" version 1

Details Page Preview

This extension adds speech to text (dictation) to the GNOME Shell using OpenAI’s Whisper automatic speech recognition model. The extension captures audio, runs transcription locally, and copies the transcribed text into your clipboard. It requires installing a companion D-Bus service (installer included). This extension accesses the clipboard to copy transcribed text.

Extension Homepage
https://github.com/kavehtehrani/gnome-speech2text/

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.1 (8) Unreviewed
1.0.4 (7) Active
1.0.0 (6) Active
1.0.0 (5) Rejected
1.0.0 (4) Rejected
1.0.0 (3) Rejected
2 Rejected
1 Rejected

Previous Reviews on this Version

JustPerfection rejected
1. Please don't use GNOME in schema id. 2. Please remove unnecessary files: - `scripts/install.sh` - `scripts/package_zip.sh` - `scripts/setup_env.sh` - `scripts/uninstall.sh` [EGO Review Guidelines: unnecessary files](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#don-t-include-unnecessary-files) 3. Use `console.*` instead of `log()`: [Port Guide 45: Logging](https://gjs.guide/extensions/upgrading/gnome-shell-45.html#logging) 4. Timeout should be removed on disable (line 260, 318, 578 `extension.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 5. Don't use sync spawn since that can freeze the shell process. You are doing so many of them and all should be fixed: [GJS Guide: Subprocesses](https://gjs.guide/guides/gio/subprocesses.html#complete-examples) 6. What's the reason for using `xdotool` (line 264, 273 and 277 `extension.js`)? You are constantly doing that and that can lead to a very bad user experience. 7. You should cancel the process on disable (line 352 `extension.js`): [Cancellable Processes](https://gjs.guide/guides/gio/subprocesses.html#cancellable-processes) 8. Move to enable and null it out in disable (line 405 `extension.js`): - [EGO Review Guidelines: Initialization](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#only-use-initialization-for-static-resources) - [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) 9. line 417-624 `extension.js` can be removed just by mentioning how to install the dependencies in the description. and if you want to keep it like that, you should move the scripts to a `.sh` file to make it more clean and readable. 10. Your script should be already executable. Remove line 516 `extension.js`. 11. You should remove the watch on disable (line 603 `extension.js`). 12. Remove line 1832-1834 `extension.js`. Not needed. 13. You cannot hold any instances of objects in the default class you are exporting and leave it there after disable: [EGO Review Guidelines: Destroy](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#destroy-all-objects) Move everything to another class and just create an instance of that class in the default one. 13. This extension is using so many spawn commands that we are against: [EGO Review Guidelines: Scripts](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#scripts-and-binaries) You have to move all of those commands and scripts to a dependency and use D-Bus instead: [D-Bus Guide](https://gjs.guide/guides/#d-bus) - [GNOME Extensions Matrix Channel](https://matrix.to/#/#extensions:gnome.org) - IRC Bridge: irc://irc.gimpnet.org/shell-extensions
kwar13 posted a review
Please discard this submission. I don't see a way to delete it but it's not the correct upload.
JustPerfection posted a review
Once you post a package here, you won't be able to delete it. Especially the packages that have been reviewed.