allow custom REPLY_INSTRUCTIONS for AsyncToolset#5739
Conversation
|
|
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
|
Hi @longcw I noticed you've worked on this area of the codebase a good amount, so I figured you might be a good person to take a look at this PR whenever you get a chance. Happy to make changes if anything's off, thanks! |
longcw
left a comment
longcw
left a comment
There was a problem hiding this comment.
thanks for the pr! looks good, something nit:
|
@longcw I don't think I have merge permission, but this should be good to merge! |
|
@longcw Hi just a friendly bump, not sure what the merge procedure is! |
|
Hi @tinalenguyen I noticed you had helped some others get their PR's merged, was hoping you could also help me out here, or let me know if there's more action I need to take in order for this to be merged. I saw I missed making it into |
|
@will-mako-ai sorry for the late reply but we are working on some refactoring on the async tool that make it default in the framework, I'll add the customization with that. |
|
@longcw Oh got it, thanks for the update! Feel free to close this PR when your changes land then. |
3 participants
Minor change to allow control of how the AsyncToolset delivers replies to the user on receiving tool updates.
Tested with
uv run pytest tests/test_tools.py