La abreviación que hiciste para ahorrar código está muy curioso, antes debo resolver problemas indicado arriba, a parte de eso, lo de pulsar una tecla y automáticamente ejecuta el envio.

Si escribo letras, se vuelve loco. Si escribo cualquier número, funciona de maravilla.

#include <iostream>
 
using namespace std;
 
int main(){
	while(true){
		int n;
		cin >> n;
		if(!cin){
			cout << "Error" << endl;
			cin.clear();
			cin.sync();
		}else{
			cout << "Ok: " << n << endl;
		}
	}
}

En primer lugar, yo separaría estoa  una función:

		while (true) {
			int n = Puerto->ReadData(lectura, 49);
			if (n > 0) {
				lectura[n + 1] = '\0';
				break;
			}
			Sleep(1);
		}

Una función que espere a que el Arduino envíe datos, y los retorne.

Lo de esperar datos solo lo vas a hacer si pulsan 1 o 2, así que llamas a esa función en cada case. Es cierto que los 2 case son idénticos salvo por la constante que envías. Podrías poner algo como:

const char *luz[2] = {"Luz_ON", "Luz_OFF"};
 
switch(opc){
    case 1:
    case 2:
        cout << "Enviando: " << luz[opc-1] << endl;
        Puerto->WriteData(luz[opc-1], strlen(luz[opc-1]));
        cout << "Respuesta: " << obtenerRespuestaArduino() << endl;
        break;
 
    default:
        // ...
}

De ese modo, solo esperará entrada del Arduino cuando sea necesario hacerlo. Al terminar el switch, puedes poner, si quieres, para leer datos del Arduino (sin esperar indefinidamente, eso sí, para que vuelva al inicio del bucle while y siga preguntando)

El problema es eso, el cómo está estructurado el código. Lo dicho: si tienes más experiencia en C#, puedes tratar de hacerlo ahí primero.