KEMBAR78
Fix OpenAIAgent function tool schema by alexey-pelykh · Pull Request #6936 · microsoft/autogen · GitHub
Skip to content

Conversation

@alexey-pelykh
Copy link
Contributor

@alexey-pelykh alexey-pelykh commented Aug 15, 2025

Why are these changes needed?

To fix OpenAIAgent's incorrect function tool schema

Related issue number

Fixes #6935

Checks

@alexey-pelykh
Copy link
Contributor Author

@microsoft-github-policy-service agree

@alexey-pelykh alexey-pelykh force-pushed the fix/autogen-ext-openai-agent-incorrect-function-schema branch from 228a8eb to d22384a Compare August 15, 2025 08:32
@alexey-pelykh alexey-pelykh force-pushed the fix/autogen-ext-openai-agent-incorrect-function-schema branch from d22384a to 001daf6 Compare August 15, 2025 08:37
@alexey-pelykh
Copy link
Contributor Author

A side note - the change may be minimized (at a loss of typing) into

                    function_schema: Dict[str, Any] = {
                        "type": "function",
                        **_convert_tool_to_function_schema(tool),
                    }
                    self._tools.append(function_schema)

@ekzhu ekzhu enabled auto-merge (squash) August 19, 2025 04:41
@codecov
Copy link

codecov bot commented Aug 19, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.80%. Comparing base (a666c65) to head (585a068).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #6936      +/-   ##
==========================================
- Coverage   80.89%   80.80%   -0.10%     
==========================================
  Files         237      237              
  Lines       18349    18350       +1     
==========================================
- Hits        14844    14828      -16     
- Misses       3505     3522      +17     
Flag Coverage Δ
unittests 80.80% <100.00%> (-0.10%) ⬇️

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ekzhu ekzhu merged commit 898d83e into microsoft:main Aug 19, 2025
71 checks passed
@alexey-pelykh alexey-pelykh deleted the fix/autogen-ext-openai-agent-incorrect-function-schema branch August 19, 2025 05:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OpenAIAgent uses incorrect tools schema

2 participants