Add configurable notification modes and desktop alerts#34
Open
fetwar wants to merge 3 commits into
Open
Conversation
notification.py & stop.py: - Replace boolean --notify with --notify tts|desktop|both|off - Add desktop push notifications via notify-send - Concise contextual messages ending with project dir openai_tts.py: - Use "verse" voice, WAV format, 1.5x speed for faster alerts
fetwar
commented
Mar 20, 2026
Comment on lines
+71
to
+73
| voice="verse", | ||
| input=text, | ||
| instructions="Speak in a cheerful, positive yet professional tone.", | ||
| response_format="mp3", | ||
| instructions="Speak fast", |
Author
There was a problem hiding this comment.
This was for personal preference, I lowkey didn't mean to include in this PR so can revert if you wish.
I find the increase in talking speed to be significantly better though, it was painfully slow
fetwar
commented
Mar 20, 2026
| instructions="Speak in a cheerful, positive yet professional tone.", | ||
| response_format="mp3", | ||
| instructions="Speak fast", | ||
| response_format="wav", |
Author
There was a problem hiding this comment.
mp3 response format did not work on linux, instead only a static-like sound could be heard as it was trying to play a compressed audio file (mp3) via LocalAudioPlayer().play() which does not expect it
Author
There was a problem hiding this comment.
See dev output docs here: https://developers.openai.com/api/docs/guides/text-to-speech#supported-output-formats
- Bare --notify (no value) defaults to tts for backward compatibility - Update settings.json to use explicit --notify tts and add --notify tts to Stop hook - Revert TTS voice instructions to cheerful professional tone, keep wav format and 1.5x speed
Avoids "Error generating completion message" when the OpenAI key lacks chat completion access.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
--notifywith--notify tts|desktop|both|offon notification.py and stop.pynotify-send(Linux)I honestly didn't mean to make this PR yet, was still fine tuning things.
If there is interest in including this functionality I will make it merge ready by:
--notify