Remove SpotlessConventionPlugin

Author: gpunto

benchmark/build.gradle.kts

@@ -13,12 +13,10 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
-import io.getstream.video.configureFlavors
-
 plugins {
     alias(libs.plugins.stream.android.test)
     alias(libs.plugins.baseline.profile)
-    id("io.getstream.spotless")
+    id("io.getstream.video.android.demoflavor")
 }
 
 android {
@@ -51,17 +49,6 @@ android {
         }
     }
 
-    // Use the same flavor dimensions as the application to allow generating Baseline Profiles on prod,
-    // which is more close to what will be shipped to users (no fake data), but has ability to run the
-    // benchmarks on demo, so we benchmark on stable data.
-    configureFlavors(this) { flavor ->
-        buildConfigField(
-            "String",
-            "APP_FLAVOR_SUFFIX",
-            "\"${flavor.applicationIdSuffix ?: ""}\""
-        )
-    }
-
     targetProjectPath = ":demo-app"
 
     testOptions.managedDevices.devices {

build-logic/convention/build.gradle.kts

@@ -34,9 +34,9 @@ gradlePlugin {
       id = "io.getstream.video.android.library"
       implementationClass = "AndroidLibraryConventionPlugin"
     }
-    register("spotless") {
-      id = "io.getstream.spotless"
-      implementationClass = "SpotlessConventionPlugin"
+    register("demoFlavorConvention") {
+      id = "io.getstream.video.android.demoflavor"
+      implementationClass = "DemoFlavorConventionPlugin"
     }
   }
 }

build-logic/convention/src/main/kotlin/DemoFlavorConventionPlugin.kt

@@ -0,0 +1,19 @@
+import com.android.build.api.dsl.ApplicationExtension
+import com.android.build.api.dsl.TestExtension
+import io.getstream.video.configureFlavors
+import org.gradle.api.Plugin
+import org.gradle.api.Project
+import org.gradle.kotlin.dsl.configure
+
+class DemoFlavorConventionPlugin : Plugin<Project> {
+    override fun apply(target: Project) {
+        with(target) {
+            pluginManager.withPlugin("com.android.application") {
+                extensions.configure<ApplicationExtension>(::configureFlavors)
+            }
+            pluginManager.withPlugin("com.android.test") {
+                extensions.configure<TestExtension>(::configureFlavors)
+            }
+        }
+    }
+}

build-logic/convention/src/main/kotlin/SpotlessConventionPlugin.kt

@@ -20,25 +20,29 @@ enum class VideoDemoFlavor(val dimension: FlavorDimension, val applicationIdSuff
     production(FlavorDimension.contentType),
 }
 
-fun configureFlavors(
+internal fun configureFlavors(
     commonExtension: CommonExtension<*, *, *, *, *, *>,
-    flavorConfigurationBlock: ProductFlavor.(flavor: VideoDemoFlavor) -> Unit = {}
 ) {
     commonExtension.apply {
         flavorDimensions += "environment"
         productFlavors {
-            VideoDemoFlavor.values().forEach {
-                create(it.name) {
+            VideoDemoFlavor.values().forEach { flavor ->
+                create(flavor.name) {
                     dimension = "environment"
-                    flavorConfigurationBlock(this, it)
+                    buildConfigField(
+                        "String",
+                        "APP_FLAVOR_SUFFIX",
+                        "\"${flavor.applicationIdSuffix.orEmpty()}\"",
+                    )
+
                     if (this@apply is ApplicationExtension && this is ApplicationProductFlavor) {
-                        if (it.applicationIdSuffix != null) {
-                            applicationIdSuffix = it.applicationIdSuffix
+                        if (flavor.applicationIdSuffix != null) {
+                            applicationIdSuffix = flavor.applicationIdSuffix
                         }
                     }
                     proguardFiles("benchmark-rules.pro")
                 }
             }
         }
     }
-}
+}

build-logic/convention/src/main/kotlin/io/getstream/video/DemoFlavor.kt

@@ -21,12 +21,12 @@ import com.github.triplet.gradle.androidpublisher.ResolutionStrategy
 import io.getstream.video.FlavorDimension
 import io.getstream.video.VideoDemoFlavor
 import io.getstream.video.android.Configuration
-import io.getstream.video.configureFlavors
 import java.io.FileInputStream
 import java.util.Properties
 
 plugins {
     id("io.getstream.video.android.application.compose")
+    id("io.getstream.video.android.demoflavor")
     id("com.google.gms.google-services")
     id(libs.plugins.firebase.crashlytics.get().pluginId)
     id(libs.plugins.kotlin.serialization.get().pluginId)
@@ -91,8 +91,6 @@ android {
         }
     }
 
-    configureFlavors(this)
-
     buildTypes {
         getByName("debug") {
             versionNameSuffix = "-DEBUG"

demo-app/build.gradle.kts

@@ -10,7 +10,7 @@ cameraCamera2 = "1.3.4"
 kover = "0.8.3"
 sonarqube = "6.0.1.5171"
 spotless = "6.21.0"
-streamConventions = "local-test"
+streamConventions = "0.3.0"
 mavenPublish = "0.34.0"
 kotlin = "1.9.25"
 ksp = "1.9.25-1.0.20"