feat: update task run status during local session lifecycle#901
feat: update task run status during local session lifecycle#901
Conversation
7461ec7 to
2d7bbe3
Compare
| notifyPromptComplete(session.taskTitle, stopReason); | ||
| } | ||
|
|
||
| // Update remote task run status when agent turn completes with no pending work |
There was a problem hiding this comment.
How should this work for interactive runs in Twig? I think this belongs to the agent package, and that for interactive runs I am not sure how we decide to transition to "completed" since the agent will regularly stop and then I prompt it again, probably the only reliable sign is PR merged. Thoughts?
There was a problem hiding this comment.
Yeah the more I thought about your comment, the more it makes sense!
For interactive runs I was thinking of handling "completed" on disconnect/logout path but that would be a fake "completed" since user would go back and continue, ending up in the back and forth status that you mentioned ^.
I decided to keep the in_progress on session create and prompt send and keeping this for session lifetime for now, added signalComplete to the agent package.
I agree the only reliable signal of true completion would be PR merged, that is going to be a follow up though since we haven't got anything on that logic as of yet (also no webhook logic also).
Basically this boils down to a not-that-useful-PR, feel free to trash :D
153c3ae to
02d0322
Compare
02d0322 to
b27ba17
Compare
Local task runs now show as "in progress" in the task list, added signalComplete() to the agent package as infrastructure for future use