BridgeJS: Remove JSException storage boxing once swiftlang/swift#89715 lands

Offen

#767 erstellt am 11.6.2026 0 Kommentare 0 Reaktionen 1 zugewiesene Person Auf GitHub ansehen

@krodak arbeitet bereits daran.

Seit 11.6.2026.

Bewertung

Dieses Issue wurde noch nicht bewertet.

Beschreibung

Tracks removal of the JSException storage boxing introduced in #766.

Context

swiftlang/swift#89320 miscompiles Wasm calls to captureless async throws(JSException) closure values when the typed error exceeds the direct error convention: the thrown error is corrupted across the async unwind, so the rejected Promise receives garbage. JSException (~36 bytes) always took the affected indirect-error path.

#766 works around this at the library level by boxing JSException's stored properties (thrownValue, description, stack) into a private final class, shrinking the struct to a single stored reference so it travels in the direct error convention and the broken path is never taken. The public API is unchanged; the cost is one heap allocation per thrown exception. The original exploration is in https://github.com/PassiveLogic/JavaScriptKit/pull/13.

The codegen-level counterpart for zero-parameter async throwing exports is tracked separately in #761.

What to do once the compiler fix ships

The proper fix is swiftlang/swift#89715 (IRGen: fix async typed throws miscompiles on Wasm). Once it is available in the oldest Swift toolchain JavaScriptKit supports:

Move thrownValue, description, and stack back to stored properties on JSException and delete the Storage class. This is a library-internal layout change only, not a breaking API change.
Keep the async closure reject end-to-end tests as the regression guard that the reject path still works without the boxing.

Vorherrschende Sprache: Swift
Sterne: 986
Forks: 76
Ø Merge: 21 Std. 11 Min.
Gemergte PRs (30 T.): 4

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
Forken Sie das Repository und arbeiten Sie in einem Branch.
Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus swiftwasm/JavaScriptKit

BridgeJS: Add diagnostics for unsupported `from: .global` / `from: .module(...)` parameters for imported symbols Offen

Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100

swiftwasm/JavaScriptKit#793 · Erstellt am 23.7.2026
BridgeJS: Same-named `@JS` types across modules collide silently in generated glue Offen

swiftwasm/JavaScriptKit#785 · 1 zugewiesene Person · Erstellt am 06.7.2026
[SwiftBuild] Preserve duplicate archive members when SwiftBuild expands static archives for linking Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100

swiftwasm/JavaScriptKit#781 · 3 Kommentare · Erstellt am 23.6.2026
☂️ `--build-system swiftbuild` compatibility issues Offen

Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 25/100

swiftwasm/JavaScriptKit#773 · Erstellt am 23.6.2026
BridgeJS: Remove forced capture workaround in async thunk body closures once swiftlang/swift#89715 lands Offen

swiftwasm/JavaScriptKit#761 · 1 zugewiesene Person · Erstellt am 10.6.2026

Alle Issues in swiftwasm/JavaScriptKit