Using directive with URL + query parameters (VirtusLab#3720) (VirtusLab#3835)

Author: jgoday

modules/build/src/main/scala/scala/build/CrossSources.scala

@@ -382,6 +382,16 @@ object CrossSources {
     crossSources -> finalInputs
   }
 
+  extension (uri: java.net.URI)
+    def asString: String =
+      java.net.URI(
+        uri.getScheme,
+        uri.getAuthority,
+        uri.getPath,
+        null,
+        uri.getFragment
+      ).toString
+
   /** @return
     *   the resource directories that should be added to the classpath
     */
@@ -405,7 +415,7 @@ object CrossSources {
     download(pUri.value.toString).left.map(
       new UsingFileFromUriError(pUri.value, pUri.positions, _)
     ).map(content =>
-      Seq(Virtual(pUri.value.toString, content))
+      Seq(Virtual(pUri.value.asString, content))
     )
 
   type CodeFile = os.Path | java.net.URI

modules/build/src/test/scala/scala/build/tests/SourcesTests.scala

@@ -125,6 +125,39 @@ class SourcesTests extends TestUtil.ScalaCliBuildSuite {
     }
   }
 
+  test("dependencies in .scala - using URL with query parameters") {
+    val testInputs = TestInputs(
+      os.rel / "something.scala" ->
+        """| //> using file http://github.com/VirtusLab/scala-cli/blob/main/modules/dummy/amm/src/main/scala/AmmDummy.scala?version=3
+           |
+           |object Main {
+           |}
+           |""".stripMargin
+    )
+    testInputs.withInputs { (root, inputs) =>
+      val (crossSources, _) =
+        CrossSources.forInputs(
+          inputs,
+          preprocessors,
+          TestLogger(),
+          SuppressWarningOptions(),
+          download = _ => Right(Array.empty[Byte])
+        ).orThrow
+      val scopedSources = crossSources.scopedSources(BuildOptions()).orThrow
+      val sources       =
+        scopedSources.sources(
+          Scope.Main,
+          crossSources.sharedOptions(BuildOptions()),
+          root,
+          TestLogger()
+        ).orThrow
+
+      expect(sources.paths.length == 1)
+      expect(sources.inMemory.length == 1)
+      expect(sources.inMemory(0).generatedRelPath.last == "AmmDummy.scala")
+    }
+  }
+
   test("dependencies in .test.scala - using") {
     val testInputs = TestInputs(
       os.rel / "something.test.scala" ->