Merge branch 'release/0.19.0'

Author: pmlopes

.github/workflows/graalvm-dev.yml

@@ -0,0 +1,41 @@
+name: GraalVM CI (dev)
+on:
+  push:
+    branches:
+      - develop
+  pull_request:
+    branches:
+      - develop
+jobs:
+  build:
+    name: ${{ matrix.version }} on ${{ matrix.os }}
+    runs-on: ${{ matrix.os }}
+    strategy:
+      matrix:
+        version: [dev]
+        os: [macos-latest, windows-latest, ubuntu-latest]
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v2
+      - name: Cache JVM
+        id: cache-java
+        uses: actions/cache@v2
+        with:
+          path: |
+            ${{ runner.temp }}/java_package.tar.gz
+            ~/.m2/repository
+          key: ${{ runner.os }}-java-${{ hashFiles('**/pom.xml') }}
+          restore-keys: |
+            ${{ runner.os }}-java-
+      - name: Setup GraalVM
+        uses: graalvm/setup-graalvm@v1
+        with:
+          version: ${{ matrix.version }}
+          java-version: '11'
+          components: 'js'
+      - name: Run tests (UNIX)
+        run: ./mvnw clean verify
+        if: runner.os != 'Windows'
+      - name: Run tests (Windows)
+        run: .\mvnw.cmd clean verify
+        if: runner.os == 'Windows'

.github/workflows/graalvm.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ${{ matrix.os }}
     strategy:
       matrix:
-        version: ['22.1.0', dev]
+        version: ['22.1.0']
         os: [macos-latest, windows-latest, ubuntu-latest]
     steps:
       - name: Checkout

CHANGELOG.md

@@ -5,7 +5,13 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/)
 and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html).
 
-## [0.18.0] -
+## [0.19.0] -
+* Bumped vertx to 4.3.2
+* Fixed bug where spaces were not allowed on CWD
+* Update CI as latest graal will not include `js` by default
+* Update docs for latest graal
+
+## [0.18.0] - 2022-04-28
 * Added new module `@vertx/uri-template`
 * Force commonjs loader to be synchronized
 * Bumped graal to 22.1.0

codegen/pom.xml

@@ -4,14 +4,14 @@
   <parent>
     <groupId>io.reactiverse</groupId>
     <artifactId>es4x-parent</artifactId>
-    <version>0.18.0</version>
+    <version>0.19.0</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
   <modelVersion>4.0.0</modelVersion>
 
   <artifactId>es4x-codegen</artifactId>
-  <version>0.18.0</version>
+  <version>0.19.0</version>
 
   <dependencies>
     <dependency>

docs/README.md

@@ -35,6 +35,6 @@ npm start # OR yarn start
 ```
 
 ::: tip COMPATIBILITY NOTE
-ES4X requires [GraalVM](https://www.graalvm.org) or Java >= 11. If you're not sure which version is installed on your
-system, consider using [jabba](https://github.com/shyiko/jabba).
+ES4X requires [GraalVM](https://www.graalvm.org) or Java >= 11. GraalVM 22.2 and up will require the installation of the
+JavaScript module with `gu install js`.
 :::

docs/advanced/vertx.md

@@ -85,11 +85,15 @@ Can be used as a `Thenable`:
 
 ```js
 try {
-  let server = await vertx
-    .createHttpServer()
-    .listen(0);
-
-  console.log('Server Ready!');
+  (async function futureTest1 () {
+    let server = await vertx
+        .createHttpServer()
+        .requestHandler(req => {
+        })
+        .listen(0);
+
+    console.log('Server Ready!');
+  })();
 } catch (err) {
   console.log('Server startup failed!')
 }

es4x/pom.xml

@@ -4,14 +4,14 @@
   <parent>
     <groupId>io.reactiverse</groupId>
     <artifactId>es4x-parent</artifactId>
-    <version>0.18.0</version>
+    <version>0.19.0</version>
     <relativePath>../pom.xml</relativePath>
   </parent>
 
   <modelVersion>4.0.0</modelVersion>
 
   <artifactId>es4x</artifactId>
-  <version>0.18.0</version>
+  <version>0.19.0</version>
 
   <properties>
     <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>

es4x/src/main/java/io/reactiverse/es4x/impl/ImportMapper.java

@@ -22,11 +22,9 @@ public class ImportMapper {
   private static final Logger LOGGER = LoggerFactory.getLogger(ImportMapper.class);
 
   public ImportMapper(JsonObject config) throws MalformedURLException {
-    this(config, new File(VertxFileSystem.getCWD()).toURI());
-  }
-
-  public ImportMapper(JsonObject config, URI baseURI) throws MalformedURLException {
-    this(config, baseURI.toURL());
+    this(
+      config,
+      new URL("file", "", Utils.slashify(VertxFileSystem.getCWD(), true)));
   }
 
   public ImportMapper(JsonObject config, URL baseURL) {

es4x/src/main/java/io/reactiverse/es4x/impl/Utils.java

@@ -1,11 +1,9 @@
 package io.reactiverse.es4x.impl;
 
 import java.io.*;
-import java.net.HttpURLConnection;
-import java.net.URI;
-import java.net.URISyntaxException;
-import java.net.URL;
+import java.net.*;
 import java.nio.charset.StandardCharsets;
+import java.nio.file.Files;
 import java.security.MessageDigest;
 import java.security.NoSuchAlgorithmException;
 import java.util.Enumeration;
@@ -61,7 +59,7 @@ public static void downloadTo(URL url, File target) throws IOException {
             throw new RuntimeException("Failed to mkdirs: " + parent);
           }
         }
-        try (BufferedOutputStream writer = new BufferedOutputStream(new FileOutputStream(target))) {
+        try (BufferedOutputStream writer = new BufferedOutputStream(Files.newOutputStream(target.toPath()))) {
           byte[] buffer = new byte[4096];
           int bytesRead;
           while ((bytesRead = reader.read(buffer)) != -1) {
@@ -93,10 +91,10 @@ public static String getManifestAttribute(String attribute) throws IOException {
     return null;
   }
 
-  public static URI fileToURI(File file) {
+  public static URL fileToURL(File file) {
     try {
-      return new URI("file://" + slashify(file.getPath(), file.isDirectory()));
-    } catch (URISyntaxException e) {
+      return new URL("file://" + slashify(file.getPath(), file.isDirectory()));
+    } catch (MalformedURLException e) {
       throw new IllegalArgumentException("Cannot convert to URI: " + file, e);
     }
   }

es4x/src/main/java/io/reactiverse/es4x/impl/VertxFileSystem.java

@@ -80,7 +80,7 @@ public VertxFileSystem(final Vertx vertx, String importMap, String... extensions
     // resolve the well known roots
     try {
       cwd = getCWD();
-      URI cwdUrl = fileToURI(new File(this.cwd).getCanonicalFile());
+      URL cwdUrl = fileToURL(new File(this.cwd).getCanonicalFile());
 
       if (importMap == null) {
         mapper = new ImportMapper(