tracking: Deno streams to web streams conversion

[crowlKats]

There is also the separate effort to replace any code that uses various deno ns classes that implement Deno.Reader & Deno.Writer that still uses the read & write functions over the readable & writable properties.

io

• buffer.ts

  • Buffer (feat: streams based Buffer #1970)
  • BufReader: replaced by ReadableStreamBYOBReader.read(, {atLeast})
  • BufWriter: unnecessary (doesnt make much sense for web streams)
  • readDelim: replaced by streams/delimiter.ts's DelimiterStream
  • readStringDelim (feat(streams): TextDelimiterStream #2006)
  • readLines: replaced by streams/delimiter.ts's TextLineStream

• files.ts

  • readRange replaced by streams/byte_slice_stream.ts's ByteSliceStream

• readers.ts

  • StringReader: replaced by TextDecoderStream
  • MultiReader: replaced by streams/merge.ts
  • LimitedReader (feat(streams): LimitedTransformStream & LimitedBytesTransformStream #2007)

• util.ts
  Do we even want these?

  • copyN
  • readShort
  • readInt
  • readLong

• writers.ts
  unnecessary.

archive

• tar (feat: web streams based archive/tar #1985)

encoding

• binary.ts

  • readExact: replaced by ReadableStreamBYOBReader.read(, {atLeast})
  • getNBytes: unecessary
  • readVarnum: doesnt this somewhat clash with io/util.ts functions?
  • readVarbig
  • writeVarnum
  • writeVarbig

• csv.ts

  • uses BufReader (feat: web streams based encoding/csv #1993, feat(encoding/csv): sync parse #2491)

log

• handler.ts chore(log): rework std/log to work without deprecated Deno.Writer #4021
  • WriterHandler
  • RotatingFileHandler

mime

• multipart mime was removed in chore(mime): remove mime module #2336

textproto

not sure what to do with this one. Update: textproto is removed in #2757

please let me know if I missed any.

[iuioiua]

I'll give readRange from files.ts a crack, if that's still something we want. I'm thinking RangedTransformStream.

Also, update:

1. std/mime is deprecated
2. std/textproto is deprecated
3. Streams-based CSV parsing is implemented

[iuioiua]

io

• buffer.ts

  • Buffer (feat: streams based Buffer #1970)
  • BufReader: replaced by ReadableStreamBYOBReader.read(, {atLeast})
  • BufWriter: unnecessary (doesnt make much sense for web streams)
  • readDelim: replaced by streams/delimiter.ts's DelimiterStream
  • readStringDelim (feat(streams): TextDelimiterStream #2006)
  • readLines: replaced by streams/delimiter.ts's TextLineStream

Is the Deno team in favour of deprecating these APIs?

[lino-levan]

@crowlKats could you update this issue according to iuioiua's comment?

[iuioiua]

CC @crowlKats

[crowlKats]

@iuioiua this seems already to have been updated by @kt3k

[iuioiua]

files.ts

• readRange

This is done in streams/ByteSliceStream.

[iuioiua]

Should we aim to have this issue complete by v1?

[lino-levan]

That would be nice.

[iuioiua]

Update: copyN(), readShort(), readInt() and readLong() are deprecated, and can be ticked off. This leaves std/archive and std/log.

[kt3k]

std/log has been addressed in #4021

[iuioiua]

Closing as this work has essentially been completed. The remaining work on @std/archive can be tracked in #1658. Thank you to all that helped!