fix(fake-browser): ensure nested objects are not mutated on set operation

[Timeraa]

This PR fixes a nasty bug that I came across when writing storage tests inside of wxt and I couldn't figure out why at the time and now my friend @Bas950 came across the same issue and it turns out that the fakebrowser package doesn't remove the references when updating the object.

I tried using structuredClone but it didn't quite fix the issue in some deeply nested objects / arrays.

Idk if you remember this @aklinker1 https://discord.com/channels/1212416027611365476/1224505556346212515/1308235414582071358 but this is the fix of that issue.

The bun lock probably got updated because I have the latest version of it.

---

This pull request includes changes to the fake-browser package to improve the handling of nested objects in the storage API. The most important changes include adding a new test case to ensure nested objects are not mutated and modifying the set method to deep clone the items before setting them.

Improvements to storage API:

• packages/fake-browser/src/apis/storage.test.ts: Added a test case to ensure that setting a nested object does not mutate the original object.

Codebase modification:

• packages/fake-browser/src/apis/storage.ts: Modified the set method in defineStorageArea to deep clone the items using JSON.parse(JSON.stringify(items)) before setting them.

[aklinker1]

Good find! Are there any other APIs that need this as well?

[Timeraa]

Good find! Are there any other APIs that need this as well?

I am not quite sure, I've so far only really used the storage of fake browser

[aklinker1]

Now that I know messaging is weird for recursive objects, should probably implement that behavior as well. Not in this PR though, I'll do it separately

[aklinker1]

Released in https://github.com/aklinker1/webext-core/releases/tag/fake-browser-v1.3.2