coop/rose-ash
1
0
Fork 0
Code Issues Pull Requests Actions 7 Packages Projects Releases Wiki Activity

Restore _driveAsync in sandbox host-callback

Browse Source 
With the pending_cek snapshot fix, _driveAsync no longer causes
duplicate resume chains. Needed for event-triggered suspensions
(btn.click → handler → perform) where the suspension propagates
through addEventListener, invisible to the outer eval.

Sandbox bytecode test: 6/6 io-sleep suspensions confirmed.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
giles
2026-04-08 21:40:17 +00:00
parent 6456bd927a
commit 52165f6a2a
1 changed files with 3 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tests/playwright/sx-inspect.js
View File

@@ -1391,10 +1391,9 @@ async function modeSandbox(page, expr, files, setup, stack, bytecode) {
if (typeof fn === 'function' && fn.__sx_handle === undefined) return fn; if (typeof fn === 'function' && fn.__sx_handle === undefined) return fn;
if (fn && fn.__sx_handle !== undefined) { if (fn && fn.__sx_handle !== undefined) {
return function() { return function() {
// Do NOT call _driveAsync here — the suspension propagates through const r = K.callFn(fn, Array.from(arguments));
// the normal CEK/VM return path. The outer callFn handles it. window._driveAsync(r);
// Calling _driveAsync would create a duplicate resume chain. return r;
return K.callFn(fn, Array.from(arguments));
}; };
} }
return function() {}; return function() {};