fix: demote SystemMessage to UserMessage in HandoffMessage context - #8260
MOHAMMED WASIM KHAN (wasim-builds) wants to merge 2 commits into
Conversation
| with open(file, "w") as f: | ||
| f.write(content) | ||
| else: | ||
| print(f"Failed to find old code in {file}") |
There was a problem hiding this comment.
Is patch_8238.py meant to be part of the repo? It looks like a one-off migration/editing script used to apply the same change to three files, while the actual source edits are already included in this PR. If it is not needed at runtime or for tests, it may be better to drop it from the final change.
There was a problem hiding this comment.
Thanks Pramod Ubbala (@upramod)! Good catch — patch_8238.py was indeed a temporary migration helper used during development and is not needed at runtime or in tests. It has been removed from the branch.
|
Hi maintainers, thank you for the review on this handoff message handling PR! The temporary patch script has been removed and the role demotion logic has been verified with existing test suites. I'm very enthusiastic about multi-agent workflows in AutoGen and would love to contribute further to the core agent abstractions. I'm also actively open to freelance/contract engagements, full-time roles, or joining as a team contributor. More of my open-source projects can be found at https://github.com/wasim-builds. |
Fixes #8238