feat: add continue button
Supported the feature to display 'Continue' while awaiting user input (currently lacking logic for controlling button visibility, to be completed before merging #1487
---update:
IMHO, the continue behavior can be implicitly embedded in sending a message. it will be more natural for UX. WDYT?
@iFurySt Do you think we should directly change the send button into "[continue]" when (1) the agent is awaiting user, (2) the input box is empty? And when user types anything into the input box, the "[continue]" button will go back to what it looks like (upward arrow)?
start a new session/task:
- send a message (to start a task)
- (user is waiting for completion, providing info, or continuing task based on prev result)
- send a message (to provide more info to help the agent complete the task or to continue the next task based on the tasks already completed)
step 2. and step 3. may loop multiple times until the user stops or restarts a new fresh session.
@xingyaoww wdyt?
@iFurySt Since https://github.com/OpenDevin/OpenDevin/pull/1290 is merged, we already support awaiting user input (e.g., user sends follow-up messages instead of re-start), check demo:
https://github.com/OpenDevin/OpenDevin/assets/38853559/d0239ed5-8521-4df0-b6d4-0ab36f128aa5
This PR just adds a functionality that, when use don't want to manually type "continue" for agent to continue working on the task, click the button would sends a user message "continue" to the agent on user's behalf, similar to OpenAI ChatGPT's continue generation button.
It aims to augment the "user input", not to replace it IMO.
i think the @openartist has finished the UI design. @09473ZH is it possible to take some time to refactor it 😄
i think the @openartist has finished the UI design. @09473ZH is it possible to take some time to refactor it 😄
Oh, that's nice! I wonder why there's a refresh icon next to 'Continue'?
@09473ZH good question--maybe it should be more like a forward arrow ->
Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:
:exclamation: No coverage uploaded for pull request base (
main@f8d4b1a). Click here to learn what that means.
:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@ Coverage Diff @@
## main #1508 +/- ##
=======================================
Coverage ? 62.80%
=======================================
Files ? 96
Lines ? 3839
Branches ? 0
=======================================
Hits ? 2411
Misses ? 1428
Partials ? 0
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
This is looking great!
A few minor style tweaks:
- Can we get it to float over the message (without obscuring text) like in the designs?
- Can we make it a bit smaller?
Approving as-is, but if feasible would love to tweak the styles per my suggestions above before merging
Thanks for catching that oversight. I've updated my code. Let me know if there's anything else I should change. :)
Can we make it a bit smaller
I've changed the font size from 14px to 12px. Do you think this works?
i think the @openartist has finished the UI design. @09473ZH is it possible to take some time to refactor it 😄