Greptile Summary

This PR fixes a bug (#309) where enabling compression together with the NoSideEffects append retry policy (which activates a FrameSignal) caused requests to fail. The root cause was an ordering issue: with_monitored_body converted the request body into a streaming type, after which execute_unary_with's call to compress_body correctly rejected it with "streaming request bodies cannot be compressed".

Key changes:

• sdk/src/api.rs: In the retry loop, compression is now applied before wrapping the body with RequestFrameMonitorBody. This preserves the body as a buffered (non-streaming) type at the time of compression.
• sdk/src/client.rs: A new compress() method on Request eagerly compresses the body, inserts the Content-Encoding header, and resets compression to Compression::None so that the existing compress_body call inside execute_unary_with is a guaranteed no-op — preventing any risk of double-compression.
• sdk/tests/stream_ops.rs: Adds compression_with_no_side_effects_unary to directly cover the fixed code path (both Gzip and Zstd with AppendRetryPolicy::NoSideEffects).
• sdk/tests/common/mod.rs: Defensively installs the aws-lc-rs rustls crypto provider during test setup to avoid a panic when both aws-lc-rs and ring features are active (e.g., via --all-features).

Confidence Score: 5/5

• This PR is safe to merge — it fixes a clear ordering bug, avoids double-compression via the Compression::None reset, and adds regression tests for the exact failure scenario.
• The fix is minimal and surgical: the two-line reordering in api.rs directly addresses the root cause, the new compress() method is straightforward with no observable side effects on the non-FrameSignal code path, and a dedicated integration test is included. No existing behaviour is changed for requests that don't use a FrameSignal.
• No files require special attention.

Important Files Changed

Sequence Diagram

sequenceDiagram
    participant C as Caller
    participant RB as RequestBuilder (api.rs)
    participant R as Request (client.rs)
    participant EU as execute_unary_with

    C->>RB: send()
    RB->>R: try_clone()
    alt frame_signal present (NoSideEffects retry)
        Note over RB: NEW: compress BEFORE monitoring
        RB->>R: compress() [sets compression=None, adds Content-Encoding header]
        R-->>RB: Request{body=compressed, compression=None}
        RB->>R: with_monitored_body(signal)
        R-->>RB: Request{body=StreamingMonitored, compression=None}
    end
    RB->>EU: execute_unary(request)
    EU->>EU: compress_body(body, Compression::None) → no-op
    EU->>EU: build_http_request(headers already has Content-Encoding)
    EU-->>RB: UnaryResponse

_{Last reviewed commit: 59dd131}