Kronecker/NotesAgentic
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 10 Wiki Activity

feat(frontend): 第二阶段前端审阅修复（Trace 建树 / 主题包 CSS / 引用定位 / 插件命令） #23

Closed
saint wants to merge 0 commits from feat/frontend-phase2-themes-trace-mermaid into main
pull from: feat/frontend-phase2-themes-trace-mermaid
merge into: :main
Conversation 3 Commits - Files Changed -
saint commented 2026-09-05 10:08:00 +08:00
Collaborator
Copy Link
Copy Source

背景

上一版前端第二阶段 PR（#18）被审阅关闭，结论「请求修改，暂不合并」，共 6 项 P2 问题。本 PR 在同一分支上逐项修复并补充回归测试，功能范围与 #18 一致（Agent Trace 可视化、主题清单/导入、Mermaid 渲染、插件命令与设置 UI）。

审阅版本 12869b5，本次提交 f273fef。

审阅问题修复（6 项 P2）

1. 主题导入未读取用户提供的 CSS

inspectThemePackage 原先只解析 YAML 清单，ThemesView.confirmInstall 另外调用 generateThemeCss() 生成硬编码演示配色 —— 用户提供的 CSS 被整份丢弃。

  • 定义并落实单文件主题包格式：YAML 清单 + 独占一行的 --- + 主题 CSS，新增 parseThemePackage() 取出真实 CSS，安装时原样使用，已删除 generateThemeCss。
  • 浏览器端无解压能力，明确拒绝 ZIP 而不是当文本解析出乱码再报「清单无效」：按内容嗅探 PK\x03\x04 与 base64 的 UEsDB（文件选择器的 accept 只是提示，不能依赖），返回 THEME_PACKAGE_UNSUPPORTED_FORMAT。
  • accept 收窄为 .yaml,.yml,.theme，统一 readAsText。
  • CSS 安全校验提前到预览阶段：不合规的包在「预览」就被拒，不再等到用户点安装；installTheme 内保留 validateCssSafety 必须先于 applyThemeCss 的约束与注释（未校验的 CSS 一律不进页面）。
  • 预览面板新增 <details> 展示将要安装的 CSS 原文与字符数，便于用户核对。

2. 刷新页面后无法恢复自定义主题

initTheme 原先 void loadCustomThemes() 未 await，applyTheme(saved) 因主题不在 allThemes 里而静默 return，页面停在完全没有 data-theme 的裸状态。

  • applyTheme 改为返回 boolean，主题不存在时返回 false 而非静默失败。
  • 拆成两步：先同步落一个内置主题兜底（{ persist: false }，避免把用户存的自定义主题 id 冲掉，也不让首屏干等接口），await loadCustomThemes() 之后再切到真正保存的那个。
  • 保存的主题已被卸载时回退默认主题、清除失效记录，并通过新增的 themeLoadWarning 告知用户；loadCustomThemes 的 catch 不再吞掉异常。

3. Trace 建树逻辑与真实后端事件顺序不一致

核对 backend/app/agent/runtime.py 后确认：真实顺序是 ModelCallStarted → ModelCallCompleted → Usage → ToolCall/ToolResult，工具在模型调用完成之后才执行，且多个工具并发跑（asyncio.gather + Semaphore）会交错到达 —— 事件相邻性根本不可用。

  • 归属关系全部改为 id 关联：model_call_id / parent_model_call_id / tool_call_id。
  • ToolResult 不再单独成节点，而是回填对应 ToolCall 的状态与耗时，结束后不会再停在 running；结果数据合并进调用节点供展开查看。
  • ModelCallCompleted/Failed 同样只更新对应节点状态。
  • SSE 断点恢复只拿到后半段时，配不上父节点的孤立事件退回顶层而不是被丢弃；此时落单的失败 ToolResult 也按 success 显示 error，不再一律算成功。
  • 顺带修正 Usage 展示字段：后端只发累计的 token_usage，原实现读的 input_tokens/output_tokens/total_tokens 都不存在。

4. 树形视图叶子节点无法展开详情

行的 click 是 children.length && toggleExpand(id)，而详情 v-if 又要求 children.length === 0 —— 两个条件互斥，叶子详情永远打不开。

  • 拆成 expandedNodes（子树展开）与 detailNodes（查看本节点数据）两个独立状态集合，任何节点含叶子都能查看自身数据。
  • 展开箭头改为独立 <button> + @click.stop + aria-label；行本身 role="button" tabindex="0"，支持 Enter/Space，aria-expanded 跟随详情开合。

5. 引用点击未接入笔记定位流程

  • 抽出 navigateToCitation()（纯函数 + 依赖注入，可脱离 Vue 上下文测试）与 useCitationNavigation() 组合式封装。
  • 调用顺序固化并注明原因：必须先 await loadFile 再 highlightBlock —— stores/editor.ts 的 loadFile 末尾会把 highlightBlockId 清空，顺序反了高亮会被自己冲掉；loadFile 失败时抛出且不跳转，避免把用户从未保存的编辑器里弹走。
  • AgentView 接上 @open-citation 并把失败原因显示给用户；顺带消除原有三处重复实现：ChatView 改为委托同一函数，AppShell 里那份既无调用方又漏了 loadFile 的副本一并修正。
  • Trace 引用节点新增「定位」按钮。

6. 新命令页缺少参数输入和实际效果执行

排查后发现 PluginsView 的命令页是 PluginMcpPanel 已有 schema 驱动表单的劣化副本，因此按建议复用而非各自改进：

  • 抽出共享组件 PluginCommandPanel.vue（loadVersion 防竞态、重新加载时重置表单避免用旧值提交非法参数、when 条件判断），PluginMcpPanel 与 PluginsView 均改为复用，删除两份重复实现与相关孤立 CSS。
  • 新增纯逻辑模块 pluginCommandForm.ts：commandFields / initialArguments / coerceArgument / missingRequiredFields / cleanArguments / applyCommandEffect。
  • effect 真的执行：navigate 走路由白名单（未知路由只提示，不会 router.push(undefined)）、refresh 按 scope 实际派发（新增 workspaceStore.refreshFileTree()），不再只拼成说明文本。
  • 同时修掉原 PluginMcpPanel 里两个潜在问题：必填字段标了「必填」却无客户端校验（只能靠后端拒绝才知道），以及布尔下拉框显示「否」但参数对象里没有该键 —— UI 与实际提交值不一致。

测试

  • 回归测试 +64 项（相关 spec 由 25 项增至 89 项）：
    • traceService.spec.ts —— 按真实后端事件顺序重写建树用例，覆盖并发工具交错到达、ToolResult 回填状态、SSE 断点恢复孤立事件。
    • theme.spec.ts —— 覆盖 await 后恢复、首屏兜底不覆盖已存 id、主题失效回退与告警、列表加载失败告警。
    • themePackageService.spec.ts —— 新增 parseThemePackage 全套用例（真实 CSS 提取、CSS 内含 --- 不被截断、CRLF、缺分隔行、空 CSS、ZIP 与 base64 ZIP 拒绝），并补「注入的 CSS 与包内内容逐字一致」。
    • 新增 pluginCommandForm.spec.ts、useCitationNavigation.spec.ts、TraceTimeline.spec.ts。
  • 对第 2、3、4 项做了变异验证：把修复分别回退成原写法后重跑，7 个对应用例确实失败，确认测试不是空转。
  • 全量：vue-tsc -b --force 通过，vitest 32 个文件 / 182 项全部通过，vite build 成功。

已知项 / 待确认

  • 仍存在大体积 chunk 警告（MarkdownContent 约 1.5 MB，主因 mermaid + katex）。可改为动态 import 拆分，但会影响首次渲染时序，倾向单独一个 PR 处理，本轮未动。
  • tsconfig.app.json 的 skipLibCheck: true 是绕开 mermaid 打包缺陷（它把 type-fest 泄漏进发布类型却只声明为自身 devDependency）。另一条路是自行补 type-fest@^4 devDependency（^5 需要 ES2024，与本项目 ES2022 目标冲突）。听审阅意见取舍。
  • 尚未进行真实浏览器端到端联调（与 #18 相同，仍是待补项）。

范围

仅 frontend/src 下第二阶段前端相关文件，未改动 backend 与契约实现（contracts/index.ts 仅新增 ThemePackageInspection.css 字段与两个主题错误码）。

## 背景 上一版前端第二阶段 PR（#18）被审阅关闭，结论「请求修改，暂不合并」，共 6 项 P2 问题。本 PR 在同一分支上逐项修复并补充回归测试，功能范围与 #18 一致（Agent Trace 可视化、主题清单/导入、Mermaid 渲染、插件命令与设置 UI）。 审阅版本 `12869b5`，本次提交 `f273fef`。 ## 审阅问题修复（6 项 P2） **1. 主题导入未读取用户提供的 CSS** `inspectThemePackage` 原先只解析 YAML 清单，`ThemesView.confirmInstall` 另外调用 `generateThemeCss()` 生成硬编码演示配色 —— 用户提供的 CSS 被整份丢弃。 - 定义并落实单文件主题包格式：**YAML 清单 + 独占一行的 `---` + 主题 CSS**，新增 `parseThemePackage()` 取出真实 CSS，安装时原样使用，已删除 `generateThemeCss`。 - 浏览器端无解压能力，**明确拒绝 ZIP** 而不是当文本解析出乱码再报「清单无效」：按内容嗅探 `PK\x03\x04` 与 base64 的 `UEsDB`（文件选择器的 `accept` 只是提示，不能依赖），返回 `THEME_PACKAGE_UNSUPPORTED_FORMAT`。 - `accept` 收窄为 `.yaml,.yml,.theme`，统一 `readAsText`。 - **CSS 安全校验提前到预览阶段**：不合规的包在「预览」就被拒，不再等到用户点安装；`installTheme` 内保留 `validateCssSafety` 必须先于 `applyThemeCss` 的约束与注释（未校验的 CSS 一律不进页面）。 - 预览面板新增 `<details>` 展示将要安装的 CSS 原文与字符数，便于用户核对。 **2. 刷新页面后无法恢复自定义主题** `initTheme` 原先 `void loadCustomThemes()` 未 await，`applyTheme(saved)` 因主题不在 `allThemes` 里而静默 `return`，页面停在完全没有 `data-theme` 的裸状态。 - `applyTheme` 改为返回 `boolean`，主题不存在时返回 `false` 而非静默失败。 - 拆成两步：**先同步落一个内置主题兜底**（`{ persist: false }`，避免把用户存的自定义主题 id 冲掉，也不让首屏干等接口），`await loadCustomThemes()` 之后再切到真正保存的那个。 - 保存的主题已被卸载时回退默认主题、清除失效记录，并通过新增的 `themeLoadWarning` 告知用户；`loadCustomThemes` 的 catch 不再吞掉异常。 **3. Trace 建树逻辑与真实后端事件顺序不一致** 核对 `backend/app/agent/runtime.py` 后确认：真实顺序是 `ModelCallStarted → ModelCallCompleted → Usage → ToolCall/ToolResult`，工具在模型调用**完成之后**才执行，且多个工具并发跑（`asyncio.gather` + `Semaphore`）会交错到达 —— 事件相邻性根本不可用。 - 归属关系全部改为 id 关联：`model_call_id` / `parent_model_call_id` / `tool_call_id`。 - `ToolResult` 不再单独成节点，而是**回填对应 `ToolCall` 的状态与耗时**，结束后不会再停在 `running`；结果数据合并进调用节点供展开查看。 - `ModelCallCompleted/Failed` 同样只更新对应节点状态。 - SSE 断点恢复只拿到后半段时，配不上父节点的孤立事件**退回顶层而不是被丢弃**；此时落单的失败 `ToolResult` 也按 `success` 显示 error，不再一律算成功。 - 顺带修正 Usage 展示字段：后端只发累计的 `token_usage`，原实现读的 `input_tokens/output_tokens/total_tokens` 都不存在。 **4. 树形视图叶子节点无法展开详情** 行的 click 是 `children.length && toggleExpand(id)`，而详情 `v-if` 又要求 `children.length === 0` —— 两个条件互斥，叶子详情永远打不开。 - 拆成 `expandedNodes`（子树展开）与 `detailNodes`（查看本节点数据）两个独立状态集合，任何节点含叶子都能查看自身数据。 - 展开箭头改为独立 `<button>` + `@click.stop` + `aria-label`；行本身 `role="button" tabindex="0"`，支持 Enter/Space，`aria-expanded` 跟随详情开合。 **5. 引用点击未接入笔记定位流程** - 抽出 `navigateToCitation()`（纯函数 + 依赖注入，可脱离 Vue 上下文测试）与 `useCitationNavigation()` 组合式封装。 - **调用顺序固化并注明原因**：必须先 `await loadFile` 再 `highlightBlock` —— `stores/editor.ts` 的 `loadFile` 末尾会把 `highlightBlockId` 清空，顺序反了高亮会被自己冲掉；`loadFile` 失败时抛出且不跳转，避免把用户从未保存的编辑器里弹走。 - `AgentView` 接上 `@open-citation` 并把失败原因显示给用户；顺带消除原有三处重复实现：`ChatView` 改为委托同一函数，`AppShell` 里那份既无调用方又漏了 `loadFile` 的副本一并修正。 - Trace 引用节点新增「定位」按钮。 **6. 新命令页缺少参数输入和实际效果执行** 排查后发现 `PluginsView` 的命令页是 `PluginMcpPanel` 已有 schema 驱动表单的**劣化副本**，因此按建议复用而非各自改进： - 抽出共享组件 `PluginCommandPanel.vue`（`loadVersion` 防竞态、重新加载时重置表单避免用旧值提交非法参数、`when` 条件判断），`PluginMcpPanel` 与 `PluginsView` 均改为复用，删除两份重复实现与相关孤立 CSS。 - 新增纯逻辑模块 `pluginCommandForm.ts`：`commandFields` / `initialArguments` / `coerceArgument` / `missingRequiredFields` / `cleanArguments` / `applyCommandEffect`。 - **effect 真的执行**：`navigate` 走路由白名单（未知路由只提示，不会 `router.push(undefined)`）、`refresh` 按 scope 实际派发（新增 `workspaceStore.refreshFileTree()`），不再只拼成说明文本。 - 同时修掉原 `PluginMcpPanel` 里两个潜在问题：必填字段标了「必填」却无客户端校验（只能靠后端拒绝才知道），以及布尔下拉框显示「否」但参数对象里没有该键 —— UI 与实际提交值不一致。 ## 测试 - 回归测试 **+64 项**（相关 spec 由 25 项增至 89 项）： - `traceService.spec.ts` —— 按真实后端事件顺序重写建树用例，覆盖并发工具交错到达、`ToolResult` 回填状态、SSE 断点恢复孤立事件。 - `theme.spec.ts` —— 覆盖 await 后恢复、首屏兜底不覆盖已存 id、主题失效回退与告警、列表加载失败告警。 - `themePackageService.spec.ts` —— 新增 `parseThemePackage` 全套用例（真实 CSS 提取、CSS 内含 `---` 不被截断、CRLF、缺分隔行、空 CSS、ZIP 与 base64 ZIP 拒绝），并补「注入的 CSS 与包内内容逐字一致」。 - 新增 `pluginCommandForm.spec.ts`、`useCitationNavigation.spec.ts`、`TraceTimeline.spec.ts`。 - 对第 2、3、4 项做了**变异验证**：把修复分别回退成原写法后重跑，7 个对应用例确实失败，确认测试不是空转。 - 全量：`vue-tsc -b --force` 通过，`vitest` **32 个文件 / 182 项全部通过**，`vite build` 成功。 ## 已知项 / 待确认 - 仍存在大体积 chunk 警告（`MarkdownContent` 约 1.5 MB，主因 mermaid + katex）。可改为动态 import 拆分，但会影响首次渲染时序，倾向单独一个 PR 处理，本轮未动。 - `tsconfig.app.json` 的 `skipLibCheck: true` 是绕开 mermaid 打包缺陷（它把 `type-fest` 泄漏进发布类型却只声明为自身 devDependency）。另一条路是自行补 `type-fest@^4` devDependency（`^5` 需要 ES2024，与本项目 ES2022 目标冲突）。听审阅意见取舍。 - 尚未进行真实浏览器端到端联调（与 #18 相同，仍是待补项）。 ## 范围 仅 `frontend/src` 下第二阶段前端相关文件，未改动 backend 与契约实现（`contracts/index.ts` 仅新增 `ThemePackageInspection.css` 字段与两个主题错误码）。
Kronecker commented 2026-09-05 15:32:31 +08:00
Owner
Copy Link
Copy Source

审阅结论：请求修改，暂不合并

审阅 PR #23，版本 f273fef，目标分支 main 为 1c7b5b4。

本轮发现两个已复现的问题，另有 8 个文件存在合并冲突。建议完成以下修复并解决冲突后，再进行合并验证。

[P2] 未启用主题的 CSS 仍会影响当前页面

位置：frontend/src/services/themePackageService.ts:235–240

安装主题时会立即注入 CSS，加载主题列表时又会注入所有已安装主题的 CSS。当前校验仅对缺少主题作用域给出警告，因此包内的 body、:root 等规则会直接影响当前页面，即使用户尚未启用该主题。

已复现：

  1. 当前主题保持为 light。
  2. 安装包含 body { color: rgb(1, 2, 3) !important; } 的自定义主题。
  3. 自定义主题未启用，但页面 body 颜色已发生变化。

切回内置主题也不会移除这些规则，多个已安装主题之间可能互相覆盖。

建议只挂载当前主题的 CSS，并在切换时移除旧主题样式；或者强制隔离主题包中的所有规则，确保未启用主题不会影响页面。

建议补充回归测试：

  • 安装未启用主题不改变当前页面样式。
  • 切换回内置主题后，自定义主题样式不再生效。
  • 多个已安装主题之间不会互相污染。

[P2] 保存响应会错误清除请求期间的新编辑状态

位置：frontend/src/features/plugins/PluginSettingsPanel.vue:56–62

设置保存期间输入框仍可编辑，但请求成功后会无条件执行 hasChanges = false。这会把请求发出后产生的新编辑错误地标记为已保存。

已复现：

  1. 输入 first edit 并点击保存。
  2. 保存请求尚未返回时，将内容改为 second edit。
  3. 请求返回，后端保存的是 first edit。
  4. 界面仍显示 second edit，但保存按钮已禁用，未保存提示也已消失。

此时用户无法直接保存最新输入，离开页面还可能丢失该修改。

建议使用编辑版本或提交快照判断是否仍有未保存内容；也可以在保存期间禁用编辑，确保显示状态与实际持久化结果一致。

建议补充回归测试：

  • 保存期间继续编辑，响应返回后仍保留未保存状态。
  • 再次保存时提交最新内容。
  • 保存失败后保留用户输入并允许重试。

合并冲突

当前 PR 与最新 main 存在以下 8 个文件冲突：

  • frontend/src/components/common/CommandPalette.vue
  • frontend/src/features/agent/AgentView.vue
  • frontend/src/features/chat/ChatView.vue
  • frontend/src/features/plugins/PluginMcpPanel.vue
  • frontend/src/features/plugins/PluginsView.vue
  • frontend/src/features/themes/ThemesView.vue
  • frontend/src/main.ts
  • frontend/src/stores/theme.ts

解决冲突时需保留 main 已合入的中英文支持、聊天记录持久化及会话并发修复，随后对合并结果重新运行测试和构建。

验证记录

  • PR 原分支：182 项测试通过。
  • 类型检查及生产构建通过。
  • 本轮补充的两个复现测试均失败，确认上述问题。
  • 尚未进行真实浏览器端到端验收。
  • 当前通过结果针对 PR 原分支，不代表解决冲突后的合并版本。

请修复上述问题、补充回归测试并解决合并冲突后再次请求审阅。

## 审阅结论：请求修改，暂不合并 审阅 PR #23，版本 `f273fef`，目标分支 main 为 `1c7b5b4`。 本轮发现两个已复现的问题，另有 8 个文件存在合并冲突。建议完成以下修复并解决冲突后，再进行合并验证。 ### [P2] 未启用主题的 CSS 仍会影响当前页面 位置：`frontend/src/services/themePackageService.ts:235–240` 安装主题时会立即注入 CSS，加载主题列表时又会注入所有已安装主题的 CSS。当前校验仅对缺少主题作用域给出警告，因此包内的 `body`、`:root` 等规则会直接影响当前页面，即使用户尚未启用该主题。 已复现： 1. 当前主题保持为 `light`。 2. 安装包含 `body { color: rgb(1, 2, 3) !important; }` 的自定义主题。 3. 自定义主题未启用，但页面 body 颜色已发生变化。 切回内置主题也不会移除这些规则，多个已安装主题之间可能互相覆盖。 建议只挂载当前主题的 CSS，并在切换时移除旧主题样式；或者强制隔离主题包中的所有规则，确保未启用主题不会影响页面。 建议补充回归测试： - 安装未启用主题不改变当前页面样式。 - 切换回内置主题后，自定义主题样式不再生效。 - 多个已安装主题之间不会互相污染。 ### [P2] 保存响应会错误清除请求期间的新编辑状态 位置：`frontend/src/features/plugins/PluginSettingsPanel.vue:56–62` 设置保存期间输入框仍可编辑，但请求成功后会无条件执行 `hasChanges = false`。这会把请求发出后产生的新编辑错误地标记为已保存。 已复现： 1. 输入 `first edit` 并点击保存。 2. 保存请求尚未返回时，将内容改为 `second edit`。 3. 请求返回，后端保存的是 `first edit`。 4. 界面仍显示 `second edit`，但保存按钮已禁用，未保存提示也已消失。 此时用户无法直接保存最新输入，离开页面还可能丢失该修改。 建议使用编辑版本或提交快照判断是否仍有未保存内容；也可以在保存期间禁用编辑，确保显示状态与实际持久化结果一致。 建议补充回归测试： - 保存期间继续编辑，响应返回后仍保留未保存状态。 - 再次保存时提交最新内容。 - 保存失败后保留用户输入并允许重试。 ### 合并冲突 当前 PR 与最新 main 存在以下 8 个文件冲突： - `frontend/src/components/common/CommandPalette.vue` - `frontend/src/features/agent/AgentView.vue` - `frontend/src/features/chat/ChatView.vue` - `frontend/src/features/plugins/PluginMcpPanel.vue` - `frontend/src/features/plugins/PluginsView.vue` - `frontend/src/features/themes/ThemesView.vue` - `frontend/src/main.ts` - `frontend/src/stores/theme.ts` 解决冲突时需保留 main 已合入的中英文支持、聊天记录持久化及会话并发修复，随后对合并结果重新运行测试和构建。 ### 验证记录 - PR 原分支：182 项测试通过。 - 类型检查及生产构建通过。 - 本轮补充的两个复现测试均失败，确认上述问题。 - 尚未进行真实浏览器端到端验收。 - 当前通过结果针对 PR 原分支，不代表解决冲突后的合并版本。 请修复上述问题、补充回归测试并解决合并冲突后再次请求审阅。
Kronecker closed this pull request 2026-09-05 15:32:31 +08:00
Kronecker commented 2026-09-05 17:42:48 +08:00
Owner
Copy Link
Copy Source

变更说明

整合最新 main，并修复 frontend phase2 前两轮审阅发现的问题。

主题管理

  • 安装和加载主题列表时不再注入 CSS，仅挂载当前启用主题的样式。
  • 切换主题时移除旧样式，返回内置主题后不残留自定义 CSS。
  • 未安装的社区主题可在隔离预览窗口中展示实际 CSS，不安装主题、不修改当前主题。
  • 移除预览结束后的延迟回滚逻辑，避免覆盖用户后续选择。

插件设置

  • 普通设置保存使用编辑版本判断，保留请求期间产生的新修改。
  • 密钥保存使用提交快照，仅清空未变化的输入。
  • 保存失败保留输入并允许重试，阻止重复提交。
  • 切换插件或卸载组件后忽略旧请求响应。

与 main 整合

  • 解决 9 个冲突文件。
  • 保留 main 的聊天持久化、会话并发修复、中英文支持及完整 Shiki 语言与图标能力。
  • 保留 phase2 的 Trace 可视化、引用导航、主题包、Mermaid 和共享插件命令表单。

验证

  • 40 个前端测试文件、233 项测试通过。
  • 类型检查和生产构建通过。
  • 回归测试覆盖主题样式隔离、全部社区主题预览、保存期间继续编辑、失败重试和旧响应隔离。
  • 浏览器验证主题切换、预览窗口隔离、延迟保存，以及 Mermaid 与 Shiki 渲染。

限制

  • 构建仍有大体积 chunk 提示。
  • 插件请求的浏览器验证使用测试接口，未执行生产插件后端端到端验收。
## 变更说明 整合最新 main，并修复 frontend phase2 前两轮审阅发现的问题。 ### 主题管理 - 安装和加载主题列表时不再注入 CSS，仅挂载当前启用主题的样式。 - 切换主题时移除旧样式，返回内置主题后不残留自定义 CSS。 - 未安装的社区主题可在隔离预览窗口中展示实际 CSS，不安装主题、不修改当前主题。 - 移除预览结束后的延迟回滚逻辑，避免覆盖用户后续选择。 ### 插件设置 - 普通设置保存使用编辑版本判断，保留请求期间产生的新修改。 - 密钥保存使用提交快照，仅清空未变化的输入。 - 保存失败保留输入并允许重试，阻止重复提交。 - 切换插件或卸载组件后忽略旧请求响应。 ### 与 main 整合 - 解决 9 个冲突文件。 - 保留 main 的聊天持久化、会话并发修复、中英文支持及完整 Shiki 语言与图标能力。 - 保留 phase2 的 Trace 可视化、引用导航、主题包、Mermaid 和共享插件命令表单。 ## 验证 - 40 个前端测试文件、233 项测试通过。 - 类型检查和生产构建通过。 - 回归测试覆盖主题样式隔离、全部社区主题预览、保存期间继续编辑、失败重试和旧响应隔离。 - 浏览器验证主题切换、预览窗口隔离、延迟保存，以及 Mermaid 与 Shiki 渲染。 ## 限制 - 构建仍有大体积 chunk 提示。 - 插件请求的浏览器验证使用测试接口，未执行生产插件后端端到端验收。
Kronecker reopened this pull request 2026-09-05 17:42:48 +08:00
Kronecker commented 2026-09-05 17:43:07 +08:00
Owner
Copy Link
Copy Source

审阅结论：同意合并。

此前提出的主题 CSS 泄漏、保存状态竞态、密钥新输入丢失和社区主题预览无效问题均已修复，并补充回归测试。

本轮复审未发现新的阻塞问题。最新 main 的聊天持久化、会话并发修复、中英文支持和 Shiki 能力得到保留。

验证结果：233 项前端测试、类型检查及生产构建通过；浏览器已验证主题隔离、实际预览与保存行为。

生产插件后端端到端验收尚未覆盖，构建体积提示可作为后续优化项。

审阅结论：同意合并。 此前提出的主题 CSS 泄漏、保存状态竞态、密钥新输入丢失和社区主题预览无效问题均已修复，并补充回归测试。 本轮复审未发现新的阻塞问题。最新 main 的聊天持久化、会话并发修复、中英文支持和 Shiki 能力得到保留。 验证结果：233 项前端测试、类型检查及生产构建通过；浏览器已验证主题隔离、实际预览与保存行为。 生产插件后端端到端验收尚未覆盖，构建体积提示可作为后续优化项。
Kronecker closed this pull request 2026-09-05 17:43:07 +08:00
Kronecker reopened this pull request 2026-09-05 17:43:11 +08:00
Kronecker closed this pull request 2026-09-05 17:45:12 +08:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
No items
No labels
Milestone
No items
No Milestone
Projects
Clear projects
No projects
Assignees
Clear assignees
saint yxx Kronecker (Kronecker)
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Kronecker/NotesAgentic#23