feat: improve dataValidation function

[Blakko]

From my tests on a 3k nodes graph, the execution time of this function went from 400ms down to 1ms

[Aarebecca]

Although it is faster to using for than map loop, there shouldn't be such a large gap at 3k nodes

[Blakko]

The performance gain comes from changing
ids.includes(edge.source) || !ids.includes(edge.target)
into
return !ids.has(edge.source) || !ids.has(edge.target);

I'll test removing the for loops in favor of a .each line, I agree there shouldn't be a big difference

[Blakko]

Using a forEach to populate the set (rather than the for loop) did show a negligible performance impact, so I made the code tidier

[Aarebecca]

It looks like there's something wrong with ci in v4 and this PR will be merged after we fix it