Don't implement functional interfaces

This isn't allowed in JavaScript.
cashapp · May 15, 2024 · 61aa473 · 61aa473
1 parent a6b742f
commit 61aa473
Showing 1 changed file with 2 additions and 4 deletions.
diff --git a/...ooling-codegen/src/main/kotlin/app/cash/redwood/tooling/codegen/protocolHostGeneration.kt b/...ooling-codegen/src/main/kotlin/app/cash/redwood/tooling/codegen/protocolHostGeneration.kt
@@ -386,12 +386,12 @@ internal fun generateProtocolNode(
                       }
                       if (trait.parameterTypes.isEmpty()) {
                         addStatement(
-                          "%L(json, change.id, eventSink)",
+                          "%L(json, change.id, eventSink)::invoke",
                           trait.eventHandlerName,
                         )
                       } else {
                         addStatement(
-                          "%L(json, change.id, eventSink, %L)",
+                          "%L(json, change.id, eventSink, %L)::invoke",
                           trait.eventHandlerName,
                           arguments.joinToCode(),
                         )
@@ -528,11 +528,9 @@ private fun generateEventHandler(
 ): TypeSpec {
   val constructor = FunSpec.constructorBuilder()
   val invoke = FunSpec.builder("invoke")
-    .addModifiers(OVERRIDE)
 
   val classBuilder = TypeSpec.classBuilder(trait.eventHandlerName)
     .addModifiers(PRIVATE)
-    .addSuperinterface(trait.lambdaType.copy(nullable = false))
 
   addConstructorParameterAndProperty(classBuilder, constructor, "json", KotlinxSerialization.Json)
   addConstructorParameterAndProperty(classBuilder, constructor, "id", Protocol.Id)