This is due to not leaving the smithed.prevent_aggression team when they should.
As a somewhat recent change, villagers maintain their team when being zombified/cured, so they will turn into zombie villagers, receive the team, and then never leave.
As for Johnny illagers, they should attack everything, so they should also leave the team.
I've personally fixed this in TCC using the following two lines after the 1s clock that gives the team.
## Fixes for smithed.prevent_aggression
team leave @e[type=minecraft:vindicator,name=Johnny,team=smithed.prevent_aggression]
team leave @e[type=minecraft:villager,team=smithed.prevent_aggression]