Skip to content

Preserve background task notifications after end_turn#57

Open
Gujiassh wants to merge 1 commit intoshareAI-lab:mainfrom
Gujiassh:fix/background-task-results
Open

Preserve background task notifications after end_turn#57
Gujiassh wants to merge 1 commit intoshareAI-lab:mainfrom
Gujiassh:fix/background-task-results

Conversation

@Gujiassh
Copy link

@Gujiassh Gujiassh commented Mar 6, 2026

Summary

  • keep the s08_background_tasks.py loop alive until background task results are delivered instead of exiting immediately on end_turn
  • mirror the same behavior in s_full.py so the capstone agent does not drop background completions either
  • add regression tests covering both entrypoints

Testing

  • python3 -m unittest tests/test_background_notifications.py -v
  • python3 -m py_compile agents/s08_background_tasks.py agents/s_full.py tests/test_background_notifications.py

Closes #46

@vercel
Copy link

vercel bot commented Mar 6, 2026

@Gujiassh is attempting to deploy a commit to the crazyboym's projects Team on Vercel.

A member of the Team first needs to authorize it.

@Gujiassh
Copy link
Author

Gujiassh commented Mar 9, 2026

The only visible blocker on this PR right now looks like the Vercel team authorization check, rather than a code/test regression. If preview deploys matter here, someone on the team will need to authorize it; otherwise I’m happy for maintainers to ignore that status and review the patch on its own merits.

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.

Background task results may be lost when agent loop exits early

1 participant