Skip to content

[JSC] ShadowRealm: wrap a generic target's return value for the caller's realm - #590

Open
robobun wants to merge 1 commit into
mainfrom
robobun/b1492092/shadow-realm-wrapper-caller-realm
Open

robobun wants to merge 1 commit into
mainfrom
robobun/b1492092/shadow-realm-wrapper-caller-realm

[JSC] ShadowRealm: wrap a generic target's return value for the calle…

c7520f6
Select commit
Loading
Failed to load commit list.
Claude / Claude Code Review completed Sep 8, 2026 in 8m 16s

Code review found 1 potential issue

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

Details

Severity Count
🔴 Important 0
🟡 Nit 1
🟣 Pre-existing 0
Severity File:Line Issue
🟡 Nit JSTests/stress/shadow-realm-wrapped-function-return-value-realm.js:27 nit: The inner loop hardcodes '200' iterations instead of using 'testLoopCount', which JSTests/README.md (imported by JS