Skip to content

stream: normalize Broadcast.from() byte inputs#64082

Open
trivikr wants to merge 1 commit into
nodejs:mainfrom
trivikr:stream-iter-broadcast-from-byte-input
Open

stream: normalize Broadcast.from() byte inputs#64082
trivikr wants to merge 1 commit into
nodejs:mainfrom
trivikr:stream-iter-broadcast-from-byte-input

Conversation

@trivikr

@trivikr trivikr commented Jun 23, 2026

Copy link
Copy Markdown
Member

Fixes: #64081

Broadcast.from() now routes non-Broadcastable inputs through from()
before pumping them into the broadcast writer.

This makes raw strings and ArrayBuffer views normalize as byte inputs,
matching from() behavior, instead of rejecting strings or iterating typed
arrays as numbers.


Assisted-by: openai:gpt-5.5

Route non-Broadcastable inputs through from(). This makes strings
and ArrayBuffer views byte inputs instead of generic iterables.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: openai:gpt-5.5
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/streams

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. stream Issues and PRs related to the stream subsystem. labels Jun 23, 2026
@trivikr trivikr added the request-ci Add this label to start a Jenkins CI on a PR. label Jun 23, 2026
@trivikr trivikr requested a review from jasnell June 27, 2026 00:08
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Jun 27, 2026
@nodejs-github-bot

This comment was marked as outdated.

@trivikr trivikr added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 27, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. needs-ci PRs that need a full CI run. stream Issues and PRs related to the stream subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

stream/iter: Broadcast.from() does not normalize byte inputs like from()

3 participants