Add a suggestion to Architect to summarize the current session and continue with Coder #15512
Conversation
Co-authored-by: Jonas Helming <jhelming@eclipsesource.com>
Co-authored-by: Jonas Helming <jhelming@eclipsesource.com>
Co-authored-by: Jonas Helming <jhelming@eclipsesource.com>
JonasHelming
changed the title
|
|
||
| import { Command } from '@theia/core'; | ||
|
|
||
| export const AI_SUMMARIZE_SESSION_AS_TASK_FOR_CODER: Command = { |
There was a problem hiding this comment.
Going with a command even though I set up the callback system!? ;-)
There was a problem hiding this comment.
I did this, because I wanted to have the nice progress bar, so it felt natural. I would use call back for actions that stay within the session for example pre-defined user message. Does this make sense to you?
There was a problem hiding this comment.
The command approach was my preference, so I'm happy either way. Re: the progress bar, its display is part of the behavior of TaskContextService.summarize, so as long as you call that, whether in a command or a callback, you should get the progress bar.
There was a problem hiding this comment.
I think it is good to support both!
Do you have any other comments on this PR?
There was a problem hiding this comment.
No - not sure if it makes sense to approve it while #15427 is still pending? Or would you like this to go in instead?
Co-authored-by: colin-grant-work <cgrant.ext@eclipsesource.com>
Co-authored-by: colin-grant-work <cgrant.ext@eclipsesource.com>
Co-authored-by: colin-grant-work <cgrant.ext@eclipsesource.com>
Co-authored-by: colin-grant-work <cgrant.ext@eclipsesource.com>
What it does
based on #15427
How to test
Follow-ups
Coder could immediatly start to implement or at least provide this as a suggestion
Breaking changes
Attribution
Review checklist
Reminder for reviewers