fix: 'reason' works for project dependencies.

Author: autonomousapps

src/functionalTest/groovy/com/autonomousapps/jvm/ReasonSpec.groovy

@@ -86,7 +86,7 @@ final class ReasonSpec extends AbstractJvmSpec {
 
     then:
     assertThat(result.output).contains(
-      "> Coordinates 'com.squareup.okio:oki' matches more than 1 dependency [com.squareup.okio:okio-jvm:3.0.0, com.squareup.okio:okio:3.0.0]")
+      "> Coordinates 'com.squareup.okio:oki' matches more than 1 dependency: [com.squareup.okio:okio-jvm:3.0.0, com.squareup.okio:okio:3.0.0]")
 
     where:
     gradleVersion << gradleVersions()

src/main/kotlin/com/autonomousapps/internal/UsageContainer.kt

@@ -0,0 +1,28 @@
+// Copyright (c) 2025. Tony Robalik.
+// SPDX-License-Identifier: Apache-2.0
+package com.autonomousapps.internal
+
+import com.autonomousapps.model.Coordinates
+import com.autonomousapps.model.internal.intermediates.Usage
+
+internal data class UsageContainer(val usages: List<UsagePair>) {
+  fun toMap(): Map<Coordinates, Set<Usage>> {
+    return usages.associate { (coordinates, usages) ->
+      coordinates to usages
+    }
+  }
+
+  companion object {
+    fun of(map: Map<Coordinates, Set<Usage>>): UsageContainer {
+      return UsageContainer(map.entries.map(UsagePair::of))
+    }
+  }
+
+  internal data class UsagePair(val coordinates: Coordinates, val usages: Set<Usage>) {
+    companion object {
+      fun of(entry: Map.Entry<Coordinates, Set<Usage>>): UsagePair {
+        return UsagePair(entry.key, entry.value)
+      }
+    }
+  }
+}

src/main/kotlin/com/autonomousapps/internal/reason/DependencyAdviceExplainer.kt

@@ -21,8 +21,8 @@ import com.autonomousapps.tasks.ReasonTask
 @Suppress("UnstableApiUsage") // guava
 internal class DependencyAdviceExplainer(
   private val project: ProjectCoordinates,
+  private val buildPath: String,
   private val requested: Coordinates,
-  private val target: Coordinates,
   private val requestedCapability: String,
   private val usages: Set<Usage>,
   private val advice: Set<Advice>,
@@ -129,22 +129,22 @@ internal class DependencyAdviceExplainer(
 
   // TODO(tsr): what are the valid scenarios? How many traces could there be for a single target?
   private fun findTrace(): BundleTrace? = bundleTraces.find { trace ->
-    trace.top.gav() == target.gav() || trace.bottom.gav() == target.gav()
+    trace.top.gav() == requested.gav() || trace.bottom.gav() == requested.gav()
   }
 
   private fun StringBuilder.printGraph(graphView: DependencyGraphView) {
     val name = graphView.configurationName
 
     // Find the complete Coordinates (including variant identification) in the graph (if available)
-    val targetInGraph = graphView.graph.nodes().firstOrNull { coordinates ->
-      coordinates.identifier == target.identifier && matchesTargetCapabilities(coordinates)
+    val targetInGraph = graphView.graph.nodes().firstOrNull { node ->
+      node.normalized(buildPath) == requested
     }
 
     if (targetInGraph == null) {
       appendReproducibleNewLine()
       append(Colors.BOLD)
       appendReproducibleNewLine(
-        "There is no path from ${project.printableName()} to ${printableIdentifier(target)} for $name"
+        "There is no path from ${project.printableName()} to ${printableIdentifier(requested)} for $name"
       )
       appendReproducibleNewLine(Colors.NORMAL)
       return
@@ -168,13 +168,6 @@ internal class DependencyAdviceExplainer(
     }
   }
 
-  private fun matchesTargetCapabilities(coordinates: Coordinates): Boolean {
-    // If their GVIs exactly match
-    return coordinates.gradleVariantIdentification == target.gradleVariantIdentification
-      // Or if the target isn't requesting on a capability and the coordinates have only the default capability.
-      || (target.gradleVariantIdentification.capabilities.isEmpty() && coordinates.hasDefaultCapability())
-  }
-
   private fun StringBuilder.printCapabilities(node: Coordinates) {
     val capabilities = node.gradleVariantIdentification.capabilities.filterNot { it == node.identifier }
 

src/main/kotlin/com/autonomousapps/internal/utils/CoordinatesString.kt

@@ -77,7 +77,7 @@ public sealed class Coordinates(
     }.sorted()
 
   internal fun hasDefaultCapability(): Boolean {
-    return gradleVariantIdentification.capabilities.singleOrNull { it == identifier } != null
+    return gradleVariantIdentification.capabilities.isEmpty() || gradleVariantIdentification.capabilities.singleOrNull { it == identifier } != null
   }
 
   /**

src/main/kotlin/com/autonomousapps/model/Coordinates.kt

@@ -1156,7 +1156,6 @@ internal class ProjectPlugin(private val project: Project) {
     }
 
     reasonTask = tasks.register("reason", ReasonTask::class.java) {
-      it.rootProjectName.set(rootProject.name)
       it.projectPath.set(theProjectPath)
       it.buildPath.set(buildPath(buildscript.configurations.named("classpath")))
       it.dependencyMap.set(dagpExtension.dependenciesHandler.map)

src/main/kotlin/com/autonomousapps/subplugin/ProjectPlugin.kt

@@ -4,9 +4,9 @@ package com.autonomousapps.tasks
 
 import com.autonomousapps.extension.DependenciesHandler
 import com.autonomousapps.internal.Bundles
+import com.autonomousapps.internal.UsageContainer
 import com.autonomousapps.internal.transform.StandardTransform
 import com.autonomousapps.internal.utils.*
-import com.autonomousapps.internal.utils.CoordinatesString.Companion.toStringCoordinates
 import com.autonomousapps.model.*
 import com.autonomousapps.model.internal.DependencyGraphView
 import com.autonomousapps.model.internal.declaration.Bucket
@@ -221,9 +221,8 @@ public abstract class ComputeAdviceTask @Inject constructor(
       )
 
       output.bufferWriteJson(projectAdvice)
-      // These must be transformed so that the Coordinates are Strings for serialization
-      dependencyUsagesOut.bufferWriteJsonMap(toStringCoordinates(dependencyUsages, buildPath))
-      annotationProcessorUsagesOut.bufferWriteJsonMap(toStringCoordinates(annotationProcessorUsages, buildPath))
+      dependencyUsagesOut.bufferWriteJson(UsageContainer.of(dependencyUsages))
+      annotationProcessorUsagesOut.bufferWriteJson(UsageContainer.of(annotationProcessorUsages))
       bundleTraces.bufferWriteJsonSet(dependencyAdviceBuilder.bundledTraces)
     }