Skip to content

Fix chat retrieval payload guard#133

Merged
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/chat-retrieval-payload-guard
Jul 5, 2026
Merged

Fix chat retrieval payload guard#133
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/chat-retrieval-payload-guard

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • Bound model-facing merged retrieval evidence by namespace/topK and cap long retrieval text before it is passed back into Gemini.
  • Return only newly retrieved chunks/assets from each harness retrieve tool result instead of replaying the full accumulated evidence ledger.
  • Cap Knowhere response log arrays while preserving raw result/reference counts.

Verification

  • pnpm exec vitest run src/agent-harness/runtime.test.ts
  • pnpm exec vitest run src/domains/chat/index.test.ts
  • pnpm run typecheck
  • pnpm run lint
  • pnpm test

@vercel

vercel Bot commented Jul 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
knowhere-notebook-staging Ready Ready Preview, Comment Jul 5, 2026 7:10pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
knowhere-notebook Ignored Ignored Jul 5, 2026 7:10pm

Request Review

@suguanYang
suguanYang merged commit e45cb54 into staging Jul 5, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant