feat: add hyprwhspr speech-to-text waybar module#1738
Conversation
# Conflicts: # Configs/.config/waybar/config.jsonc
|
No actionable comments were generated in the recent review. π βΉοΈ Recent review infoβοΈ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: π Files selected for processing (2)
β Files skipped from review due to trivial changes (1)
π§ Files skipped from review as they are similar to previous changes (1)
π WalkthroughWalkthroughThis PR integrates hyprwhspr speech-to-text functionality into HyDE by adding the package declaration and wiring a Waybar module that polls the status script every 2 seconds and exposes toggle/restart controls. Changeshyprwhspr Speech-to-Text Feature
Estimated code review effortπ― 1 (Trivial) | β±οΈ ~3 minutes Poem
π₯ Pre-merge checks | β 5β Passed checks (5 passed)
βοΈ Tip: You can configure your own custom pre-merge checks in the settings. β¨ Finishing Touchesπ§ͺ Generate unit tests (beta)
Tip π¬ Introducing Slack Agent: The best way for teams to turn conversations into code.Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.
Built for teams:
One agent for your entire SDLC. Right inside Slack. π Get started Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
π€ Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Configs/.local/share/wayle/modules/hyprwhspr.toml`:
- Around line 4-7: The README/instruction line and the module id are
inconsistent: the instruction tells users to add "custom-hyprwhspr" but the
module block uses id = "hyprwhspr"; make them match by either updating the
instruction to say add "hyprwhspr" to your bar layout or renaming the module id
to "custom-hyprwhspr" (change the id value in the [[modules.custom]] block).
Ensure the instruction text and the id in the module block ("hyprwhspr" vs
"custom-hyprwhspr") are identical.
πͺ Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
βΉοΈ Review info
βοΈ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 1516bc35-1158-424f-a4ef-bab858549288
π Files selected for processing (3)
Configs/.local/share/waybar/modules/custom-hyprwhspr.jsoncConfigs/.local/share/wayle/modules/hyprwhspr.tomlScripts/pkg_extra.lst
There was a problem hiding this comment.
Actionable comments posted: 1
π€ Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@Configs/.local/share/wayle/modules/sensorsinfo.toml`:
- Around line 15-16: The scroll direction bindings in sensorsinfo.toml are
likely inverted: the keys scroll-up and scroll-down are wired to the commands
"hyde-shell sensorsinfo --prev" and "--next" respectively; if the intended UX is
that scrolling up should advance to the next item and scrolling down should go
to the previous item, swap the command values for the scroll-up and scroll-down
keys so scroll-up uses "--next" and scroll-down uses "--prev" (adjust the values
for the scroll-up and scroll-down entries in the file).
πͺ Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
βΉοΈ Review info
βοΈ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: c4a46b62-61ce-4ecb-af83-bd545567a352
π Files selected for processing (4)
Configs/.local/share/wayle/modules/gpuinfo.tomlConfigs/.local/share/wayle/modules/keybindhint.tomlConfigs/.local/share/wayle/modules/sensorsinfo.tomlConfigs/.local/share/wayle/modules/updates.toml
β Files skipped from review due to trivial changes (2)
- Configs/.local/share/wayle/modules/updates.toml
- Configs/.local/share/wayle/modules/gpuinfo.toml
Add waybar custom module for hyprwhspr (native Linux speech-to-text). Polls the hyprwhspr tray script for recording/ready/stopped/error states with left-click toggle and right-click restart actions. Also adds hyprwhspr as an optional package in pkg_extra.lst. Closes HyDE-Project#1719
4a6033d to
1b5a568
Compare
|
I am planning to freeze the current |
|
another languaje to learn π hahhaa |
|
Haha, another language to learn π . Even though Lua is procedural, my mental model for it is kind of OOP-style β I treat everything like structs with methods. I'll do a PR soon. But I really want to make our installer better. On a related note, Iβm working on a dotfile manager (deez-dots) https://github.com/hyde-project/deez-dots with a monofile design so it can be curl | sh friendly. Iβve been considering adding uv or pip+venv for cleaner maintenance, and maybe modularizing it. If we go that route, porting to Go could make sense too, especially for building a single binary with interactive UI libraries. |
|
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 10 days. |
Summary
Adds hyprwhspr integration to HyDE β a native speech-to-text tool for Linux with support for Whisper, Cohere, Parakeet and more.
custom-hyprwhspr.jsonc): polls the hyprwhspr tray script every 2s, shows recording/ready/stopped/error states with JSON output, left-click toggles service, right-click restartspkg_extra.lst(commented out by default)Details
The waybar module uses hyprwhspr's built-in tray script (
/usr/lib/hyprwhspr/config/hyprland/hyprwhspr-tray.sh) which outputs JSON withtext,class, andtooltipfields. Theclassfield enables CSS styling per state (recording,ready,stopped,error,unloaded).Files changed
Configs/.local/share/waybar/modules/custom-hyprwhspr.jsoncScripts/pkg_extra.lsthyprwhspras optional package (commented out)Closes #1719
Summary by CodeRabbit