Skip to content

Bump JsRuntimeHost for V8 asynchronous WebAssembly task handling - #1870

Merged
bkaradzic-microsoft merged 1 commit into
masterfrom
bump/jsruntimehost-20260910
Sep 11, 2026
Merged

bkaradzic-microsoft merged 1 commit into
masterfrom
bump/jsruntimehost-20260910

Conversation

@bkaradzic-microsoft

Copy link
Copy Markdown
Member

Updates the JsRuntimeHost pin from a6b98eaa1a9887b35adceed21f8c7d44c4f38e43 to f47991dd5729fac1d6d477e66ed8a843da1d4832, the latest upstream main.

Includes BabylonJS/JsRuntimeHost#232:

  • Dispatches V8 foreground tasks through AppRuntime so asynchronous WebAssembly compilation promises settle instead of hanging.
  • Shares the delayed-task scheduler between V8 foreground work and the scheduling polyfill, with explicit ownership and shutdown handling.

This is a one-line dependency-pin update based on current Babylon Native master.

The standard-stream logger follow-up, BabylonJS/JsRuntimeHost#236, is still open and is not included in this revision.

Update to f47991dd5729fac1d6d477e66ed8a843da1d4832, the latest
JsRuntimeHost main revision. This includes BabylonJS/JsRuntimeHost#232,
which dispatches V8 foreground tasks through AppRuntime so asynchronous
WebAssembly compilation promises can settle.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot-Session: 88569c10-a7ff-4373-9a58-afa9c68b8c09
Copilot AI lite review requested due to automatic review settings September 11, 2026 00:08

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 Needs a closer look

The runtime and scheduler changes warrant final human validation.

Pull request overview

Updates the JsRuntimeHost dependency for asynchronous WebAssembly task handling.

Changes:

  • Bumps the JsRuntimeHost commit pin.
  • Includes upstream PR #232’s task-dispatch and scheduler lifecycle fixes.
  • Excludes the open standard-stream logger follow-up.
File summaries
File Description
CMakeLists.txt Updates the JsRuntimeHost dependency revision.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@bkaradzic-microsoft
bkaradzic-microsoft merged commit d649994 into master Sep 11, 2026
35 checks passed
@bkaradzic-microsoft
bkaradzic-microsoft deleted the bump/jsruntimehost-20260910 branch September 11, 2026 02:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants