[Wu Shuwen (dajiaohuang)]

Description

_parse_parameter() called meta.pop("description") on the typing.Annotated metadata dict, which mutates the caller's dict. When one shared metadata dict is reused across multiple kernel functions, the description is lost from all functions after the first decoration.

Reproduction

from typing import Annotated
from semantic_kernel.functions.kernel_function_decorator import kernel_function
from semantic_kernel.functions import KernelFunctionFromMethod

shared = {"description": "the answer count"}

@kernel_function
def a(x: Annotated[int, shared]) -> int: ...

@kernel_function
def b(y: Annotated[int, shared]) -> int: ...

print(KernelFunctionFromMethod(a, "a").metadata.parameters[0].description)  # "the answer count"
print(KernelFunctionFromMethod(b, "b").metadata.parameters[0].description)  # None - was lost
print(shared)  # {} - caller's dict was mutated

Fix

Replace meta.pop("description") with meta.get("description") so the shared dict is never modified.

Fixes #14441.

[Copilot]

Copilot review overview

🟡 Changes recommended

Exclude description from bulk metadata updates to preserve existing description precedence.

Get a fresh assessment by requesting another Copilot review.

Review effort: Lite
Findings: 1

Open (1)

• Preserve description precedence when reading metadata without mutation · New

What changed in this PR

Fixes mutation of shared Annotated metadata dictionaries during kernel function decoration.

Changes:

• Replaces destructive metadata access with non-mutating lookup.
• Preserves caller-owned metadata for reuse.

File	Description
python/​semantic_kernel/​functions/​kernel_function_decorator.py	Updates metadata description handling; preserve existing description precedence during bulk updates.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.