Fixed the issue that is throwing StringIndexOutOfBoundsException when duplicated replacements are used to edit the file. In other words, trying to replace single dependency more than once which is not valid.

Author: Seetaramayya

modules/core/src/main/scala/org/scalasteward/core/edit/EditAlg.scala

@@ -106,7 +106,7 @@ final class EditAlg[F[_]](implicit
       repoDir <- workspaceAlg.repoDir(data.repo)
       replacementsByPath = updateReplacements.groupBy(_.position.path).toList
       _ <- replacementsByPath.traverse { case (path, replacements) =>
-        fileAlg.editFile(repoDir / path, Substring.Replacement.applyAll(replacements))
+        fileAlg.editFile(repoDir / path, Substring.Replacement.applyAll(replacements.toSet))
       }
       _ <- reformatChangedFiles(data)
       msgTemplate = data.config.commits.messageOrDefault

modules/core/src/main/scala/org/scalasteward/core/edit/update/data/Substring.scala

@@ -41,10 +41,10 @@ object Substring {
   final case class Replacement(position: Position, replacement: String)
 
   object Replacement {
-    def applyAll(replacements: List[Replacement])(source: String): String = {
+    def applyAll(replacements: Set[Replacement])(source: String): String = {
       var start = 0
       val sb = new java.lang.StringBuilder(source.length)
-      replacements.sortBy(_.position.start).foreach { r =>
+      replacements.toSeq.sortBy(_.position.start).foreach { r =>
         val before = source.substring(start, r.position.start)
         start = r.position.start + r.position.value.length
         sb.append(before).append(r.replacement)

modules/core/src/test/scala/org/scalasteward/core/edit/RewriteTest.scala

@@ -1,10 +1,11 @@
 package org.scalasteward.core.edit
 
+import cats.data.NonEmptyList
 import cats.effect.unsafe.implicits.global
 import munit.FunSuite
 import org.scalasteward.core.TestInstances.dummyRepoCache
 import org.scalasteward.core.TestSyntax._
-import org.scalasteward.core.data.{Repo, RepoData, Update}
+import org.scalasteward.core.data.{CrossDependency, GroupId, Repo, RepoData, Update, Version}
 import org.scalasteward.core.mock.MockContext.context._
 import org.scalasteward.core.mock.MockState
 import org.scalasteward.core.repoconfig.RepoConfig
@@ -931,6 +932,37 @@ class RewriteTest extends FunSuite {
     runApplyUpdate(update, original, expected)
   }
 
+  test("duplicate updates should be successful") {
+    val dependency = "com.pauldijou".g % "jwt-play-json".a % "5.0.0"
+
+    val artifactId = Update.ForArtifactId(
+      CrossDependency(dependency),
+      newerVersions = NonEmptyList.of(Version("9.2.0")),
+      newerGroupId = Some(GroupId("com.github.jwt-scala")),
+      newerArtifactId = Some("jwt-play-json")
+    )
+    val duplicatedUpdates = Update.ForGroupId(NonEmptyList.of(artifactId, artifactId))
+    val buildSbtContent =
+      """
+        | lazy val root = (project in file("."))
+        |  .settings(
+        |    scalafmtOnCompile := true,
+        |    scalaVersion := scala213,
+        |    libraryDependencies ++= Seq(
+        |      "com.pauldijou"                %% "jwt-play-json"           % "5.0.0", // JWT parsing
+        |      "org.scalatestplus"            %% "mockito-3-4"             % "3.2.10.0" % Test
+        |    ),
+        |    crossScalaVersions := supportedScalaVersions
+        |  )
+        |""".stripMargin
+    val original = Map("build.sbt" -> buildSbtContent)
+    val expectedSbtContent = buildSbtContent
+      .replaceAll("com.pauldijou", "com.github.jwt-scala")
+      .replaceAll("5.0.0", "9.2.0")
+    val expected = Map("build.sbt" -> expectedSbtContent)
+    runApplyUpdate(duplicatedUpdates, original, expected)
+  }
+
   private def runApplyUpdate(
       update: Update.Single,
       files: Map[String, String],