FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

stream: normalize fused stateless transform results by trivikr · Pull Request #65367 · nodejs/node · GitHub

/ node Public

stream: normalize fused stateless transform results - #65367

Merged
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
trivikr:stream-iter-stateless-transform-intermediate-output
Aug 25, 2026
Merged

stream: normalize fused stateless transform results#65367
nodejs-github-bot merged 1 commit into
nodejs:mainfrom
trivikr:stream-iter-stateless-transform-intermediate-output

Conversation

trivikr commented Aug 18, 2026

Copy link
Copy Markdown
Member

Fixes: #65366

Normalize each stateless transform result before passing it to the next transform in a fused run. This ensures that subsequent transforms always receive Uint8Array[] batches in both synchronous and asynchronous pipelines.


Assisted-by: codex:gpt-5.6-sol

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/streams

nodejs-github-bot added needs-ci PRs that need a full CI run. stream Issues and PRs related to the stream subsystem. labels Aug 18, 2026
Normalize each stateless transform result before passing it to
the next transform in a fused run. This ensures that subsequent
transforms always receive Uint8Array[] batches in both synchronous
and asynchronous pipelines.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: codex:gpt-5.6-sol
trivikr force-pushed the stream-iter-stateless-transform-intermediate-output branch from f42db5e to a780551 Compare August 18, 2026 03:01
jasnell added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 18, 2026
github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 18, 2026

This comment was marked as resolved.

codecov Bot commented Aug 18, 2026
edited
Loading

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 51.51515% with 16 lines in your changes missing coverage. Please review.
✅ Project coverage is 90.11%. Comparing base (92ee8b7) to head (a780551).
⚠️ Report is 103 commits behind head on main.

Files with missing lines Patch % Lines
lib/internal/streams/iter/pull.js 51.51% 14 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #65367      +/-   ##
==========================================
- Coverage   90.15%   90.11%   -0.04%     
==========================================
  Files         752      752              
  Lines      251884   251907      +23     
  Branches    47368    47373       +5     
==========================================
- Hits       227078   227009      -69     
- Misses      16164    16236      +72     
- Partials     8642     8662      +20     
Files with missing lines Coverage Δ
lib/internal/streams/iter/pull.js 83.28% <51.51%> (-0.87%) ⬇️

... and 36 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Collaborator

trivikr added author ready PRs that have at least one approval, no outstanding review comments, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Aug 23, 2026
nodejs-github-bot merged commit 4b5e86c into nodejs:main Aug 25, 2026
75 checks passed

Copy link
Copy Markdown
Collaborator

Landed in 4b5e86c

nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Aug 25, 2026
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Normalize each stateless transform result before passing it to
the next transform in a fused run. This ensures that subsequent
transforms always receive Uint8Array[] batches in both synchronous
and asynchronous pipelines.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: codex:gpt-5.6-sol
PR-URL: #65367
Fixes: #65366
Reviewed-By: James M Snell <jasnell@gmail.com>
aduh95 pushed a commit that referenced this pull request Aug 25, 2026
Normalize each stateless transform result before passing it to
the next transform in a fused run. This ensures that subsequent
transforms always receive Uint8Array[] batches in both synchronous
and asynchronous pipelines.

Signed-off-by: Kamat, Trivikram <16024985+trivikr@users.noreply.github.com>
Assisted-by: codex:gpt-5.6-sol
PR-URL: #65367
Fixes: #65366
Reviewed-By: James M Snell <jasnell@gmail.com>
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

Labels

author ready PRs that have at least one approval, no outstanding review comments, 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: fused stateless transforms do not normalize output between transforms

4 participants


Back | FazBrowse Home | New Git URL