Fix: notifications broken on Windows (writeFileSync /dev/tty)

[harryalbert]

warpNotify writes the OSC 777 escape sequence via writeFileSync("/dev/tty", ...). /dev/tty is a Unix-only device — it doesn't exist on native Windows (PowerShell, CMD). The catch block silently swallows the error, so no notifications ever reach Warp on Windows. This also means the setup chip never disappears, since Warp never receives a session_start event.

The fix is to fall back to process.stdout.write when the first write attempt fails. This matches the pattern used by OpenCode's own clipboard PR and the opencode-terminal-bell-notifier community plugin

Should fix#11 (I reached out to folks in the issue thread to ask them to confirm)

[harryalbert]

• Fix: notifications broken on Windows (writeFileSync /dev/tty) #12 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[arijitcodes]

Any update on when this will be merged and released?
Also worth noting that even with this fix, there's a startup timing issue (see #18) — session.created fires before the plugin registers its handlers, so the session_start notification that dismisses the setup badge is never sent. A deferred session_start after init (as described in #18) might fix it.

[harryalbert]

Any update on when this will be merged and released? Also worth noting that even with this fix, there's a startup timing issue (see #18) — session.created fires before the plugin registers its handlers, so the session_start notification that dismisses the setup badge is never sent. A deferred session_start after init (as described in #18) might fix it.

Hey @arijitcodes thanks for pinging! I don't love the approach in this PR so was holding off on merging, and it fell off my radar after that. I think I'll close this PR for now, but if you have a solution for WSL in mind, please feel free to submit a PR and tag me as a reviewer! I'm happy to TAL. It seems like there's a possible solution described in this issue.

I also don't have a WSL machine to test on, so if you do end up submitting a fix, could you please attatch a video of the fix WAI? Thank you!

[arijitcodes]

Any update on when this will be merged and released? Also worth noting that even with this fix, there's a startup timing issue (see #18) — session.created fires before the plugin registers its handlers, so the session_start notification that dismisses the setup badge is never sent. A deferred session_start after init (as described in #18) might fix it.

Hey @arijitcodes thanks for pinging! I don't love the approach in this PR so was holding off on merging, and it fell off my radar after that. I think I'll close this PR for now, but if you have a solution for WSL in mind, please feel free to submit a PR and tag me as a reviewer! I'm happy to TAL. It seems like there's a possible solution described in this issue.

I also don't have a WSL machine to test on, so if you do end up submitting a fix, could you please attatch a video of the fix WAI? Thank you!

Hey @harryalbert , Thanks for taking the time to respond.
Well I was facing the issue on my Windows 11 npm install - non WSL.
I installed with npm i -g, and notifications werent working.

After doing some AI Research, trial and error, i got it working by replacingwriteFileSync("/dev/tty", sequence); with process.stdout.write(sequence); in the notify.js Line 13.

And after a nice OC restart, it started working - Both as Windows Toast Notification, as well as the Warp In-app notification tray (along with tab icons etc).

NOTE: This didnt solve the #18 Notification Badge thing, but atleast got my notifications working.

[harryalbert]

ah got it, that makes sense. Thanks for clarifying! so would the fix in this PR have solved you issue?

[arijitcodes]

ah got it, that makes sense. Thanks for clarifying! so would the fix in this PR have solved you issue?

Well yes (probably).
I just didn't bother with the additional try-catch and went raw.

I am attaching some screenshots here that shows the notify.js code and the Warp + OC notifications working.
Hope it helps (?).

@harryalbert

Screenshots:

1. NPM Install (I use NVM - Node Version Manager for Windows):
   
   
   
2. Notify.js Code:
   
   
   
3. Warp In-App Toast Alert, Notification Tray Dot Icon & Tab Activity Icon working:
   
   
   
4. Windows OS Default Toast Notification when Warp is out of focus working:

[harryalbert]

nice, this makes sense! I think I'll probably merge this PR then, because it seems like it might address your issue

[arijitcodes]

nice, this makes sense! I think I'll probably merge this PR then, because it seems like it might address your issue

Yes, would appreciate a merge and release tbh lol
Otherwise I will have to re-do the edit everytime theres a plugin re-install, or cache deletion or plugin update.

Also, thanks a ton for your super fast response, appreciate it a ton 🔥 🙌🏻

[harryalbert]

nice, this makes sense! I think I'll probably merge this PR then, because it seems like it might address your issue

Yes, would appreciate a merge and release tbh lol Otherwise I will have to re-do the dit everytime theres a plugin re-install, or cache deletion or plugin update.

Also, thanks a ton for your super fast response, appreciate it a ton 🔥 🙌🏻

of course! Happy to help — thanks for re-raising this!

[arijitcodes]

nice, this makes sense! I think I'll probably merge this PR then, because it seems like it might address your issue

Yes, would appreciate a merge and release tbh lol Otherwise I will have to re-do the dit everytime theres a plugin re-install, or cache deletion or plugin update.
Also, thanks a ton for your super fast response, appreciate it a ton 🔥 🙌🏻

of course! Happy to help — thanks for re-raising this!

Ayyyy merged it already, nice. Hope we get a new release soon 😼

[harryalbert]

nice, this makes sense! I think I'll probably merge this PR then, because it seems like it might address your issue

Yes, would appreciate a merge and release tbh lol Otherwise I will have to re-do the dit everytime theres a plugin re-install, or cache deletion or plugin update.
Also, thanks a ton for your super fast response, appreciate it a ton 🔥 🙌🏻

of course! Happy to help — thanks for re-raising this!

Ayyyy merged it already, nice. Hope we get a new release soon 😼

​Just published the new version!

[arijitcodes]

nice, this makes sense! I think I'll probably merge this PR then, because it seems like it might address your issue

Yes, would appreciate a merge and release tbh lol Otherwise I will have to re-do the dit everytime theres a plugin re-install, or cache deletion or plugin update.
Also, thanks a ton for your super fast response, appreciate it a ton 🔥 🙌🏻

of course! Happy to help — thanks for re-raising this!

Ayyyy merged it already, nice. Hope we get a new release soon 😼

 [
![gif](https://camo.githubusercontent.com/0386fb17a15515803cdbb80592c34c4bb159d568d56dad5bc994cdd3ba7298cb/68747470733a2f2f6d656469612e67697068792e636f6d2f6d656469612f635736347045455a6530595a612f67697068792e676966)
](https://camo.githubusercontent.com/0386fb17a15515803cdbb80592c34c4bb159d568d56dad5bc994cdd3ba7298cb/68747470733a2f2f6d656469612e67697068792e636f6d2f6d656469612f635736347045455a6530595a612f67697068792e676966)
[
](https://camo.githubusercontent.com/0386fb17a15515803cdbb80592c34c4bb159d568d56dad5bc994cdd3ba7298cb/68747470733a2f2f6d656469612e67697068792e636f6d2f6d656469612f635736347045455a6530595a612f67697068792e676966)

​Just published the new version!

Seems like its working now! Legit 🔥
Thanks a tonn mate 🙌🏻