Складывается ощущение, что он просто никогда на самом деле не работал ни с Jackson, ни с JPA, ни с чем-то еще... У вообще не писал именно бизнес-приложений. От того и не понимание, я думаю.

Вообще это очень часто я замечаю у авторов фреймворков и библиотек. У того же спринга тоже бывают с этим проблемы. Довольно крутые ребята, которые пишут код так, что я завидую, иногда показывают признаки того, что они на самом деле этот свой фреймворк ни разу не использовали в реальном сложном проекте. А только лишь на демо, в консалтинге, на презентациях и прочее.

Складывается ощущение, что он просто никогда на самом деле не работал ни с Jackson, ни с JPA, ни с чем-то еще... У вообще не писал именно бизнес-приложений. От того и не понимание, я думаю.

Вообще это очень часто я замечаю у авторов фреймворков и библиотек. У того же спринга тоже бывают с этим проблемы. Довольно крутые ребята, которые пишут код так, что я завидую, иногда показывают признаки того, что они на самом деле этот свой фреймворк ни разу не использовали в реальном сложном проекте. А только лишь на демо, в консалтинге, на презентациях и прочее.

Логику по типу этой следует оставлять в контроллерах или переносить в сервисы?

В сервисы, и в одну цепочку оформить

Да. findby().map( ).orElse( )