From 88ccc7ab81b65e8c41f53411e7433aaaa1bbc7c4 Mon Sep 17 00:00:00 2001
From: Stanislaw Grams <stanislawgrams@gmail.com>
Date: Sun, 8 Feb 2026 23:09:51 +0100
Subject: [PATCH] [feat](trx-frontend-http): add decoder toggle/clear UI and
 endpoints

Add POST endpoints for toggle_aprs_decode, toggle_cw_decode,
clear_aprs_decode, and clear_cw_decode. Add toggle buttons in APRS
and CW tabs. Render decoder enabled state from SSE updates. Clear
button now also resets server-side decoder state.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Signed-off-by: Stanislaw Grams <stanislawgrams@gmail.com>
---
 .../trx-frontend-http/assets/web/app.js       | 18 +++++++++
 .../trx-frontend-http/assets/web/index.html   |  2 +
 .../assets/web/plugins/aprs.js                |  7 +++-
 .../assets/web/plugins/cw.js                  |  7 +++-
 .../trx-frontend/trx-frontend-http/src/api.rs | 38 +++++++++++++++++++
 5 files changed, 70 insertions(+), 2 deletions(-)

diff --git a/src/trx-client/trx-frontend/trx-frontend-http/assets/web/app.js b/src/trx-client/trx-frontend/trx-frontend-http/assets/web/app.js
index 2ca326e..1fb221a 100644
--- a/src/trx-client/trx-frontend/trx-frontend-http/assets/web/app.js
+++ b/src/trx-client/trx-frontend/trx-frontend-http/assets/web/app.js
@@ -362,6 +362,24 @@ function render(update) {
   if (typeof update.clients === "number") {
     document.getElementById("about-clients").textContent = update.clients;
   }
+  // Decoder toggle buttons
+  const aprsToggleBtn = document.getElementById("aprs-decode-toggle-btn");
+  const cwToggleBtn = document.getElementById("cw-decode-toggle-btn");
+  if (aprsToggleBtn) {
+    const aprsOn = !!update.aprs_decode_enabled;
+    aprsToggleBtn.textContent = aprsOn ? "Disable APRS" : "Enable APRS";
+    aprsToggleBtn.style.borderColor = aprsOn ? "#00d17f" : "";
+    aprsToggleBtn.style.color = aprsOn ? "#00d17f" : "";
+    window.aprsDecodeEnabled = aprsOn;
+  }
+  if (cwToggleBtn) {
+    const cwOn = !!update.cw_decode_enabled;
+    cwToggleBtn.textContent = cwOn ? "Disable CW" : "Enable CW";
+    cwToggleBtn.style.borderColor = cwOn ? "#00d17f" : "";
+    cwToggleBtn.style.color = cwOn ? "#00d17f" : "";
+    window.cwDecodeEnabled = cwOn;
+  }
+
   powerHint.textContent = readyText();
   lastLocked = update.status && update.status.lock === true;
   lockBtn.textContent = lastLocked ? "Unlock" : "Lock";
diff --git a/src/trx-client/trx-frontend/trx-frontend-http/assets/web/index.html b/src/trx-client/trx-frontend/trx-frontend-http/assets/web/index.html
index 428fdb0..b878bde 100644
--- a/src/trx-client/trx-frontend/trx-frontend-http/assets/web/index.html
+++ b/src/trx-client/trx-frontend/trx-frontend-http/assets/web/index.html
@@ -145,6 +145,7 @@
       </div>
       <div id="subtab-aprs" class="sub-tab-panel" style="display:none;">
         <div class="aprs-controls">
+          <button id="aprs-decode-toggle-btn" type="button">Enable APRS</button>
           <button id="aprs-clear-btn" type="button">Clear</button>
           <small id="aprs-status" style="color:var(--text-muted);">Waiting for server decode</small>
         </div>
@@ -152,6 +153,7 @@
       </div>
       <div id="subtab-cw" class="sub-tab-panel" style="display:none;">
         <div class="cw-controls">
+          <button id="cw-decode-toggle-btn" type="button">Enable CW</button>
           <button id="cw-clear-btn" type="button">Clear</button>
           <small id="cw-status" style="color:var(--text-muted);">Waiting for server decode</small>
           <div id="cw-signal-indicator" class="cw-signal-off"></div>
diff --git a/src/trx-client/trx-frontend/trx-frontend-http/assets/web/plugins/aprs.js b/src/trx-client/trx-frontend/trx-frontend-http/assets/web/plugins/aprs.js
index 389c4f7..5dcdde0 100644
--- a/src/trx-client/trx-frontend/trx-frontend-http/assets/web/plugins/aprs.js
+++ b/src/trx-client/trx-frontend/trx-frontend-http/assets/web/plugins/aprs.js
@@ -72,10 +72,15 @@ function addAprsPacket(pkt) {
   }
 }
 
-document.getElementById("aprs-clear-btn").addEventListener("click", () => {
+document.getElementById("aprs-decode-toggle-btn").addEventListener("click", async () => {
+  try { await postPath("/toggle_aprs_decode"); } catch (e) { console.error("APRS toggle failed", e); }
+});
+
+document.getElementById("aprs-clear-btn").addEventListener("click", async () => {
   aprsPacketsEl.innerHTML = "";
   aprsPacketHistory = [];
   saveSetting("aprsPackets", []);
+  try { await postPath("/clear_aprs_decode"); } catch (e) { console.error("APRS clear failed", e); }
 });
 
 // Restore saved packets and map markers on page load
diff --git a/src/trx-client/trx-frontend/trx-frontend-http/assets/web/plugins/cw.js b/src/trx-client/trx-frontend/trx-frontend-http/assets/web/plugins/cw.js
index 8a03536..1b5820f 100644
--- a/src/trx-client/trx-frontend/trx-frontend-http/assets/web/plugins/cw.js
+++ b/src/trx-client/trx-frontend/trx-frontend-http/assets/web/plugins/cw.js
@@ -6,8 +6,13 @@ const cwToneInput = document.getElementById("cw-tone");
 const cwSignalIndicator = document.getElementById("cw-signal-indicator");
 const CW_MAX_LINES = 200;
 
-document.getElementById("cw-clear-btn").addEventListener("click", () => {
+document.getElementById("cw-decode-toggle-btn").addEventListener("click", async () => {
+  try { await postPath("/toggle_cw_decode"); } catch (e) { console.error("CW toggle failed", e); }
+});
+
+document.getElementById("cw-clear-btn").addEventListener("click", async () => {
   cwOutputEl.innerHTML = "";
+  try { await postPath("/clear_cw_decode"); } catch (e) { console.error("CW clear failed", e); }
 });
 
 // --- Server-side CW decode handler ---
diff --git a/src/trx-client/trx-frontend/trx-frontend-http/src/api.rs b/src/trx-client/trx-frontend/trx-frontend-http/src/api.rs
index 4b76640..222ec52 100644
--- a/src/trx-client/trx-frontend/trx-frontend-http/src/api.rs
+++ b/src/trx-client/trx-frontend/trx-frontend-http/src/api.rs
@@ -264,6 +264,38 @@ pub async fn set_tx_limit(
     send_command(&rig_tx, RigCommand::SetTxLimit(query.limit)).await
 }
 
+#[post("/toggle_aprs_decode")]
+pub async fn toggle_aprs_decode(
+    state: web::Data<watch::Receiver<RigState>>,
+    rig_tx: web::Data<mpsc::Sender<RigRequest>>,
+) -> Result<HttpResponse, Error> {
+    let enabled = state.get_ref().borrow().aprs_decode_enabled;
+    send_command(&rig_tx, RigCommand::SetAprsDecodeEnabled(!enabled)).await
+}
+
+#[post("/toggle_cw_decode")]
+pub async fn toggle_cw_decode(
+    state: web::Data<watch::Receiver<RigState>>,
+    rig_tx: web::Data<mpsc::Sender<RigRequest>>,
+) -> Result<HttpResponse, Error> {
+    let enabled = state.get_ref().borrow().cw_decode_enabled;
+    send_command(&rig_tx, RigCommand::SetCwDecodeEnabled(!enabled)).await
+}
+
+#[post("/clear_aprs_decode")]
+pub async fn clear_aprs_decode(
+    rig_tx: web::Data<mpsc::Sender<RigRequest>>,
+) -> Result<HttpResponse, Error> {
+    send_command(&rig_tx, RigCommand::ResetAprsDecoder).await
+}
+
+#[post("/clear_cw_decode")]
+pub async fn clear_cw_decode(
+    rig_tx: web::Data<mpsc::Sender<RigRequest>>,
+) -> Result<HttpResponse, Error> {
+    send_command(&rig_tx, RigCommand::ResetCwDecoder).await
+}
+
 pub fn configure(cfg: &mut web::ServiceConfig) {
     cfg.service(index)
         .service(status_api)
@@ -277,6 +309,10 @@ pub fn configure(cfg: &mut web::ServiceConfig) {
         .service(set_mode)
         .service(set_ptt)
         .service(set_tx_limit)
+        .service(toggle_aprs_decode)
+        .service(toggle_cw_decode)
+        .service(clear_aprs_decode)
+        .service(clear_cw_decode)
         .service(crate::server::audio::audio_ws)
         .service(favicon)
         .service(logo)
@@ -405,6 +441,8 @@ async fn wait_for_view(mut rx: watch::Receiver<RigState>) -> Result<RigSnapshot,
         server_version: state.server_version,
         server_latitude: state.server_latitude,
         server_longitude: state.server_longitude,
+        aprs_decode_enabled: state.aprs_decode_enabled,
+        cw_decode_enabled: state.cw_decode_enabled,
     })
 }