BREAKING(streams): remove iterateReader, readableStreamFromReader, readerFromIterable, readerFromStreamReader, writableStreamFromWriter, and writerFromStreamWriter

[kt3k]

This PR removes deprecated APIs in @std/streams

What's changed

iterateReader, readableStreamFromReader, readerFromStreamReader, writableStreamFromWriter, and writerFromStreamWriter are removed in favor of the similar APIs in @std/io.

readerFromIterable are removed in favor of the combination of readerFromStreamReader(ReadableStream.from().getReader()).

Why was this change been made?

We are in the process of migrating from Reader/Writer based I/O APIs to Web Stream I/O APIs. As part of that process, we decided to move all Reader/Writer related APIs to std/io.

Now std/streams has helpers for Web Streams APIs. This is now recommended way to perform IO in Deno.

All APIs in std/io are related to legacy Reader/Writer based APIs. These are now not recommended way to perform IO in Deno.

This move tries to make the above distinction of 2 sets of APIs clear.

Migration

Import these from std/io

-import {
-  iterateReader,
-  readableStreamFromReader,
-  readerFromStreamReader,
-  writableStreamFromWriter,
-  writerFromStreamWriter
-} from "@std/streams";
+import {
+  iterateReader,
+  toReadableStream as readableStreamFromReader,
+  readerFromStreamReader,
+  toWritableStream as writableStreamFromWriter,
+  writerFromStreamWriter
+} from "@std/io";

For readerFromIterable(iter), use readerFromStreamReader(ReadablerStream.from(iter).getReader())

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.22%. Comparing base (935d665) to head (b346bb6).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5060      +/-   ##
==========================================
- Coverage   92.22%   92.22%   -0.01%     
==========================================
  Files         487      481       -6     
  Lines       38859    38757     -102     
  Branches     5444     5436       -8     
==========================================
- Hits        35838    35742      -96     
+ Misses       2966     2960       -6     
  Partials       55       55              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[iuioiua]

LGTM once nit is addressed.

[iuioiua]

This test for iterateReader() no longer uses iterateReader().

[kt3k]

The next version of streams is planned to be 1.0.0-rc.1 (#4600). Landing this now.