Skip to content

Fix chat harness finalization budget#134

Merged
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/chat-finalize-step-budget
Jul 5, 2026
Merged

Fix chat harness finalization budget#134
suguanYang merged 1 commit into
stagingfrom
fix/wangbinqi/chat-finalize-step-budget

Conversation

@suguanYang

Copy link
Copy Markdown
Contributor

Summary

  • raise the chat harness loop cap from 10 to 13 steps
  • force a finalization pass at step 12 using existing tool results
  • stop the loop once the finalize tool is called

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 8:05pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
knowhere-notebook Ignored Ignored Jul 5, 2026 8:05pm

Request Review

@suguanYang
suguanYang merged commit 98a3278 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