Skip to content

Upgrade to upstream WebKit 0e86b49069a5#280

Open
robobun wants to merge 842 commits into
mainfrom
bun/upgrade-to-0e86b49069a5
Open

Upgrade to upstream WebKit 0e86b49069a5#280
robobun wants to merge 842 commits into
mainfrom
bun/upgrade-to-0e86b49069a5

Merge fork main (4895f45d) to pick up #283 and the _LINKED_INTO cmake…

51b5559
Select commit
Loading
Failed to load commit list.
Claude / Claude Code Review completed Jul 14, 2026 in 20m 53s

Code review found 1 potential issue

Found 2 candidates, confirmed 1. See review comments for details.

Details

Severity Count
🔴 Important 0
🟡 Nit 0
🟣 Pre-existing 1
Severity File:Line Issue
🟣 Pre-existing JSTests/stress/typedarray-constructor-jit-compliant-toindex-firstarg-int16array.js:19 int16array test file uses Uint16Array instead of Int16Array

Annotations

Check notice on line 19 in JSTests/stress/typedarray-constructor-jit-compliant-toindex-firstarg-int16array.js

See this annotation in the file changed.

@claude claude / Claude Code Review

int16array test file uses Uint16Array instead of Int16Array

This test file is named for Int16Array but constructs `new Uint16Array(size)` at line 19, making it a byte-for-byte duplicate of `typedarray-constructor-jit-compliant-toindex-firstarg-uint16array.js` and leaving Int16Array uncovered for this JIT compliance path. This is a pre-existing upstream copy-paste error brought in verbatim by the merge — not blocking, but worth reporting to upstream WebKit.