Skip to content

feat: update task run status during local session lifecycle#901

Open
tatoalo wants to merge 5 commits intomainfrom
feat/twig/local-runs-task-progress
Open

feat: update task run status during local session lifecycle#901
tatoalo wants to merge 5 commits intomainfrom
feat/twig/local-runs-task-progress

Conversation

@tatoalo
Copy link
Contributor

@tatoalo tatoalo commented Feb 11, 2026

Local task runs now show as "in progress" in the task list, added signalComplete() to the agent package as infrastructure for future use

@tatoalo tatoalo force-pushed the feat/twig/local-runs-task-progress branch from 7461ec7 to 2d7bbe3 Compare February 11, 2026 18:49
@tatoalo tatoalo requested review from a team February 11, 2026 18:51
notifyPromptComplete(session.taskTitle, stopReason);
}

// Update remote task run status when agent turn completes with no pending work
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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?

Copy link
Contributor Author

@tatoalo tatoalo Feb 12, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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

@tatoalo tatoalo force-pushed the feat/twig/local-runs-task-progress branch 2 times, most recently from 153c3ae to 02d0322 Compare February 12, 2026 10:56
@tatoalo tatoalo force-pushed the feat/twig/local-runs-task-progress branch from 02d0322 to b27ba17 Compare February 12, 2026 10:58
@tatoalo tatoalo requested a review from a team February 17, 2026 15:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants