Allow rejected tool function to be async in order to ask user for a reason #2217
augustocdias
started this conversation in
Ideas
Replies: 2 comments 1 reply
-
Awesome idea. I'll put that on my list |
Beta Was this translation helpful? Give feedback.
0 replies
-
This has been added in #2249 |
Beta Was this translation helpful? Give feedback.
1 reply
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.
-
One thing I find useful in Avante is when you reject an execution of a tool call to provide for a reason to the LLM, so it can decide to proceed, abort or even retry with different inputs. It would be nice to let the
rejected
function in theoutput
to handle async in the same fashion as the actual tool so that we can callvim.ui.input
to ask the user for a reason.Another suggestion would be to add a flag for code_companion to handle this internally by itself.
Beta Was this translation helpful? Give feedback.
All reactions