🤖 fix: suppress SSH warnings that pollute command output #981
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Move LogLevel=ERROR outside the identityFile conditional so it applies to all SSH connections. This prevents SSH ControlMaster warnings like 'mux_client_request_session' and 'ControlSocket already exists' from appearing in git command output (e.g., as fake untracked files in the Review tab). Also refactors exec() to use buildSSHArgs() helper, eliminating ~30 lines of duplicated SSH arg building code.
ammar-agent
added a commit
that referenced
this pull request
Dec 8, 2025
The previous fix (#981) added LogLevel=ERROR to suppress SSH log messages, but mux_client_request_session and ControlSocket warnings come from the SSH multiplexing client code and write directly to stderr regardless of log level. This fix adds a TransformStream filter that removes these specific SSH multiplexing messages from stderr before they reach consumers: - mux_client_request_session: session request failed: Session open refused by peer - ControlSocket ... already exists, disabling multiplexing - Shared connection to ... closed. These messages were appearing as fake untracked files in the Review tab because git ls-files output was being polluted with SSH client noise.
ammar-agent
added a commit
that referenced
this pull request
Dec 8, 2025
The previous fix (#981) used LogLevel=ERROR, but mux_client_request_session messages are at ERROR level in OpenSSH. Using LogLevel=FATAL suppresses these while still showing fatal connection failures.
ammario
pushed a commit
that referenced
this pull request
Dec 8, 2025
The previous fix (#981) used `LogLevel=ERROR`, but `mux_client_request_session` messages are at ERROR level in OpenSSH. Using `LogLevel=FATAL` suppresses these while still showing fatal connection failures. ## One-line fix ```diff - args.push("-o", "LogLevel=ERROR"); + args.push("-o", "LogLevel=FATAL"); ``` _Generated with `mux`_
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Move
LogLevel=ERRORoutside theidentityFileconditional so it applies to all SSH connections. This prevents SSH ControlMaster warnings likemux_client_request_sessionandControlSocket already existsfrom appearing in git command output (e.g., as fake untracked files in the Review tab).Also refactors
exec()to usebuildSSHArgs()helper, eliminating ~30 lines of duplicated SSH arg building code.Generated with
mux