proposal: Introduce the Brotli Compression Algorithm

[BlackAsLight]

proposal: Introduce the Brotli Compression Algorithm

[iuioiua]

Either way, I like the idea of a BrotliCompressionStream() and BrotliDecompressionStream().

[iuioiua]

We don't want to overwrite globals. It should be an exported class, if anything.

[iuioiua]

To reiterate, I suggested std/streams just so there are no blockers to getting the ball rolling with an implementation. I think such an implementation should exist in std/compress (or similar). However, more discussion is needed before committing to a whole new package.

[iuioiua]

How about implementing extended versions of CompressionStream and DecompressionStream that also support br as the argument? If constructed with anything other than br, it delegates to using the runtime.

Whichever way we go, we'd gladly take a look at a PR that does this in @std/compress.

[BlackAsLight]

How about implementing extended versions of CompressionStream and DecompressionStream that also support br as the argument? If constructed with anything other than br, it delegates to using the runtime.

Whichever way we go, we'd gladly take a look at a PR that does this in @std/compress.

You mean something like this?

self.CompressionStream = function () {
    const compressionStream = CompressionStream
    return class {
        #readable: ReadableStream<Uint8Array>
        #writable: WritableStream<Uint8Array>
        constructor(format: string) {
            const { readable, writable } = format === 'br' ? new BrotliCompressionStream() : new compressionStream(format)
            this.#readable = readable
            this.#writable = writable
        }

        get readable(): ReadableStream<Uint8Array> {
            return this.#readable
        }

        get writable(): WritableStream<Uint8Array> {
            return this.#writable
        }
    }
}()