Skip to content

feat: add client tool abort signal#786

Merged
blove merged 1 commit into
mainfrom
blove/client-tools-m2-abort-signal
Jul 8, 2026
Merged

feat: add client tool abort signal#786
blove merged 1 commit into
mainfrom
blove/client-tools-m2-abort-signal

Conversation

@blove

@blove blove commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary:

  • Add FunctionToolHandlerContext with an AbortSignal for function client-tool handlers.
  • Pass handler context through executeFunctionTool while preserving existing one-argument handlers.
  • Abort in-flight function client tools on agent stop and injector teardown, and suppress resolve after abort.
  • Regenerate chat API docs for the new public type.

Test Plan:

  • npx vitest run src/lib/client-tools/execute.spec.ts src/lib/client-tools/tools.type-spec.ts --config vite.config.mts (from libs/chat; red then green)
  • npx vitest run src/lib/client-tools/client-tool-executor.spec.ts --config vite.config.mts (from libs/chat; red then green)
  • npm run generate-api-docs
  • npx nx test chat
  • npx nx test ag-ui
  • npx nx test langgraph
  • Lint error count: 0 for chat, ag-ui, and langgraph
  • git diff --check
  • git diff --cached --check
  • post-rebase npx nx test chat
  • post-rebase git diff --check origin/main...HEAD

@vercel

vercel Bot commented Jul 8, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated (UTC)
threadplane Ready Ready Preview, Comment Jul 8, 2026 3:21pm

Request Review

@blove blove enabled auto-merge (squash) July 8, 2026 15:08
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@blove blove merged commit 1e5a637 into main Jul 8, 2026
46 of 56 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