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