Skip to content

Python: Don't return code output via on_intermediate_msg callback#13058

Merged
moonbox3 merged 1 commit into
microsoft:mainfrom
moonbox3:assistant-agent-code-return
Sep 4, 2025
Merged

Python: Don't return code output via on_intermediate_msg callback#13058
moonbox3 merged 1 commit into
microsoft:mainfrom
moonbox3:assistant-agent-code-return

Conversation

@moonbox3

@moonbox3 moonbox3 commented Sep 3, 2025

Copy link
Copy Markdown
Collaborator

Motivation and Context

Current, while using an OpenAIAssistantAgent, we are returning code output via the on_intermediate_message callback. This shouldn't be happening right now, as all other agents return code via TextContent with a metadata {"code": True} key-value pair in the dictionary.

Description

Contribution Checklist

@moonbox3 moonbox3 requested a review from a team as a code owner September 3, 2025 08:39
@moonbox3 moonbox3 self-assigned this Sep 3, 2025
@moonbox3 moonbox3 added agents python Pull requests for the Python Semantic Kernel labels Sep 3, 2025
@moonbox3 moonbox3 enabled auto-merge September 3, 2025 08:39
@moonbox3

moonbox3 commented Sep 3, 2025

Copy link
Copy Markdown
Collaborator Author

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
agents/open_ai
   assistant_thread_actions.py3385783%107, 110, 116, 195, 197, 236–242, 416, 474, 487, 500, 599–600, 602–603, 609–610, 612–613, 615–618, 620–621, 623–624, 626–627, 641–645, 648–650, 697, 699–700, 704–708, 710–711, 725–726, 797, 802, 834
TOTAL27111472882% 

Python Unit Test Overview

Tests Skipped Failures Errors Time
3715 22 💤 0 ❌ 0 🔥 1m 39s ⏱️

@moonbox3 moonbox3 added this pull request to the merge queue Sep 4, 2025
@github-merge-queue github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Sep 4, 2025
@moonbox3 moonbox3 added this pull request to the merge queue Sep 4, 2025
Merged via the queue into microsoft:main with commit 06d6494 Sep 4, 2025
32 checks passed
@moonbox3 moonbox3 deleted the assistant-agent-code-return branch September 4, 2025 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agents python Pull requests for the Python Semantic Kernel

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Python: Bug: Assistant Agent puts code interpreter streaming chunks to on_intermediate_message

3 participants