This code is really hard to read. The logic there is a dense complicated block of code, which would take a lot of time for anyone to read and understand. Could you spend some time refactoring it into e.g. functions (with descriptive names and clearly defined scope)? Or at least, think through this code, check which parts might be confusing and add long comments explaining what's going on.

I can see that there's a draft label here. I guess we've agreed that we should use drafts only for arch review... but I might be wrong. Nevertheless, what's the difference here between draft and non-draft?