Skip to content

Drain suppressed binary responses before copying to clipboard#514

Merged
ryanfowler merged 1 commit into
mainfrom
fix-copy-binary-truncation
May 12, 2026
Merged

Drain suppressed binary responses before copying to clipboard#514
ryanfowler merged 1 commit into
mainfrom
fix-copy-binary-truncation

Conversation

@ryanfowler
Copy link
Copy Markdown
Owner

Summary

  • Drain binary responses to io.Discard when terminal output is suppressed and --copy is active, so the tee captures the full body instead of only the sniffed prefix.
  • Keep the existing binary warning behavior for terminal output.
  • Add a regression test that verifies clipboard capture sees the full response body when binary output is suppressed.

Testing

  • go test -v ./internal/fetch
  • go test -v ./...

@ryanfowler ryanfowler enabled auto-merge May 12, 2026 00:59
@ryanfowler ryanfowler merged commit 78c7afe into main May 12, 2026
10 checks passed
@ryanfowler ryanfowler deleted the fix-copy-binary-truncation branch May 12, 2026 01:00
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