From 31e52832775330e74fc584dc5864d7ec2510e078 Mon Sep 17 00:00:00 2001
From: imalasong <imalasong@qq.com>
Date: Sat, 28 Mar 2026 17:06:01 +0800
Subject: [PATCH] fix(web): merge _secret edit fields when GET omits public
 secret keys

---
 .../components/channels/channel-config-page.tsx  | 16 ++++++++++++++++
 1 file changed, 16 insertions(+)

diff --git a/web/frontend/src/components/channels/channel-config-page.tsx b/web/frontend/src/components/channels/channel-config-page.tsx
index 6af821ac9..cb93b7aaa 100644
--- a/web/frontend/src/components/channels/channel-config-page.tsx
+++ b/web/frontend/src/components/channels/channel-config-page.tsx
@@ -105,6 +105,22 @@ function buildSavePayload(
     payload[key] = value
   }
 
+  // When GET config omits secret fields (redacted), editConfig may only have `_secret`
+  // keys; the loop above skips `_` keys and never sees a public `app_secret` etc., so
+  // merge those edit-buffer values here. Only touch secrets that appear in edit state
+  // so we do not inject unrelated empty secret keys for every channel type.
+  for (const [secretKey, editKey] of Object.entries(SECRET_FIELD_MAP)) {
+    if (Object.prototype.hasOwnProperty.call(payload, secretKey)) {
+      continue
+    }
+    if (!(editKey in editConfig) && !(secretKey in editConfig)) {
+      continue
+    }
+    const incoming = asString(editConfig[editKey])
+    const stored = asString(editConfig[secretKey])
+    payload[secretKey] = incoming !== "" ? incoming : stored
+  }
+
   if (channel.name === "whatsapp_native") {
     payload.use_native = true
   }