feat(graalvm): add polyfilled fetch support

feat(runtime): add polyfill for `fetch` api chore(graalvm): rebuild polyfills and modules Signed-off-by: Sam Gammon <sam@elide.dev>
elide-dev · Feb 23, 2025 · 1c66a5d · 1c66a5d
1 parent c00e776
commit 1c66a5d
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 9 deletions.
diff --git a/packages/cli/src/main/kotlin/elide/tool/cli/Elide.kt b/packages/cli/src/main/kotlin/elide/tool/cli/Elide.kt
@@ -115,6 +115,7 @@ import elide.tool.io.RuntimeWorkdirManager
       NativeEngine.boot(RuntimeWorkdirManager.acquire()) {
         listOf(
           "elide.js.vm.enableStreams" to "true",
+          "jdk.httpclient.allowRestrictedHeaders" to "Host,Content-Length",  // needed for fetch
           "io.netty.allocator.maxOrder" to "3",
           "io.netty.serviceThreadPrefix" to "elide-svc",
           "io.netty.native.deleteLibAfterLoading" to "true",  // reversed bc of bug (actually does not delete)

diff --git a/packages/graalvm/src/main/kotlin/elide/runtime/core/internals/graalvm/GraalVMEngine.kt b/packages/graalvm/src/main/kotlin/elide/runtime/core/internals/graalvm/GraalVMEngine.kt
@@ -101,6 +101,7 @@ import org.graalvm.polyglot.HostAccess as PolyglotHostAccess
       .allowCreateProcess(true)
       .allowHostClassLoading(true)
       .allowNativeAccess(true)
+      .allowHostClassLookup { true }
       .engine(engine)
 
     // allow plugins to customize the context on creation

diff --git a/packages/graalvm/src/main/kotlin/elide/runtime/plugins/js/JavaScript.kt b/packages/graalvm/src/main/kotlin/elide/runtime/plugins/js/JavaScript.kt
@@ -128,22 +128,23 @@ import elide.runtime.plugins.js.JavaScriptVersion.*
       "js.string-lazy-substrings",
       "js.shadow-realm",
       "js.zone-rules-based-time-zones",
+      // Enabled for use by polyfills or for experimental features:
+      "js.java-package-globals",
+      "js.graal-builtin",
+      "js.polyglot-evalfile",
+      "js.load",
+      "js.polyglot-builtin",
     )
 
     disableOptions(
       "js.console",
-      "js.graal-builtin",
       "js.interop-complete-promises",
-      "js.java-package-globals",
-      "js.load",
-      "js.polyglot-evalfile",
       "js.print",
       "js.regexp-static-result",
       "js.scripting",
       "js.syntax-extensions",
       // Experimental:
       "js.operator-overloading",
-      "js.polyglot-builtin",
     )
 
     setOptions(

diff --git a/packages/graalvm/src/main/resources/META-INF/elide/embedded/runtime/js/polyfills.js b/packages/graalvm/src/main/resources/META-INF/elide/embedded/runtime/js/polyfills.js
diff --git a/packages/graalvm/src/test/kotlin/elide/runtime/intrinsics/js/JsGlobalsTest.kt b/packages/graalvm/src/test/kotlin/elide/runtime/intrinsics/js/JsGlobalsTest.kt
@@ -241,8 +241,6 @@ private const val ENABLE_SUPPRESSIONS = true
     "TextDecoderStream",  // not yet implemented
     "TextEncoderStream",  // not yet implemented
     "DOMException",  // not yet implemented
-  ).plus(
-    fetchGlobals
   )
 
   // Types which are expected to be provided by JS polyfills.
@@ -255,6 +253,8 @@ private const val ENABLE_SUPPRESSIONS = true
     "TransformStreamDefaultController",
     "WritableStreamDefaultController",
     "WritableStreamDefaultWriter",
+  ).plus(
+    fetchGlobals
   )).toSortedSet()
 
   // Globals which are expected not to be found host-side.

diff --git a/runtime b/runtime
+1 −0		elide/runtime/js/polyfills/BUILD.bazel
+930 −0		elide/runtime/js/polyfills/fetch.js
+1 −0		elide/runtime/js/polyfills/polyfills.js