[code-review] wailskit/im.go: editing a disabled IM adapter silently re-enables it (Enabled missing from #107 preserve list)

[topcheer]

Description

SaveIMAdapter (desktop/wailskit/im.go L119-183) silently flips a disabled adapter back to enabled whenever any other field is edited.

• L131: enabled := values["enabled"] != "false" — defaults to true when the key is absent.
• L155-175 (the [business-logic] desktop SaveIMAdapter loses Targets/Args/Env/AllowFrom on update — editing adapter via UI silently drops channel bindings #107 "preserve fields not set via UI" fix) preserves Args, Env, AllowFrom, OutputMode, Targets, Extra — but not Enabled.
• The edit dialog (frontend/src/components/IMManagement.tsx L168-178) sends only { platform, ...editFields } — never enabled.

Trigger scenario

1. User disables adapter via SetIMAdapterEnabled(name, false) (cost/noise/debug reasons)
2. User opens Edit dialog, changes only the token field, saves
3. values["enabled"] missing → adapterCfg.Enabled = true
4. Preserve list omits Enabled → persisted Enabled flips false → true
5. On next start the adapter is unexpectedly online

Expected vs actual

• Expected: editing a token does not change enablement (same intent as [business-logic] desktop SaveIMAdapter loses Targets/Args/Env/AllowFrom on update — editing adapter via UI silently drops channel bindings #107)
• Actual: every edit of a disabled adapter re-enables it

Fix

Add Enabled to the preserve list — e.g. only override when values["enabled"] is explicitly "true"/"false"; otherwise keep existing.Enabled.

Severity

High — deterministic silent state loss on a routine UI action; user-intent violation with operational impact (adapter comes online unexpectedly).

Verified by independent review subagent (sa-1, static analysis + full call-chain verification).

[topcheer]

已修复并推送（commit ff7a701）。

根因

SaveIMAdapter 中 enabled := values["enabled"] != "false" 在 key 缺失时默认为 true，而 #107 的保留字段列表（Args/Env/AllowFrom/OutputMode/Targets/Extra）不含 Enabled——编辑对话框只发送 { platform, ...editFields }，导致每次编辑禁用的 adapter 都会静默重新启用。

修复方案（desktop/wailskit/im.go）

Enabled 现在默认保留现有 adapter 的状态，仅当 values 中显式出现 "true"/"false" 时才覆盖：

enabled := true
if existing, exists := cfg.IM.Adapters[name]; exists {
    enabled = existing.Enabled
}
switch values["enabled"] {
case "true":  enabled = true
case "false": enabled = false
}

测试（desktop/wailskit/im_test.go）

• TestSaveIMAdapterPreservesDisabledState：禁用后仅编辑 token 字段，验证 Enabled 保持 false 且 token 更新生效
• TestSaveIMAdapterExplicitEnable：显式 enabled=true 仍可重新启用

两个包完整测试通过（-tags goolm）。