From ea46fa7eda85508eb530e1016e5a9492074efc9d Mon Sep 17 00:00:00 2001
From: kenji yoshida <6b656e6a69@gmail.com>
Date: Wed, 24 Jan 2024 07:06:56 +0900
Subject: [PATCH] Apply suggestions from code review

Co-authored-by: Brice Jaglin <bjaglin@gmail.com>
---
 .../scalafix/internal/sbt/ScalafixInterface.scala   | 13 -------------
 src/main/scala/scalafix/sbt/ScalafixPlugin.scala    |  9 ++++++---
 2 files changed, 6 insertions(+), 16 deletions(-)

diff --git a/src/main/scala/scalafix/internal/sbt/ScalafixInterface.scala b/src/main/scala/scalafix/internal/sbt/ScalafixInterface.scala
index 2460cf0c..85b0907b 100644
--- a/src/main/scala/scalafix/internal/sbt/ScalafixInterface.scala
+++ b/src/main/scala/scalafix/internal/sbt/ScalafixInterface.scala
@@ -123,19 +123,6 @@ object ScalafixInterface {
 
   private[scalafix] val defaultLogger: Logger = ConsoleLogger(System.out)
 
-  def fromToolClasspath(
-      scalafixBinaryScalaVersion: String,
-      scalafixDependencies: Seq[ModuleID],
-      scalafixCustomResolvers: Seq[Repository],
-      logger: Logger = defaultLogger
-  ): () => ScalafixInterface =
-    fromToolClasspath(
-      scalafixBinaryScalaVersion = scalafixBinaryScalaVersion,
-      scalafixDependencies = scalafixDependencies,
-      scalafixCustomResolvers = scalafixCustomResolvers,
-      logger = logger,
-      callback = new ScalafixLogger(logger)
-    )
 
   def fromToolClasspath(
       scalafixBinaryScalaVersion: String,
diff --git a/src/main/scala/scalafix/sbt/ScalafixPlugin.scala b/src/main/scala/scalafix/sbt/ScalafixPlugin.scala
index 0241a02e..cdd518b0 100644
--- a/src/main/scala/scalafix/sbt/ScalafixPlugin.scala
+++ b/src/main/scala/scalafix/sbt/ScalafixPlugin.scala
@@ -77,7 +77,10 @@ object ScalafixPlugin extends AutoPlugin {
       )
 
     val scalafixCallback: SettingKey[ScalafixMainCallback] =
-      settingKey[ScalafixMainCallback]("")
+      settingKey[ScalafixMainCallback](
+        "The handler for the diagnostics emitted during scalafix execution. Can be set in ThisBuild or at project-level. " +
+          "Defaults to a wrapper around `sbt.Logger`."
+      )
 
     val scalafixSemanticdb: ModuleID =
       scalafixSemanticdb(BuildInfo.scalametaVersion)
@@ -162,13 +165,12 @@ object ScalafixPlugin extends AutoPlugin {
         SettingKey[Boolean]("bspEnabled") := false
       )
     ),
-    scalafixCallback := new ScalafixLogger(ScalafixInterface.defaultLogger),
     scalafixInterfaceProvider := ScalafixInterface.fromToolClasspath(
       scalafixScalaBinaryVersion.value,
       scalafixDependencies = scalafixDependencies.value,
       scalafixCustomResolvers = scalafixResolvers.value,
       logger = ScalafixInterface.defaultLogger,
-      callback = scalafixCallback.value
+      callback = (ThisBuild / scalafixCallback).value
     ),
     update := {
       object SemanticdbScalac {
@@ -206,6 +208,7 @@ object ScalafixPlugin extends AutoPlugin {
   )
 
   override lazy val globalSettings: Seq[Def.Setting[_]] = Seq(
+    scalafixCallback := new ScalafixLogger(ScalafixInterface.defaultLogger),
     scalafixConfig := None, // let scalafix-cli try to infer $CWD/.scalafix.conf
     scalafixOnCompile := false,
     scalafixCaching := true,