Repository navigation
Show the call's describe_intent sentence as the heading of the tool approval card #17873
goljavieglc
started this conversation in
Feature Requests & Suggestions
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
What. When a tool has
describe_intentenabled, the model writes a one-sentence label for each call in anintentargument (e.g. "Creating a ticket for Juan"). The tool approval card currently shows the tool name as its heading (e.g.create_work_package_mcp_openproject) and leavesintentinside the raw JSON body.Proposal. When the request carries a non-empty
intentstring, use it as the card heading, show the tool name underneath in secondary text, and leaveintentout of the arguments body. Withoutintent, nothing changes.Why. Approvers are usually not the people who configured the tools. "Creating the PN-101 work package in HTB Guion Manufacturing" is something a user can approve. A tool name plus a JSON blob is not, and people end up approving on trust. The sentence already exists, because the model writes it for the activity label, so this only reuses it.
Scope / areas affected. Approval preview building (on
dev,buildApprovalPreviewin@librechat/chat) and the review panel (client/src/components/Chat/approval/Review.tsx). The intent needs its own character bound, like the other labels, and counts toward the batch preview budget. No backend or API changes.Status. We run this as a local patch on v0.8.9-rc1 (before the preview moved into
packages/chat), with tests inpreview.test.tsandReview.test.tsx. We can share the patch or port it todevif the approach is welcome.All reactions