Everything looks good. The only thing I recommend, which I think is a blocker is to at least have a cleanup function to unregister the chart from the global state onUnMount or onBeforeUnmount

hmm, yeah that's probably fair. That also would help with the state getting polluted in long running sessions.

fixed. 3d8f6e6

Could datasets be empty?

Should guard against allTimestamps reducing to [] -> actualMinStamp = 0 -> actualMaxStamp = 0

I'm having trouble tracing in the code if that might cause any trouble down stream may be have a test around empty datasets?

yeah, it could return 0 in that case. But if your dataset is empty then it won't be rendering anything anyways. I'm not sure I can prove that without a test though, so I'll add one.

ah, this one is a little tricky and I had handled this already, but it's not at all obvious. I added a test 3d8f6e6.

The trick is that syncedMinStamp is doing this logic:

 const syncedMinStamp = computed<number>(() => {
 return Object.values(state.minStamps)
 .reduce((min, stamp): number => {
 return min === 0 ? stamp : Math.min(min, stamp)
 }, 0)
 })

To be clear, return min === 0 ? stamp : Math.min(min, stamp) means if we ever encounter a 0 we ignore it in favor of the first actual timestamp we find. Granted, this means I'm disqualifying Wed Dec 31 1969 19:00:00 GMT-0500 as a start time, but... I think that's okay.