Add config support for suppress warning flag (#1779)

Author: MaciejG604

modules/cli/src/main/scala/scala/cli/commands/bsp/Bsp.scala

@@ -59,7 +59,7 @@ object Bsp extends ScalaCommand[BspOptions] {
                 () => buildOptions0.javaHome().value.javaCommand
               ),
               persistentLogger,
-              sharedOptions.suppress.suppressDirectivesInMultipleFilesWarning
+              buildOptions0.suppressWarningOptions.suppressDirectivesInMultipleFilesWarning
             ).map(_._2).getOrElse(initialInputs)
 
           Build.updateInputs(allInputs, buildOptions(sharedOptions))

modules/cli/src/main/scala/scala/cli/commands/dependencyupdate/DependencyUpdate.scala

@@ -36,7 +36,7 @@ object DependencyUpdate extends ScalaCommand[DependencyUpdateOptions] {
           () => buildOptions.javaHome().value.javaCommand
         ),
         logger,
-        options.shared.suppress.suppressDirectivesInMultipleFilesWarning
+        buildOptions.suppressWarningOptions.suppressDirectivesInMultipleFilesWarning
       ).orExit(logger)
 
     val scopedSources = crossSources.scopedSources(buildOptions).orExit(logger)

modules/cli/src/main/scala/scala/cli/commands/shared/SharedOptions.scala

@@ -273,7 +273,7 @@ final case class SharedOptions(
     }
     bo.BuildOptions(
       suppressWarningOptions =
-        bo.SuppressWarningOptions(suppress.suppressDirectivesInMultipleFilesWarning),
+        bo.SuppressWarningOptions(suppressDirectivesInMultipleFilesWarning),
       scalaOptions = bo.ScalaOptions(
         scalaVersion = scalaVersion
           .map(_.trim)
@@ -420,6 +420,18 @@ final case class SharedOptions(
     }
   }
 
+  def suppressDirectivesInMultipleFilesWarning: Option[Boolean] =
+    suppress.suppressDirectivesInMultipleFilesWarning.filter(identity).orElse(
+      configDb.map(_.get(Keys.suppressDirectivesInMultipleFilesWarning))
+        .map {
+          case Right(opt) => opt
+          case Left(ex) =>
+            logger.debug(ConfigDbException(ex))
+            None
+        }
+        .getOrElse(None)
+    )
+
   def configDb: Either[BuildException, ConfigDb] =
     ConfigDb.open(directories.directories.dbPath.toNIO)
       .wrapConfigException

modules/config/src/main/scala/scala/cli/config/Keys.scala

@@ -21,6 +21,9 @@ object Keys {
 
   val interactive = new Key.BooleanEntry(Seq.empty, "interactive")
 
+  val suppressDirectivesInMultipleFilesWarning =
+    new Key.BooleanEntry(Seq("suppress-warning"), "directives-in-multiple-files")
+
   val proxyAddress  = new Key.StringEntry(Seq("httpProxy"), "address")
   val proxyUser     = new Key.PasswordEntry(Seq("httpProxy"), "user")
   val proxyPassword = new Key.PasswordEntry(Seq("httpProxy"), "password")
@@ -40,6 +43,7 @@ object Keys {
     ghToken,
     globalInteractiveWasSuggested,
     interactive,
+    suppressDirectivesInMultipleFilesWarning,
     pgpPublicKey,
     pgpSecretKey,
     pgpSecretKeyPassword,

modules/integration/src/test/scala/scala/cli/integration/RunTestDefinitions.scala

@@ -733,6 +733,36 @@ abstract class RunTestDefinitions(val scalaVersionOpt: Option[String])
     }
   }
 
+  test("suppress multiple using directives warning message with global config") {
+    val inputs = TestInputs(
+      os.rel / "Foo.scala" ->
+        s"""//> using scala "3.2.0"
+           |
+           |object Foo extends App {
+           |  println("Foo")
+           |}
+           |""".stripMargin,
+      os.rel / "Bar.scala"  -> "",
+      os.rel / "Hello.java" -> "//> using packaging.output \"out\""
+    )
+    inputs.fromRoot { root =>
+      val warningMessage = "Using directives detected in"
+
+      os.proc(TestUtil.cli, "config", "suppress-warning.directives-in-multiple-files", "true")
+        .call(cwd = root)
+
+      val output =
+        os.proc(TestUtil.cli, ".").call(
+          cwd = root,
+          stderr = os.Pipe
+        ).err.trim()
+      expect(!output.contains(warningMessage))
+
+      os.proc(TestUtil.cli, "config", "suppress-warning.directives-in-multiple-files", "false")
+        .call(cwd = root)
+    }
+  }
+
   def sudoTest(): Unit = {
     val fileName = "simple.sc"
     val message  = "Hello"