-
Notifications
You must be signed in to change notification settings - Fork 46.2k
feat(frontend/builder): Add sub-graph update UX #11631
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: dev
Are you sure you want to change the base?
feat(frontend/builder): Add sub-graph update UX #11631
Conversation
β Deploy Preview for auto-gpt-docs-dev canceled.
|
|
Important Review skippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the β¨ Finishing touchesπ§ͺ Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
β Deploy Preview for auto-gpt-docs canceled.
|
|
Here's the code health analysis summary for commits Analysis Summary
|
|
Thanks for implementing this sub-graph update feature! This will definitely improve the user experience when working with sub-agents in the graph editor. The code looks good with well-structured components for handling updates, incompatibility detection, and resolution mode. I particularly like the clear visual indicators for broken connections and the detailed incompatibility dialog. Before mergingYou need to complete your test plan by verifying all the test scenarios you've outlined:
Once you've completed all the tests and received design approval, please update the checklist to reflect this. Minor notes
Please let us know when you've completed the remaining tests so we can merge this improvement. |
|
Thank you for this PR that addresses the sub-graph update UX! The implementation looks thorough and well-designed. Looking Good
Before MergingYour test plan needs completion:
Please complete the remaining test plan items to ensure this feature works as expected in all scenarios. Once those are checked off, this should be ready to merge. Is there anything you need help with to finish testing these scenarios? |
|
Thanks for working on improving the sub-graph update experience! This is a valuable enhancement that will make the builder UX much better. FeedbackYour implementation looks well thought out, with good separation of concerns between the UI components and the core logic in the However, before we can merge this PR:
Once these two items are addressed, the PR should be ready for merging. The code implementation itself looks solid and well-structured. |
OPEN-2743: Ability to Update Sub-Agents in Graph (Without Re-Adding)
Updating sub-graphs is a cumbersome experience at the moment, this should help. :)
Changes ποΈ
CustomNode>IncompatibilityDialog+SubAgentUpdateBarsub-componentsuseSubAgentUpdatehookChecklist π
For code changes: