vi wird immer zweimal ausgeführt

Ganz einfach: Es finden zwei Ereignisse "Wertänderung" statt. Einmal bei Drücken des Tasters, einmal beim Loslassen.
Stelle das Schaltverhalten um auf "Latch beim Drücken" oder "Latch beim Loslassen". Das Rücksetzen des Tasters erfolgt dann automatisch beim Lesen des Tasters. Da das aber kein Bedieneingriff ist, löst das kein zweites Ereignis aus.
Ich selbst bevorzuge "Latch beim Loslassen". Grund: bei "Latch beim Drücken" fehlt meist das "Bedienerlebnis". Der Taster wird oft im selben Moment wieder rückgesetzt, wenn er gedrückt wurde. Der Benutzer sieht also gar nicht, dass der Taster je gedrückt wurde.
Das Wait links ist überflüssig: Die Ereignissruktur wartet bereits bis zu unendlich lange auf die Bedienereignisse.