Merge pull request #2 from coderphonui/1-helpcant-repair-json-missing-closed-quote-in-value

Fix missing end quote of value

Author: coderphonui

src/main/java/com/cdpn/jsonautorepair/internal/EscapeProcessor.java

@@ -3,6 +3,8 @@
 public class EscapeProcessor {
     private static final char EOF = '\0';
     private static final char TAB_CHAR = '\t';
+    private static final char COMMA = ',';
+    private static final char CLOSED_BRACKET = '}';
     private static final char BREAK_LINE_CHAR = '\n';
     private static final char SPACE_CHAR = ' ';
     private static final char DOUBLE_QUOTE_CHAR = '\"';
@@ -23,7 +25,7 @@ public String process() {
         for (int i = 0; i < inputString.length(); i++) {
             char currentChar = inputString.charAt(i);
             if (currentChar != DOUBLE_QUOTE_CHAR) {
-                handleNonQuoteCharacter(currentChar);
+                handleNonQuoteCharacter(currentChar, i);
             }
             else {
                 handleQuoteCharacter(currentChar, i);
@@ -72,7 +74,23 @@ private boolean hasNextQuoteRightAfterCurrentQuoteWithoutComma(int position) {
         return findNextNonSpaceChar(position + 1) == DOUBLE_QUOTE_CHAR;
     }
 
-    private void handleNonQuoteCharacter(char currentChar) {
+    private void handleNonQuoteCharacter(char currentChar, int position) {
+        if (currentChar == COMMA) {
+            char nextNonSpaceChar = findNextNonSpaceChar(position + 1);
+            if (inQuotes) {
+                if (nextNonSpaceChar == DOUBLE_QUOTE_CHAR ) {
+                    escapedJson.append(DOUBLE_QUOTE_CHAR);
+                    inQuotes = false;
+                }
+                if (nextNonSpaceChar == CLOSED_BRACKET ) {
+                    escapedJson.append(DOUBLE_QUOTE_CHAR);
+                    inQuotes = false;
+                    return;
+                }
+            }
+            escapedJson.append(currentChar);
+            return;
+        }
         if (!inQuotes) {
             escapedJson.append(currentChar);
             return;

src/test/java/com/cdpn/jsonautorepair/JSONAutoRepairerTest.java

@@ -37,6 +37,33 @@ public void repair_should_cleanup_markdown_code_block_when_string_contain_json_c
 
     }
 
+    @Test
+    public void repair_should_fix_missing_end_quote_of_value() {
+        String originalJSON = """
+                {
+                    "name": "Alice,
+                    "age": 30
+                }
+                """;
+        assertEquals(JsonParser.parseString("""
+                {
+                    "name": "Alice",
+                    "age": 30
+                }
+                """).toString(), jsonAutoRepairer.repair(originalJSON));
+
+        originalJSON = """
+                {
+                    "name": "Alice,
+                }
+                """;
+        assertEquals(JsonParser.parseString("""
+                {
+                    "name": "Alice"
+                }
+                """).toString(), jsonAutoRepairer.repair(originalJSON));
+    }
+
 
 
     @Test