Merge pull request #56 from skydoves/plugin/failOnStabilityChange

Implement failOnStabilityChange property for the gradle plugin

Author: skydoves

stability-gradle/api/stability-gradle.api

@@ -23,6 +23,7 @@ public final class com/skydoves/compose/stability/gradle/StabilityAnalyzerGradle
 public abstract class com/skydoves/compose/stability/gradle/StabilityCheckTask : org/gradle/api/DefaultTask {
 	public fun <init> ()V
 	public final fun check ()V
+	public abstract fun getFailOnStabilityChange ()Lorg/gradle/api/provider/Property;
 	public abstract fun getIgnoredClasses ()Lorg/gradle/api/provider/ListProperty;
 	public abstract fun getIgnoredPackages ()Lorg/gradle/api/provider/ListProperty;
 	public abstract fun getProjectName ()Lorg/gradle/api/provider/Property;
@@ -43,6 +44,7 @@ public abstract class com/skydoves/compose/stability/gradle/StabilityDumpTask :
 public abstract class com/skydoves/compose/stability/gradle/StabilityValidationConfig {
 	public fun <init> (Lorg/gradle/api/file/ProjectLayout;Lorg/gradle/api/model/ObjectFactory;)V
 	public final fun getEnabled ()Lorg/gradle/api/provider/Property;
+	public final fun getFailOnStabilityChange ()Lorg/gradle/api/provider/Property;
 	public final fun getIgnoredClasses ()Lorg/gradle/api/provider/ListProperty;
 	public final fun getIgnoredPackages ()Lorg/gradle/api/provider/ListProperty;
 	public final fun getIgnoredProjects ()Lorg/gradle/api/provider/ListProperty;

stability-gradle/src/main/kotlin/com/skydoves/compose/stability/gradle/StabilityAnalyzerExtension.kt

@@ -132,4 +132,12 @@ public abstract class StabilityValidationConfig @Inject constructor(
    */
   public val ignoredClasses: ListProperty<String> =
     objects.listProperty(String::class.java).convention(emptyList())
+
+  /**
+   * Whether to fail the build when stability changes are detected.
+   * When false, stability changes will be logged as warnings instead.
+   * Default: true
+   */
+  public val failOnStabilityChange: Property<Boolean> =
+    objects.property(Boolean::class.javaObjectType).convention(true)
 }

stability-gradle/src/main/kotlin/com/skydoves/compose/stability/gradle/StabilityAnalyzerGradlePlugin.kt

@@ -89,6 +89,7 @@ public class StabilityAnalyzerGradlePlugin : KotlinCompilerPluginSupportPlugin {
       stabilityDir.set(extension.stabilityValidation.outputDir)
       ignoredPackages.set(extension.stabilityValidation.ignoredPackages)
       ignoredClasses.set(extension.stabilityValidation.ignoredClasses)
+      failOnStabilityChange.set(extension.stabilityValidation.failOnStabilityChange)
     }
 
     // Make check task depend on stabilityCheck if enabled (only if check task exists)

stability-gradle/src/main/kotlin/com/skydoves/compose/stability/gradle/StabilityCheckTask.kt

@@ -62,6 +62,12 @@ public abstract class StabilityCheckTask : DefaultTask() {
   @get:Input
   public abstract val projectName: Property<String>
 
+  /**
+   * Whether to fail the build when stability changes are detected.
+   */
+  @get:Input
+  public abstract val failOnStabilityChange: Property<Boolean>
+
   init {
     group = "verification"
     description = "Check composable stability against reference file"
@@ -110,8 +116,6 @@ public abstract class StabilityCheckTask : DefaultTask() {
 
     if (differences.isNotEmpty()) {
       val message = buildString {
-        appendLine("❌ Stability check failed!")
-        appendLine()
         appendLine("The following composables have changed stability:")
         appendLine()
         differences.forEach { diff ->
@@ -123,10 +127,16 @@ public abstract class StabilityCheckTask : DefaultTask() {
             "to update the stability file.",
         )
       }
-      throw GradleException(message)
-    }
 
-    logger.lifecycle("✅ Stability check passed.")
+      if (failOnStabilityChange.get()) {
+        throw GradleException("❌ Stability check failed!\n\n$message")
+      } else {
+        logger.warn("⚠️  Stability changes detected:\n\n$message")
+        logger.lifecycle("✓ Stability check completed with warnings (failOnStabilityChange=false)")
+      }
+    } else {
+      logger.lifecycle("✅ Stability check passed.")
+    }
   }
 
   private fun parseStabilityFromCompiler(