Skip to content

Conversation

@krissetto
Copy link
Contributor

@krissetto krissetto commented Jan 20, 2026

Draft because it depends on #1443

Allow users to give a reason when rejecting a tool call. This can help steer the model towards the right solutions and avoid some logical loops models can sometimes gets into


Screencast

Screencast.From.2026-01-19.21-03-18.mp4

@krissetto krissetto changed the title Reject tool with reason Reject tools with a reason Jan 20, 2026
simonferquel
simonferquel previously approved these changes Jan 20, 2026
@krissetto krissetto dismissed simonferquel’s stale review January 20, 2026 13:50

The merge-base changed after approval.

includes a new multi-select dialog compoent, used for this feature but reusable elsewhere

Signed-off-by: Christopher Petito <[email protected]>
@krissetto krissetto force-pushed the reject-tool-with-reason branch from fdec9a7 to 76e45f9 Compare January 20, 2026 13:56
@krissetto krissetto marked this pull request as ready for review January 20, 2026 13:56
@krissetto krissetto requested a review from a team as a code owner January 20, 2026 13:56
@dgageot dgageot merged commit 36cde33 into docker:main Jan 20, 2026
5 checks passed
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.

3 participants