fix(desktop): 优化用量历史模型与 harness 配色 - #4164
Conversation
Signed-off-by: kirozeng <kirozeng@gmail.com>
Signed-off-by: kirozeng <kirozeng@gmail.com>
Signed-off-by: kirozeng <kirozeng@gmail.com>
Signed-off-by: kirozeng <kirozeng@gmail.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
| Filename | Overview |
|---|---|
| apps/desktop/src/renderer/components/settings/usage/usageHistoryColors.ts | 新增 harness 固定身份色,并使用五个主题种子色为后续模型派生 OKLCH 色相。 |
| apps/desktop/src/renderer/components/settings/usage/UsageHistorySection.tsx | 配色顺序改由完整历史的 token 聚合结果生成,避免日期筛选导致颜色重排。 |
| apps/desktop/src/renderer/components/settings/usage/UsageTokenBars.tsx | 使用完整颜色索引保留每个已登记模型的独立柱图片段。 |
| apps/desktop/src/renderer/components/settings/usage/UsageBreakdownTables.tsx | Agent 与模型表的色块、占比条统一消费新的分类颜色映射。 |
| apps/desktop/src/renderer/components/settings/usage/tests/usageHistoryColors.test.tsx | 覆盖 1000 个派生颜色、35 个模型分段及 harness 排名反转时的颜色一致性。 |
| docs/design-evidence/2026-09-09/usage-category-colors.md | 登记配色视觉验证过程,但验证结果中的 harness computed 色值仍是旧版方案。 |
Flowchart
%%{init: {'theme': 'neutral'}}%%
flowchart LR
H[完整用量历史] --> R[按 token 聚合并排序模型]
R --> O[稳定的完整 colorOrder]
O --> T[筛选范围内模型表]
O --> B[近 30 天每日柱图]
T --> C[同一模型使用同一颜色]
B --> C
A[Agent / harness 类型] --> I[固定身份色映射]
I --> S[顶部占比条、行前色块、占比短条]
Prompt To Fix All With AI
### Issue 1
docs/design-evidence/2026-09-09/usage-category-colors.md:23
**验证色值已过期**
这里仍记录旧版青绿、靛蓝、琥珀的验证结果,但当前实现和本文前文已经改为 Claude 陶土橙、Codex 蓝、pi 青绿。这会让设计证据无法准确证明最终提交版本,请同步更新 computed 色值。
```suggestion
- Agent 顶部占比条、行前色块与占比短条一致。Light:`rgb(217, 119, 87)` / `rgb(122, 157, 255)` / `rgb(20, 184, 166)`;Dark:`rgb(217, 119, 87)` / `rgb(122, 157, 255)` / `rgb(45, 212, 191)`。
```
---
For each issue above, determine whether it is valid and should be fixed. If so, fix it directly.Reviews (1): Last reviewed commit: "Merge origin/main into fix/usage-categor..." | Re-trigger Greptile
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 377baf24de
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
「验证结果」仍记录初版青绿/靛蓝/琥珀的 computed 色值,与最终实现不一致; 按 review 反馈更新为当前值:Claude rgb(217, 119, 87)、Codex rgb(122, 157, 255)(双模式同值),pi Light rgb(20, 184, 166) / Dark rgb(45, 212, 191);初版实测值改为明确标记的历史对照。 「截图与记录」同步指向 pi 版栅格证据,登记源文件 blob 与 PR head 逐项一致的复核结论,并按治理合同 §6 如实登记 PR 附件上传与设计师 验收缺口。 本轮仅改本文档,不改运行时代码;纯文档改动按 development-workflow.md 跳过 workspace 单测。 Signed-off-by: Kirozeng <kirozeng@gmail.com>
|
Codex usage limits have been reached for code reviews. Please check with the admins of this repo to increase the limits by adding credits. |
|
本轮已修复 review 指出的证据过期问题(commit 改了什么:
「截图与记录」一节同步改为指向 pi 版栅格证据文件,并补上 blob 复核结论。 核对过的证据:上述数值与 视觉验收还差两步人工动作(合入前):Light/Dark 真实 Electron 截图(35 个模拟模型、三 harness、40 天数据)已在证据索引登记的忽略目录 dispatch:9e261b73d865d13c |
|
已核实本轮两条反馈,无需代码改动, CI 的「Windows unit tests 缺席」:这个 required context 是汇总门禁 job( Codex 配额通知:这条评论是 Codex 云端 code review 的用量提示,需要仓库管理员在 ChatGPT Codex 的 code review 设置里启用 credits 才能继续,与 PR 代码无关,PR 侧无需处理。 dispatch:65c1a2998cb5ade8 |
Signed-off-by: Kirozeng <kirozeng@gmail.com>
|
已把 冲突在哪:main 侧合入了 DS-6(#4135)等更新,与本分支同时改了 怎么解的:
验证: dispatch:439031655cb36ce8 |
|
已核实本轮 CI 告警,无需代码改动, 原因同上一轮:required context「Windows unit tests」是 终态已核验: dispatch:5e601cc97641d7dd |
这次改了什么
摘要
优化设置「用量历史」的分类配色:Agent/harness 的顶部占比条、行前色块和占比短条统一为 Claude 陶土橙、Codex 蓝、pi 青绿;模型超过五个后继续分配颜色,模型表和每日柱图保持对应,不再将后续模型全部显示为灰色。
变更类型
fix缺陷修复范围
UI 变化
docs/design-rules/DESIGN.md §2、§4、§5、docs/design-rules/usage-history-charts.md,以及docs/design-rules/design-governance.md §3.4、§6、§7、§13。docs/design-evidence/2026-09-09/usage-category-colors.md。当前截图为pi-light.png、pi-dark.png;源码以pi-manifest.json的 blob 绑定。图片保存在本地忽略目录,未上传 PR 附件。2026-09-09,项目所有者 / 设计负责人 kirozeng 明确确认当前配色视觉验收通过,并仅对本 PR 豁免截图上传要求;由 Agent 代记,非 AI 自我批准。验收及豁免记录。此记录更新证据索引中此前的待验收状态,不改变其他 PR 的证据要求。怎么验证的
自动验证
覆盖 1000 个颜色分配、35 个模型的片段/表格对应、Agent 排名反转,以及超过 30 天的历史模型。既有单日筛选、空白点击和最近七天强调等回归测试继续保留。
手工验证
Agent 在独立 macOS Electron 沙箱中核对 Light/Dark:三个 harness 标记位置颜色相同;35 个模型均有颜色且与柱图片段对应;单日点击筛选仍生效,柱高及四角 2px 不变。默认范围恢复为最近 30 天。按所有者要求由主 Agent 自查完整 diff,不启动额外独立审查席。
未执行的验证
风险
风险分类
影响与回滚
提交前检查
git commit -s)