Kotlin reflections support

Author: slinkydeveloper

bytebuddy-proxy-support/src/main/java/dev/restate/bytebuddy/proxysupport/ByteBuddyProxyFactory.java

@@ -51,7 +51,14 @@ public final class ByteBuddyProxyFactory implements ProxyFactory {
   public <T> @Nullable T createProxy(Class<T> clazz, MethodInterceptor interceptor) {
     // Cannot proxy final classes
     if (Modifier.isFinal(clazz.getModifiers())) {
-      throw new IllegalArgumentException("Class " + clazz + " is final, cannot be proxied.");
+      if (ReflectionUtils.isKotlinClass(clazz)) {
+        throw new IllegalArgumentException(
+            clazz
+                + " is not open, cannot be proxied. Make it open to be able to proxy it, or refactor it extracting the restate interface out of it.");
+      }
+      throw new IllegalArgumentException(
+          clazz
+              + " is final, cannot be proxied. Remove the final keyword, or refactor it extracting the restate interface out of it.");
     }
 
     try {
@@ -87,7 +94,7 @@ public Method getMethod() {
 
       return proxyInstance;
     } catch (Exception e) {
-      throw new IllegalArgumentException("Cannot create proxy for class " + clazz, e);
+      throw new IllegalArgumentException("Cannot create proxy for " + clazz, e);
     }
   }
 
@@ -99,10 +106,11 @@ private <T> Class<?> generateProxyClass(Class<T> clazz) throws NoSuchFieldExcept
           ReflectionUtils.getUniqueDeclaredMethods(
               clazz,
               method ->
-                  ReflectionUtils.findAnnotation(method, Handler.class) != null
-                      || ReflectionUtils.findAnnotation(method, Shared.class) != null
-                      || ReflectionUtils.findAnnotation(method, Workflow.class) != null
-                      || ReflectionUtils.findAnnotation(method, Exclusive.class) != null);
+                  !Modifier.isStatic(method.getModifiers())
+                      && (ReflectionUtils.findAnnotation(method, Handler.class) != null
+                          || ReflectionUtils.findAnnotation(method, Shared.class) != null
+                          || ReflectionUtils.findAnnotation(method, Workflow.class) != null
+                          || ReflectionUtils.findAnnotation(method, Exclusive.class) != null));
       for (var method : methods) {
         validateMethod(method);
       }
@@ -114,10 +122,12 @@ private <T> Class<?> generateProxyClass(Class<T> clazz) throws NoSuchFieldExcept
             : byteBuddy.subclass(clazz);
 
     var annotationMatcher =
-        isAnnotatedWith(Handler.class)
-            .or(isAnnotatedWith(Exclusive.class))
-            .or(isAnnotatedWith(Shared.class))
-            .or(isAnnotatedWith(Workflow.class));
+        not(isStatic())
+            .and(
+                isAnnotatedWith(Handler.class)
+                    .or(isAnnotatedWith(Exclusive.class))
+                    .or(isAnnotatedWith(Shared.class))
+                    .or(isAnnotatedWith(Workflow.class)));
     try (var unloaded =
         builder
             // Add a field to store the interceptor

client-kotlin/build.gradle.kts

@@ -9,5 +9,7 @@ dependencies {
   api(project(":client")) { exclude("dev.restate", "sdk-serde-jackson") }
   api(project(":sdk-serde-kotlinx"))
 
+  implementation(project(":common-kotlin"))
   implementation(libs.kotlinx.coroutines.core)
+  implementation(kotlin("reflect"))
 }