Поругайте код (NodeJS/AngularJS)

Зачем держать node_modules в репозитории? Есть же shrinkwrap.

Поругайте код

nodejs

Можно ругать даже в него не заглядывая,

https://github.com/Neoromantique/data-driven-front-end-application/blob/maste...
Ну ты бы вообще каждую строчку прокомментировал ещё
код надо писать так, чтобы было понятно что происходит.
А комментировать надо не действия, а причину принятия решений.

https://github.com/Neoromantique/data-driven-front-end-application/blob/maste...
Стиль комментария не соблюдён

Ну и в коде именно каша из пробелов и табов. Используй для индентации что-нибудь одно.

За ангуляр не скажу. Про бакенд:

- отдельно bin нет смысла делать, кидай в основной файл. Я б скорее код задвинул в lib чтобы корень не распухал, но дело вкуса.
- node_modules из репа убери
- я б вместо ejs брал jade, но дело вкуса
- нет обработчика uncaughtException, если что-то напутаешь - грохнется сервер https://nodejs.org/api/process.html#process_event_uncaughtexception. Аналогично для https://nodejs.org/api/process.html#process_event_unhandledrejection
- если поддержка старых нод не нужна, посмотри koa вместо express https://github.com/koajs/koa
- eslint прикрути
- тестов нет, и travic-ci не прикручен. Не хочешь тесты делать - линт хотя бы туда впиши
- писалки логов не нашел, будет трудно разбираться что случилось
- у монгуза не проверяется, соединился с базой или нет

как-то так...

Писал сначала в Sublime потом в виме, да надо привести к uniform.

что это такое? я в книгах читал, что лучше npm install и bower install и package/bower.json

Эта штука позволяет залочить версии зависимостей.

https://docs.npmjs.com/cli/shrinkwrap