-
Notifications
You must be signed in to change notification settings - Fork 46.2k
fix(backend): use marketplace name/description for downloaded agents in library #11347
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?
fix(backend): use marketplace name/description for downloaded agents in library #11347
Conversation
…in library Introduces optional 'name' and 'description' fields to the LibraryAgent model, allowing marketplace or custom overrides. Updates database schema, migration, and model logic to prioritize these fields over AgentGraph defaults when present.
✅ Deploy Preview for auto-gpt-docs-dev canceled.
|
✅ Deploy Preview for auto-gpt-docs 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 |
|
Here's the code health analysis summary for commits Analysis Summary
|
Renamed the migration file to update its timestamp for better chronological organization of migrations related to library agent custom name and description.
|
Thank you for this PR that fixes agents downloaded from the marketplace showing the creator's graph name instead of the marketplace name in the user's library. The implementation approach looks solid! A few notes:
Once you've completed your testing plan and checked those boxes, this PR should be ready for merging. |
…library-is-different-from-name
|
@cursor please try and fix the ci pipleines errors on the backend |
|
Made a quick pass at the backend CI failure and found the culprit: every I’ve updated the migration to operate on the default schema, which matches the rest of our history: |
|
Thank you for this PR addressing the marketplace agent naming issue! The implementation looks good conceptually, but there are a few things to address before it's ready to merge:
Once you've completed your testing and addressed these points, please update the PR. |
Co-authored-by: nicholas.tindle <[email protected]>
|
Cursor Agent can help with this pull request. Just |
|
Thank you for this PR to fix the marketplace agent name/description issue in the library. The implementation looks good, adding optional name/description fields to LibraryAgent and properly handling the override logic. The code changes match the stated intention in the PR description. Before this can be approved, please complete the remaining test item in your test plan:
Once you've completed these tests and checked the remaining item, this PR should be ready to merge. The changes are focused and address the specific issue mentioned. |
Co-authored-by: nicholas.tindle <[email protected]>
Changes
Fixes agents downloaded from marketplace showing creator's graph name instead of marketplace name in user's library.
Problem: When downloading a marketplace agent, the library entry used
AgentGraph.name/description(creator's version) instead of the marketplace listing name/description.Solution: Added optional
nameanddescriptionoverride fields toLibraryAgent. When adding store agents, these are populated from the marketplace listing. The model layer falls back to graph values if not set (backward compatible).Modified Files
schema.prisma: Added optionalname/descriptiontoLibraryAgentbackend/server/v2/library/db.py: Set marketplace name/description when adding store agentsbackend/server/v2/library/model.py: Use override fields with fallback to graph valuesI still need to fully test this
For code changes: