BREAKING(json): remove writableStrategy and readableStrategy options

[iuioiua]

What's changed

The writableStrategy and readableStrategy options have been removed from the constructors from ConcatenatedJsonParseStream, JsonParseStream and JsonStringifyStream.

Why this change was made

These options were removed because they aren't used in most use cases. For this reason, none of the other TransformStream implementations in the Standard Library have these options.

Who this affects

Only those who use these options.

Migration guide

If you require the use of these options, please implement a custom class that uses them.

class CustomJsonStringifyStream extends TransformStream<unknown, string> {
  constructor(
    writableStrategy: QueuingStrategy<unknown>,
    readableStrategy: QueuingStrategy<string>,
  ) {
    super(
      {
        transform(chunk, controller) {
          controller.enqueue(JSON.stringify(chunk));
        },
      },
      writableStrategy,
      readableStrategy,
    );
  }
}

Related

Closes #4113

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.70%. Comparing base (15ec8df) to head (f43490f).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5097      +/-   ##
==========================================
- Coverage   92.70%   92.70%   -0.01%     
==========================================
  Files         482      482              
  Lines       38713    38703      -10     
  Branches     5462     5462              
==========================================
- Hits        35888    35878      -10     
  Misses       2770     2770              
  Partials       55       55              

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

[kt3k]

LGTM