feat(backend): Enable executing store agents without agent ownership #9267
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Enable the tests in `monitor.spec.ts`. * Remove `test.describe.skip` to enable the tests. * Ensure the tests are now running and passing successfully. --- For more details, open the [Copilot Workspace session](https://copilot-workspace.githubnext.com/Significant-Gravitas/AutoGPT?shareId=XXXX-XXXX-XXXX-XXXX).
…wnership (#9179) This PR enables the execution of store agents even if they are not owned by the user. Key changes include handling store-listed agents in the `get_graph` logic, improving execution flow, and ensuring version-specific handling. These updates support more flexible agent execution. - **Graph Retrieval:** Updated `get_graph` to check store listings for agents not owned by the user. - **Version Handling:** Added `graph_version` to execution methods for consistent version-specific execution. - **Execution Flow:** Refactored `scheduler.py`, `rest_api.py`, and other modules for clearer logic and better maintainability. - **Testing:** Updated `test_manager.py` and other test cases to validate execution of store-listed agents added test for accessing graph --------- Co-authored-by: Reinier van der Leer <[email protected]> Co-authored-by: Zamil Majdy <[email protected]>
…ts/open-2276-add-ability-to-execute-store-agents-without-agent-ownership
Pwuts
requested review from
Swiftyos and
majdyz
and removed request for
a team
✅ Deploy Preview for auto-gpt-docs-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
github-actions
bot
added
platform/frontend
5 tasks
✅ Deploy Preview for auto-gpt-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
3 tasks
ntindle
requested changes
Jan 15, 2025
…agents-without-agent-ownership
Pwuts
force-pushed
the
pwuts/open-2276-add-ability-to-execute-store-agents-without-agent-ownership
branch
from
af6be24 to
03df229
Compare
…agents-without-agent-ownership
Without this constraint, `review_store_submission` fails.
ntindle
previously approved these changes
Jan 26, 2025
This comment was marked as resolved.
This comment was marked as resolved.
Pwuts
commented
Jan 27, 2025
…agents-without-agent-ownership
ntindle
approved these changes
Jan 27, 2025
…agents-without-agent-ownership
Pwuts
deleted the
pwuts/open-2276-add-ability-to-execute-store-agents-without-agent-ownership
branch
waterstark
pushed a commit
to waterstark/AutoGPT
that referenced
this pull request
Jan 30, 2025
…ignificant-Gravitas#9267) This re-introduces PR Significant-Gravitas#9179 with some fixes. This PR enables the execution of store agents even if they are not owned by the user. Key changes include handling store-listed agents in the `get_graph` logic, improving execution flow, and ensuring version-specific handling. These updates support more flexible agent execution. ### Changes 🏗️ (copied from Significant-Gravitas#9179) - **Graph Retrieval:** Updated `get_graph` to check store listings for agents not owned by the user. - **Version Handling:** Added `graph_version` to execution methods for consistent version-specific execution. - **Execution Flow:** Refactored `scheduler.py`, `rest_api.py`, and other modules for clearer logic and better maintainability. - **Testing:** Updated `test_manager.py` and other test cases to validate execution of store-listed agents added test for accessing graph Out-of-scope changes: - Add logic to pretty-print Pydantic validation error responses to backend API client in frontend --------- Co-authored-by: Nicholas Tindle <[email protected]> Co-authored-by: Nicholas Tindle <[email protected]> Co-authored-by: Swifty <[email protected]> Co-authored-by: Zamil Majdy <[email protected]>
github-merge-queue bot
pushed a commit
that referenced
this pull request
Feb 13, 2025
- Blocked by #9267 This re-introduces changes from the following PRs with fixes: - #9218 - #9211 ### Changes 🏗️ - See #9218 - See #9211 Fixes: - Fix Prisma query statements in `v2.library.db` - Fix creation of (library) agents - Fix test cleanup of (library) agents - Fix handling and passing of `node_input` parameters ### Checklist 📋 #### For code changes: - [x] I have clearly listed my changes in the PR description - [x] I have made a test plan - [x] I have tested my changes according to the test plan: - [x] Create & run a new agent - [x] Update & run an existing agent
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This re-introduces PR #9179 with some fixes.
This PR enables the execution of store agents even if they are not owned by the user. Key changes include handling store-listed agents in the
get_graphlogic, improving execution flow, and ensuring version-specific handling. These updates support more flexible agent execution.Changes 🏗️
(copied from #9179)
get_graphto check store listings for agents not owned by the user.graph_versionto execution methods for consistent version-specific execution.scheduler.py,rest_api.py, and other modules for clearer logic and better maintainability.test_manager.pyand other test cases to validate execution of store-listed agents added test for accessing graphOut-of-scope changes: