mafintosh / mafintosh/streamx

"text-decoder" dep does not support browsers and is slow

Open
#113 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
239
Forks
27
Avg merge
27m
Merged PRs (30d)
1

Description

See upstream replies:

* Lack of browser support: https://github.com/holepunchto/text-decoder/issues/8#issuecomment-3897893674
* Performance being not a goal: https://github.com/holepunchto/text-decoder/issues/13#issuecomment-3899552787

It is also untested and did not properly decode UTF-8 until just-released 1.2.6
So this likely needs a dep bump even before replacement

## Performance

That is about 5x slower on UTF-8 in both Node.js and React Native that a fast codepath

See comparison in https://github.com/holepunchto/text-decoder/pull/14

Contributor guide

No contributing guide indexed for this repository

Research direction

Start by reading the linked upstream replies and the comparison in text-decoder pull request 14, then inspect how streamx uses the text-decoder dependency. Check whether version 1.2.6 addresses UTF-8 correctness; done requires browser support and improved UTF-8 performance in Node.js and React Native.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript, node.js, react-native
Domain
performance, web-dev
Issue type
Bug
Difficulty
4/5
Estimated time
3-5 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.