¿Este circuito tiene el aislamiento adecuado?

Su circuito está bien y se aislaría si el suelo no estuviera unido. Sería apropiado si tuviera una fuente de 12 V flotante separada de la que quisiera probar la presencia.

Sin embargo, dado que ambos lados tienen una tierra común, los suministros de 12 V y 3,3 V no están aislados entre sí. Como no están aislados, puedes usar una conexión más directa. Su optoaislador aún funcionaría, pero agrega costos y complicaciones innecesarios.

Un enfoque simple es:

Esto se invierte de la lógica que tienes. La entrada del microcontrolador será baja cuando 12 V esté presente y alta de lo contrario. Como es una entrada de microcontrolador, generalmente hace lo que es conveniente para el hardware y maneja la polaridad en el firmware en consecuencia.

Si necesita aislamiento de tierra, necesita diferentes fuentes de alimentación. Sin embargo, si su única preocupación es el nivel de voltaje, es decir, no desea dañar el microcontrolador, no necesita aislamiento de tierra. Todo lo que necesita hacer es garantizar que el voltaje máximo que llega a los pines no exceda el máximo.

Esto, como usted ha sugerido, puede lograrse a través del aislamiento óptico; Sin embargo, hay formas más baratas de hacerlo. Un ejemplo sería usar un divisor de voltaje.

This debería ayudarlo más.