Investigate and fix possible memory leaks in store2

[dai-shi]

#2463 (comment)

[dai-shi]

@backbone87 Do you have any idea for the scenario where only store2 fails with #2463 (comment) ?

[backbone87]

the downstream deps (t) must only be tracked for mounted downstream deps

[dai-shi]

because it leaks memory, or any other reason?

[backbone87]

yes because it leaks memory. you could keep the non-mounted downstream in weakrefs but I think that is more expensive than just always "pulling" unmounted atoms on read

[dai-shi]

Do you think you can write a test that passes with store.ts and fails with store2.ts?

[dai-shi]

Do you think you can write a test that passes with store.ts and fails with store2.ts?

I was able to do it and am working on it: #2533