From 337cb72974a03972bb1bd6e9f4ed8325b4f5b1a7 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Thu, 26 Mar 2026 06:27:01 +0000
Subject: [PATCH] [refactor](trx-server): use state data constructors for
 pub(crate) fields

Migrate ready_data_from_state and transmitting_data_from_state to use
the new ReadyStateData::new() and TransmittingStateData::new()
constructors instead of direct struct field initialization.

https://claude.ai/code/session_01XzurkeuUmamBuhQwxVy7T4
Signed-off-by: Claude <noreply@anthropic.com>
---
 src/trx-server/src/rig_task.rs | 30 +++++++++++++++---------------
 1 file changed, 15 insertions(+), 15 deletions(-)

diff --git a/src/trx-server/src/rig_task.rs b/src/trx-server/src/rig_task.rs
index ace7029..b49dfe4 100644
--- a/src/trx-server/src/rig_task.rs
+++ b/src/trx-server/src/rig_task.rs
@@ -1033,29 +1033,29 @@ fn desired_machine_state(state: &RigState) -> RigMachineState {
 }
 
 fn ready_data_from_state(state: &RigState, rig_info: trx_core::rig::RigInfo) -> ReadyStateData {
-    ReadyStateData {
+    ReadyStateData::new(
         rig_info,
-        freq: state.status.freq,
-        mode: state.status.mode.clone(),
-        vfo: state.status.vfo.clone(),
-        rx: state.status.rx.clone(),
-        tx_limit: state.status.tx.as_ref().and_then(|tx| tx.limit),
-        locked: lock_state_from(state),
-    }
+        state.status.freq,
+        state.status.mode.clone(),
+        state.status.vfo.clone(),
+        state.status.rx.clone(),
+        state.status.tx.as_ref().and_then(|tx| tx.limit),
+        lock_state_from(state),
+    )
 }
 
 fn transmitting_data_from_state(
     state: &RigState,
     rig_info: trx_core::rig::RigInfo,
 ) -> TransmittingStateData {
-    TransmittingStateData {
+    TransmittingStateData::new(
         rig_info,
-        freq: state.status.freq,
-        mode: state.status.mode.clone(),
-        vfo: state.status.vfo.clone(),
-        tx: state.status.tx.clone(),
-        locked: lock_state_from(state),
-    }
+        state.status.freq,
+        state.status.mode.clone(),
+        state.status.vfo.clone(),
+        state.status.tx.clone(),
+        lock_state_from(state),
+    )
 }
 
 fn lock_state_from(state: &RigState) -> bool {