Skip to content

Fix gRPC dry-run stdin reuse#689

Merged
ryanfowler merged 1 commit into
mainfrom
fix-grpc-dryrun-stdin-reuse
Jun 1, 2026
Merged

Fix gRPC dry-run stdin reuse#689
ryanfowler merged 1 commit into
mainfrom
fix-grpc-dryrun-stdin-reuse

Conversation

@ryanfowler

Copy link
Copy Markdown
Owner

Summary

  • Materialize dry-run request bodies once for stdin-backed gRPC streams and reuse the result for output.
  • Preserve the existing preview path for non-streaming bodies while routing binary decisions through a shared helper.
  • Add a regression test covering client-streaming gRPC dry-run with stdin input.

Testing

  • cargo test --all-features --test grpc grpc_dry_run_materializes_streaming_stdin_body_once
  • cargo test --all-features --test grpc

@ryanfowler ryanfowler enabled auto-merge June 1, 2026 04:44
@ryanfowler ryanfowler merged commit 176d2ec into main Jun 1, 2026
4 checks passed
@ryanfowler ryanfowler deleted the fix-grpc-dryrun-stdin-reuse branch June 1, 2026 04:47
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