-
Notifications
You must be signed in to change notification settings - Fork 43.7k
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
Move app code to app subpackage. #4962
Move app code to app subpackage. #4962
Conversation
✅ Deploy Preview for auto-gpt-docs canceled.
|
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## master #4962 +/- ##
==========================================
- Coverage 51.02% 50.65% -0.37%
==========================================
Files 118 119 +1
Lines 4898 4894 -4
Branches 649 648 -1
==========================================
- Hits 2499 2479 -20
- Misses 2215 2232 +17
+ Partials 184 183 -1
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can/should this be moved to autogpt/prompts
?
This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request. |
Background
One of the core Re-arch goals is to separate out Auto-GPT library code from application code. This is a first very crude push in that direction.
Changes
autogpt.app
namespace and makes corresponding import changes.autogpt.cli
->autogpt.app.cli
autogpt.confugrator
->autogpt.app.configurator
autogpt.main
->autogpt.app.main
autogpt.prompts.prompt
->autogpt.app.prompt
autogpt.setup
->autogpt.app.setup
COMMAND_CATEGORIES
global fromautogpt.main
toautogpt.commands
.DEFAULT_TRIGGERING_PROMPT
fromautogpt.prompts.prompt
toautogpt.prompts.default_prompts
command_registry
attribute from theAIConfig
main.py
file.construct_full_prompt
method from theAIConfig
to be a standalone function inautogpt.app.prompt
, pushing theAIConfig
towards being a relatively clean (with a little leftover cruft) data structure.Documentation
X
Test Plan
CI with appropriate adjustments to test imports plus manual testing.
PR Quality Checklist