chore: release v0.16.3

internal/cat/omnirig.go

@@ -177,14 +177,21 @@ func (o *OmniRig) ReadState() (RigState, error) {
 		BandFromHz(freqA) == BandFromHz(freqB)
 
 	if genuineSplit {
-		// OmniRig's Vfo enum is RX-letter then TX-letter (AB = RX A, TX B).
-		// ADIF: FreqHz = TX, RxFreqHz = RX.
+		// ADIF: FreqHz = TX, RxFreqHz = RX. Determine which VFO is RX from the
+		// ACTIVE frequency (OmniRig's generic Freq — the VFO you're listening on):
+		// RX = the active VFO, TX = the other one. This is far more reliable than
+		// trusting OmniRig's Vfo AB/BA enum, which several rigs (e.g. Yaesu FTDX10)
+		// report inverted — the split then showed TX/RX swapped.
 		s.Split = true
-		switch s.Vfo {
-		case "BA":
-			s.FreqHz, s.RxFreqHz = freqA, freqB // TX A, RX B
-		default: // "AB" and the usual "TX on the other VFO" case
-			s.FreqHz, s.RxFreqHz = freqB, freqA // TX B, RX A
+		switch {
+		case freqMain != 0 && freqMain == freqA:
+			s.RxFreqHz, s.FreqHz = freqA, freqB // listening on A → TX on B
+		case freqMain != 0 && freqMain == freqB:
+			s.RxFreqHz, s.FreqHz = freqB, freqA // listening on B → TX on A
+		case s.Vfo == "BA":
+			s.FreqHz, s.RxFreqHz = freqA, freqB // fall back to the Vfo enum
+		default:
+			s.FreqHz, s.RxFreqHz = freqB, freqA
 		}
 	} else {
 		// Simplex: the operating frequency is OmniRig's generic Freq (the active