CLI Bus.unsubscribe has same splice(-1) bug as desktop app

[deepak0x]

Description

Same bug as the desktop app's bus — the unsubscribe closure uses splice(indexOf(handler), 1) without guarding against indexOf returning -1. A double-unsubscribe removes the wrong handler.

Steps to reproduce

1. Subscribe two handlers to the same runId via CLI bus
2. Call the first handler's unsubscribe twice
3. Second handler gets removed

Expected behavior

Double-unsubscribe should be a no-op.

Actual behavior

splice(-1, 1) removes the last element — the wrong handler.

[deepak0x]

@ramnique @arkml happy to take this one too if it's up for grabs.

[fuleinist]

Fixed in PR #504

[deepak0x]

Hey @fuleinist
How dare you worked on my issue🙂
I kept it opened to first approve from maintainer.