Hi,
I will answer the question anyway:
It is not possible to say whether it is bad or not (that is the problem with foam). Of course, the joints must be sealed so that the concrete does not leak out, but this must be done using formwork that is flush with the exterior, so that the concrete cross-section meets the minimum dimension everywhere. The problem with foam (and therefore regardless of the type) is that you cannot control how far the foam penetrates into the interior of the component. This reduces, uncontrollably, the component thickness of a waterproof concrete component at its most critical point – the joint. As it looks here, they have at least done the foaming quite neatly, so one can initially assume that nothing will happen.
However, there is of course a reason why the waterproof concrete guideline has been significantly tightened, especially with regard to waterproof concrete construction using element walls, and is still hotly debated.
In my opinion, this is simply not a professionally correct execution, which is why I am pointing it out. The answer that it has often been done this way does not change that.
If I always drive my car at 80 where 50 is allowed and no one stops me, I may not get into trouble, but that does not suddenly make it correct.
But to finally respond to your question:
1. The foam must be removed
2. I would avoid placing profiles at the inner corners as much as possible, but rather seal them, since we expect annual movement here that leads to cracks. With sealing, you can best "touch it up" again.
3. At the inner external corner, I would like to see how the elements come together there to assess where the crack is most likely to occur.
Good luck going forward