Parabéns pelo projeto e por deixá-lo publico. Muita gente já falou coisas positivas sobre seu código, então não vou repetir :) Pensando em alguém que não está familiarizado com o código e passa o olho nesse trecho.
if (createLinkResponse.isLeft())
return httpResponseUtils.badRequest(createLinkResponse.value);
Left ou Right não é uma coisa que fica claro pra mim, sei que é um jto Haskell de programar, eu acho confuso (o problema pode ser eu), sugestão:
if (createLinkResponse.failed())
return httpResponseUtils.badRequest(createLinkResponse.value);
Para mim ficaria mais fácil a leitura, não sei o que os outros colegas acham, e o que vc acha tb?
A estrutra do código em si é clara, fácil de se localizar, métodos curtos, classes curtas, testes (super importante)
Concordo com você, uma pessoa que nunca viu esse tipo de tratamento de erro pode estranhar(não que seja o seu caso kk), porém isso já é meio que uma convenção.
Agradeço pela sugestão. 🤝