Skip to content

[Improve] Clarify integration tool approval prompts - #3074

Merged
daniel-lxs merged 2 commits into
developfrom
feature/humanize-tool-approval-card-1asze1wjw3tbf
Sep 21, 2026
Merged

daniel-lxs merged 2 commits into
developfrom
feature/humanize-tool-approval-card-1asze1wjw3tbf

Conversation

@roomote-roomote

Copy link
Copy Markdown
Contributor

​Opened on behalf of @daniel-lxs. Follow up by mentioning @roomote-roomote, in the web UI, or in Slack.

What changed

  • Replaces raw integration and tool identifiers in the approval prompt with a human-readable request, using catalog display names when available and a conservative generic fallback for arbitrary tools.
  • Moves raw integration names, tool names, and redacted argument JSON behind an accessible Details disclosure, including the clearer empty state "No additional details."
  • Clarifies the action hierarchy: Allow once is primary, Allow for this session is secondary, and Deny is a quiet ghost action.
  • Uses an intentional mobile layout with full-width stacked allow actions and a compact Deny action instead of wrapped controls.

Why this change was made

The approval card exposed internal runtime vocabulary and made three choices compete visually. This keeps the decision understandable at a glance while preserving technical details for people who want to inspect the exact request.

Impact

Only the pending integration-tool approval card changes. Its transcript placement, requester-only checks, session-scoped permission behavior, and decision payloads remain unchanged.

Screenshots

Desktop

Approval card with primary and secondary allow actions plus a quiet Deny action

Mobile

Approval card with stacked mobile allow actions and a compact Deny action

@roomote-community

roomote-community Bot commented Sep 21, 2026 •

Copy link
Copy Markdown
Contributor

No code issues found. See task

Reviewed 133ae75

@roomote-roomote
roomote-roomote Bot marked this pull request as ready for review September 21, 2026 21:52
@daniel-lxs
daniel-lxs merged commit 3ff7c8c into develop Sep 21, 2026
17 checks passed
@daniel-lxs
daniel-lxs deleted the feature/humanize-tool-approval-card-1asze1wjw3tbf branch September 21, 2026 21:53
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