No. Valentine is actually quite intelligent, and it is an important reveal in the story that he has been playing up the "dumb jock" stereotype in his career choices largely because that is what is expected of him, not because that is who he actually is. And McNab never gave off "dangerous" vibes even for a second. He actually embodies the "patient father" role in how he interacts with the group, and in how he stopped Valentine from getting revenge over Zaina's death. Neither of them are aggressive at all outside of the ring, or combat, where it is totally appropriate for them to do so.
Again, not even close. Vi's family issues stem from her brother's abandonment of her after they lost their parents, and the way he isolated her from Valentine (hiding his letters, etc). Her brother's death was actually more of a relief to her than a traumatic experience. And you're leaving out the dog attack and physical scars that are Vi's true trauma. Those are what often cause her to become paralyzed and unable to function. Ignoring that major event in her life is very strange. And revealing perhaps in its omission, because it was a situation where Valentine's physical strength, courage, and assertiveness, likely saved Vi's life from a dangerous animal. That kind of masculinity is seen as very positive and desirable in the eyes of the opposite sex, as that particular threat proves.
The one-sided and pejorative narratives you suggest are simply too forced to be believed. They don't fit the events of the story, or the characterizations, and they fall into a hypoagency vs hyperagency trap for each sex that is honestly insulting.