Fix AttributeError: 'NoneType' object has no attribute 'name'

[drewgillson]

There is a bug related to the interplay between the society of mind agent and the FileLogger, probably somewhere around https://github.com/microsoft/autogen/blob/main/autogen/agentchat/contrib/society_of_mind_agent.py#L181, which causes an empty name property to be sent. The FileLogger should gracefully handle this circumstance and not return an error to the function caller.

Why are these changes needed?

Related issue number

Checks

• I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
• I've added tests (if relevant) corresponding to the changes introduced in this PR.
• I've made sure all auto checks have passed.

[sonichi]

There is an error in the change. It'll be good to write a simple test to cover the case.

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 42.45%. Comparing base (6aaa238) to head (9396ef6).
Report is 4 commits behind head on main.

Files with missing lines	Patch %	Lines
autogen/logger/file_logger.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3127      +/-   ##
==========================================
+ Coverage   33.24%   42.45%   +9.21%     
==========================================
  Files          99       99              
  Lines       11016    11016              
  Branches     2365     2537     +172     
==========================================
+ Hits         3662     4677    +1015     
+ Misses       7026     5906    -1120     
- Partials      328      433     +105     

Flag	Coverage Δ
unittests	42.41% <0.00%> (+9.16%)	⬆️

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

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

[Hk669]

@drewgillson thanks for spotting the bug.

[gitguardian]

️✅ There are no secrets present in this pull request anymore.

If these secrets were true positive and are still valid, we highly recommend you to revoke them.
Once a secret has been leaked into a git repository, you should consider it compromised, even if it was deleted immediately.
Find here more information about risks.

---

^{_{🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.}}

[jackgerrits]

Thanks for the fix @drewgillson !