LukasMeine / vigilance

node_modules/json-buffer/index.js

//TODO: handle reviver/dehydrate function like normal
//and handle indentation, like normal.
//if anyone needs this... please send pull request.

exports.stringify = function stringify (o) {
  if('undefined' == typeof o) return o

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

  if(o && Buffer.isBuffer(o))

The variable Buffer seems to be never declared. If this is a global, consider adding a /** global: Buffer */ comment.

    return JSON.stringify(':base64:' + o.toString('base64'))

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

  if(o && o.toJSON)
    o =  o.toJSON()

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

  if(o && 'object' === typeof o) {
    var s = ''
    var array = Array.isArray(o)
    s = array ? '[' : '{'
    var first = true

    for(var k in o) {

A for in loop automatically includes the property of any prototype object, consider checking the key using hasOwnProperty.

      var ignore = 'function' == typeof o[k] || (!array && 'undefined' === typeof o[k])
      if(Object.hasOwnProperty.call(o, k) && !ignore) {
        if(!first)
          s += ','

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

        first = false
        if (array) {
          if(o[k] == undefined)

Comparing o.k to undefined using the == operator is not safe. Consider using === instead.

            s += 'null'

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

          else
            s += stringify(o[k])
        } else if (o[k] !== void(0)) {

Consider using undefined instead of void(0). It is equivalent and more straightforward to read.

          s += stringify(k) + ':' + stringify(o[k])
        }
      }
    }

    s += array ? ']' : '}'

    return s
  } else if ('string' === typeof o) {
    return JSON.stringify(/^:/.test(o) ? ':' + o : o)
  } else if ('undefined' === typeof o) {
    return 'null';
  } else
    return JSON.stringify(o)

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

}

exports.parse = function (s) {
  return JSON.parse(s, function (key, value) {
    if('string' === typeof value) {
      if(/^:base64:/.test(value))
        return new Buffer(value.substring(8), 'base64')

The variable Buffer seems to be never declared. If this is a global, consider adding a /** global: Buffer */ comment.

Curly braces around statements make for more readable code and help prevent bugs when you add further statements.

      else
        return /^:/.test(value) ? value.substring(1) : value 

else is not necessary here since all if branches return, consider removing it to reduce nesting and make code more readable.

    }
    return value
  })
}