Review of "Speech2Text" version 1.2 (9)

Details Page Preview

This extension adds speech to text (dictation) 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 (see instructions on github). This extension accesses the clipboard to copy transcribed text.

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

No comments.

Diff Against

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.4 (10) Unreviewed
1.2 (9) Rejected
1.1 (8) Rejected
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

kwar13 posted a review
Hello JustPerfection, 1-I removed the installation script entirely since that seems to be the sticking point here. Now the user only gets guidance how to install it themselves and further info is available on github. 2- While I find ai-assisted coding extremely valuable, the extension is far from "ai generated" and I have spent several weeks coding it and making sure that it performant. I have found AI to be quite valuable in the documentation and showing a path forward when I've been dealing with unfamiliar technology, and given that this is my first gnome extension it definitely has helped with troubleshooting and structuring it. 3- The other extension (https://extensions.gnome.org/review/65946) that you have highlighted as "that is doing the same thing without using those services" is because it is a fork my code. You can see that https://github.com/bcelary/gnome-speech2text/ it is a fork of my repo and basically swapped the native openai whisper implementation with whisper.cpp. No issue there but I didn't want to rely on an unofficial implementation but I'm glad that users can have a choice of their implementation. Nonetheless, the main difference seems to be the installation script which should be resolved by now. I trust these should resolve all your concerns but happy to take further constructive feedback. Thanks, Kaveh
JustPerfection rejected
1. Timeout should be removed before creating a new one since that function can be called multiple times (line 613 `lib/recordingDialog.js`). 2. Timeout should be removed on destroy (line 713 `lib/setupDialog.js`): [EGO Review Guidelines: Timeout](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#remove-main-loop-sources) 3. Not easy to track for cleanup (line 51 `lib/resourceUtils.js`). 4. The instruction should be on the git repo, not to be generated in the extension (line 602 `lib/setupDialog.js`). 5. Avoid sending large diff. It's hard to review. 6. Avoid using unnecessary try-catch: - line 137-151, 169-245 `lib/resourceUtils.js` - line 301 `lib/setupDialog.js` - line 149-218 `lib/uiManager.js` I know about the fork as I mentioned about that in the other [extension review](https://extensions.gnome.org/review/65938). Please avoid sending unnecessary code and keep the code clean, readable and easy follow for the reviewers.
JustPerfection posted a review
Also, please don't use GNOME in extension name: [EGO Review Guidelines: Trademarks](https://gjs.guide/extensions/review-guidelines/review-guidelines.html#copyrights-and-trademarks)
kwar13 posted a review
Hi JustPerfection, I have made the changes and also removed the "GNOME" from the extension's name entirely. Now when I upload the zip file, there is no way for me to link it to this extension and it shows up as a new one: https://extensions.gnome.org/extension/9131/speech2text/ It is basically showing up as a new submission. Is there a way for me to link it to this extension so it counts as an update to this? I believe it is because the URL is still reading: "https://extensions.gnome.org/extension/8238/gnome-speech2text/" Please advise.
kwar13 posted a review
If it is matching by UUID I can restore that back to what it was (gnome-speech2text@kaveh.page) so that it matches the UUID of this extension. I basically removed 'gnome' from everywhere including the github's repo name.
JustPerfection posted a review
You didn't need to change the uuid though. If you want to keep this one, you can request for rejection for the other one.