Skip to content

Python: Fix non-string KernelArguments being converted to strings in prompt template function calls #13292

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 3 commits into from
Oct 30, 2025
Merged

Python: Fix non-string KernelArguments being converted to strings in prompt template function calls #13292

merged 3 commits into from
Oct 30, 2025
+127 −1

Conversation

@moonbox3
Copy link
Collaborator

Motivation and Context

Fixes #13199 - Non-string kernel arguments are now preserved when passed to functions in prompt templates. Adds unit tests.

Contribution Checklist

@moonbox3 moonbox3 requested a review from a team as a code owner October 24, 2025 05:12
@markwallace-microsoft markwallace-microsoft added the python Pull requests for the Python Semantic Kernel label Oct 24, 2025
@markwallace-microsoft
Copy link
Member

markwallace-microsoft commented Oct 24, 2025 •
edited
Loading

Python Test Coverage

Python Test Coverage Report •
FileStmtsMissCoverMissing
template_engine/blocks
   code_block.py84198%119
   var_block.py41197%98
TOTAL27309456183% 

Python Unit Test Overview

Tests Skipped Failures Errors Time
3763 23 💤 0 ❌ 0 🔥 1m 39s ⏱️

@moonbox3 moonbox3 enabled auto-merge October 28, 2025 05:54
@moonbox3 moonbox3 requested a review from dmytrostruk October 28, 2025 10:08
@moonbox3 moonbox3 added this pull request to the merge queue Oct 28, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 28, 2025
@moonbox3 moonbox3 added this pull request to the merge queue Oct 29, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 29, 2025
@moonbox3 moonbox3 added this pull request to the merge queue Oct 29, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 29, 2025
@moonbox3 moonbox3 enabled auto-merge October 30, 2025 04:31
@moonbox3 moonbox3 added this pull request to the merge queue Oct 30, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 30, 2025
@moonbox3 moonbox3 added this pull request to the merge queue Oct 30, 2025
Merged via the queue into microsoft:main with commit 05d659d Oct 30, 2025
42 of 48 checks passed
@moonbox3 moonbox3 deleted the fix-13199 branch October 30, 2025 06:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@TaoChenOSU TaoChenOSU TaoChenOSU approved these changes

@dmytrostruk dmytrostruk dmytrostruk approved these changes

Assignees

No one assigned

Labels

python Pull requests for the Python Semantic Kernel

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Python: Bug: [Python] - Using Non-String KernelArguments in Prompt Template function parameters fails

4 participants

@moonbox3 @markwallace-microsoft @TaoChenOSU @dmytrostruk