fix: failure when usage is none

[aa875982361]

User description

Features

• ImportError 当运行 startup.py #233

Feature Docs

Influence

Result

Other

---

PR Type

Bug fix

---

Description

• Fixed an issue where the application could encounter errors if chunk.usage was present but had a None value. Now, the code checks that chunk.usage is truthy before proceeding to use it.

---

Changes walkthrough 📝

	Relevant files
Bug fix	openai_api.py Fix handling of `None` value in `chunk.usage`                        metagpt/provider/openai_api.py Added a condition to check if chunk.usage is not only present but also truthy before using it. +1/-1

---

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

[codiumai-pr-agent-pro]

PR Description updated to latest commit (8695629)

• Copy walkthrough table to "Files Changed" Tab

[codiumai-pr-agent-pro]

PR Review 🔍

⏱️ Estimated effort to review [1-5]	2, because the PR involves a simple conditional check to ensure that 'chunk.usage' is not only present but also truthy before proceeding. The change is minimal and straightforward, making the review process relatively quick and easy.
🧪 Relevant tests	No
⚡ Possible issues	Possible Bug: The code checks if 'chunk.usage' is truthy, which is generally fine. However, if 'chunk.usage' can be a valid falsy value (like 0 or an empty list, depending on the context), this might unintentionally skip valid usage data. It would be beneficial to clarify what types of values 'chunk.usage' can hold and if any of these might be falsy yet valid.
🔒 Security concerns	No

[codiumai-pr-agent-pro]

PR Code Suggestions ✨

Category	Suggestions
Enhancement	Simplify attribute checking and value retrieval using getattr. Replace the check hasattr(chunk, "usage") and chunk.usage with a more Pythonic approach using getattr. This avoids accessing the attribute twice and handles potential None values more cleanly. metagpt/provider/openai_api.py [103] -if hasattr(chunk, "usage") and chunk.usage: +if getattr(chunk, "usage", None):

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 70.19%. Comparing base (f201b2f) to head (a473c74).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1253      +/-   ##
==========================================
- Coverage   70.22%   70.19%   -0.04%     
==========================================
  Files         316      316              
  Lines       18860    18860              
==========================================
- Hits        13245    13239       -6     
- Misses       5615     5621       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[garylin2099]

LGTM

[chg0901]

nice