Includes invalid annotation name in debug message with KSP.

RELNOTES=n/a PiperOrigin-RevId: 508473051
google · Feb 9, 2023 · 47abf56 · 47abf56
1 parent f347c8a
commit 47abf56
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/java/dagger/internal/codegen/xprocessing/XAnnotations.java b/java/dagger/internal/codegen/xprocessing/XAnnotations.java
@@ -101,6 +101,12 @@ public static String toStableString(XAnnotation annotation) {
       if (annotation.getType().isError()) {
         return "@" + annotation.getName(); // SUPPRESS_GET_NAME_CHECK
       }
+      // TODO(b/264089557): Non-annotation elements can be incorrectly treated as annotation in KSP,
+      // therefore calling getAnnotationValues() can cause confusing error.
+      if (getProcessingEnv(annotation).getBackend() == XProcessingEnv.Backend.KSP
+          && annotation.getTypeElement().getConstructors().size() != 1) {
+        return String.format("@%s", getClassName(annotation).canonicalName());
+      }
       return annotation.getAnnotationValues().isEmpty()
           // If the annotation doesn't have values then skip the empty parenthesis.
           ? String.format("@%s", getClassName(annotation).canonicalName())