Prośba o sprawdzenie kodu

Napisałem z nudów takie coś https://github.com/Taraj2/Simple-Chat/tree/master/NodejsWebApp1 niby wszystko działa tak jak zaplanowałem i jest ok ale pisząc to natknąłem się na parę problemów np. tworząc elementy lepiej użyć createElement() czy po prostu innerHTML w rezultacie otrzymujemy identyczny wynik więc która opcja jest lepsza. Jeżeli ktoś ma jakieś rady co zrobić żeby to było bardziej "poprawnie" napisane chętnie je usłyszę xD

Jak dla mnie:

1. Za duże pliki js (w sumie to plik). Pliki 100 do 150 linijek czyta się o wiele lepiej.
2. Podobnie funkcje (ale widziałem znacznie gorsze).
3. If'ologia - za dużo używasz tam if..else, a potrójne zagłębienie to już przesada.
4. Pliki less - jak dla mnie też za dużo zagłębień. źle się to czyta.
5. Zewnętrzne pliki/biblioteki trzymaj w osobnym folderze/folderach niż twoje pliki.
6. innerHTML - bardziej wydajny, do dodawania mniejszych elementów, jak chcesz dodać jakiś większy element to raczej lepiej createElement()