I think that "is this unfriendly-or-unkind?""is this unfriendly or unkind?" is the wrong question, because it accepts the existing framework which is less than ideal. I want to be more nuanced:
Is there a social issue with phrasing comments this way? Absolutely yes. Some might be surprised to hear me say it (I have a reputation for bluntness), but to me this kind of commentary crosses a line. It's no longer firmly stating that something is a terrible idea; it implies that the OP should know better and should feel bad for having written the code. (Some OPs should know better, but I feel it's better to express this plainly.)
Would Stack Overflow be perceived as a friendlier place if such commentary were wiped as a matter of course, easy to convince moderators to remove, and much less often made in the first place? Again, yes. We don't want people coming here for code review in the first place, so unsolicited code review is at best a distraction and at worst negative feedback to the idea of asking a question at all. It doesn't say anything about improving the question - and anyway, questions don't have to involve "best practice" to meet standards. It's fine to have questions motivated by an XY problem, exactly because it might not be an XY problem for someone else.
Should such commentary merit a 100 reputation penalty and a direct talking-to from mods? No. However, it does merit deletion. More importantly for the current discussion, though, it merits tracking to see if someone has a pattern of such flowery language. People who are abrasive but not directly insulting, should be counseled on ways to improve how they come across. The latter doesn't, to my understanding, generally happen with a "no longer needed" flag. But I don't know if that's a technical problem (a need for a different flag tier) or a social one (asking mods to please keep this sort of thing in mind).