From bba3878aced2a921f2697b817700085382e1d0e3 Mon Sep 17 00:00:00 2001
From: Hiram Chirino <hiram@hiramchirino.com>
Date: Mon, 2 Jun 2025 09:09:00 -0400
Subject: [PATCH] feat: upgrade Chicory runtime to v1.4.0 and improve build
 configuration
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

• Upgrade Chicory WebAssembly runtime from version 1.3.0 to 1.4.0
• Update Chicory dependency artifact names to new stable packages:
  - Replace experimental aot-experimental with annotations
  - Replace host-module-annotations-experimental with chicory-compiler-maven-plugin
  - Replace host-module-processor-experimental with annotations-processor
• Improve Spotless configuration with enhanced wildcard import detection:
  - Replace simple regex-based wildcard import removal with Groovy-based validation
  - Add clear error messages for wildcard imports requiring manual expansion
• Add not-ci profile for automatic code formatting during local development
• Update documentation references to point to current Chicory runtime compiler docs

This upgrade modernizes the project's WebAssembly runtime dependencies and improves
the developer experience by providing better build validation and automatic code
formatting. The Chicory 1.4.0 upgrade brings performance improvements and stability
enhancements to WebAssembly execution.

Signed-off-by: Hiram Chirino <hiram@hiramchirino.com>
---
 pom.xml                                       | 47 ++++++++++++++++---
 proxy-wasm-java-host/pom.xml                  |  6 +--
 .../java/io/roastedroot/proxywasm/Plugin.java |  5 +-
 .../roastedroot/proxywasm/internal/ABI.java   |  4 +-
 4 files changed, 49 insertions(+), 13 deletions(-)

diff --git a/pom.xml b/pom.xml
index de1412b..c89e2af 100644
--- a/pom.xml
+++ b/pom.xml
@@ -72,7 +72,7 @@
     <junit.version>5.12.2</junit.version>
 
     <!-- runtime versions -->
-    <chicory.version>1.3.0</chicory.version>
+    <chicory.version>1.4.0</chicory.version>
     <jersey.version>3.1.10</jersey.version>
     <jetty.version>11.0.25</jetty.version>
 
@@ -112,11 +112,18 @@
                 <formatJavadoc>false</formatJavadoc>
               </googleJavaFormat>
               <importOrder/>
-              <replaceRegex>
-                <name>Remove wildcard imports</name>
-                <searchRegex>import\s+(?:static\s+)?[^\*\s]+\*;(\r\n|\r|\n)</searchRegex>
-                <replacement>$1</replacement>
-              </replaceRegex>
+              <jsr223>
+                <name>Wildcard Imports Not Allowed</name>
+                <dependency>org.apache.groovy:groovy-jsr223:4.0.27</dependency>
+                <engine>groovy</engine>
+                <script>def pattern = ~/import\s+(?:static\s+)?[^\*\s]+\*;(\r\n|\r|\n)/
+                                    def matcher = pattern.matcher(source)
+                                    if (matcher.find()) {
+                                    def importText = matcher.group().trim()
+                                    throw new Exception("Wildcard imports not allowed:\n\n    " + importText + "\n\nPlease fully expand the imports.\n")
+                                    }
+                                    source</script>
+              </jsr223>
               <removeUnusedImports/>
             </java>
             <pom>
@@ -230,6 +237,34 @@
   </build>
 
   <profiles>
+    <profile>
+      <id>not-ci</id>
+      <activation>
+        <property>
+          <name>!ci</name>
+        </property>
+      </activation>
+      <build>
+        <plugins>
+          <plugin>
+            <groupId>com.diffplug.spotless</groupId>
+            <artifactId>spotless-maven-plugin</artifactId>
+            <configuration>
+              <checkSkip>true</checkSkip>
+            </configuration>
+            <executions>
+              <execution>
+                <goals>
+                  <goal>apply</goal>
+                </goals>
+                <phase>verify</phase>
+              </execution>
+            </executions>
+          </plugin>
+        </plugins>
+      </build>
+    </profile>
+
     <profile>
       <id>release</id>
       <build>
diff --git a/proxy-wasm-java-host/pom.xml b/proxy-wasm-java-host/pom.xml
index 3ec1f8b..887e889 100644
--- a/proxy-wasm-java-host/pom.xml
+++ b/proxy-wasm-java-host/pom.xml
@@ -16,12 +16,12 @@
   <dependencies>
     <dependency>
       <groupId>com.dylibso.chicory</groupId>
-      <artifactId>aot-experimental</artifactId>
+      <artifactId>annotations</artifactId>
       <version>${chicory.version}</version>
     </dependency>
     <dependency>
       <groupId>com.dylibso.chicory</groupId>
-      <artifactId>host-module-annotations-experimental</artifactId>
+      <artifactId>chicory-compiler-maven-plugin</artifactId>
       <version>${chicory.version}</version>
     </dependency>
     <dependency>
@@ -93,7 +93,7 @@
           <annotationProcessorPaths>
             <path>
               <groupId>com.dylibso.chicory</groupId>
-              <artifactId>host-module-processor-experimental</artifactId>
+              <artifactId>annotations-processor</artifactId>
               <version>${chicory.version}</version>
             </path>
           </annotationProcessorPaths>
diff --git a/proxy-wasm-java-host/src/main/java/io/roastedroot/proxywasm/Plugin.java b/proxy-wasm-java-host/src/main/java/io/roastedroot/proxywasm/Plugin.java
index 55e9879..73a5dc6 100644
--- a/proxy-wasm-java-host/src/main/java/io/roastedroot/proxywasm/Plugin.java
+++ b/proxy-wasm-java-host/src/main/java/io/roastedroot/proxywasm/Plugin.java
@@ -301,9 +301,10 @@ public Builder withImportMemory(ImportMemory memory) {
          * The {@link Machine} controls the low-level execution of WASM instructions.
          * By default, an interpreter-based machine is used.
          * Providing a custom factory allows using alternative execution strategies, such as
-         * Ahead-Of-Time (AOT) compilation to improve execution performance.
+         * wasm to bytecode compilation to improve execution performance.
          *
-         * <p>See the Chicory documentation (https://chicory.dev/docs/experimental/aot) for more details on Aot compilation.
+         * <p>See the Chicory documentation (https://chicory.dev/docs/usage/runtime-compiler) for more details
+         * on WASM to bytecode compilation and execution.
          *
          * @param machineFactory A function that takes a WASM {@link Instance} and returns a {@link Machine}.
          * @return this {@code Builder} instance for method chaining.
diff --git a/proxy-wasm-java-host/src/main/java/io/roastedroot/proxywasm/internal/ABI.java b/proxy-wasm-java-host/src/main/java/io/roastedroot/proxywasm/internal/ABI.java
index 16fddca..6ba2d46 100644
--- a/proxy-wasm-java-host/src/main/java/io/roastedroot/proxywasm/internal/ABI.java
+++ b/proxy-wasm-java-host/src/main/java/io/roastedroot/proxywasm/internal/ABI.java
@@ -5,8 +5,8 @@
 import static io.roastedroot.proxywasm.internal.Helpers.split;
 import static io.roastedroot.proxywasm.internal.Helpers.string;
 
-import com.dylibso.chicory.experimental.hostmodule.annotations.HostModule;
-import com.dylibso.chicory.experimental.hostmodule.annotations.WasmExport;
+import com.dylibso.chicory.annotations.HostModule;
+import com.dylibso.chicory.annotations.WasmExport;
 import com.dylibso.chicory.runtime.ExportFunction;
 import com.dylibso.chicory.runtime.Instance;
 import com.dylibso.chicory.runtime.Memory;