in transform stream transform method, `callback(null, null)` doesn't end stream but `this.push(null); callback()` does

Version

latest main branch

Platform

irrelevant

Subsystem

stream

What steps will reproduce the bug?

https://nodejs.org/api/stream.html#transform-transformchunk-encoding-callback

callback(null, null) doesn't end stream but this.push(null); callback() does

How often does it reproduce? Is there a required condition?

Always

What is the expected behavior? Why is that the expected behavior?

callback(null, null) and this.push(null); callback() have the same effect (the stream ends).

What do you see instead?

callback(null, null) is different from this.push(null); callback()

Additional information

https://github.com/nodejs/node/blob/main/lib/internal/streams/transform.js#L177-L179

Would it be possible to change this to the following?

if (val !== undefined) {
  this.push(val)
}

Bắt đầu tại lib/internal/streams/transform.js ở các dòng 177-179 và so sánh hành vi callback của Transform được mô tả trong tài liệu với phần triển khai stream hiện có. Tái hiện sự khác biệt giữa callback(null, null) và this.push(null); callback(), sau đó kiểm tra các bài kiểm thử stream liên quan. Được xem là hoàn thành khi cả hai dạng đều kết thúc stream một cách nhất quán như mô tả.