fix: bundleTraces is a sortedSet and BundleTrace is Comparable.

Author: autonomousapps

src/main/kotlin/com/autonomousapps/model/internal/intermediates/BundleTrace.kt

@@ -11,7 +11,48 @@ import dev.zacsweers.moshix.sealed.annotations.TypeLabel
 internal sealed class BundleTrace(
   val top: Coordinates,
   val bottom: Coordinates
-) {
+) : Comparable<BundleTrace> {
+
+  override fun compareTo(other: BundleTrace): Int {
+    return when (this) {
+      is DeclaredParent -> {
+        when (other) {
+          is DeclaredParent -> {
+            compareBy(DeclaredParent::parent)
+              .thenBy(DeclaredParent::child)
+              .compare(this, other)
+          }
+
+          else -> 1
+        }
+      }
+
+      is UsedChild -> {
+        when (other) {
+          is UsedChild -> {
+            compareBy(UsedChild::parent)
+              .thenBy(UsedChild::child)
+              .compare(this, other)
+          }
+
+          is DeclaredParent -> -1
+          else -> 1
+        }
+      }
+
+      is PrimaryMap -> {
+        when (other) {
+          is PrimaryMap -> {
+            compareBy(PrimaryMap::primary)
+              .thenBy(PrimaryMap::subordinate)
+              .compare(this, other)
+          }
+
+          else -> -1
+        }
+      }
+    }
+  }
 
   @TypeLabel("parent")
   @JsonClass(generateAdapter = false)
@@ -34,4 +75,3 @@ internal sealed class BundleTrace(
     val subordinate: Coordinates
   ) : BundleTrace(primary, subordinate)
 }
-

src/main/kotlin/com/autonomousapps/tasks/ComputeAdviceTask.kt

@@ -282,7 +282,7 @@ internal class DependencyAdviceBuilder(
   val advice: Set<Advice>
 
   /** Dependencies that are removed from [advice] because they match a bundle rule. Used by **Reason**. */
-  val bundledTraces = mutableSetOf<BundleTrace>()
+  val bundledTraces: MutableSet<BundleTrace> = sortedSetOf()
 
   init {
     advice = computeDependencyAdvice(projectPath)
@@ -333,29 +333,29 @@ internal class DependencyAdviceBuilder(
 
           advice.isAdd() && bundles.hasParentInBundle(originalCoordinates) -> {
             val parent = bundles.findParentInBundle(originalCoordinates)!!
-            bundledTraces += BundleTrace.DeclaredParent(parent = parent, child = originalCoordinates)
+            bundledTraces.add(BundleTrace.DeclaredParent(parent = parent, child = originalCoordinates))
             null
           }
 
           // Optionally map given advice to "primary" advice, if bundle has a primary
           advice.isAdd() -> {
             val p = bundles.maybePrimary(advice, originalCoordinates)
             if (p != advice) {
-              bundledTraces += BundleTrace.PrimaryMap(primary = p.coordinates, subordinate = originalCoordinates)
+              bundledTraces.add(BundleTrace.PrimaryMap(primary = p.coordinates, subordinate = originalCoordinates))
             }
             p
           }
 
           advice.isRemove() && bundles.hasUsedChild(originalCoordinates) -> {
             val child = bundles.findUsedChild(originalCoordinates)!!
-            bundledTraces += BundleTrace.UsedChild(parent = originalCoordinates, child = child)
+            bundledTraces.add(BundleTrace.UsedChild(parent = originalCoordinates, child = child))
             null
           }
 
           // If the advice has a used child, don't change it
           advice.isAnyChange() && bundles.hasUsedChild(originalCoordinates) -> {
             val child = bundles.findUsedChild(originalCoordinates)!!
-            bundledTraces += BundleTrace.UsedChild(parent = originalCoordinates, child = child)
+            bundledTraces.add(BundleTrace.UsedChild(parent = originalCoordinates, child = child))
             null
           }