Fix "TypeError: Converting circular structure to JSON" when passing hyperformulaInstance to hotSettings

[tonextone]

Context

Fixes the issue #8728 (comment) .

For JSON.stringify() is used to compare hotSettings (current ⇔ new) ,
any circular structure ( like hyperformulaInstance ) within hotSettings
causes this: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Errors/Cyclic_object_value

How has this been tested?

Added a test case.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature or improvement (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)
• Additional language file or change to the existing one (translations)

Related issue(s):

1. Fix [9.0.0+] Adding HyperFormula leads to "TypeError: Converting circular structure to JSON" #8728

Affected project(s):

• handsontable
• @handsontable/angular
• @handsontable/react
• @handsontable/vue
• @handsontable/vue3

Checklist:

• I have reviewed the guidelines about Contributing to Handsontable and I confirm that my code follows the code style of this project.
• I have signed the Contributor License Agreement
• My change requires a change to the documentation.

[AMBudnik]

Thank you very much for creating the pull request, signing CLA, and adding the test scenario.
I am adding the ready for review label. We will proceed with the code review as soon as possible.

[171h]

why not merge? I have the same problem.

cc: @kirszenbaum @AMBudnik @adrianszymanski89

@handsontable_vue3.js:127  Uncaught (in promise) TypeError: Converting circular structure to JSON
    --> starting at object with constructor 'Rule2'
    |     property 'definition' -> object with constructor 'Array'
    |     index 0 -> object with constructor 'Alternation2'
    |     ...
    |     index 1 -> object with constructor 'NonTerminal2'
    --- property 'referencedRule' closes the circle
    at JSON.stringify (<anonymous>)
    at simpleEqual (@handsontable_vue3.js:127:15)
    at prepareSettings (@handsontable_vue3.js:115:128)
    at Proxy.handler (@handsontable_vue3.js:141:24)
    at callWithErrorHandling (chunk-J7TRJLPB.js:213:18)
    at callWithAsyncErrorHandling (chunk-J7TRJLPB.js:221:17)
    at job (chunk-J7TRJLPB.js:1810:9)
    at callWithErrorHandling (chunk-J7TRJLPB.js:213:32)
    at flushJobs (chunk-J7TRJLPB.js:411:9)
    at flushJobs (chunk-J7TRJLPB.js:421:7)

[budnix]

@tonextone Can I ask you to add a changelog file? You can generate that by running npm run changelog entry in the project's root dir. Then, I'll be able to merge the PR.

[tonextone]

@budnix Thank you for the follow-up. I just added a changelog entry.

( It's my first time, so I'm not sure if it's perfect. )

[AMBudnik]

Thank you again for your contribution @tonextone
The changes are published.