fix: add missing reflection hints for type used with ReflectTools (#23059) (CP: 24.9) (#23069)

* fix: register reflection and resource hints for error handlers in native builds (#23054)

Extract and register exception types from HasErrorParameter<E> generic
parameter for reflection. Register resource hints for ClassPathScanner
to discover error handler classes at runtime in GraalVM native images.

Fixes #23041

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: Zhe Sun <31067185+ZheSun88@users.noreply.github.com>

* fix: add missing reflection hints for type used with ReflectTools (#23059)

WebComponentExporter, I18NProvider, and MenuAccessControl might be instantiated
via ReflectTools.createInstance and were missing reflection hints in
VaadinBeanFactoryInitializationAotProcessor, which could cause issues at runtime
in native executables.

Fixes #23060

---------

Co-authored-by: Claude Opus 4.5 <noreply@anthropic.com>
Co-authored-by: Zhe Sun <31067185+ZheSun88@users.noreply.github.com>

Author: 3 people

vaadin-spring/src/main/java/com/vaadin/flow/spring/springnative/VaadinBeanFactoryInitializationAotProcessor.java

@@ -47,14 +47,17 @@
 import com.vaadin.flow.component.Component;
 import com.vaadin.flow.component.ComponentEvent;
 import com.vaadin.flow.component.UI;
+import com.vaadin.flow.component.WebComponentExporter;
 import com.vaadin.flow.component.page.AppShellConfigurator;
+import com.vaadin.flow.i18n.I18NProvider;
 import com.vaadin.flow.router.HasErrorParameter;
 import com.vaadin.flow.router.HasUrlParameter;
 import com.vaadin.flow.router.Layout;
 import com.vaadin.flow.router.Route;
 import com.vaadin.flow.router.RouteAlias;
 import com.vaadin.flow.router.RouterLayout;
 import com.vaadin.flow.server.PWA;
+import com.vaadin.flow.server.auth.MenuAccessControl;
 
 public class VaadinBeanFactoryInitializationAotProcessor
         implements BeanFactoryInitializationAotProcessor {
@@ -121,6 +124,10 @@ public BeanFactoryInitializationAotContribution processAheadOfTime(
                 registerSubTypes(hints, reflections, HasUrlParameter.class);
                 registerSubTypes(hints, reflections,
                         "com.vaadin.flow.data.converter.Converter");
+                registerSubTypes(hints, reflections,
+                        WebComponentExporter.class);
+                registerSubTypes(hints, reflections, I18NProvider.class);
+                registerSubTypes(hints, reflections, MenuAccessControl.class);
             }
         };
     }

vaadin-spring/src/test/java/com/vaadin/flow/spring/springnative/VaadinBeanFactoryInitializationAotProcessorTest.java

@@ -42,7 +42,10 @@
 import com.vaadin.flow.component.ComponentEvent;
 import com.vaadin.flow.component.Tag;
 import com.vaadin.flow.component.UI;
+import com.vaadin.flow.component.WebComponentExporter;
 import com.vaadin.flow.component.page.AppShellConfigurator;
+import com.vaadin.flow.component.webcomponent.WebComponent;
+import com.vaadin.flow.i18n.I18NProvider;
 import com.vaadin.flow.router.BeforeEnterEvent;
 import com.vaadin.flow.router.BeforeEvent;
 import com.vaadin.flow.router.ErrorParameter;
@@ -53,6 +56,7 @@
 import com.vaadin.flow.router.RouteAlias;
 import com.vaadin.flow.router.RouterLayout;
 import com.vaadin.flow.server.PWA;
+import com.vaadin.flow.server.auth.MenuAccessControl;
 
 import static org.assertj.core.api.Assertions.assertThat;
 import static org.mockito.ArgumentMatchers.any;
@@ -621,6 +625,76 @@ void getSubtypesOf_excludesParentType() {
                 .isNotEmpty().doesNotContain(RouterLayout.class);
     }
 
+    @Test
+    void getSubtypesOf_findsWebComponentExporterSubclasses() {
+        VaadinBeanFactoryInitializationAotProcessor processor = new VaadinBeanFactoryInitializationAotProcessor();
+
+        String packageName = getClass().getPackageName();
+        Collection<Class<?>> exporters = processor.getSubtypesOf(
+                new Reflections(packageName), WebComponentExporter.class);
+
+        assertThat(exporters).as("Should find WebComponentExporter subclasses")
+                .contains(TestWebComponentExporter.class);
+    }
+
+    @Test
+    void getSubtypesOf_findsI18NProviderImplementations() {
+        VaadinBeanFactoryInitializationAotProcessor processor = new VaadinBeanFactoryInitializationAotProcessor();
+
+        String packageName = getClass().getPackageName();
+        Collection<Class<?>> providers = processor.getSubtypesOf(
+                new Reflections(packageName), I18NProvider.class);
+
+        assertThat(providers).as("Should find I18NProvider implementations")
+                .contains(TestI18NProvider.class);
+    }
+
+    @Test
+    void getSubtypesOf_findsMenuAccessControlImplementations() {
+        VaadinBeanFactoryInitializationAotProcessor processor = new VaadinBeanFactoryInitializationAotProcessor();
+
+        String packageName = getClass().getPackageName();
+        Collection<Class<?>> accessControls = processor.getSubtypesOf(
+                new Reflections(packageName), MenuAccessControl.class);
+
+        assertThat(accessControls)
+                .as("Should find MenuAccessControl implementations")
+                .contains(TestMenuAccessControl.class);
+    }
+
+    @Test
+    void processAheadOfTime_webComponentExporterSubtype_reflectionHintRegistered() {
+        RuntimeHints hints = processAotForHintsWithSubtypes(
+                TestWebComponentExporter.class, WebComponentExporter.class);
+
+        assertThat(RuntimeHintsPredicates.reflection()
+                .onType(TestWebComponentExporter.class))
+                .as("WebComponentExporter subtype should be registered for reflection")
+                .accepts(hints);
+    }
+
+    @Test
+    void processAheadOfTime_i18nProviderSubtype_reflectionHintRegistered() {
+        RuntimeHints hints = processAotForHintsWithSubtypes(
+                TestI18NProvider.class, I18NProvider.class);
+
+        assertThat(RuntimeHintsPredicates.reflection()
+                .onType(TestI18NProvider.class))
+                .as("I18NProvider subtype should be registered for reflection")
+                .accepts(hints);
+    }
+
+    @Test
+    void processAheadOfTime_menuAccessControlSubtype_reflectionHintRegistered() {
+        RuntimeHints hints = processAotForHintsWithSubtypes(
+                TestMenuAccessControl.class, MenuAccessControl.class);
+
+        assertThat(RuntimeHintsPredicates.reflection()
+                .onType(TestMenuAccessControl.class))
+                .as("MenuAccessControl subtype should be registered for reflection")
+                .accepts(hints);
+    }
+
     @Test
     void getAnnotatedClasses_findsClassesWithSpecificAnnotation() {
         VaadinBeanFactoryInitializationAotProcessor processor = new VaadinBeanFactoryInitializationAotProcessor();
@@ -903,4 +977,41 @@ public static class RouteWithUILayout extends Component {
     @Tag("div")
     public static class RouteWithDefaultLayout extends Component {
     }
+
+    public static class TestWebComponentExporter
+            extends WebComponentExporter<Component> {
+        public TestWebComponentExporter() {
+            super("test-exporter");
+        }
+
+        @Override
+        protected void configureInstance(WebComponent<Component> webComponent,
+                Component component) {
+        }
+    }
+
+    public static class TestI18NProvider implements I18NProvider {
+        @Override
+        public java.util.List<java.util.Locale> getProvidedLocales() {
+            return java.util.List.of(java.util.Locale.ENGLISH);
+        }
+
+        @Override
+        public String getTranslation(String key, java.util.Locale locale,
+                Object... params) {
+            return key;
+        }
+    }
+
+    public static class TestMenuAccessControl implements MenuAccessControl {
+        @Override
+        public void setPopulateClientSideMenu(
+                PopulateClientMenu populateClientMenu) {
+        }
+
+        @Override
+        public PopulateClientMenu getPopulateClientSideMenu() {
+            return PopulateClientMenu.AUTOMATIC;
+        }
+    }
 }