Você não está comparando datas e sim strings
A pergunta é: Como isso funciona afinal? Como o javascript compara duas strings e retorna true ou false dependendo do valor nelas?
2018-09-07 | 050 048 049 056 045 048 057 045 048 055
2018-05-08 | 050 048 049 056 045 048 053 045 048 056
---
A melhor solução mesmo para seu caso é seguir as orientações constantes no comentário do @Bacco.
A única solução real é esquecer tudo que tem no código da pergunta e começar certo do zero. Se vai usar strings, use a ordem AAAAMMDD e não faça nada de conversão. Se o formato já é data, compare como data ou numericamente (usando getTime). Se são 2 formatos diferentes, converta pra um formato só que faça sentido (ou string na ordem certa, ou inteiro/timestamp)