Code Review：fix(assistant): use dynamic backend list in assistant editor dropdown (#1798)

变更概述

本 PR 将 AssistantEditDrawer.tsx 中硬编码的 6 个后端选项替换为从 ACP_BACKENDS_ALL 动态生成，使所有已检测到的 ACP 后端（如 iFlow、Goose、Kimi 等）都能出现在 Main Agent 下拉列表中。同时新增了一个单元测试文件验证该逻辑。

方案评估

结论：✅ 方案合理

用 ACP_BACKENDS_ALL 作为唯一数据源生成 label，彻底消除了检测到后端但下拉缺项的问题，与项目架构一致。对未知 backend ID 回退为 id 作为 label，兜底处理得当。测试文件直接提取并测试核心映射逻辑，覆盖了关键场景。

问题清单

🔵 LOW — 双重 .map() 可合并为一次

文件：src/renderer/pages/settings/AgentSettings/AssistantManagement/AssistantEditDrawer.tsx，第 265–274 行

问题代码：

{Array.from(availableBackends)
  .map((id) => {
    const config = ACP_BACKENDS_ALL[id as keyof typeof ACP_BACKENDS_ALL];
    return { value: id, label: config?.name ?? id };
  })
  .map((opt) => (
    <Select.Option key={opt.value} value={opt.value}>
      {opt.label}
    </Select.Option>
  ))}

问题说明：两次 .map() 遍历同一数组，可以合并为一次，减少中间数组分配。

修复建议：

{Array.from(availableBackends).map((id) => {
  const config = ACP_BACKENDS_ALL[id as keyof typeof ACP_BACKENDS_ALL];
  const label = config?.name ?? id;
  return (
    <Select.Option key={id} value={id}>
      {label}
    </Select.Option>
  );
})}

汇总

结论

✅ 批准合并 — 无阻塞性问题，修复方案正确，测试覆盖充分，LOW 级别优化不影响合并。

本报告由本地 pr-review skill 生成，包含完整项目上下文，无截断限制。

CONCLUSION: APPROVED
IS_CRITICAL_PATH: false
PR_NUMBER: 1798

✅ 已自动 review，无阻塞性问题，正在触发自动合并。