Support escaping commas in extra flags

save-common/src/commonMain/kotlin/com/saveourtool/save/core/plugin/ExtraFlagsExtractor.kt

@@ -29,7 +29,8 @@ class ExtraFlagsExtractor(private val generalConfig: GeneralConfig,
      * @return [ExtraFlags] or null if no match occurred
      */
     internal fun extractExtraFlagsFrom(line: String) = line
-        .split(",", ", ")
+        .splitByNonEscaped(',')
+        .map { it.replace("\\,", ",") }
         .associate { part ->
             val pair = part.split("=", limit = 2).map {
                 it.replace("\\=", "=")
@@ -95,3 +96,22 @@ fun resolvePlaceholdersFrom(
             }
         }
 }
+
+/**
+ * Split [this] string by [delimiter] unless it's prepended by `\`.
+ */
+internal fun String.splitByNonEscaped(delimiter: Char): List<String> {
+    val indicesToSplit = mapIndexed { index, c -> index to c }
+        .filter { (index, c) ->
+            c == delimiter && (index == 0 || get(index - 1) != '\\')
+        }
+        .map { (index, _) -> index }
+    val result = mutableListOf<String>()
+    var currentOffset = 0
+    indicesToSplit.forEach {
+        result.add(substring(currentOffset, it))
+        currentOffset = it + 1
+    }
+    result.add(substring(currentOffset, length))
+    return result
+}

save-common/src/commonNonJsTest/kotlin/com/saveourtool/save/core/ExtraFlagsExtractorTest.kt

@@ -1,10 +1,6 @@
 package com.saveourtool.save.core
 
-import com.saveourtool.save.core.plugin.ExtraFlags
-import com.saveourtool.save.core.plugin.ExtraFlagsExtractor
-import com.saveourtool.save.core.plugin.GeneralConfig
-import com.saveourtool.save.core.plugin.filterAndJoinBy
-import com.saveourtool.save.core.plugin.resolvePlaceholdersFrom
+import com.saveourtool.save.core.plugin.*
 
 import okio.fakefilesystem.FakeFileSystem
 
@@ -26,6 +22,9 @@ class ExtraFlagsExtractorTest {
             "Unparseable nonsense" to ExtraFlags.empty,
             "args1=--flag --opt,args2=-debug --flag2" to ExtraFlags("--flag --opt", "-debug --flag2"),
             "args1=--flag\\=value,args2=--foo=bar" to ExtraFlags("--flag=value", "--foo=bar"),
+            "args1=option1\\,option2,args2=option3\\,option4" to ExtraFlags("option1,option2", "option3,option4"),
+            "args1=option1\\,option2" to ExtraFlags("option1,option2", ""),
+            "args2=option3\\,option4" to ExtraFlags("", "option3,option4"),
         )
             .forEach { (line, extraFlags) ->
                 assertEquals(extraFlags, extraFlagsExtractor.extractExtraFlagsFrom(line))
@@ -105,6 +104,52 @@ class ExtraFlagsExtractorTest {
                 "another-cmd --flag"
             )
         )
+
+        checkMultilineDirectives(
+            Regex("""// RUN: (.*([^\\]=)?.*)\\?"""),
+            listOf(
+                "// RUN: command --flag=option\\,\\",
+                "// RUN: another-option --another-flag",
+                "// RUN: another-cmd\\=\\",
+                "// RUN: --flag=option\\,another-option",
+            ),
+            listOf(
+                "command --flag=option\\,another-option --another-flag",
+                "another-cmd\\=--flag=option\\,another-option"
+            )
+        )
+    }
+
+    @Test
+    fun `test for splitByNonEscaped`() {
+        assertEquals(
+            listOf("this string\\, not split"),
+            "this string\\, not split".splitByNonEscaped(','),
+        )
+        assertEquals(
+            listOf("this string", " but split"),
+            "this string, but split".splitByNonEscaped(','),
+        )
+        assertEquals(
+            listOf("this string\\, not split", " but here - it's split"),
+            "this string\\, not split, but here - it's split".splitByNonEscaped(','),
+        )
+        assertEquals(
+            listOf("", ""),
+            ",".splitByNonEscaped(','),
+        )
+        assertEquals(
+            listOf("", "text"),
+            ",text".splitByNonEscaped(','),
+        )
+        assertEquals(
+            listOf("\\,"),
+            "\\,".splitByNonEscaped(','),
+        )
+        assertEquals(
+            listOf("\\,text"),
+            "\\,text".splitByNonEscaped(','),
+        )
     }
 
     private fun checkMultilineDirectives(