make trailing comma more robust against in-string replacements

avoid that e.g. "}" inside string literals will be replaced with "},"

Author: aholtkamp

jangaroo-maven/jangaroo-maven-plugin/src/main/java/net/jangaroo/jooc/mvnplugin/WorkspaceConverterMojo.java

@@ -1097,14 +1097,12 @@ private boolean ignoreDependency(Dependency dependency) {
     return "net.jangaroo__jangaroo-browser".contains(String.format("%s__%s", dependency.getGroupId(), dependency.getArtifactId()));
   }
 
-  private String convertJangarooConfig(String jangarooConfig) {
+  public static String convertJangarooConfig(String jangarooConfig) {
     return jangarooConfig
-            .replaceAll("\"([a-zA-Z_$][0-9a-zA-Z_$]*)\":", "$1:")
-            .replace("}", "},")
-            .replace("]", "],")
-            .replace("\"\n", "\",\n")
-            .replace(",,", ",")
-            .replace(",);", ");");
+            .replaceAll("(\n\\s*)\"([a-zA-Z_$][0-9a-zA-Z_$]*)\":", "$1$2:")
+            .replace("}\n", "},\n")
+            .replace("]\n", "],\n")
+            .replace("\"\n", "\",\n");
   }
 
   private boolean isValidVersion(String version) {

jangaroo-maven/jangaroo-maven-plugin/src/test/java/net/jangaroo/jooc/mvnplugin/WorkspaceConverterMojoTest.java

@@ -0,0 +1,9 @@
+package net.jangaroo.jooc.mvnplugin;
+
+import junit.framework.TestCase;
+
+public class WorkspaceConverterMojoTest extends TestCase {
+  public void testConvertJangarooConfig() {
+    assertEquals("{\n  hello: \"{world}\",\n}", WorkspaceConverterMojo.convertJangarooConfig("{\n  \"hello\": \"{world}\"\n}"));
+  }
+}