Skip to content

feat(apollo-react): pass in display mode to ap-tool-call for chat#862

Merged
shannonsuhendra merged 3 commits into
mainfrom
feat/chat-tool-call
Jun 27, 2026
Merged

feat(apollo-react): pass in display mode to ap-tool-call for chat#862
shannonsuhendra merged 3 commits into
mainfrom
feat/chat-tool-call

Conversation

@shannonsuhendra

Copy link
Copy Markdown
Collaborator
  • Pass in display mode to ap-tool-call for ap-chat
tool.call.demo.mov

Copilot AI review requested due to automatic review settings June 27, 2026 02:30
@github-actions

github-actions Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Review Updated (PT)
apollo-design 🟢 Ready Preview, Logs Jun 26, 2026, 07:51:13 PM
apollo-docs 🟢 Ready Preview, Logs Jun 26, 2026, 07:51:13 PM
apollo-landing 🟢 Ready Preview, Logs Jun 26, 2026, 07:51:13 PM
apollo-vertex 🟢 Ready Preview, Logs Jun 26, 2026, 07:51:13 PM

@github-actions

Copy link
Copy Markdown
Contributor

Dependency License Review

  • 1945 package(s) scanned
  • ✅ No license issues found
  • ⚠️ 2 package(s) excluded (see details below)
License distribution
License Packages
MIT 1715
ISC 89
Apache-2.0 55
BSD-3-Clause 27
BSD-2-Clause 23
BlueOak-1.0.0 8
MPL-2.0 4
MIT-0 3
CC0-1.0 3
MIT OR Apache-2.0 2
(MIT OR Apache-2.0) 2
Unlicense 2
LGPL-3.0-or-later 1
Python-2.0 1
CC-BY-4.0 1
(MPL-2.0 OR Apache-2.0) 1
Unknown 1
Artistic-2.0 1
(WTFPL OR MIT) 1
(BSD-2-Clause OR MIT OR Apache-2.0) 1
CC-BY-3.0 1
0BSD 1
(MIT OR CC0-1.0) 1
MIT AND ISC 1
Excluded packages
Package Version License Reason
@img/sharp-libvips-linux-x64 1.2.4 LGPL-3.0-or-later LGPL pre-built binary, not linked
khroma 2.1.0 Unknown MIT per GitHub repo, missing license field in package.json

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds support for passing a displayMode through chat tool-call messages so the ApToolCall renderer can switch between “tool name only”, “inputs/outputs”, and “full trace” views in both demos and the ap-chat dev harness.

Changes:

  • Forward meta.displayMode from tool-call chat messages into ApToolCall.
  • Update ap-chat dev UI to send tool-call messages for each display mode, including a full-trace span tree payload.
  • Update the apollo-react chat story support demo to send tool-call messages for each display mode.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
web-packages/ap-chat/src/dev.ts Adds display-mode-aware tool call demo payloads and UI buttons in the ap-chat dev harness.
packages/apollo-react/src/material/stories/chat-story-support.tsx Updates the chat story demo to emit tool-call messages for each display mode (including full trace).
packages/apollo-react/src/material/components/ap-chat/components/message/chat-message-content.tsx Forwards meta.displayMode to ApToolCall when rendering tool-call messages.

Comment thread packages/apollo-react/src/material/stories/chat-story-support.tsx
Comment thread web-packages/ap-chat/src/dev.ts Outdated
@github-actions

github-actions Bot commented Jun 27, 2026

Copy link
Copy Markdown
Contributor

📊 Coverage + size by package

Per-package coverage and bundle size on this PR. New-line coverage = of the source lines this PR adds or changes, the % hit by tests.

Package Coverage New-line coverage Packed (gzip) Unpacked vs main
@uipath/apollo-core 9.0% 43.82 MB 57.31 MB ±0
@uipath/apollo-react 34.7% 0.0% (0/9) 7.27 MB 27.59 MB +1.8 KB
@uipath/apollo-wind 40.1% 392.4 KB 2.55 MB +14 B
@uipath/ap-chat 85.8% — (105 untracked) 43.41 MB 55.85 MB +81 B

"Coverage" is each package's own coverage.include scope (e.g. apollo-core instruments only scripts/). "Packed"/"Unpacked" come from npm pack --dry-run and only cover built packages — "—" means not measured this run (package not affected / not built). "vs main" is the packed (gzipped) delta against the last successful main build (the package-sizes artifact from the Release workflow); "—" there means no main baseline was available this run. The baseline is main's latest build, not this PR's exact merge-base, so it includes any drift since the branch diverged. Packages with no vitest config are omitted.

Copilot AI review requested due to automatic review settings June 27, 2026 02:46

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

Comment thread packages/apollo-react/src/material/stories/chat-story-support.tsx
@shannonsuhendra shannonsuhendra merged commit d51ace3 into main Jun 27, 2026
44 checks passed
@shannonsuhendra shannonsuhendra deleted the feat/chat-tool-call branch June 27, 2026 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants