[BridgeJS] Pass String parameters unretained by default

Offen

#677 erstellt am 26.2.2026 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit: 5/5
Geschätzter Aufwand: Über eine Woche
Anfängerfreundlichkeit: 28/100

Issue-Typ: Refactoring
Klarheit: Größtenteils klar
Aktivitätsstatus: Veraltet
Tech-Stack: javascript, swift, wasm
Bereich: api, performance, web-dev

Rechercherichtung

Start by tracing _swift_js_make_js_string, bridgeJSLowerParameter, and the generated bjs_hello thunk shown in the issue, then inspect how JSString references are represented. Done means ordinary String parameters cross the bridge without a JS memory-store retain/release cycle while JSString still supports caller-controlled by-reference reuse; verify the impact against the ElementaryUI performance benchmarks.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

When calling a JS function through BridgeJS with String parameters, every String is

lowered by calling _swift_js_make_js_string
which decodes and retains in JS object store (new entry, new ref = 1)
the object store ref is passed as wasm parameter
the JS thunk immediately get the object and releases (count--, remove from store)

this is very wasteful, and makes retain the number one bottleneck in current ElementaryUI performance benchmarks.

Example:

@JSFunction func hello(_ v: String) throws

// generates
func _$hello(_ v: String) throws(JSException) -> Void {
    let vValue = v.bridgeJSLowerParameter()
    bjs_hello(vValue)
    if let error = _swift_js_take_exception() {
        throw error
    }
}

function bjs_hello(v) {
    try {
        const vObject = swift.memory.getObject(v);
        swift.memory.release(v);
        imports.hello(vObject);
    } catch (error) {
        setException(error);
    }
}

I suggest:

passing String always "in-line" (ie: as address + length) without retaining in the JS memory store
still support JSString by-ref to control Swift caller caching (ie: retain once and use same ref multiple times)

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 JSException storage boxing once swiftlang/swift#89715 lands Offen

swiftwasm/JavaScriptKit#767 · 1 zugewiesene Person · Erstellt am 11.6.2026

Alle Issues in swiftwasm/JavaScriptKit