logo
7
16
WeChat Login

fix: 将 Issue 描述注入 user message,修复太子读不到 issue 描述的问题 (#67)#68

Merged

DeepSeek-TUI-pro

NPC
created 1 weeks ago
main
auto/fix-issue-67-context-in-user-message-e898a23d
Edit
OverviewCommits
2
Files changed
2
Attachments

DeepSeek-TUI-pro

NPC

问题

Closes #67

根因

Issue #67@npc/mandate-engine(太子) 后,太子只能看到 @mention 的原始文本,看不到 Issue 的描述内容。虽然 issueDetail 已预加载到 system prompt,但 LLM 对 system prompt 中嵌入的上下文关注度不足。

修复

在 src/agent.ts 中将 Issue 的标题、描述、作者、标签等信息注入到 user role 的 message 中,让 LLM 像读聊天记录一样自然获取 Issue 上下文,而非依赖 system prompt。

改动

  • 仅修改 src/agent.ts:在构建 LLM 消息时,若 ctx.issueDetail 存在,将其格式化后拼接到 user message 前面

NPC
1

@CodeBuddy 你来CR一下这个PR

1
79,568 tokens, 5.780 credits, ¥0.289
reviewed

@alibaba(宋冬冬🦕) CR 完成 ✅

评审结果:needs_modification

变更概述

本次改动(src/agent.ts,+22/-1)将 Issue/PR 描述信息从 system prompt 迁移到 user message 中注入,修复 #67(天子/角色读不到 issue 描述)。

发现的问题

级别文件说明
⚠️ warningsrc/agent.ts:52-66Issue 详情与 prompts.ts:82-97 的 system prompt 注入存在信息重复,浪费 Token 且造成上下文冗余

建议

推荐方案 A:移除 prompts.tsbuildSystemPrompt 里重复的 preloadedContext 注入逻辑,只在 user message 中保留一处注入即可。

其他方面

  • ✅ 无安全漏洞
  • ✅ 无潜在 Bug(issueDetail 做了可选判断)
  • ✅ 代码结构清晰,filter(Boolean) 处理空描述合理

@CodeBuddy 你来处理一下CR提到的问题

1
178,282 tokens, 10.510 credits, ¥0.525

@alibaba(宋冬冬🦕) CR 问题已处理 ✅

修改内容:移除 src/prompts.tsbuildSystemPrompt 里重复的 preloadedContext 注入逻辑(-17 行)。

Issue/PR 上下文信息(标题、描述、作者、标签)现在只在 src/agent.ts 的 user message 中注入一处,消除了与 system prompt 的信息重复,减少 Token 消耗且避免上下文冗余。TypeScript 编译验证通过。

is using the merge method to merge into5e1e524f
合并来自 auto/fix-issue-67-context-in-user-message-e898a23d 的合并请求 #68

NPC
referenced pull request

Successfully merged and closed

Reviewer
(宋冬冬🦕)
Assignee
None yet
Label
None yet
Participant