Agents sessions: indicate need for user input like approvals
Testing #274477
When an agent session requests permission, we may want to render this info on the list view so users know they need to come back to the agent session to provide feedback.
I wonder if we should introduce a new state along side running/cancelled/finished to indicate this. Basically a 4th state for input by user needed? And we would use this in many situations:
- confirmation for tool call
- follow up question
Besides the UX piece, this needs some kind of new API and adoption for all providers.
fyi @jo-oikawa @mrleemurray our designs need to include this state: a session is in progress but also stuck on user input
related: https://github.com/microsoft/vscode/issues/280617
See feedback also from https://github.com/microsoft/vscode/issues/281590 (as part of https://github.com/microsoft/vscode-internalbacklog/issues/6388).
For now I merged the fix that surfaces the confirmation to the user. We can explore something else in the future about making it actionable from the same view
I really think we need something stronger. Lets work on it in December/January, this should surface to the user with a stronger indicator on a session card.
//cc @jo-oikawa @eli-w-king
I have some existing stuff I want to clean up around tool confirmation state (like moving the window notifications out of the rendering logic.) And as part of this additional work I would love to have the tool requiring input shown on hover on the session.
So I would be more than happy to take this on for dec/jan :)
Warning icon in yellow + update subheader to something like "Input needed"?
@roblourens , @osortega and me will drive this. Me from UX, others from core.
@osortega can you backport your changes to recovery?