[Bug] Runtime Error when using save() to get snapshot in a sheet that have a chart

[kubothegsd]

Before you submit this issue, have you checked the following

• Is this really a problem?
• I have searched the Github Issues for similar issues, but did not find anything.

Affected packages and versions

0.5.5

Reproduction link

I set up a sheet with a chart using the preset

presets: [
     UniverSheetsAdvancedPreset({
        license: <'txt content'>,
      }),
]

Then I use the save() method to get a snapshot

const activeWorkbook = univerAPI.getActiveWorkbook();
const snapshot = activeWorkbook.save();

Expected behavior

Should get a snapshot as normal without throwing exception

Actual behavior

Unhandled Runtime Error
TypeError: Converting circular structure to JSON
--> starting at object with constructor 'SafeSubscriber'
| property '_finalizers' -> object with constructor 'Array'
| index 0 -> object with constructor 'OperatorSubscriber'
--- property '_parentage' closes the circle

Call Stack
JSON.stringify
<anonymous>
k._serializeChartForUnit
webpack-internal:/node_modules/.pnpm/@[email protected]_@[email protected][email protected][email protected]/node_modules/@univerjs-pro/sheets-chart/lib/es/index.js
Object.toJson
webpack-internal:/node_modules/.pnpm/@[email protected]_@[email protected][email protected][email protected]/node_modules/@univerjs-pro/sheets-chart/lib/es/index.js
eval
webpack-internal:/node_modules/.pnpm/@[email protected]_@[email protected][email protected][email protected]/node_modules/@univerjs/core/lib/es/index.js
Array.map
<anonymous>
VE.getResourcesByType
webpack-internal:/node_modules/.pnpm/@[email protected]_@[email protected][email protected][email protected]/node_modules/@univerjs/core/lib/es/index.js
VE.getResources
webpack-internal:/node_modules/.pnpm/@[email protected]_@[email protected][email protected][email protected]/node_modules/@univerjs/core/lib/es/index.js
ui.saveUnit
webpack-internal:/node_modules/.pnpm/@[email protected]_@[email protected][email protected][email protected]/node_modules/@univerjs/core/lib/es/index.js
f.save
webpack-internal:/node_modules/.pnpm/@[email protected]_@[email protected][email protected][email protected]/node_modules/@univerjs/sheets/lib/es/facade.js

System information

Using Trial License in JavaScript/TypeScript Projects (without server)

[VicKun4937]

sorry , can you create a demo to reproduce this problem, I can't reproduce this one.

[kubothegsd]

This is the demo. https://stackblitz.com/~/github.com/kubothegsd/sheets-vite-demo

snapshot_error.mp4

Thanks for helping, @VicKun4937

[VicKun4937]

thank you for your reproduce, it really a bug , and we will fix it in next version.

And I find that you only have three types of chart, you can test it with a license , you can get more support chart type. As you know , Chinese New Year is coming, may be you question can't be replied timely. You can leave messages in git or discard, we will answer you later days.

[VicKun4937]

It is caused by local reference in json, fixed