feat(index): round fix + improved quote fixer

`json-fixer` can now fix several issues instead of one until it went through the heuristical-based
number of fixes needed for particular data. The quote fixer now doesn't includ commas or any
meaningful JSON tokens within the quotes.

index.js

@@ -10,6 +10,13 @@ const {parse} = require('./src/json.pjs')
 } */
 
 const psw = d => process.stdout.write(`${d}\n`)
+let fixRounds = 0
+let roundThreshold = 20
+
+const setFixThreshold = data => {
+  const lineCount = data.split('\n').length
+  roundThreshold = Math.max(data.length / lineCount, lineCount)
+}
 
 const doubleCheck = (data, verbose = false) => {
   /* eslint-disable no-console */
@@ -22,6 +29,8 @@ const doubleCheck = (data, verbose = false) => {
       psw('Nearly fixed data:')
       data.split('\n').forEach((l, i) => psw(`${chalk.yellow(i)} ${l}`))
     }
+    // eslint-disable-next-line no-use-before-define
+    if (fixRounds < roundThreshold) return fixJson(err, data, verbose)
     console.error(chalk.red(`There's still an error!`))
     throw new Error(err.message)
   }
@@ -95,7 +104,7 @@ const fixMissingQuotes = ({start, fixedData, verbose}) => {
   if (verbose) psw(chalk.magenta('Missing quotes'))
   const targetLine = start.line - 1
   const brokenLine = removeLinebreak(fixedData[targetLine])
-  const NO_RH_QUOTES = /(":\s*)([\s\S]+)/g
+  const NO_RH_QUOTES = /(":\s*)([^,{}[\]]+)/g
   const NO_LH_QUOTES = /(^[^"]\S[\S\s]+)(:\s*["\w{[])/g
   let fixedLine = NO_RH_QUOTES.test(brokenLine)
     ? brokenLine.replace(NO_RH_QUOTES, '$1"$2"')
@@ -170,6 +179,7 @@ const fixComment = ({start, fixedData, verbose}) => {
 
 /*eslint-disable no-console */
 const fixJson = (err, data, verbose) => {
+  ++fixRounds
   const lines = data.split('\n')
   if (verbose) {
     psw(`Data:`)
@@ -231,6 +241,8 @@ const checkJson = (data, verbose = false) => {
       }
     }
   } catch (err) {
+    fixRounds = 0
+    setFixThreshold(data)
     return {
       data: fixJson(err, data, verbose),
       changed: true,

src/__tests__/index.js

@@ -236,3 +236,30 @@ describe('comments', () => {
     })
   })
 })
+
+//ops and concats
+
+describe('multi rounds', () => {
+  it('x2', () => {
+    const json = fs.readFileSync('./test/samples/twoErrs.json', 'utf-8')
+    const {data, changed} = jf(json)
+    expect(changed).toBeTruthy()
+    expect(data).toEqual({
+      name: 'sample #19',
+      type: 'JSON',
+      error: '2 errors',
+      version: 19,
+    })
+  })
+  it('x3', () => {
+    const json = fs.readFileSync('./test/samples/threeErrs.json', 'utf-8')
+    const {data, changed} = jf(json)
+    expect(changed).toBeTruthy()
+    expect(data).toEqual({
+      name: 'sample #21',
+      type: 'JSON',
+      error: '3 errors',
+      version: 21,
+    })
+  })
+})

test/samples/ops.json

@@ -0,0 +1,6 @@
+{
+  "name": "sample #20",
+  "type": "JSON",
+  "error": "comment",
+  "version": 2 * 10
+}

test/samples/threeErrs.json

@@ -0,0 +1,6 @@
+{
+  "name": "sample #21",
+  "type": JSON,
+  "error": "3 errors"
+  "version": x15
+}

test/samples/twoErrs.json

@@ -0,0 +1,6 @@
+{
+  "name": "sample #19",
+  "type": 'JSON',
+  "error": "2 errors",
+  "version": 19 //16
+}