Skip to content

Add client tool continuation policy#793

Merged
blove merged 1 commit into
mainfrom
blove/client-tools-m5-continuation-policy
Jul 8, 2026
Merged

Add client tool continuation policy#793
blove merged 1 commit into
mainfrom
blove/client-tools-m5-continuation-policy

Conversation

@blove

@blove blove commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add a max-turn continuation policy with typed limit diagnostics for client tools
  • surface typed lifecycle props to view/ask components while preserving existing status props
  • extend the client-tools cockpit demos with terminal view coverage and abort-aware local tool scaffolding

Verification

  • npm run generate-api-docs
  • NX_DAEMON=false npx nx test chat --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx run chat:type-tests --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx lint chat --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx build chat --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx test ag-ui --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx test langgraph --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx build cockpit-ag-ui-client-tools-angular --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx build cockpit-langgraph-client-tools-angular --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx e2e cockpit-ag-ui-client-tools-angular --skip-nx-cache --outputStyle=static
  • NX_DAEMON=false npx nx e2e cockpit-langgraph-client-tools-angular --skip-nx-cache --outputStyle=static
  • git diff --check

Notes

  • chat lint exits 0 with existing warnings in type-spec and spec files
  • cockpit builds exit 0 with existing bundle budget/CommonJS warnings

@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 10:40pm

Request Review

@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 enabled auto-merge (squash) July 8, 2026 22:37
@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 622b07c into main Jul 8, 2026
23 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