CI 检查未通过

以下 job 在本次自动化 review 时未通过，请修复：

本次自动化 review 暂缓，待 CI 全部通过后将重新处理。

Code Review：feat(guid): refresh sidebar actions and skills market entry (#1945)

变更概述

本 PR 刷新了侧栏顶部操作区域（新会话、搜索、批量管理）在折叠/展开状态下的布局和交互，新增桌面端侧栏拖拽调整宽度功能，将助手详情编辑能力内嵌到 Guid 页面的 AssistantSelectionArea 中，同时精简了 Skills Market Banner 的交互形态。改动涉及 Layout、Sider、GuidPage、AssistantSelectionArea、ConversationSearchPopover、SkillsMarketBanner 等多个模块，并移除了 remote agent 独立路径（合并到 ACP 统一路径）和 warmup 预热调用。

方案评估

结论：⚠️ 方案有缺陷

整体方案方向正确——侧栏拖拽、折叠/展开自适应、双击防抖等都是合理的 UX 改进。将 remote agent 路径合并到 ACP 统一路径也是好的简化。但 AssistantSelectionArea 从纯展示组件膨胀为包含完整编辑状态管理（5 个 modal/drawer）的重组件，且同一套 modal 树在两个分支中完整重复了约 180 行，存在明显的代码重复和职责过重问题。

问题清单

🟠 HIGH — AssistantSelectionArea 中 modal/drawer 树完整重复

文件：src/renderer/pages/guid/components/AssistantSelectionArea.tsx，第 298–389 行 与 第 449–539 行

问题代码：

// 第一处（isPresetAgent 分支，约第 298 行起）
<AssistantEditDrawer editVisible={editor.editVisible} ... />
<DeleteAssistantModal ... />
<AddSkillsModal ... />
<SkillConfirmModals ... />
<AddCustomPathModal ... />

// 第二处（列表视图分支，约第 449 行起）— 完全相同的 props
<AssistantEditDrawer editVisible={editor.editVisible} ... />
<DeleteAssistantModal ... />
<AddSkillsModal ... />
<SkillConfirmModals ... />
<AddCustomPathModal ... />

问题说明：两个 return 分支中完整复制了 ~180 行一模一样的 JSX（5 个 modal/drawer 及其所有 props）。这违反了 DRY 原则，任何 props 变更都需要在两处同步修改，极易引发不一致。

修复建议：

将 modal 树提取到组件底部的一个公共片段中，两个分支共享同一份渲染：

const modalTree = (
  <>
    {agentMessageContext}
    <AssistantEditDrawer ... />
    <DeleteAssistantModal ... />
    <AddSkillsModal ... />
    <SkillConfirmModals ... />
    <AddCustomPathModal ... />
  </>
);

// 在两个 return 中：
return (
  <div className='mt-12px w-full'>
    {/* 各自的内容 */}
    {modalTree}
  </div>
);

🟠 HIGH — SkillsMarketBanner enabled 状态硬编码 light-mode 颜色

文件：src/renderer/pages/guid/components/SkillsMarketBanner.tsx，第 93–94 行

问题代码：

style={
  enabled
    ? {
        backgroundColor: 'rgba(255, 255, 255, 0.45)',
        borderColor: 'rgba(0, 0, 0, 0.08)',
      }
    : undefined
}

问题说明：rgba(255,255,255,0.45) 和 rgba(0,0,0,0.08) 是纯白底/纯黑边的硬编码值，在深色主题下背景会显示为不协调的白色半透明块。项目约定要求使用语义 token（var(--color-*) 或 uno.config.ts 中的 token）。

修复建议：

style={
  enabled
    ? {
        backgroundColor: 'color-mix(in srgb, var(--color-bg-2) 45%, transparent)',
        borderColor: 'var(--color-border-1)',
      }
    : undefined
}

🟡 MEDIUM — Layout.tsx 拖拽未清理 body style 的边界情况

文件：src/renderer/components/layout/Layout.tsx，第 364–365 行

问题代码：

document.body.style.cursor = 'col-resize';
document.body.style.userSelect = 'none';

问题说明：如果用户在拖拽过程中鼠标移出窗口（Electron BrowserWindow 之外）并释放鼠标，mouseup 事件可能不会被接收到（取决于 Electron 的 mouseleave 行为），导致 cursor 和 userSelect 永久停留在拖拽状态。

修复建议：

在 effect 中增加 blur 事件监听作为安全清理：

const handleBlur = () => endDrag();
window.addEventListener('blur', handleBlur);
return () => {
  window.removeEventListener('mousemove', handleMouseMove);
  window.removeEventListener('mouseup', endDrag);
  window.removeEventListener('blur', handleBlur);
  endDrag();
};

🟡 MEDIUM — openAssistantDetails useCallback 依赖了未稳定化的 editor 对象

文件：src/renderer/pages/guid/components/AssistantSelectionArea.tsx，第 193 行

问题代码：

}, [agentMessage, assistants, customAgents, editor, selectedAgentInfo?.customAgentId, selectedAgentKey, t]);

问题说明：editor 是 useAssistantEditor 返回的对象，如果该 hook 每次渲染都返回新的引用（未 useMemo），则 openAssistantDetails 会在每次渲染时重建，进而触发 useLayoutEffect 重新注册，导致不必要的重渲染。应只依赖 editor.handleEdit 而非整个 editor 对象。

修复建议：

}, [agentMessage, assistants, customAgents, editor.handleEdit, selectedAgentInfo?.customAgentId, selectedAgentKey, t]);

汇总

结论

⚠️ 有条件批准 — 存在代码重复和暗色模式兼容性问题，修复后可合并。

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

CONCLUSION: CONDITIONAL
IS_CRITICAL_PATH: false
PR_NUMBER: 1945

PR Fix 验证报告

原始 PR: #1945
修复方式: 直接推送到 feat/sidebar-skills-market-ui-pr

总结： ✅ 已修复 4 个 | ❌ 未能修复 0 个