No stderr capture on toggleProcess / exitNodeProcess (lost diagnostic output) #25

Closed
opened 2026-05-19 04:22:54 +00:00 by vybe · 4 comments
Collaborator

Problem

toggleProcess and exitNodeProcess only capture stdout. Stderr is not collected or inspected. Any diagnostic output from tailscale (e.g., "must be root", authentication prompts, permission errors) is silently discarded. The only user-visible feedback on failure is the generic error toast.

Impact

Users have no way to see the real reason a tailscale up/down/set command failed. Debugging is limited to the exit code alone.

## Problem `toggleProcess` and `exitNodeProcess` only capture stdout. Stderr is not collected or inspected. Any diagnostic output from `tailscale` (e.g., "must be root", authentication prompts, permission errors) is silently discarded. The only user-visible feedback on failure is the generic error toast. ## Impact Users have no way to see the real reason a `tailscale up/down/set` command failed. Debugging is limited to the exit code alone.
Owner

This issue should probably be looked at more generically as well. In the event of unexpected output or command failure, how does the user know what happened? Some guidance is needed. Either the toasts need to catch everything and detail what the error state is, or there needs to be some kind of log.

If I later have to troubleshoot problems between versions, or when command output changes, I want to know what I'm looking at.

This issue should probably be looked at more generically as well. In the event of unexpected output or command failure, how does the user know what happened? Some guidance is needed. Either the toasts need to catch everything and detail what the error state is, or there needs to be some kind of log. If I later have to troubleshoot problems between versions, or when command output changes, I want to know what I'm looking at.
Author
Collaborator

Related: #13 (split error handling on statusCheck), #29 (fragmented error paths). Same pattern: processes only check exitCode, stderr is ignored.

Related: #13 (split error handling on statusCheck), #29 (fragmented error paths). Same pattern: processes only check exitCode, stderr is ignored.
Author
Collaborator

Added stderr: StdioCollector to toggle/exitNode processes. Merged to vibes.

Added stderr: StdioCollector to toggle/exitNode processes. Merged to vibes.
Author
Collaborator

Resolved via merge daab237. The plugin is confirmed working by @jtmorris. Human code review required before merging to testing.

Resolved via merge daab237. The plugin is confirmed working by @jtmorris. Human code review required before merging to testing.
vybe closed this issue 2026-05-21 07:16:54 +00:00
Sign in to join this conversation.
No labels
No milestone
No project
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: jtmorris/dms_tailscalectl#25
No description provided.