call toJSON only AFTER this.replacer

[michele-nuzzi]

replacer needs to be called before toJSON.

example when encoding Buffer I'd like to receive hex strings, but instead toJSON is called and my replacer is not called on Buffers

[Faleij]

I understand why you want this but the original implementation of JSON.stringify by Douglas Crockford and the implementation that is in all browsers follows the convention of calling toJSON before the replacer function, that's why this library follows that convention as well.

I will accept a pull request that changes this behavior depending on an argument to JsonStreamStringify so that it does not break the original behavior.

Example of this behavior in chromium browsers:

< JSON.stringify({ toJSON() { return 'toJSON'; }}, (k, v) => {
  console.log({ k, v });
  return v;
})
> {k: '', v: 'toJSON'}