refactor(forge): Clean `forge`

[kcze]

Background

Should be merged after:

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

Follow-up after Component-based Agents

Changes 🏗️

This PR removes unused forge code and merges autogpt and forge code.

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

[github-actions[bot]]

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

[netlify[bot]]

✅ Deploy Preview for auto-gpt-docs canceled.

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

[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.

[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]]

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

[codecov[bot]]

Codecov Report

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

Project coverage is 22.63%. Comparing base (e8d7dfa) to head (637ff67). Report is 1 commits behind head on master.

:exclamation: Current head 637ff67 differs from pull request most recent head e401d14

Please upload reports for the commit e401d14 to get more accurate results.

Files	Patch %	Lines
...ogpts/autogpt/autogpt/app/agent_protocol_server.py	0.00%	9 Missing :warning:
autogpts/autogpt/autogpt/app/main.py	50.00%	2 Missing :warning:
...pts/autogpt/autogpt/agent_factory/configurators.py	0.00%	1 Missing :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7117      +/-   ##
==========================================
+ Coverage   22.53%   22.63%   +0.09%     
==========================================
  Files          66       66              
  Lines        2702     2669      -33     
  Branches      307      299       -8     
==========================================
- Hits          609      604       -5     
+ Misses       2060     2035      -25     
+ Partials       33       30       -3     

Flag	Coverage Δ
Linux	22.63% <25.00%> (+0.09%)	:arrow_up:
Windows	22.51% <25.00%> (+0.09%)	:arrow_up:
autogpt-agent	22.63% <25.00%> (+0.09%)	:arrow_up:
macOS	22.63% <25.00%> (+0.09%)	: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.

[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 Description updated to latest commit (https://github.com/Significant-Gravitas/AutoGPT/commit/1999bfbcac23ff6462de2e6b2928b0389dab6cad)

• [ ] Copy walkthrough table to "Files Changed" Tab

[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, due to the extensive changes across multiple files, including refactoring, removal of unused code, and reorganization of modules. The PR touches on various aspects of the system, from database interactions to API routing, which requires a thorough understanding of the overall architecture and the specific functionalities being modified.
🧪 Relevant tests	No
⚡ Possible issues	Possible Bug: The removal of extensive test cases for AI profiles and utilities without corresponding new tests for the refactored or newly added functionalities might leave parts of the codebase untested, potentially leading to undetected bugs.
	Performance Concern: The restructuring and renaming of modules and methods could lead to confusion and errors if not properly documented and communicated to all team members, potentially affecting the development velocity and increasing the risk of bugs in future changes.
🔒 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
Possible bug	Add a check to ensure task_obj is not None before accessing its attributes Consider adding a check to ensure that task_obj is not None before accessing its attributes in the convert_to_task function. This will prevent potential AttributeError exceptions if task_obj is unexpectedly None. autogpts/forge/forge/agent_protocol/database/db.py [92-93] +if task_obj is None: + logger.error("task_obj is None in convert_to_task") + raise ValueError("task_obj cannot be None") logger.debug(f"Converting TaskModel to Task for task_id: {task_obj.task_id}") task_artifacts = [convert_to_artifact(artifact) for artifact in task_obj.artifacts] Suggestion importance[1-10]: 8 Why: This suggestion correctly identifies a potential bug where task_obj could be None, which would lead to an AttributeError when trying to access its attributes. Adding a check enhances robustness and prevents runtime errors.	8
Security	Sanitize the task_request before logging to prevent exposure of sensitive information To avoid potential security issues, consider sanitizing the task_request before logging it. This will prevent any sensitive information from being exposed in the logs. autogpts/forge/forge/agent_protocol/api_router.py [106] -logger.exception(f"Error whilst trying to create a task: {task_request}") +sanitized_task_request = sanitize_task_request(task_request) +logger.exception(f"Error whilst trying to create a task: {sanitized_task_request}") Suggestion importance[1-10]: 8 Why: This suggestion correctly identifies a security risk related to logging sensitive data and proposes a practical solution to sanitize the data before logging.	8
Possible issue	Add exception handling for the write_file method in the create_artifact function The write_file method in the create_artifact function should handle potential exceptions to ensure robustness. autogpts/forge/forge/agent/agent.py [189] -await self.workspace.write_file(file_path, data) +try: + await self.workspace.write_file(file_path, data) +except Exception as e: + logger.error(f"Failed to write file {file_path}: {e}") + raise Suggestion importance[1-10]: 8 Why: Proper exception handling for file operations is crucial to prevent crashes and ensure robust error management in file I/O operations.	8
	Add exception handling for the read_file method in the get_artifact function The read_file method in the get_artifact function should handle potential exceptions to ensure robustness. autogpts/forge/forge/agent/agent.py [211] -retrieved_artifact = self.workspace.read_file(file_path) +try: + retrieved_artifact = self.workspace.read_file(file_path) +except Exception as e: + logger.error(f"Failed to read file {file_path}: {e}") + raise Suggestion importance[1-10]: 8 Why: Similar to writing, reading operations can fail due to various reasons, and handling these exceptions is essential for robustness and error transparency.	8