Repository navigation
feat(csv-parse): expose native browser Web Streams - #535
Draft
Huzaifa-Asif wants to merge 1 commit into
Draft
Huzaifa-Asif wants to merge 1 commit into
Huzaifa-Asif wants to merge 1 commit into
Conversation
Expose a browser-native Web Streams entry and normalize native byte chunks. Include DOM-only declarations, focused regressions, browser usage documentation, and existing polyfill notices. AI-assisted implementation and tests; independently reviewed by a separate agent.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Browser users cannot use
csv-parse/streambecause it importsnode:stream/web, and there is no browser Web Streams export. Addcsv-parse/browser/esm/streambacked by native browser streams:Normalize native Uint8Array chunks before the shared parser buffers them, retain the Buffer polyfill notices, and generate isolated DOM declarations from the existing parser options. Native Web Streams support is required. This is scoped to csv-parse.
Closes #449
Validation on Node24.18.1/macOS and Chrome154:
an booleanwhile source emitsa boolean). The full suite is not green.Existing index/sync generated bundles are left out; the build also exposes unrelated base generated-file drift. No full operating-system/runtime matrix is claimed.
AI disclosure: OpenAI Codex researched and implemented this change and ran the local checks. A separate agent reviewed the frozen patch before submission. Human maintainer review is requested; no claim of personal human code review is made.