Very detailed article with examples! You can tell you did something right because the self proclaimed '10x' developers are nitpicking things, lol.
These are definitely worst practices which is why I think we have seen a lot of evolution in JavaScript in the past decade to bring the syntax closer to something that resembles other familiar classical OOP languages. This is also why TypeScript has gained so much steam.
I predict we will see further evolution for ES in this direction where we will have declared types and the proper error handling to support them, as well as other familiar classical OOP concepts like interfaces and abstract class types.
The only reason we never needed these things before was that JavaScript wasn't designed to be used the way we use it today.
To all you nitpicky JS hipsters, get with the times and start programming in a way that transcends language and stop pushing ES's esoteric and confusing 'best practices' of a decade ago.