fix: check for Vaadin native process (#1297) (CP: 24.6)

.github/workflows/validation-native.yaml

@@ -61,7 +61,7 @@ jobs:
         id: set-matrix
         run: |
           test_modules=$(find integration-tests -maxdepth 1 -mindepth 1 -type d -name "*-tests")
-          test_modules=$(while IFS= read -r mod; do grep -q "%test-security." $mod/src/main/resources/application.properties; echo "{\"name\": \"${mod/integration-tests\//}\", \"path\": \"$mod\", \"security\": $? }"; done <<< "${test_modules}" | jq -s -c '.')
+          test_modules=$(while IFS= read -r mod; do test -f $mod/src/main/resources/application.properties && grep -q "%test-security." $mod/src/main/resources/application.properties; echo "{\"name\": \"${mod/integration-tests\//}\", \"path\": \"$mod\", \"security\": $? }"; done <<< "${test_modules}" | jq -s -c '.')
           echo "modules=${test_modules}" >> "$GITHUB_OUTPUT"
   build-and-test-native:
     name: Native tests
@@ -98,7 +98,7 @@ jobs:
           set -x -e -o pipefail
           mvn -V -e -B -ntp verify -am -DtrimStackTrace=false -Dselenide.browserBinary=${{ steps.setup-chrome.outputs.chrome-path }} -Pit-tests,production -Dnative -pl ${{ matrix.module.path }}
       - name: Native Security Test
-        if: ${{ matrix.module.security == 1 }}
+        if: ${{ matrix.module.security == 0 }}
         run: |
           set -x -e -o pipefail
           mvn -V -e -B -ntp verify -am -DtrimStackTrace=false -Dselenide.browserBinary=${{ steps.setup-chrome.outputs.chrome-path }} -Pit-tests,production -Dnative -Dquarkus.profile=test-security -pl ${{ matrix.module.path }}

commons/deployment/src/main/java/com/github/mcollovati/quarkus/hilla/deployment/QuarkusHillaNativeProcessor.java

@@ -110,13 +110,13 @@
 
 public class QuarkusHillaNativeProcessor {
 
-    @BuildStep(onlyIf = IsNativeBuild.class)
+    @BuildStep(onlyIf = IsNativeBuild.class, onlyIfNot = IsVaadinNativeProcessorAvailable.class)
     void patchAtmosphere(CombinedIndexBuildItem index, BuildProducer<BytecodeTransformerBuildItem> producer) {
         AtmospherePatches patcher = new AtmospherePatches(index.getComputingIndex());
         patcher.apply(producer);
     }
 
-    @BuildStep(onlyIf = IsNativeBuild.class)
+    @BuildStep(onlyIf = IsNativeBuild.class, onlyIfNot = IsVaadinNativeProcessorAvailable.class)
     @Record(ExecutionTime.RUNTIME_INIT)
     @Produce(DefaultRouteBuildItem.class)
     void deferAtmosphereInit(
@@ -156,7 +156,7 @@ void removeHillaSpringBasedClasses(Capabilities capabilities, BuildProducer<Remo
      * application is build with subscription key and the license-checker is not
      * configured as explicit project dependency.
      */
-    @BuildStep(onlyIf = IsNativeBuild.class)
+    @BuildStep(onlyIf = IsNativeBuild.class, onlyIfNot = IsVaadinNativeProcessorAvailable.class)
     void generateDummyDauClassesIfLicenseCheckerIsNotPresent(
             BuildProducer<GeneratedNativeImageClassBuildItem> producer) {
         String dauIntegration = "com.vaadin.pro.licensechecker.dau.DauIntegration";
@@ -223,6 +223,7 @@ void hillaNativeSupport(
         nativeImageResource.produce(NativeImageResourcePatternsBuildItem.builder()
                 .includePatterns("hilla-openapi\\.json", "hilla-engine-configuration\\.json", "file-routes\\.json")
                 .includePatterns("META-INF/microprofile-config\\.properties")
+                .includePatterns("META-INF/maven/com.github.mcollovati/quarkus-hilla-commons/pom\\.properties")
                 .build());
 
         IndexView index = combinedIndex.getComputingIndex();
@@ -286,7 +287,7 @@ private Set<ClassInfo> getJsonClasses(IndexView index) {
         return classes;
     }
 
-    @BuildStep
+    @BuildStep(onlyIfNot = IsVaadinNativeProcessorAvailable.class)
     void vaadinNativeSupport(
             CombinedIndexBuildItem combinedIndex,
             BuildProducer<RuntimeInitializedPackageBuildItem> runtimeInitializedPackage,
@@ -298,7 +299,7 @@ void vaadinNativeSupport(
         nativeImageResource.produce(NativeImageResourcePatternsBuildItem.builder()
                 .includeGlobs("META-INF/VAADIN/**", "com/vaadin/**", "vaadin-i18n/**")
                 .includePatterns("org/atmosphere/util/version\\.properties")
-                .includePatterns("META-INF/maven/com.github.mcollovati/quarkus-hilla-commons/pom\\.properties")
+                .includePatterns("META-INF/maven/com.vaadin/vaadin-core/pom\\.properties")
                 .build());
 
         runtimeInitializedPackage.produce(new RuntimeInitializedPackageBuildItem("org.atmosphere.util.analytics"));
@@ -440,4 +441,20 @@ public boolean getAsBoolean() {
             return nativeConfig.enabled();
         }
     }
+
+    public static class IsVaadinNativeProcessorAvailable implements BooleanSupplier {
+
+        @Override
+        public boolean getAsBoolean() {
+            try {
+                Class.forName(
+                        "com.vaadin.quarkus.deployment.VaadinQuarkusNativeProcessor",
+                        false,
+                        Thread.currentThread().getContextClassLoader());
+                return true;
+            } catch (ClassNotFoundException e) {
+                return false;
+            }
+        }
+    }
 }