Cupid — it’s not just about the name; the Greek Cupid doesn’t really fit with light-aligned angels. IMO, something like “acolytes” would look better.
Fighter — it’s basically fine, but “warrior” would sound more elegant.
Illumination — also not on theme; replace it with “saint” at least, or “blessed” perhaps.
And also, it would be better to place Angels/Archangels somewhere in the middle, and in the replaced castle actually replace them with paladins (like in 2), also remove monks and put them on the lower tiers of the celestial castle.
Thus, you get two castles — one pseudo-European, the other pseudo-celestial — allied with each other. This would look very organic and cohesive.