fix: preserve URL query strings in invoke output #54
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.
Summary
kernel invokeoutput escaping&as\\u0026(and similarly<,>), which breaks copy/paste of URLs returned in invocation output (e.g. replay URLs).SetEscapeHTML(false).Why
Go's default
json.MarshalIndentescapes HTML-sensitive characters. When the CLI re-marshals invocation output for pretty printing, URLs like...?jwt=...&replay_id=...can show up as...?jwt=...\\u0026replay_id=....Test plan
go test ./...Note
Prevents HTML character escaping in pretty-printed invoke results by using a custom JSON encoder to preserve URL characters.
cmd/invoke.go:printResultwithSetEscapeHTML(false)and indentation to avoid escaping&,<,>in pretty-printed output; trims trailing newline.Written by Cursor Bugbot for commit f3502a4. This will update automatically on new commits. Configure here.