I'm inclined to merge this as a temp fix but I'm not sure how I feel about the fact that we're claiming NodeError is thread-safe when it really isn't. Does @preconcurrency import mitigate the need for this?
Merging this for now since @preconcurrency doesn't always seem to cut it — though I'll def be working on improving concurrency support going forward, since NodeError isn't really Sendable. Thanks!
I'm inclined to merge this as a temp fix but I'm not sure how I feel about the fact that we're claiming
NodeError
is thread-safe when it really isn't. Does@preconcurrency import
mitigate the need for this?