- Notifications
You must be signed in to change notification settings - Fork950
remove pretty print json flag#179
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Pull Request Overview
This PR reverts the changes introduced in PR#114, restoring the previous JSON output behavior that separates JSON messages by newlines.
- Reverts deletion of newline-separated JSON to comply with protocol requirements
- Removes the "pretty-print-json" flag from configuration and associated code changes
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
Thank you!
936b24c intomainUh oh!
There was an error while loading.Please reload this page.
Reverts#114
This change breaks the protocol, which expects json to be separated by newlines. To avoid confusion, this is being reverted.
Instead, users are encouraged to use:
mcpcurlscript/prettyprint-logto format log results