Skip to content

feat: add client tool execution claims#789

Merged
blove merged 1 commit into
mainfrom
blove/client-tools-m35-claim-before-execute
Jul 8, 2026
Merged

feat: add client tool execution claims#789
blove merged 1 commit into
mainfrom
blove/client-tools-m35-claim-before-execute

Conversation

@blove

@blove blove commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add an opt-in client tool execution guard boundary for claim-before-execute function tools.
  • Add idempotent: true action metadata to bypass durable pre-claims for safe re-run handlers.
  • Wire the guard through the function executor, coordinator, and chat component input while preserving the existing no-guard path.
  • Export the new guard/store types and regenerate chat API docs.

Test Plan

  • npx vitest run src/lib/client-tools/client-tool-execution-guard.spec.ts --config vite.config.mts
  • npx vitest run src/lib/client-tools/client-tool-executor.spec.ts --config vite.config.mts
  • npx vitest run src/lib/client-tools/client-tools-coordinator.spec.ts src/lib/compositions/chat/chat.component.client-tools.spec.ts --config vite.config.mts
  • npm run generate-api-docs
  • npx nx test chat
  • npx nx lint chat
  • npx nx build chat
  • npx nx run-many -t build --projects=chat,langgraph,ag-ui,render,a2ui,licensing,telemetry --configuration=production
  • node scripts/check-dx-coverage.mjs
  • git diff --check

@blove blove enabled auto-merge (squash) July 8, 2026 17:24

@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.

@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 5:28pm

Request Review

@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.

@blove blove merged commit a52d8ff into main Jul 8, 2026
24 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