Skip to content

Prevent recursive standard output logging - #214

Merged
AvdLee merged 1 commit into
mainfrom
fix/standard-output-recursion
Sep 11, 2026
Merged

AvdLee merged 1 commit into
mainfrom
fix/standard-output-recursion

Conversation

@AvdLee

@AvdLee AvdLee commented Sep 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • stop replaying captured stdout/stderr after the original output destination fails
  • avoid writing replay errors back into the captured streams
  • continue persisting captured output to the diagnostics log
  • add regression coverage for a closed output destination

Root cause

A detached long-running app could lose its original terminal. FileHandle.write then raised an Objective-C exception, and the catch block used print, feeding the error back into the captured pipe indefinitely.

Testing

  • swift test (57 tests passed)

@AvdLee
AvdLee merged commit bc2837a into main Sep 11, 2026
1 check passed
@AvdLee
AvdLee deleted the fix/standard-output-recursion branch September 11, 2026 06:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant