CodeCompanion.nvim 插件中聊天消息可见性控制的缺陷分析

在 Neovim 插件 CodeCompanion.nvim 的最新版本中发现了一个关于聊天消息可见性控制的缺陷。该缺陷影响了用户在聊天界面中对消息显示的控制能力。

问题本质

该插件提供了一个聊天功能，允许用户与 AI 进行交互。在聊天界面中，不同类型的消息被标记为不同的角色（role），包括"system"、"user"和"assistant"等。插件设计上允许通过设置 opts.visible = false 来控制某些消息是否在聊天缓冲区中显示。

然而，当前实现中存在一个逻辑缺陷：对于非"system"角色的消息（特别是"user"角色的消息），即使明确设置了 visible = false 选项，这些消息内容仍然会被强制显示在聊天缓冲区中。

技术细节分析

问题的根源在于 add_messages_to_buf(msgs) 函数的实现逻辑。该函数在处理消息时，没有对所有角色类型的消息统一应用可见性检查，而是仅对"system"角色的消息进行了特殊处理。

正确的实现应该：

1. 对所有角色的消息都检查 opts.visible 设置
2. 只有当 visible 为真或未设置时才将消息内容添加到缓冲区
3. 保持消息元数据的完整性，无论内容是否显示

影响范围

这个缺陷影响了以下使用场景：

• 希望隐藏某些用户提示的开发者
• 需要控制聊天界面信息密度的用户
• 依赖消息可见性设置来实现特定工作流的配置

解决方案

修复方案相对简单直接：在消息处理逻辑中，对所有角色类型的消息都应用相同的可见性检查规则。具体实现可以参考以下伪代码：

for _, msg in ipairs(msgs) do
    if not msg.opts or msg.opts.visible ~= false then
        -- 添加消息到缓冲区
    end
end

这种修改保持了向后兼容性，同时修复了可见性控制的缺陷。

最佳实践建议

对于插件使用者，在修复发布前可以采取以下临时解决方案：

1. 在消息发送后手动从缓冲区删除不希望显示的内容
2. 使用自定义函数包装消息发送逻辑
3. 暂时避免依赖非"system"角色的消息隐藏功能

对于插件开发者，建议在实现类似功能时：

1. 保持不同角色类型消息处理的一致性
2. 明确文档中关于可见性控制的说明
3. 考虑添加日志输出以帮助调试类似问题

这个缺陷的修复将增强插件在消息显示控制方面的灵活性和一致性，为用户提供更精确的界面定制能力。