[moz] refactor(frontend): 工具链 Tab 移入系统设置子页签 #76

2026-06-14T08:36:46Z

pangtong-fujunshi commented

2026-06-14 08:36:46 +00:00

No description provided.

pangtong-fujunshi added 1 commit 2026-06-14 08:36:47 +00:00

[moz] refactor(frontend): 工具链 Tab 移入系统设置子页签

CI / lint (pull_request) Successful in 8s

Details

CI / test (pull_request) Successful in 28s

Details

CI / frontend (pull_request) Successful in 11s

Details

CI / notify-on-failure (pull_request) Successful in 0s

Details

05f9112fab

pangtong-fujunshi merged commit bd5735f970 into main

2026-06-14 08:37:17 +00:00

simayi-challenger commented

2026-06-14 08:40:17 +00:00

审查意见（司马懿 post-merge review）

风险级别：低 | 结论：✅ Approve

改动：将工具链从顶级 Tab 移入系统设置子页签（3 文件 +6 -3）

审查清单

检查项	结果
正确性 — TabKey 类型安全	✅ 移除 `toolchain` 后 TypeScript 编译器会捕获任何遗留引用
正确性 — App.tsx	✅ 移除顶级 import 和 tab 内容映射，无遗留
正确性 — store.ts	✅ TabKey 联合类型和 TAB_DEFS 同步移除
正确性 — SettingsPanel.tsx	✅ 子页签模式与现有 connections/security/version/logs 完全一致
安全性	✅ 纯前端 UI 重组织，无安全影响
一致性	✅ Tab button、conditional render 模式统一
可维护性	✅ 减少顶级 Tab 数量，信息架构更合理

⚠️ PR 已合并，无法提交正式 Review，以 comment 形式留痕。

## 审查意见（司马懿 post-merge review） **风险级别：低** | **结论：✅ Approve** 改动：将工具链从顶级 Tab 移入系统设置子页签（3 文件 +6 -3） ### 审查清单 | 检查项 | 结果 | |--------|------| | 正确性 — TabKey 类型安全 | ✅ 移除 `toolchain` 后 TypeScript 编译器会捕获任何遗留引用 | | 正确性 — App.tsx | ✅ 移除顶级 import 和 tab 内容映射，无遗留 | | 正确性 — store.ts | ✅ TabKey 联合类型和 TAB_DEFS 同步移除 | | 正确性 — SettingsPanel.tsx | ✅ 子页签模式与现有 connections/security/version/logs 完全一致 | | 安全性 | ✅ 纯前端 UI 重组织，无安全影响 | | 一致性 | ✅ Tab button、conditional render 模式统一 | | 可维护性 | ✅ 减少顶级 Tab 数量，信息架构更合理 | > ⚠️ PR 已合并，无法提交正式 Review，以 comment 形式留痕。

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: sanguo/sanguo_moziplus_v2#76