refactor!: Remove agent breakpoint and agent snapshot#11202
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
Coverage reportClick to see where and how coverage changed
This report was generated by python-coverage-comment-action |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
…gent-tool-invoker
…gent-tool-invoker
Code reviewNo issues found. Checked for bugs and CLAUDE.md compliance. |
anakin87
left a comment
There was a problem hiding this comment.
Happy about the simplification!
I left a minor comment.
In addition, we should track documentation changes to be done. But maybe MIGRATION.md is sufficient. WDYT?
Please wait before merging: I want to understand if 3.0.0-rc0 version and tag are needed before...
Good point. I think |
anakin87
left a comment
There was a problem hiding this comment.
Once you've applied the minor change I requested, feel free to merge: version tag has already been placed.
Related Issues
Proposed Changes:
Remove break point and snapshot support from the internals of Agent. Motivation is that it's not heavily used and adds a lot of complexity to the Agent internals.
Open to adding this back in the future if we see a strong need (e.g. HITL + Breakpoints to enable long running Agents). And if we decide to do so I think there is a simpler way to re-add it than in it's current form.
Updated MIGRATION.md and added an upgrade release note describing the breaking changes.
How did you test it?
Updated existing tests.
Notes for the reviewer
Checklist
fix:,feat:,build:,chore:,ci:,docs:,style:,refactor:,perf:,test:and added!in case the PR includes breaking changes.