lib: narrow ReadableStreamBYOBRequest.view return type to Uint8Array

[Jah-yee]

Description

Follow WHATWG streams spec update: whatwg/streams#1367

ReadableStreamBYOBRequest.view is always constructed as a Uint8Array, and the specification will be updated to enforce this. This changes the documented return type from ArrayBufferView to Uint8Array.

Fix

Changed the JSDoc type annotation for view getter from @type {ArrayBufferView} to @type {Uint8Array} in lib/internal/webstreams/readablestream.js.

Relation to #62958

This PR replaces #62958, which was closed after the commit message lint check failed. The commit message was corrected (per feedback from @MattiasBuelens) and resubmitted as this new PR with the fix already applied.

Fixes #62952

[Jah-yee]

Hi @MattiasBuelens, thank you for your review on #62958! 👋

This PR replaces #62958 with the corrected commit message (per your feedback on the line-length lint issue — the commit message was adjusted accordingly).

The change is identical: a 1-line JSDoc type narrowing from ArrayBufferView to Uint8Array, following the WHATWG streams spec update.

The CI checks were cancelled (not failed) and appear to be pending. Could you please review this PR when you have a moment? Happy to address any feedback.

Also tagging @gurgunday — thank you for the earlier feedback on #62958 as well! 🙏

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.65%. Comparing base (2a601eb) to head (e6b5c86).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #63017      +/-   ##
==========================================
- Coverage   89.65%   89.65%   -0.01%     
==========================================
  Files         708      708              
  Lines      220374   220374              
  Branches    42264    42265       +1     
==========================================
- Hits       197584   197570      -14     
+ Misses      14656    14651       -5     
- Partials     8134     8153      +19     

Files with missing lines	Coverage Δ
lib/internal/webstreams/readablestream.js	98.54% <100.00%> (ø)

... and 27 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.

[MattiasBuelens]

Your commit message still doesn't pass all checks. Note that we recently added a new requirement that requires all commits to have a Signed-off-by line, see point 6 in the guidelines.

You can run these checks locally before pushing by using core-validate-commit. For example:

git rev-parse HEAD | npx core-validate-commit

You can update your commit message with git commit --amend. Afterwards, instead of opening a new pull request like you did last time, you should git push --force to update your existing PR branch (and replace the old commit).

If you're not sure how to do this, just let us know. We're here to help you, or we can do it for you if needed. 😉

[Jah-yee]

Thank you so much for the detailed guidance! 🙏

I'll fix the commit message with the line and force-push to update this PR directly — no need to open a new PR.

Will update shortly.

[Jah-yee]

Thank you so much for the detailed guidance! 🙏

I will fix the commit message with the Signed-off-by line and force-push to update this PR directly — no need to open a new PR.

Will update shortly.

[Jah-yee]

Thanks for the review! I've added the Signed-off-by line to the commit message per the contributor guidelines (point 6). The commit now includes:

Signed-off-by: Jah-yee <166608075+Jah-yee@users.noreply.github.com>

Commit SHA updated: 8cdb57ef (same changes, only commit message modified)