fix: Improve BodyReadable type definition

[umarov]

Updates the body to use BodyReadable. Currently, body was using an intersection between Readable and BodyMixin.

This actually exposes the dump function in TypeScript now.

Let me know if the resume and abort are actually nullable. I looked at the code and abort is null initially. resume seems to be optional for Readable

Fixes #1256

[codecov-commenter]

Codecov Report

Merging #1259 (84251c2) into main (901f9d6) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1259   +/-   ##
=======================================
  Coverage   94.10%   94.10%           
=======================================
  Files          44       44           
  Lines        4089     4089           
=======================================
  Hits         3848     3848           
  Misses        241      241           

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 901f9d6...84251c2. Read the comment docs.

[mcollina]

we tests our types using tsd. Could you add a test?

[umarov]

@mcollina does it make sense to add types from the BodyMixin into this BodyReadable? I tried adding type tests for those, but the class type I created does not have them. I saw that the BodyMixin already has some types. I wonder if it's redundant to have them in two places.

[umarov]

I can try pushing up what I have

[mcollina]

lgtm