Skip to content

fix: prefer tool title in app view chrome #749

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Chipagosfinest wants to merge 1 commit into
base: main
Choose a base branch
from
+8 −2
Open

fix: prefer tool title in app view chrome #749

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 7 additions & 2 deletions examples/basic-host/src/index.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -339,9 +339,14 @@ function ToolCallInfoPanel({ toolCallInfo, isDestroying, onRequestClose, onClose
className={styles.toolCallInfoPanel}
style={isDestroying ? { opacity: 0.5, pointerEvents: "none" } : undefined}
>
{/* Row 1: Header with server:tool name and close button */}
{/* Row 1: Header with server:tool title and close button */}
<div className={styles.appHeader}>
<span>{toolCallInfo.serverInfo.name}:<span className={styles.toolName}>{toolCallInfo.tool.name}</span></span>
<span>
{toolCallInfo.serverInfo.name}:
<span className={styles.toolName}>
{toolCallInfo.tool.title ?? toolCallInfo.tool.name}
</span>
</span>
{onRequestClose && !isDestroying && (
<button
className={styles.closeButton}
Expand Down
1 change: 1 addition & 0 deletions specification/draft/apps.mdx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -409,6 +409,7 @@ Example (app-only tool, hidden from model):
- Resource MUST exist on the server
- Host MUST use `resources/read` to fetch the referenced resource URI
- Host MAY prefetch and cache UI resource content for performance optimization
- When labeling host-provided chrome for a tool-instantiated View, Host SHOULD use the Tool's human-readable `title` when present, falling back to `name`
- Since UI resources are primarily discovered through tool metadata, Servers MAY omit UI-only resources from `resources/list` and `notifications/resources/list_changed`

#### Visibility:
Expand Down