Skip to content

feat(web): 优化聚合 tool use 展示与聊天背景设置#619

Merged
tiann merged 3 commits into
tiann:mainfrom
junxin367:feat/web-tool-group-summary-colors
May 13, 2026
Merged

feat(web): 优化聚合 tool use 展示与聊天背景设置#619
tiann merged 3 commits into
tiann:mainfrom
junxin367:feat/web-tool-group-summary-colors

Conversation

@junxin367
Copy link
Copy Markdown
Contributor

变更概述

  • 将聚合 Tool Use 卡片的顶部标题和展开行摘要改成多语言友好单行文案,避免直接暴露 raw 路径 / 完整命令
  • 新增聊天背景色本地设置:支持配置聚合 Tool Use 背景和用户消息背景,提供默认颜色 / 柔和蓝 / 柔和绿 / 柔和黄 / 原生取色器
  • 补充 grouped presentation、聊天配色 hook、设置页渲染的回归测试,并提交对应 spec / plan 文档

原因

  • 现有聚合 Tool Use 展示对路径和命令过于直出,体验不够友好
  • 需要给用户消息和聚合 Tool Use 卡片增加可配置、但不过度突出的背景色能力

影响范围

  • web/src/components/ToolCard/ToolGroupCard.tsx
  • web/src/components/ToolCard/groupedPresentation.ts
  • web/src/hooks/useChatSurfaceColors.ts
  • web/src/routes/settings/index.tsx
  • web/src/components/AssistantChat/messages/user-bubble.tsx
  • docs/superpowers/specs/*
  • docs/superpowers/plans/*

验证结果

  • cd web && bun run test -- src/components/ToolCard/groupedPresentation.test.ts src/hooks/useChatSurfaceColors.test.ts src/components/ToolCard/ToolGroupCard.test.tsx src/routes/settings/index.test.tsx
  • cd web && bun run typecheck
  • cd web && bun run build
  • 全仓 bun run test(当前仓库存在 CLI 侧历史失败,本 PR 未处理)

风险 / 回滚

  • 风险:聚合摘要的命令归类依赖轻量 heuristics,极少数命令可能落到通用描述
  • 风险:聊天背景色通过 localStorage + CSS 变量覆盖,需关注极端主题下的视觉一致性
  • 回滚:直接回退本 PR 的 3 个提交即可恢复

Reviewer Notes

  • 重点看 groupedPresentation.ts 的命令归类和文案优先级是否符合预期
  • 重点看 useChatSurfaceColors.ts 的持久化与主题切换时变量刷新逻辑
  • 本次 UI 改动未附截图,建议本地重点验收会话页聚合卡片和 Settings > Chat 新配置项

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Findings
未发现需要阻塞合并的问题。

Summary
Review mode: initial
已完整检查本次 web diff(聚合 tool use 摘要、聊天背景色设置、i18n、CSS 变量与相关测试)。残余风险主要在命令归类仍是启发式逻辑,少数复杂命令可能落到较泛化的展示文案;未发现会导致数据丢失、安全问题或明显回归的改动。

Testing

  • cd web && bun run test -- src/components/ToolCard/groupedPresentation.test.ts src/hooks/useChatSurfaceColors.test.ts src/components/ToolCard/ToolGroupCard.test.tsx src/routes/settings/index.test.tsx(28 passed)
  • cd web && bun run typecheck

HAPI Bot

@tiann tiann merged commit 60af983 into tiann:main May 13, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants