TUI: messagesBeforeRevert/messagesFromRevert duplicate revert boundary logic

已关闭 适合新手
#56,179 1 条评论 0 个 reaction 已指派 0 人 在 GitHub 查看

维护者通常 1 天内回复

还没有人认领这个 Issue。

评估

难度
2/5
预计耗时
1-3 小时
新手友好度
75/100
Issue 类型
重构
描述清晰度
描述清楚
活跃度
活跃
技术栈
typescript
领域
frontend

调研方向

阅读 routes/session/index.tsx 第 134–174 行附近的代码,并追踪 messagesBeforeRevert 和 messagesFromRevert 如何使用 revert 边界。将它们共用的边界查找提取到一个 helper 中,然后检查会话视图在有和没有匹配的 revert 消息时的行为;当两个函数都使用共用的查找并保留当前结果时,工作即告完成。

由索引模型根据 Issue 内容生成。

描述

Summary

routes/session/index.tsx:134-174 — messagesBeforeRevert and messagesFromRevert both search for the revert boundary — duplicate logic.

Root cause

const messagesBeforeRevert = () => {
  const messageID = session()?.revert?.messageID
  const index = messages().findIndex((message) => message.id === messageID)
  return index === -1 ? messages() : messages().slice(0, index)
}
const messagesFromRevert = () => {
  const messageID = session()?.revert?.messageID
  const index = messages().findIndex((message) => message.id === messageID)
  return index === -1 ? [] : messages().slice(index)
}

Both functions find the revert boundary index independently.

Suggested fix

Extract the revert boundary index lookup into a shared helper.

主要语言
TypeScript
星标
224k
派生
30k
平均合并
6 小时 12 分钟
30 天内合并 PR
445

环境准备

从这里开始

  1. 先读完整个 Issue,再读项目的贡献指南。
  2. 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
  3. Fork 仓库,在一个分支上完成修改。
  4. 提交 Pull Request,并在描述里引用这个 Issue 编号。

anomalyco/opencode 的其他 Issue

查看 anomalyco/opencode 的全部 Issue

相似的 Issue

更多 TypeScript Issue

把新 issue 发到你的邮箱

精选适合新手参与的 GitHub issue 摘要。