fix(ruby): dispatch every response shape to the right request adapter by andreaTP · Pull Request #8256 · microsoft/kiota

Andrea Peruffo (andreaTP) · 2026-09-22T12:57:33Z

Generated Ruby clients only ever called send_async, which reads a single object, so most
response shapes were broken: void raised factory cannot be null, binary raised on an
undefined Binary constant, scalars emitted a bare lowercase type name, and a collection of
models silently returned one empty model.

Dispatches to all five request adapter methods, mirroring the Python generator. Also fixes
enum collections, scalar and binary request bodies, the int64/double/decimal/int8/
uint8/base64 formats, and a date being read with get_time_value.

Requires the runtime gems at 0.22.0, already released, so the pins move with it.

it/ruby/spec/basic/integration_test_basic.rb has been commented out since it was written.
It now makes a real call and asserts the typed error.

Binary upload still fails in FaradayRequestAdapter, which calls .empty? on the StringIO
body. Generator side is correct, that is the remaining half of kiota-ruby#54.

fixes microsoft/kiota-ruby#42,
fixes microsoft/kiota-ruby#67
fixes microsoft/kiota-ruby#125.
Covers items 2, 3 and 6 of #7956.

… method - void, collection, scalar, enum and binary responses all went through send_async, which only reads a single object - binary is now StringIO; only a whole body is a stream, a collection of binary values is a payload - enum collections were generated as object collections, calling create_from_discriminator_value on a plain Hash - scalar request bodies went through set_content_from_parsable and failed to serialize - int64, double, decimal, int8, uint8 and the base64 formats deserialized through constants nothing defines - a date was read with get_time_value because the primitive type map collapsed Date onto Time - the request body was only set when a request configuration parameter was present - a resolved model keeps its factory even when named like a primitive, binary or void - appsettings.json moves to the 0.22.0 runtime gems

Copilot

Copilot review overview

🔵 Needs a closer look

Add dedicated writer coverage for the scalar stream-response dispatch path.

Review effort: Lite
Findings: None

Resolved since last review (4)

Binary conversion targets the wrong parameter when body and binary inputs coexi…
Binary aliases incorrectly replace resolved model types
Enum response executor dispatch lacks regression coverage
Add regression test for binary refinement rewriting and stringio import

Vincent Biret (baywet) · 2026-09-29T15:54:30Z

Andrea Peruffo (@andreaTP) I think we're getting really close here, maybe missing additional tests?

Add dedicated writer coverage for the scalar stream-response dispatch path.

Copilot

Copilot review overview

🟢 Approval recommended

All reviewed changes have been addressed and no blocking issues remain.

Review effort: Lite
Findings: None

Andrea Peruffo (andreaTP) · 2026-09-29T16:30:53Z

Vincent Biret (@baywet) seems like we are finally here 🎉

Vincent Biret (baywet)

Thank you for making the changes!

Copilot AI lite review requested due to automatic review settings September 22, 2026 12:57

Andrea Peruffo (andreaTP) requested a review from a team as a code owner September 22, 2026 12:57

Vincent Biret (baywet) previously approved these changes Sep 22, 2026

View reviewed changes

Copilot started reviewing on behalf of Andrea Peruffo (andreaTP) September 22, 2026 13:02

View session