fix(non-blocking-json): start batching after index 0

Author: christianalfoni

packages/node_modules/non-blocking-json/src/index.test.ts

@@ -48,4 +48,11 @@ test("should handle more scenarios", async () => {
   const obj = JSON.parse(string)
   const result = await stringify(obj)
   expect(result).toBe(string)
+})
+
+test("should handle crazy scenario", async () => {
+  const string = '{"foo":"\\"hey {}\\""}'
+  const obj = JSON.parse(string)
+  const result = await parse(string)
+  expect(result).toEqual(obj)
 })

packages/node_modules/non-blocking-json/src/index.ts

@@ -79,7 +79,7 @@ export function stringify(value, replacer?) {
       }
 
       for (let [key, value] of paths) {
-        if (x % ASYNC_BATCH_COUNT === 0) {
+        if (x && x % ASYNC_BATCH_COUNT === 0) {
           if (TRACK_PERF) {
             const end = performance.now()
             console.log("PERF - ", end - start)
@@ -162,15 +162,16 @@ export async function parse(value) {
   }
 
   for (let charIndex = 0; charIndex < value.length; charIndex++) {
-    if (charIndex % ASYNC_BATCH_COUNT === 0) {
+    if (charIndex && charIndex % ASYNC_BATCH_COUNT === 0) {
       if (TRACK_PERF) {
         const end = performance.now()
         console.log("PERF - ", end - start)
         start = end
       }
       await new Promise(resolve => next(resolve));
     }
-    if (value[charIndex] === '"') {
+    
+    if (value[charIndex] === '"' && value[charIndex - 1] !== '\\') {
       isInString = !isInString;
       continue;
     }
@@ -216,7 +217,7 @@ export async function parse(value) {
 
     const keys = Object.keys(parsedValue);
 
-    if (i % ASYNC_BATCH_COUNT === 0) {
+    if (i && i % ASYNC_BATCH_COUNT === 0) {
       await new Promise(resolve => next(resolve));
     }