Skip to content

agentHost: distinguish tools running on other clients#326245

Open
connor4312 wants to merge 2 commits into
mainfrom
connor4312/fix-cross-client-tool-prompts
Open

agentHost: distinguish tools running on other clients#326245
connor4312 wants to merge 2 commits into
mainfrom
connor4312/fix-cross-client-tool-prompts

Conversation

@connor4312

Copy link
Copy Markdown
Member

Fixes #324112

When an Agent Host session is observed from multiple clients, client-provided tools now distinguish their owning client. Non-owning clients show a quiet inline Running <tool> on another client... Skip? progress message instead of surfacing the owner's permission prompt or invoking the tool locally.

The inline Skip action uses the same scoped markdown-action pattern as MCP authentication. Once the tool reaches a terminal state, it returns to the standard tool-call rendering.

Validation:

  • npm run typecheck-client
  • focused Agent Host/chat unit tests (131 passing)
  • client-tools and multi-client protocol integration tests (14 passing)
  • npm run valid-layers-check
  • changed-file hygiene and pre-commit hooks
Render client-provided tools owned by another connected client as cancellable progress instead of surfacing their permission prompt. Preserve ordinary tool rendering once the invocation reaches a terminal state.\n\nFixes #324112\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Match the MCP authentication action pattern by rendering Skip? in the running message paragraph while preserving keyboard and screen-reader behavior.\n\nCo-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings July 16, 2026 22:46
@connor4312
connor4312 enabled auto-merge (squash) July 16, 2026 22:47

Copilot AI 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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@connor4312 connor4312 closed this Jul 17, 2026
auto-merge was automatically disabled July 17, 2026 00:22

Pull request was closed

@connor4312 connor4312 reopened this Jul 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants