Merged
Conversation
Test262 conformance changes
|
Rahul-2k4
force-pushed
the
fix/headers-iterable-4611
branch
from
9848a6e to
2e1480c
Compare
Member
|
If you're using AI agents to do this work, maybe you should use something like git-worktree to avoid accidentally pushing unrelated commits? Just a suggestion. |
Contributor
Author
|
yep sure @jedel1043 ill keep that in mind . i got little confused because of some uncommited changes of my other pr . and again thank you for the suggestion of worktree that it will keep my workspace clean . Will keep in mind and ill surely improve from next time and thanks for the quick reviews |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #4638 +/- ##
==========================================
+ Coverage 47.24% 57.23% +9.99%
==========================================
Files 476 550 +74
Lines 46892 60428 +13536
==========================================
+ Hits 22154 34587 +12433
- Misses 24738 25841 +1103 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
jedel1043
reviewed
Feb 23, 2026
jedel1043
reviewed
Feb 25, 2026
Rahul-2k4
force-pushed
the
fix/headers-iterable-4611
branch
from
cd41627 to
ca51b07
Compare
Rahul-2k4
force-pushed
the
fix/headers-iterable-4611
branch
from
ca51b07 to
b286eec
Compare
Contributor
Author
|
@jedel1043 yep sure |
akash-R-A-J
pushed a commit
to akash-R-A-J/boa
that referenced
this pull request
Mar 1, 2026
## Summary This PR makes `Headers` iterable in Boa runtime so it can be consumed by iterable APIs. This Pull Request closes boa-dev#4611 ### What changed - Added a regression test for `Headers` iterability: - spread: `[...new Headers([["x", "y"]])]` - `Map` construction: `new Map(new Headers([["x", "y"]]))` - Wired `Headers.prototype[Symbol.iterator]` during fetch registration. - Exposed `entries`, `keys`, and `values` as class methods on `Headers`. ### Why `Headers` should be iterable over `[name, value]` pairs. Without `@@iterator`, spread and `Map` construction throw a `TypeError`. ## Testing ```bash cargo test -p boa_runtime fetch::tests -- --nocapture All fetch tests pass, including the new regression test.
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
2 participants
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.
Summary
This PR makes
Headersiterable in Boa runtime so it can be consumed by iterable APIs.This Pull Request closes #4611
What changed
Headersiterability:[...new Headers([["x", "y"]])]Mapconstruction:new Map(new Headers([["x", "y"]]))Headers.prototype[Symbol.iterator]during fetch registration.entries,keys, andvaluesas class methods onHeaders.Why
Headersshould be iterable over[name, value]pairs. Without@@iterator, spread andMapconstruction throw aTypeError.Testing
cargo test -p boa_runtime fetch::tests -- --nocapture All fetch tests pass, including the new regression test.