refactor(agent): Remove unused `autogpt` code

[kcze]

Background

Should be merged after:

• https://github.com/Significant-Gravitas/AutoGPT/pull/7106
• https://github.com/Significant-Gravitas/AutoGPT/pull/7151
• https://github.com/Significant-Gravitas/AutoGPT/pull/7117

Follow-up after Component-based Agents

Changes 🏗️

This PR removes unused autogpt code and reorganizes its file structure.

PR Quality Scorecard ✨

• [x] Have you used the PR description template?   +2 pts
• [x] Is your pull request atomic, focusing on a single change?   +5 pts
• [ ] Have you linked the GitHub issue(s) that this PR addresses?   +5 pts
• [ ] Have you documented your changes clearly and comprehensively?   +5 pts
• [ ] Have you changed or added a feature?   -4 pts
  • [ ] Have you added/updated corresponding documentation?   +4 pts
  • [ ] Have you added/updated corresponding integration tests?   +5 pts
• [ ] Have you changed the behavior of AutoGPT?   -5 pts
  • [ ] Have you also run agbenchmark to verify that these changes do not regress performance?   +10 pts

[netlify[bot]]

✅ Deploy Preview for auto-gpt-docs canceled.

Name	Link
🔨 Latest commit	913b3e0b30f4dfe2ce425b988d7a38e7d6035efe
🔍 Latest deploy log	https://app.netlify.com/sites/auto-gpt-docs/deploys/664daec18df59b000873edc2

[github-actions[bot]]

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

[github-actions[bot]]

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

[github-actions[bot]]

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

[github-actions[bot]]

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.

[codiumai-pr-agent-pro[bot]]

PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.

PR Review 🔍

⏱️ Estimated effort to review [1-5]	4, because the PR involves a significant amount of changes across multiple files, including refactoring, renaming, and reorganization of code. The changes affect logging, database interactions, and model definitions, which are critical areas that require careful review to ensure functionality and integration are maintained.
🧪 Relevant tests	Yes
⚡ Possible issues	Possible Bug: The renaming of logging instances from LOG to logger and the changes in import paths for models and database interactions could lead to issues if not all references were correctly updated. This needs thorough testing to ensure that all parts of the application are still communicating correctly.
	Performance Concern: The restructuring of model imports and database access patterns could potentially affect performance. Benchmarking or profiling might be necessary to ensure that performance has not degraded.
🔒 Security concerns	No

[codiumai-pr-agent-pro[bot]]

PR-Agent was enabled for this repository. To continue using it, please link your git user with your CodiumAI identity here.

PR Code Suggestions ✨

Category	Suggestion	Score
Best practice	Use logger.exception to include stack traces in error logs for better debugging Consider using logger.exception instead of logger.error in the except Exception as e blocks to automatically include the stack trace in the log message. This can provide more context for debugging. autogpts/forge/forge/agent_protocol/database/db.py [178] -logger.error(f"Unexpected error while creating task: {e}") +logger.exception(f"Unexpected error while creating task: {e}") Suggestion importance[1-10]: 9 Why: Using logger.exception in exception blocks is a best practice as it includes the stack trace in the log, providing more context for debugging. This is a significant improvement for error handling.	9
Possible issue	Ensure database sessions are closed in a finally block to avoid resource leaks Consider adding a check to ensure that session.close() is called in a finally block to guarantee that the session is closed even if an exception occurs. autogpts/forge/forge/agent_protocol/database/db.py [237] -session.close() +finally: + session.close() Suggestion importance[1-10]: 8 Why: Ensuring that database sessions are closed even in the event of an error is crucial for resource management and preventing leaks. This is a valid and important improvement.	8
	Add exception handling to the coroutine decorator The coroutine decorator should handle exceptions that might occur during the execution of the coroutine to prevent the application from crashing unexpectedly. autogpts/autogpt/autogpt/app/utils.py [244-245] def wrapper(*args: P.args, **kwargs: P.kwargs): - return asyncio.run(f(*args, **kwargs)) + try: + return asyncio.run(f(*args, **kwargs)) + except Exception as e: + logger.error(f"Exception occurred in coroutine: {e}") + raise Suggestion importance[1-10]: 8 Why: Exception handling in the coroutine decorator is crucial for robustness and preventing crashes, making this a high-value suggestion.	8
	Ensure the coroutine decorator does not call asyncio.run if the event loop is already running The coroutine decorator should use asyncio.run only if the event loop is not already running to avoid runtime errors. autogpts/autogpt/autogpt/app/utils.py [244-245] def wrapper(*args: P.args, **kwargs: P.kwargs): - return asyncio.run(f(*args, **kwargs)) + if not asyncio.get_event_loop().is_running(): + return asyncio.run(f(*args, **kwargs)) + else: + return f(*args, **kwargs) Suggestion importance[1-10]: 8 Why: Ensuring asyncio.run is called only when the event loop isn't running is crucial to avoid runtime errors, thus a high-value suggestion.	8
Maintainability	Remove the unused config parameter from the split_text function The split_text function's config parameter is not used within the function. Consider removing it if it is not necessary. autogpts/forge/forge/content_processing/text.py [222-227] def split_text( text: str, - config: Config, max_chunk_length: int, tokenizer: ModelTokenizer, with_overlap: bool = True, ) -> list[str]: Suggestion importance[1-10]: 8 Why: The suggestion is valid and improves the function by removing an unused parameter, thus enhancing code clarity and maintainability.	8

[codecov[bot]]

Codecov Report

Attention: Patch coverage is 86.66667% with 2 lines in your changes are missing coverage. Please review.

Project coverage is 36.05%. Comparing base (bcc5282) to head (913b3e0).

Files	Patch %	Lines
...ogpts/autogpt/autogpt/app/agent_protocol_server.py	0.00%	1 Missing :warning:
autogpts/autogpt/autogpt/app/utils.py	90.00%	1 Missing :warning:

Additional details and impacted files

@@             Coverage Diff             @@
##           master    #7112       +/-   ##
===========================================
+ Coverage   22.63%   36.05%   +13.42%     
===========================================
  Files          66       19       -47     
  Lines        2669     1273     -1396     
  Branches      299      182      -117     
===========================================
- Hits          604      459      -145     
+ Misses       2035      786     -1249     
+ Partials       30       28        -2     

Flag	Coverage Δ
Linux	36.05% <86.66%> (+13.42%)	:arrow_up:
Windows	35.91% <86.66%> (+13.39%)	:arrow_up:
autogpt-agent	36.05% <86.66%> (+13.42%)	:arrow_up:
macOS	36.05% <86.66%> (+13.42%)	:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[github-actions[bot]]

This pull request has conflicts with the base branch, please resolve those so we can evaluate the pull request.

[github-actions[bot]]

Conflicts have been resolved! 🎉 A maintainer will review the pull request shortly.