Notifications should ignore subagents

[ThomasSteinbach]

Subagents in OpenCode work autonomously. Notifications about finished tasks from them are just distraction.

Notifications should only come from the main agent.

Before warp I used https://github.com/mohak34/opencode-notifier where this behavior works great.

[harryalbert]

Hey @ThomasSteinbach , thanks for submitting this issue! If there's a solid fix based on that notifier, please feel free to submit a PR in this repo and I'd be happy to take a look!

[arijitcodes]

Hey @ThomasSteinbach , thanks for submitting this issue! If there's a solid fix based on that notifier, please feel free to submit a PR in this repo and I'd be happy to take a look!

Hey @harryalbert , i am not 100% sure but there is PR #22 waiting for approval - which might be related to this issue.

[harryalbert]

I just merged @aqeelat 's PR (#22) which should address this issue! New plugin version will be published soon. Thanks @aqeelat for fixing, @arijitcodes for pointing me to his PR, and @ThomasSteinbach for raising this issue!

[arijitcodes]

I just merged @aqeelat 's PR (#22) which should address this issue! New plugin version will be published soon. Thanks @aqeelat for fixing, @arijitcodes for pointing me to his PR, and @ThomasSteinbach for raising this issue!

Happy to help!
BTW, @harryalbert , just a question, is it possible to enable the Discussions tab on this repository ? Helps with some questions.

For ex: I wanna know if we have found a permanent solution for the 'Notification setup instruction' badge that appears to tell us to install about this plugin, but even after we have it installed, it doesnt go away and appears on every restart of OC.

I am not sure so not opening a new issue yet. Just asking here (as we dont have the discussion panel enabled yet).

Have a look at the SS if possible.

Also, thanks a ton for your support with all these issues and PRs. Appreciate it mate.

[arijitcodes]

Hi @harryalbert
Sorry for this double-ping, but have you gotten any time to give this a thought yet ?

I just merged @aqeelat 's PR (#22) which should address this issue! New plugin version will be published soon. Thanks @aqeelat for fixing, @arijitcodes for pointing me to his PR, and @ThomasSteinbach for raising this issue!

Happy to help! BTW, @harryalbert , just a question, is it possible to enable the Discussions tab on this repository ? Helps with some questions.

For ex: I wanna know if we have found a permanent solution for the 'Notification setup instruction' badge that appears to tell us to install about this plugin, but even after we have it installed, it doesnt go away and appears on every restart of OC.

I am not sure so not opening a new issue yet. Just asking here (as we dont have the discussion panel enabled yet).

Have a look at the SS if possible.

Also, thanks a ton for your support with all these issues and PRs. Appreciate it mate.

[aqeelat]

@arijitcodes I don't have a say in this but I hate the discussion tab. Maintainers never reply there and issues get low visibility.

Discussions should be in discord or in an issue.

With that said, I'm curious to know what you wanted to discuss. What's stopping you from opening an issue?

[arijitcodes]

@arijitcodes I don't have a say in this but I hate the discussion tab. Maintainers never reply there and issues get low visibility.

Discussions should be in discord or in an issue.

With that said, I'm curious to know what you wanted to discuss. What's stopping you from opening an issue?

Apologies if it sent you a ping too.

Opening the Discussion tab was just an idea for people to communicate and help each other. The feature is there and i see some opensource projects using them pretty well. Although I understand that for maintainers its pretty difficult to look through all these, but this particular plugin and repo seems pretty chill - at-least what I have noticed in the past two weeks. So could have been a nice addition IMO.

About creating a new issue, I am not sure if its an issue on the plugin's end or warp's end. And also its not that big of a deal.