Skip to content

Conversation

@touchaponk
Copy link

This pull request introduces logging for the subprocess's standard output (stdout) and standard error (stderr) streams`. The changes ensure that output from the subprocess is captured and displayed in the current process's logs for better debugging and monitoring.

Enhancements to subprocess logging:

  • Added an asynchronous task _print_stdout to read and log lines from the subprocess's stdout stream, throttling the loop to prevent excessive CPU usage.
  • Added an asynchronous task _print_stderr to read and log lines from the subprocess's stderr stream, similarly throttling the loop for efficiency.

@touchaponk
Copy link
Author

@ehartford could you please see if we can merge the PR since right now longer mcp server response will hang up the client

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