false positive for sibling (not-circular) references

[proppy]

var safeStringify = require('json-stringify-safe')

var obj = { foo: 'bar' };

console.log(safeStringify({
    a: obj,
    b: obj
}));


console.log(JSON.stringify({
    a: obj,
    b: obj
}));

Will produce the following ouput:

{"a":{"foo":"bar"},"b":"[Circular ~.a]"}
{"a":{"foo":"bar"},"b":{"foo":"bar"}}

It seems to identify multiple sibling references to the same object as circulars.

[alexturpin]

Getting a similar problem that makes this unusable for me:

var safeStringify = require('json-stringify-safe');
var list = ["a", "b", "c"];
var obj = {list: list, foo: {bar: list}};
console.log("native", JSON.stringify(obj));
console.log("json-stringify-safe", safeStringify(obj));

Native JSON.stringify has no issue with it, but the safe stringify does. Output:

native {"list":["a","b","c"],"foo":{"bar":["a","b","c"]}}
json-stringify-safe {"list":["a","b","c"],"foo":{"bar":"[Circular ~.list]"}}

[jupiter]

+1

[moll]

Oops, time flies. I'll get to it, @jupiter. ;)

[moll]

Comrades,

I copied the implementation from Must.js, perked it up a little and added a full test suite.
Would y'all mind giving it a spin and see if it does everything you imagined it should?
There's a CHANGELOG, but in summary it just doesn't report false positives any more.

Thanks a bunch!

[moll]

v5.0.1 is now released which I should fix this issue. ;-)