Make sure we leave objects in good standing if there is an error

[mcollina]

No description provided.

[thetutlage]

After this release, the behavior for the following code has changed. It this is the expected outcome or a regression?

const safeStringify = require('fast-safe-stringify')

try {
  console.log(safeStringify({
    toJSON() {
      throw new Error('blowup')
    },
  }))
} catch (error) {
  console.log('never reached here')
  console.log(error)
}

With v2.0.7, the catch block was executed and now the code never reaches the catch block and returns a string saying "[unable to serialize, circular reference is too complex to analyze]"

Reproduction repo: https://github.com/thetutlage/fast-safe-stringify-reproduction

[mcollina]

That's expected!