Autogpt v0.5.1 fixazure

[foreee]

User description

Background

fix this issue https://github.com/Significant-Gravitas/AutoGPT/issues/7128

Changes 🏗️

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

---

PR Type

Bug fix, Enhancement

---

Description

• Enhanced response validation and error handling in one_shot.py.
• Added logging configuration to main application functions.
• Introduced LoggingConfig to Config class and updated OpenAI API key pattern.
• Added new GPT-4 model variants and loaded Azure configuration from environment.
• Expanded failure check in json_loads to include demjson3.syntax_error.
• Refactored logging configuration to support external config.
• Updated default LLM model names in .env.template.

---

Changes walkthrough 📝

	Relevant files
Enhancement	one_shot.py Enhance response validation and error handling in one_shot.py autogpts/autogpt/autogpt/agents/prompt_strategies/one_shot.py Deep copy response schema before validation. Remove command property from schema if using functions API. Improved error message for missing tool calls. +9/-2      main.py Add logging configuration to main application functions    autogpts/autogpt/autogpt/app/main.py Added logging configuration parameter to run_auto_gpt and run_auto_gpt_server. +2/-0      config.py Add logging configuration and update OpenAI API key pattern autogpts/autogpt/autogpt/config/config.py Added LoggingConfig to Config class. Updated OpenAI API key pattern to support project keys. +4/-2      openai.py Add new GPT-4 models and Azure configuration loading          autogpts/autogpt/autogpt/core/resource/model_providers/openai.py Added new GPT-4 model variants. Loaded Azure configuration from environment. +23/-2    config.py Refactor logging configuration to support external config autogpts/autogpt/autogpt/logs/config.py Added logging configuration parameter to configure_logging. Refactored logging configuration to use passed or environment config. +31/-23
Bug fix	json_utils.py Expand failure check in JSON utility function                        autogpts/autogpt/autogpt/core/utils/json_utils.py Expanded failure check in json_loads to include demjson3.syntax_error. +1/-1
Configuration changes	.env.template Update default LLM model names in .env.template                    autogpts/autogpt/.env.template Updated default LLM model names in .env.template. +4/-4

---

💡 PR-Agent usage: Comment /help on the PR to get a list of all available PR-Agent tools and their descriptions

[github-actions[bot]]

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

[qodo-code-review[bot]]

PR Description updated to latest commit (https://github.com/Significant-Gravitas/AutoGPT/commit/5f245d257c705fa7829aba7c0dc14f2e067de4f6)

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

[qodo-code-review[bot]]

PR Review 🔍

⏱️ Estimated effort to review [1-5]	4, because the PR involves multiple files with significant changes including logic modifications, configuration updates, and error handling improvements. The changes are spread across various components of the system which requires a thorough understanding of the overall architecture and potential side effects.
🧪 Relevant tests	No
⚡ Possible issues	Possible Bug: The deletion of the 'command' property in the response schema based on certain conditions could lead to unintended behavior if other parts of the system expect this property to always exist.
	Error Handling Concern: The change in error messages (e.g., from "No 'tool_calls' in assistant reply" to "Assistant did not use any tools") might affect systems or logs that rely on specific wording.
🔒 Security concerns	No

[qodo-code-review[bot]]

PR Code Suggestions ✨

Category	Suggestion	Score
Possible issue	Use the pop method to safely remove a dictionary key Instead of using del to remove the "command" property from response_schema.properties, consider using the pop method. This will avoid potential KeyError if "command" does not exist. autogpts/autogpt/autogpt/agents/prompt_strategies/one_shot.py [401] -del response_schema.properties["command"] +response_schema.properties.pop("command", None) Suggestion importance[1-10]: 8 Why: Using pop with a default value of None is safer as it handles cases where the key might not exist, preventing a KeyError.	8

[netlify[bot]]

✅ Deploy Preview for auto-gpt-docs canceled.

Name	Link
🔨 Latest commit	0233a889a58a8d5cf8273d3f49568f9214a01d78
🔍 Latest deploy log	https://app.netlify.com/sites/auto-gpt-docs/deploys/66606c5e7cd8d600080e4df0

[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 3 lines in your changes missing coverage. Please review.

Project coverage is 44.21%. Comparing base (4e02f7d) to head (31778d5). Report is 37 commits behind head on master.

:exclamation: Current head 31778d5 differs from pull request most recent head 0233a88

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

Files	Patch %	Lines
forge/forge/llm/providers/openai.py	33.33%	2 Missing :warning:
forge/forge/config/config.py	0.00%	1 Missing :warning:

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #7182      +/-   ##
==========================================
+ Coverage   36.05%   44.21%   +8.15%     
==========================================
  Files          19       63      +44     
  Lines        1273     3594    +2321     
  Branches      182      487     +305     
==========================================
+ Hits          459     1589    +1130     
- Misses        786     1939    +1153     
- Partials       28       66      +38     

Flag	Coverage Δ
Linux	44.21% <25.00%> (+8.15%)	:arrow_up:
Windows	44.30% <25.00%> (+8.38%)	:arrow_up:
autogpt-agent	?
forge	44.21% <25.00%> (?)
macOS	44.21% <25.00%> (+8.15%)	: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.

[Pwuts]

The changes outside of config.py are all either unnecessary or unrelated, so please revert those.

[github-actions[bot]]

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

[Swiftyos]

@foreee Thank you for your contribution.

Did you see Pwuts comment:

The changes outside of config.py are all either unnecessary or unrelated, so please revert those.

If you are able to revert this changes and make the small change that was suggested above we should be able to get this merged in 🚀

[Torantulino]

Getting reports that this fix doesn't work, and people are still having this problem. #7128

[Swiftyos]

@foreee Thank you for this! Sadly the change isn't working - however, we are planning on deprecating the forge soon in favour of the new system, which is located in the rnd folder at the moment.

If you'd like to help out there it would be greatly appreciated. Most of the PR's being merged are targeting projects in that folder.