Skip to content

Conversation

@contextablemark
Copy link
Collaborator

@contextablemark contextablemark commented Oct 25, 2025

Note: This fix does not fully address #568; doing this will require some more extensive refactoring.

Addresses #568 and #556

Summary

  • iterate unseen transcript messages in arrival order so tool results and user messages are handled without reordering
  • allow _start_new_execution and the background runner to accept explicit message batches, ensuring only the intended messages are marked processed
  • extend the tool-result flow tests to assert ordering and capture the batches passed through the execution helpers
  • insure that assistant messages and tool requests don't get played back

@contextablemark contextablemark marked this pull request as ready for review October 29, 2025 20:15
@maxkorp maxkorp merged commit 43a6cb5 into main Oct 29, 2025
28 of 30 checks passed
@contextablemark contextablemark deleted the contextablemark/fix556and568 branch November 10, 2025 15:42
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.

3 participants