Skip to content

Upgrade WebKit to 51cc3feb7298 - #31796

Merged
Jarred-Sumner merged 7 commits into
mainfrom
claude/webkit-upgrade-51cc3feb7298
Jun 16, 2026
Merged

Upgrade WebKit to 51cc3feb7298#31796
Jarred-Sumner merged 7 commits into
mainfrom
claude/webkit-upgrade-51cc3feb7298

Upgrade WebKit to 24362e675175 (#32105)

acf96b0
Select commit
Loading
Failed to load commit list.
Claude / Claude Code Review completed Jun 12, 2026 in 19m 51s

Code review found 1 important issue

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

Details

Severity Count
🔴 Important 1
🟡 Nit 0
🟣 Pre-existing 0
Severity File:Line Issue
🔴 Important scripts/build/deps/webkit.ts:10 WEBKIT_VERSION points to preview build, contradicting PR description

Annotations

Check failure on line 10 in scripts/build/deps/webkit.ts

See this annotation in the file changed.

@claude claude / Claude Code Review

WEBKIT_VERSION points to preview build, contradicting PR description

The PR description states `WEBKIT_VERSION` now points at the release `autobuild-5851d4722e461bae1eb5537b091f4103e192a94a` (from merged oven-sh/WebKit#248), but this diff sets it to `autobuild-preview-pr-251-92b221c1` — a preview build of a different PR. Preview-PR autobuild artifacts are ephemeral and get GC'd, so once that release is deleted `prebuiltUrl()` will 404 for everyone on the prebuilt path. Either the final bump to the release tag wasn't pushed or the description is stale; this should