Disable service-loading ApolloCompilerPlugin

libraries/apollo-compiler/src/main/kotlin/com/apollographql/apollo/compiler/ApolloCompilerPlugin.kt

@@ -11,13 +11,6 @@ import com.apollographql.apollo.compiler.ir.IrOperations
 import com.apollographql.apollo.compiler.operationoutput.OperationDescriptor
 import com.apollographql.apollo.compiler.operationoutput.OperationId
 
-@Deprecated(
-    "Use ApolloCompilerPlugin instead (don't forget to replace the META-INF.service file to `com.apollographql.apollo.compiler.ApolloCompilerPlugin`",
-    ReplaceWith("ApolloCompilerPlugin"),
-    DeprecationLevel.ERROR
-)
-typealias Plugin = ApolloCompilerPlugin
-
 /**
  * [ApolloCompilerPlugin] allows to customize the behaviour of the Apollo Compiler.
  *

libraries/apollo-gradle-plugin-external/src/main/kotlin/com/apollographql/apollo/gradle/internal/serviceloader.kt

@@ -19,7 +19,7 @@ internal fun apolloCompilerPlugin(
 
   val plugin = plugins.singleOrNull()
   if (plugin != null) {
-    return plugin
+    error("Apollo: use ApolloCompilerPluginProvider instead of ApolloCompilerPlugin directly. ApolloCompilerPluginProvider allows arguments and logging")
   }
 
   val pluginProviders = ServiceLoader.load(ApolloCompilerPluginProvider::class.java).toList()

libraries/apollo-gradle-plugin/testProjects/operationIdsWithIsolation/apollo-compiler-plugin/src/main/kotlin/apollo/plugin/MyPlugin.kt

@@ -1,12 +1,23 @@
 package apollo.plugin
 
+import com.apollographql.apollo.annotations.ApolloExperimental
 import com.apollographql.apollo.compiler.OperationOutputGenerator
 import com.apollographql.apollo.compiler.ApolloCompilerPlugin
+import com.apollographql.apollo.compiler.ApolloCompilerPluginEnvironment
+import com.apollographql.apollo.compiler.ApolloCompilerPluginProvider
 import com.apollographql.apollo.compiler.operationoutput.OperationDescriptor
 import com.apollographql.apollo.compiler.operationoutput.OperationId
 
 class MyPlugin: ApolloCompilerPlugin {
     override fun operationIds(descriptors: List<OperationDescriptor>): List<OperationId>? {
         return descriptors.map { OperationId("${it.name}CustomId", it.name) }
     }
+}
+
+@OptIn(ApolloExperimental::class)
+class MyPluginProvider: ApolloCompilerPluginProvider {
+    override fun create(environment: ApolloCompilerPluginEnvironment): ApolloCompilerPlugin {
+        return MyPlugin()
+    }
+
 }

libraries/apollo-gradle-plugin/testProjects/operationIdsWithIsolation/apollo-compiler-plugin/src/main/resources/META-INF/services/com.apollographql.apollo.compiler.ApolloCompilerPluginProvider

@@ -0,0 +1 @@
+apollo.plugin.MyPluginProvider

tests/compiler-plugins/add-field/src/main/kotlin/hooks/TestPlugin.kt

@@ -13,6 +13,14 @@ import com.apollographql.apollo.ast.responseName
 import com.apollographql.apollo.ast.rootTypeDefinition
 import com.apollographql.apollo.compiler.DocumentTransform
 import com.apollographql.apollo.compiler.ApolloCompilerPlugin
+import com.apollographql.apollo.compiler.ApolloCompilerPluginEnvironment
+import com.apollographql.apollo.compiler.ApolloCompilerPluginProvider
+
+class TestPluginProvider: ApolloCompilerPluginProvider {
+  override fun create(environment: ApolloCompilerPluginEnvironment): ApolloCompilerPlugin {
+    return TestPlugin()
+  }
+}
 
 class TestPlugin : ApolloCompilerPlugin {
   override fun documentTransform(): DocumentTransform {

tests/compiler-plugins/add-field/src/main/resources/META-INF/services/com.apollographql.apollo.compiler.ApolloCompilerPluginProvider

@@ -0,0 +1 @@
+hooks.TestPluginProvider

tests/compiler-plugins/capitalize-enum-values/src/main/kotlin/hooks/TestPlugin.kt

@@ -1,11 +1,19 @@
 package hooks
 
 import com.apollographql.apollo.compiler.ApolloCompilerPlugin
+import com.apollographql.apollo.compiler.ApolloCompilerPluginEnvironment
+import com.apollographql.apollo.compiler.ApolloCompilerPluginProvider
 import com.apollographql.apollo.compiler.Transform
 import com.apollographql.apollo.compiler.codegen.kotlin.KotlinOutput
 import com.squareup.kotlinpoet.FunSpec
 import com.squareup.kotlinpoet.TypeSpec
 
+class TestPluginProvider: ApolloCompilerPluginProvider {
+  override fun create(environment: ApolloCompilerPluginEnvironment): ApolloCompilerPlugin {
+    return TestPlugin()
+  }
+}
+
 class TestPlugin : ApolloCompilerPlugin {
   override fun kotlinOutputTransform(): Transform<KotlinOutput> {
     return object : Transform<KotlinOutput> {

tests/compiler-plugins/capitalize-enum-values/src/main/resources/META-INF/services/com.apollographql.apollo.compiler.ApolloCompilerPluginProvider

@@ -0,0 +1 @@
+hooks.TestPluginProvider

tests/compiler-plugins/custom-flatten/src/main/kotlin/hooks/TestPlugin.kt

@@ -1,9 +1,17 @@
 package hooks
 
 import com.apollographql.apollo.compiler.ApolloCompilerPlugin
+import com.apollographql.apollo.compiler.ApolloCompilerPluginEnvironment
+import com.apollographql.apollo.compiler.ApolloCompilerPluginProvider
 import com.apollographql.apollo.compiler.Transform
 import com.apollographql.apollo.compiler.ir.IrOperations
 
+class TestPluginProvider: ApolloCompilerPluginProvider {
+  override fun create(environment: ApolloCompilerPluginEnvironment): ApolloCompilerPlugin {
+    return TestPlugin()
+  }
+}
+
 class TestPlugin : ApolloCompilerPlugin {
   override fun irOperationsTransform(): Transform<IrOperations> {
     return object : Transform<IrOperations> {

tests/compiler-plugins/custom-flatten/src/main/resources/META-INF/services/com.apollographql.apollo.compiler.ApolloCompilerPluginProvider

@@ -0,0 +1 @@
+hooks.TestPluginProvider

tests/compiler-plugins/default-null-values/src/main/kotlin/hooks/TestPlugin.kt

@@ -1,12 +1,20 @@
 package hooks
 
 import com.apollographql.apollo.compiler.ApolloCompilerPlugin
+import com.apollographql.apollo.compiler.ApolloCompilerPluginEnvironment
+import com.apollographql.apollo.compiler.ApolloCompilerPluginProvider
 import com.apollographql.apollo.compiler.Transform
 import com.apollographql.apollo.compiler.codegen.kotlin.KotlinOutput
 import com.squareup.kotlinpoet.CodeBlock
 import com.squareup.kotlinpoet.KModifier
 import com.squareup.kotlinpoet.TypeSpec
 
+class TestPluginProvider: ApolloCompilerPluginProvider {
+  override fun create(environment: ApolloCompilerPluginEnvironment): ApolloCompilerPlugin {
+    return TestPlugin()
+  }
+}
+
 class TestPlugin: ApolloCompilerPlugin {
   override fun kotlinOutputTransform(): Transform<KotlinOutput> {
     return object : Transform<KotlinOutput> {

tests/compiler-plugins/default-null-values/src/main/resources/META-INF/services/com.apollographql.apollo.compiler.ApolloCompilerPluginProvider

@@ -0,0 +1 @@
+hooks.TestPluginProvider

tests/compiler-plugins/getters-and-setters/src/main/kotlin/hooks/TestPlugin.kt

@@ -1,13 +1,20 @@
 package hooks
 
 import com.apollographql.apollo.compiler.ApolloCompilerPlugin
+import com.apollographql.apollo.compiler.ApolloCompilerPluginEnvironment
+import com.apollographql.apollo.compiler.ApolloCompilerPluginProvider
 import com.apollographql.apollo.compiler.Transform
 import com.apollographql.apollo.compiler.capitalizeFirstLetter
 import com.apollographql.apollo.compiler.codegen.java.JavaOutput
 import com.squareup.javapoet.JavaFile
 import com.squareup.javapoet.MethodSpec
 import javax.lang.model.element.Modifier
 
+class TestPluginProvider: ApolloCompilerPluginProvider {
+  override fun create(environment: ApolloCompilerPluginEnvironment): ApolloCompilerPlugin {
+    return TestPlugin()
+  }
+}
 class TestPlugin : ApolloCompilerPlugin {
   override fun javaOutputTransform(): Transform<JavaOutput> {
     return object : Transform<JavaOutput> {

tests/compiler-plugins/getters-and-setters/src/main/resources/META-INF/services/com.apollographql.apollo.compiler.ApolloCompilerPluginProvider

@@ -0,0 +1 @@
+hooks.TestPluginProvider

tests/compiler-plugins/typename-interface/src/main/kotlin/hooks/TestPlugin.kt

@@ -1,12 +1,20 @@
 package hooks
 
 import com.apollographql.apollo.compiler.ApolloCompilerPlugin
+import com.apollographql.apollo.compiler.ApolloCompilerPluginEnvironment
+import com.apollographql.apollo.compiler.ApolloCompilerPluginProvider
 import com.apollographql.apollo.compiler.Transform
 import com.apollographql.apollo.compiler.codegen.kotlin.KotlinOutput
 import com.squareup.kotlinpoet.ClassName
 import com.squareup.kotlinpoet.KModifier
 import com.squareup.kotlinpoet.TypeSpec
 
+class TestPluginProvider: ApolloCompilerPluginProvider {
+  override fun create(environment: ApolloCompilerPluginEnvironment): ApolloCompilerPlugin {
+    return TestPlugin()
+  }
+}
+
 class TestPlugin : ApolloCompilerPlugin {
   private val interfaceName = "hooks.typenameinterface.HasTypeName"
 

tests/compiler-plugins/typename-interface/src/main/resources/META-INF/services/com.apollographql.apollo.compiler.ApolloCompilerPluginProvider

@@ -0,0 +1 @@
+hooks.TestPluginProvider