cleaner definition of tool_responses fixes #1174

[yenif]

Why are these changes needed?

I couldn't replicate #1174, but I could imagine how this could have created an empty tool_responses.

I could also do a defensive dict key filter on generate_oai_reply, but feels really dirty. ideally there would be built out types and then it would work to just take the keys from the dict that are needed for the type. But that is a bigger thing.

Related issue number

Closes #1174

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.

[codecov-commenter]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (2ca8aae) 30.41% compared to head (05d75a1) 48.79%.

Files	Patch %	Lines
autogen/agentchat/conversable_agent.py	80.00%	1 Missing and 1 partial ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1182       +/-   ##
===========================================
+ Coverage   30.41%   48.79%   +18.37%     
===========================================
  Files          32       32               
  Lines        4301     4300        -1     
  Branches      993     1044       +51     
===========================================
+ Hits         1308     2098      +790     
+ Misses       2899     1998      -901     
- Partials       94      204      +110     

Flag	Coverage Δ
unittests	48.69% <80.00%> (+18.33%)	⬆️

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.

[davorrunje]

I created PR #1184 today supporting streamed tool calls and I think I understand the new OpenAI API quite well now. I'll check this out carefully tomorrow, I am a bit too tired now.

[ekzhu]

Are this and #1187 both required to close the bug #1174 ?