Fully fix issue sending data from client to devtools

[jerelmiller]

Fully reverts changes from #1245 that removed JSON.stringify from the entire client payload. Irregular cache values caused some issues when serializing on all fields.

[relativeci]

Job #8: Bundle Size — 1.08MiB (~-0.01%).

42eef38(current) vs 66c05ae main#3(baseline)

Warning

Bundle contains 5 duplicate packages – View duplicate packages

Bundle metrics   2 changes 1 improvement

	Current Job #8	Baseline Job #3
Initial JS	1.04MiB(~-0.01%)	1.04MiB
Initial CSS	0B	0B
Cache Invalidation	94.56%	0%
Chunks	5	5
Assets	12	12
Modules	486	486
Duplicate Modules	32	32
Duplicate Code	7.42%	7.42%
Packages	59	59
Duplicate Packages	4	4

Bundle size by type   1 change 1 improvement

	Current Job #8	Baseline Job #3
JS	1.04MiB (~-0.01%)	1.04MiB
IMG	35.85KiB	35.85KiB
HTML	810B	810B
Other	777B	777B

---

View job #8 report View json-stringify-all-data branch activity View project dashboard

[jerelmiller]

Noting here that @phryneas had a good idea to JSON.parse(JSON.stringify(...)) in the content script before sending it. This should guarantee the value is serializable.

[jerelmiller]

I'll note here that I think a potential future improvement here is to catch errors in case the cache can't be fully stringified. It would be a much nicer experience to show an error in the devtools panel, but we'll need to figure out a way to transport that error over. That is a bit out of scope for this PR.

[phryneas]

Tested in Chrome, TypeScript compiles, looks good, let's go! :)