Perso j'y crois sur deux idées regroupées : le truc codé en interne confié à un stagiaire ou à un dev débutant en JS, et le fait que ce soit dans un intranet qui fait que ça a pu passer sous les radars jusque là.
Rien que la ligne 545, ça sent le truc pompé sur un exemple pour Node sans se demander en quoi ça pose un problème si on le copie côté client.

Quand au "true === true", il y a quelques jours j'ai trouvé dans le code de l'appli legacy sur laquelle je bosse actuellement un truc du genre :
if (abc) { do_this(); } else { do_this(); }C'est typique du code retouché en patch rapide qui n'a jamais été changé avant : un "abc === true" changé en "true === true" juste pour forcer le if, sans réfléchir à simplement virer le if.