BubbleTea 代码审查

快速参考

关键：避免误报

请先阅读 elm-architecture.md！ 最常见的审查错误是将正确模式标记为错误。

非问题（不要标记这些）

实际问题（需要标记这些）

审查清单

架构

• - [ ] Update()中无阻塞I/O（文件、网络、休眠）
• [ ] 返回 tea.Cmd 的辅助函数不被标记为阻塞
• [ ] 所有异步操作使用命令

模型与更新

• - [ ] 模型不可变（Update返回新模型，不修改原模型）
• [ ] Init返回正确的初始命令（或nil）
• [ ] Update处理所有预期的消息类型
• [ ] 处理WindowSizeMsg以实现响应式布局
• [ ] 多个命令使用tea.Batch
• [ ] 正确使用tea.Quit退出

视图与样式

• - [ ] View是纯函数（无副作用）
• [ ] Lipgloss样式定义一次，不在View中定义
• [ ] 按键绑定使用key.Matches配合help.KeyMap

组件

• - [ ] 子组件更新正确传播
• [ ] Bubbles组件使用尺寸初始化
• [ ] Huh表单通过Update循环嵌入（不使用Run()）

关键模式

模型必须不可变

go
// 错误 - 修改模型
func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
m.items = append(m.items, newItem) // 修改！
return m, nil
}

// 正确 - 返回新模型
func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
newItems := make([]Item, len(m.items)+1)
copy(newItems, m.items)
newItems[len(m.items)] = newItem
m.items = newItems
return m, nil
}

异步/IO的命令

go
// 错误 - Update中阻塞
func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
data, _ := os.ReadFile(config.json) // 阻塞UI！
m.config = parse(data)
return m, nil
}

// 正确 - 使用命令
func (m Model) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
return m, loadConfigCmd()
}

func loadConfigCmd() tea.Cmd {
return func() tea.Msg {
data, err := os.ReadFile(config.json)
if err != nil {
return errMsg{err}
}
return configLoadedMsg{parse(data)}
}
}

样式定义一次

go
// 错误 - 每次渲染创建新样式
func (m Model) View() string {
style := lipgloss.NewStyle().Bold(true).Foreground(lipgloss.Color(205))
return style.Render(Hello)
}

// 正确 - 在包级别或模型中定义样式
var titleStyle = lipgloss.NewStyle().Bold(true).Foreground(lipgloss.Color(205))

func (m Model) View() string {
return titleStyle.Render(Hello)
}

何时加载参考文档

• - 首次审查BubbleTea → elm-architecture.md（防止误报）
• 审查Update函数逻辑 → model-update.md
• 审查View函数、样式 → view-styling.md
• 审查组件层次结构 → composition.md
• 使用Bubbles组件 → bubbles-components.md

审查问题

1. 1. Update()是否没有阻塞I/O？（注意：不是cmd辅助函数是否阻塞？）
2. Update中的模型是否不可变？
3. Lipgloss样式是否定义一次，不在View中定义？
4. 是否处理了WindowSizeMsg以实现调整大小？
5. 按键绑定是否使用help.KeyMap记录？
6. Bubbles组件尺寸是否正确？