Skip to content

Comments

feat(linux): add SIGUSR1 signal for transcribe_with_post_process#759

Merged
cjpais merged 4 commits intocjpais:mainfrom
arimxyer:feat/sigusr1-post-process
Feb 16, 2026
Merged

feat(linux): add SIGUSR1 signal for transcribe_with_post_process#759
cjpais merged 4 commits intocjpais:mainfrom
arimxyer:feat/sigusr1-post-process

Conversation

@arimxyer
Copy link
Contributor

Summary

  • Adds SIGUSR1 signal handling that maps to the transcribe_with_post_process action, enabling Linux users to trigger transcription with post-processing via system signals (e.g. pkill -SIGUSR1 -x handy)
  • Mirrors the existing SIGUSR2 toggle pattern (start/stop with ManagedToggleState)
  • Signal mapping: SIGUSR1transcribe_with_post_process, SIGUSR2transcribe (unchanged)

Closes #758

Context

Linux desktop users (Pop!_OS, GNOME, Wayland) rely on pkill -SIGUSR2 -x handy for basic transcription because Tauri's global shortcut capture doesn't always work under Wayland. This adds a second signal so they can also trigger post-processing transcription without a keyboard shortcut, which also benefits Wayland/GNOME support efforts (PR #572).

Changes

  • src-tauri/src/signal_handle.rs: Import SIGUSR1, add match arm mapping it to transcribe_with_post_process
  • src-tauri/src/lib.rs: Import SIGUSR1, register it in Signals::new alongside SIGUSR2

Test plan

  • Verify pkill -SIGUSR2 -x handy still toggles basic transcription (no regression)
  • Verify pkill -SIGUSR1 -x handy toggles transcription with post-processing
  • Confirm both signals respect the toggle state (send twice = start then stop)

🤖 Generated with Claude Code

arimxyer and others added 2 commits February 9, 2026 23:01
Register SIGUSR1 alongside SIGUSR2 in the signal handler so Linux
users can trigger transcription with post-processing via system
signals (e.g. `pkill -SIGUSR1 -x handy`). This mirrors the existing
SIGUSR2 toggle pattern and maps to the `transcribe_with_post_process`
action already present in ACTION_MAP.

Closes cjpais#758

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cjpais
Copy link
Owner

cjpais commented Feb 16, 2026

Thanks, I simplified and rebased. This is a good and obvious change to have.

@cjpais cjpais merged commit 203ba1d into cjpais:main Feb 16, 2026
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add SIGUSR1 signal support for transcribe_with_post_process

2 participants