Esse problema acontece com qualquer funcionalidade que mostra o estado mudando um ícone: o ícone atual representa o estado atual ou estado futuro?

Quando vi esse PR, fui consultar alguns aplicativos de bancos que possuem esse "olho" para mostrar/esconder os valores financeiros. Abri dois (acho que XP e Nubank) e percebi que não tratam isso da mesma forma.

O ideal é ter alguma forma auxiliar de descobrir o que o botão fará. No caso dessa funcionalidade, está com um tooltip, que apesar dos seus problemas de acessibilidade, é melhor do que nada.