allow bulk external (#842)

Author: SentryMan

inject-generator/src/main/java/io/avaje/inject/generator/InjectProcessor.java

@@ -176,10 +176,16 @@ public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment
     maybeElements(roundEnv, ExternalPrism.PRISM_TYPE).stream()
       .flatMap(Set::stream)
       .forEach(e -> {
-        var type = UType.parse(e.asType());
-        type = "java.util.List".equals(type.mainType()) ? type.param0() : type;
-        ProcessingContext.addOptionalType(type.fullWithoutAnnotations(), Util.named(e));
-        ProcessingContext.addOptionalType(type.fullWithoutAnnotations(), null);
+        if (e instanceof ExecutableElement) {
+          ExecutableElement method = (ExecutableElement) e;
+          method.getParameters().forEach(p -> {
+            var type = UType.parse(p.asType());
+            addExternalType(p, type);
+          });
+        } else {
+          var type = UType.parse(e.asType());
+          addExternalType(e, type);
+        }
       });
 
     maybeElements(roundEnv, ServiceProviderPrism.PRISM_TYPE).ifPresent(this::registerSPI);
@@ -207,6 +213,12 @@ public boolean process(Set<? extends TypeElement> annotations, RoundEnvironment
     return false;
   }
 
+  private void addExternalType(Element e, UType type) {
+    type = "java.util.List".equals(type.mainType()) ? type.param0() : type;
+    ProcessingContext.addOptionalType(type.fullWithoutAnnotations(), Util.named(e));
+    ProcessingContext.addOptionalType(type.fullWithoutAnnotations(), null);
+  }
+
   private void validateQualifier(ExecutableElement method) {
     var type = APContext.asTypeElement(method.getReturnType());
     if (type == null || type.getKind() != ElementKind.ANNOTATION_TYPE) {

inject-generator/src/test/java/io/avaje/inject/generator/models/valid/external/BulkExternal.java

@@ -0,0 +1,12 @@
+package io.avaje.inject.generator.models.valid.external;
+
+import java.lang.ref.WeakReference;
+
+import io.avaje.inject.External;
+import jakarta.inject.Singleton;
+
+@Singleton
+public class BulkExternal {
+  @External
+  public BulkExternal(WeakReference<Integer> mace, Cloneable jango) {}
+}

inject/src/main/java/io/avaje/inject/External.java

@@ -1,5 +1,6 @@
 package io.avaje.inject;
 
+import static java.lang.annotation.ElementType.CONSTRUCTOR;
 import static java.lang.annotation.ElementType.FIELD;
 import static java.lang.annotation.ElementType.PARAMETER;
 import static java.lang.annotation.RetentionPolicy.SOURCE;
@@ -10,13 +11,14 @@
 import java.lang.reflect.Type;
 
 /**
- * Marks this dependency as an external bean not managed by avaje inject.
- * Compile-time validation will be disabled for this type.
- * <p>
- * The external dependency is expected to be provided by
- * {@link BeanScopeBuilder#bean(String, Type, Object)}.
+ * Marks dependencies as an external beans potentially not managed by avaje inject. Compile-time
+ * validation will be disabled for types annotated.
+ *
+ * <p>The external dependency is usually expected to be provided by {@link
+ * BeanScopeBuilder#bean(String, Type, Object)} or by other modules in cases where ordering is
+ * irregular.
  */
 @Documented
 @Retention(SOURCE)
-@Target({FIELD, PARAMETER})
+@Target({FIELD, PARAMETER, CONSTRUCTOR})
 public @interface External {}