fix(gateway,loop,tools): 失败回合必落可见回复 + knowledge 角色可信
根因（agent-191，均给出文件:行）： 1) knowledge 的 persona 在 schema 里是 optional，模型省略时工具把空串透给 memory.Retriever → 硬校验 ErrMissingPersona → 模型只能重复调用同一堵墙。 修：新增 tool/turncontext.go（loop 播种本回合角色），knowledge 依次取 「参数 → 本回合角色 → 可消化拒绝」，拒绝文案明确「重试不会成功」。 2) persistAssistantReply 对空文本直接 return，而三条正常收尾出口（空回复 / 轮次用尽 / 回合内部报错）都会给空文本 ⇒ 历史只剩 user 消息、客户端静默。 修：新增 session_turn_reply.go 的 persistTurnReply（唯一落库入口 + 按原因 的中文诚实说明）；中断 / 陈旧代次 / 静默内部流程（backthink、子代理调度） 仍不落库，真实文本行为逐字未改。 测试：knowledge_test 改断言 + turn_persona_test（新）+ session_turn_reply_test（新，7 条） 变异：去掉 loop 角色发布 ⇒ loop×2 + gateway「工具执行次数 = 0, want 2」红
This commit is contained in:
AskaEth committed
1 parent
3be70f8232
commit
18381f65fd
9 files changed
+838
-11
No files matched your search
@@ -7,17 +7,22 @@ import (
|
||||
"testing"
|
||||
|
||||
"git.yeij.top/AskaEth/Cyrene_v2/internal/config"
|
||||
"git.yeij.top/AskaEth/Cyrene_v2/internal/db"
|
||||
"git.yeij.top/AskaEth/Cyrene_v2/internal/runtime/memory"
|
||||
"git.yeij.top/AskaEth/Cyrene_v2/internal/runtime/tool"
|
||||
)
|
||||
|
||||
// stubRetriever is a fake KnowledgeRetriever for testing knowledge responses.
|
||||
// 它同时记下每次检索收到的角色，使测试能断言「角色作用域到底是谁」。
|
||||
type stubRetriever struct {
|
||||
results []memory.Result
|
||||
err error
|
||||
results []memory.Result
|
||||
err error
|
||||
personas []string
|
||||
}
|
||||
|
||||
func (s *stubRetriever) Retrieve(ctx context.Context, personaID, query string, topK int) ([]memory.Result, error) {
|
||||
_ = ctx
|
||||
s.personas = append(s.personas, personaID)
|
||||
return s.results, s.err
|
||||
}
|
||||
|
||||
@@ -28,12 +33,102 @@ func knowledgeToolForTest(t *testing.T, opts ...Option) *knowledgeTool {
|
||||
return findTool(t, buildTools(t, &config.Config{}, opts...), "knowledge").(*knowledgeTool)
|
||||
}
|
||||
|
||||
// TestKnowledge_EmptyPersonaWallsInTheStore 是切片 KB1 根因①的证据型测试：
|
||||
// 把**空角色**透给记忆检索器时，存储会以 ErrMissingPersona 硬失败（不是「查不到
|
||||
// 命中」）。修复前 knowledge 工具在模型省略可选参数 persona 时正是这么做的 ——
|
||||
// 于是模型拿到一条它无从纠正的错误（它不知道该填哪个角色 id），只能对着同一堵墙
|
||||
// 反复重试（安卓侧实测：连续两次「工具调用失败 · knowledge」）。
|
||||
func TestKnowledge_EmptyPersonaWallsInTheStore(t *testing.T) {
|
||||
t.Parallel()
|
||||
ctx := context.Background()
|
||||
conn, err := db.Open(ctx, ":memory:", silentLogger())
|
||||
if err != nil {
|
||||
t.Fatalf("db.Open: %v", err)
|
||||
}
|
||||
t.Cleanup(func() { _ = conn.Close() })
|
||||
store, err := memory.NewStore(ctx, conn, silentLogger())
|
||||
if err != nil {
|
||||
t.Fatalf("memory.NewStore: %v", err)
|
||||
}
|
||||
retriever := memory.NewRetriever(store)
|
||||
|
||||
if _, err := retriever.Retrieve(ctx, "", "W15-key-test", 3); !errors.Is(err, memory.ErrMissingPersona) {
|
||||
t.Fatalf("空角色检索的错误 = %v, want memory.ErrMissingPersona", err)
|
||||
}
|
||||
if _, err := retriever.Retrieve(ctx, "cyrene", "W15-key-test", 3); err != nil {
|
||||
t.Fatalf("有角色时不应失败: %v", err)
|
||||
}
|
||||
}
|
||||
|
||||
// TestKnowledge_MissingPersonaIsDigestibleRefusal 锁定根因①的修复：角色既不在参数
|
||||
// 里、也不在本回合 ctx 上时，工具返回一条**可消化的拒绝** —— 写明不可用并要求改道，
|
||||
// 而不是把底层的 missing persona 透出去（那正是「连撞两次同墙」的成因）。
|
||||
func TestKnowledge_MissingPersonaIsDigestibleRefusal(t *testing.T) {
|
||||
t.Parallel()
|
||||
sr := &stubRetriever{}
|
||||
kt := knowledgeToolForTest(t, WithKnowledgeRetriever(sr))
|
||||
|
||||
_, err := kt.Execute(context.Background(), map[string]any{"query": "W15-key-test"})
|
||||
if err == nil {
|
||||
t.Fatal("没有角色时应当明确拒绝，而不是静默检索")
|
||||
}
|
||||
if !errors.Is(err, ErrKnowledgeUnavailable) {
|
||||
t.Fatalf("err = %v, want ErrKnowledgeUnavailable（调用方要能以 errors.Is 分类）", err)
|
||||
}
|
||||
msg := err.Error()
|
||||
if !strings.Contains(msg, "不要重复调用本工具") {
|
||||
t.Fatalf("拒绝文案没有要求改道，模型会继续撞墙: %q", msg)
|
||||
}
|
||||
if strings.Contains(msg, "missing persona") {
|
||||
t.Fatalf("拒绝文案把底层不可纠正的错误直接透给了模型: %q", msg)
|
||||
}
|
||||
if len(sr.personas) != 0 {
|
||||
t.Fatalf("拒绝的调用不该触达检索器, got %v", sr.personas)
|
||||
}
|
||||
}
|
||||
|
||||
// TestKnowledge_UsesTurnPersonaFromContext 锁定运行时路径：模型省略 persona 参数时，
|
||||
// 工具取循环发布在 ctx 上的**本回合角色**（tool.WithTurnPersona，见
|
||||
// internal/runtime/tool/turncontext.go + loop 的发布点）。
|
||||
func TestKnowledge_UsesTurnPersonaFromContext(t *testing.T) {
|
||||
t.Parallel()
|
||||
sr := &stubRetriever{}
|
||||
kt := knowledgeToolForTest(t, WithKnowledgeRetriever(sr))
|
||||
|
||||
ctx := tool.WithTurnPersona(context.Background(), "persona-turn")
|
||||
if _, err := kt.Execute(ctx, map[string]any{"query": "回忆"}); err != nil {
|
||||
t.Fatalf("Execute: %v", err)
|
||||
}
|
||||
if len(sr.personas) != 1 || sr.personas[0] != "persona-turn" {
|
||||
t.Fatalf("检索角色 = %v, want [persona-turn]", sr.personas)
|
||||
}
|
||||
}
|
||||
|
||||
// TestKnowledge_ExplicitPersonaWins 断言显式参数优先于本回合角色：模型点名角色时
|
||||
// 尊重模型的选择（既有语义不变）。
|
||||
func TestKnowledge_ExplicitPersonaWins(t *testing.T) {
|
||||
t.Parallel()
|
||||
sr := &stubRetriever{}
|
||||
kt := knowledgeToolForTest(t, WithKnowledgeRetriever(sr))
|
||||
|
||||
ctx := tool.WithTurnPersona(context.Background(), "persona-turn")
|
||||
if _, err := kt.Execute(ctx, map[string]any{"query": "回忆", "persona": "persona-arg"}); err != nil {
|
||||
t.Fatalf("Execute: %v", err)
|
||||
}
|
||||
if len(sr.personas) != 1 || sr.personas[0] != "persona-arg" {
|
||||
t.Fatalf("检索角色 = %v, want [persona-arg]", sr.personas)
|
||||
}
|
||||
}
|
||||
|
||||
func TestKnowledge_NoKnowledgeBase(t *testing.T) {
|
||||
t.Parallel()
|
||||
kt := knowledgeToolForTest(t) // no retriever wired
|
||||
_, err := kt.Execute(context.Background(), map[string]any{"query": "cyrene"})
|
||||
if err == nil || !strings.Contains(err.Error(), "no knowledge base") {
|
||||
t.Fatalf("expected no-knowledge-base error, got %v", err)
|
||||
if !errors.Is(err, ErrKnowledgeUnavailable) {
|
||||
t.Fatalf("expected ErrKnowledgeUnavailable, got %v", err)
|
||||
}
|
||||
if !strings.Contains(err.Error(), "没有配置知识库") {
|
||||
t.Fatalf("拒绝文案没有说明原因: %v", err)
|
||||
}
|
||||
}
|
||||
|
||||
@@ -64,7 +159,8 @@ func TestKnowledge_ReturnsResults(t *testing.T) {
|
||||
func TestKnowledge_EmptyResults(t *testing.T) {
|
||||
t.Parallel()
|
||||
kt := knowledgeToolForTest(t, WithKnowledgeRetriever(&stubRetriever{}))
|
||||
got, err := kt.Execute(context.Background(), map[string]any{"query": "nothing"})
|
||||
ctx := tool.WithTurnPersona(context.Background(), "p1")
|
||||
got, err := kt.Execute(ctx, map[string]any{"query": "nothing"})
|
||||
if err != nil {
|
||||
t.Fatalf("Execute: %v", err)
|
||||
}
|
||||
@@ -83,7 +179,8 @@ func TestKnowledge_EmptyResults(t *testing.T) {
|
||||
func TestKnowledge_RetrieverError(t *testing.T) {
|
||||
t.Parallel()
|
||||
kt := knowledgeToolForTest(t, WithKnowledgeRetriever(&stubRetriever{err: errors.New("boom")}))
|
||||
_, err := kt.Execute(context.Background(), map[string]any{"query": "x"})
|
||||
ctx := tool.WithTurnPersona(context.Background(), "p1")
|
||||
_, err := kt.Execute(ctx, map[string]any{"query": "x"})
|
||||
if err == nil || !strings.Contains(err.Error(), "retrieve") {
|
||||
t.Fatalf("expected retrieve error, got %v", err)
|
||||
}
|
||||
|
||||
Reference in new issue
Block a user