how about add a '\a' bell for confirm dialog?

[yurenchen000]

When waiting for a worker, we’re not always watching the terminal; we may be in a different tab, window, or workspace.

A '\a' terminal bell (or other custom string) could be useful for bringing us back

echo -e '\a'

[meowgorithm]

Hey! We’re planning on adding notification support in #1356, which we expect to merge. It’ll work on pretty much all platforms and so it should solve for this one.

[yurenchen000]

i use crush over ssh,
does that notification cross ssh & show in client side ?

i only known terminal bell can cross ssh

[meowgorithm]

It should, yes. I’d recommend trying out the PR and seeing if it works for you (and if not, commenting accordingly).

[yurenchen000]

thanks!
I tried 8b75f66
with ping and switch to other window or workspace immediately
use terminator 2.1.3 and kitty 0.29.0, locally bash and ssh bash

I got this result:

	terminator local	kitty local	terminator ssh	kitty ssh
notify	✔️	✔️	x	x
bell	x	x	x	x

is it native (locally terminal) notification only?

[yurenchen000]

I added this print roughly, and it worked as I expected.
(I didn't distinguish whether it was a non-interactive task or an active terminal.

index 1db2b407..d6633825 100644
--- a/internal/agent/agent.go
+++ b/internal/agent/agent.go
@@ -493,6 +493,8 @@ func (a *sessionAgent) Run(ctx context.Context, call SessionAgentCall) (*fantasy
                notifBody := fmt.Sprintf("Agent's turn completed in \"%s\"", currentSession.Title)
                _ = notification.Send("Crush is waiting...", notifBody)
        }
+       fmt.Print("\a")