[SergeyMenshykh]

Summary

• Keep the first function when MCP tool names collide, warn for skipped duplicates, and dispatch calls to the exact advertised function.
• Add opt-in plugin-qualified tool names and warn when names exceed MCP's recommended length.
• Update MCP server documentation and cover naming, duplicate registration, exclusions, dispatch, and agent forwarding with tests.

Validation

• 70 focused MCP connector and agent tests passed.
• Ruff formatting passed; four changed Python files lint clean. The six Ruff property-docstring findings in agent.py also occur at the upstream base.

[github-actions]

Python Test Coverage Report •

File	Stmts	Miss	Cover	Missing
kernel.py	236	58	75%	149, 160, 164, 320–321, 323, 412, 418–419, 423–424, 477–481, 483–484, 497–506, 508, 510–511, 513–533, 535–540
agents
agent.py	342	48	85%	123, 130, 143–144, 147–149, 152–154, 163, 205, 210, 215, 220, 225, 229, 233, 237, 278–280, 308–309, 311, 315, 435, 437, 502, 516, 609, 623, 636, 688, 695–696, 699–700, 719, 766, 823, 908–909, 911–912, 966, 1037, 1052
connectors
mcp.py	496	173	65%	82, 87, 99, 109–118, 125–126, 129–130, 139–140, 147, 161–168, 176–180, 182–183, 185, 194, 303–309, 344–346, 351–353, 367–368, 371–372, 380–382, 426–427, 429, 431, 433–439, 443–445, 447–457, 461–462, 466–467, 471, 473, 477–478, 482, 490, 505, 520–528, 568–569, 587–588, 610–611, 614–615, 618–623, 627–628, 631–632, 635–641, 725, 727, 804, 806, 808, 810, 889, 891, 893, 895, 897, 965, 1022–1023, 1090, 1092, 1176–1178, 1181–1183, 1189, 1195, 1204–1205, 1207–1209, 1223–1224, 1226–1227, 1229–1231, 1234, 1241–1244, 1252, 1261–1262, 1267, 1269
TOTAL	29053	5561	80%

Python Unit Test Overview

Tests	Skipped	Failures	Errors	Time
4164	22 💤	0 ❌	0 🔥	2m 15s ⏱️

[Copilot]

Copilot review overview

🟢 Approval recommended

The implementation consistently preserves advertised tool identity and includes focused coverage for the affected behavior.

Review effort: Balanced
Findings: None

What changed in this PR

Makes Python MCP tool naming deterministic and consistent across kernels, functions, and agents.

Changes:

• Adds optional plugin-qualified tool names and length warnings.
• Retains the first duplicate and dispatches to the advertised function.
• Expands tests and MCP sample documentation.

File	Description
python/​semantic_kernel/​connectors/​mcp.py	Implements naming, collision handling, warnings, and exact dispatch.
python/​semantic_kernel/​kernel.py	Exposes the naming option through kernels.
python/​semantic_kernel/​agents/​agent.py	Forwards the naming option from agents.
python/​tests/​unit/​connectors/​mcp/​test_mcp.py	Tests collisions, exclusions, naming, limits, and dispatch.
python/​tests/​unit/​agents/​test_agent.py	Tests agent tool naming.
python/​samples/​demos/​mcp_server/​README.md	Documents default and qualified names.

---

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

[semantic-kernel-automation]

MAF Automated Review — Iteration 1

Result: Findings reported
Scope: full PR (1 commit(s)): 1b3d445d8387
Model: gpt-5.6-sol

Overview

The change consistently binds advertised MCP schemas to the exact function invoked, rejects unexposed names, preserves bare-name exclusions, and adds focused coverage for duplicate ordering, qualified names, length warnings, and wrapper forwarding. Two less-common but valid plugin configurations remain broken: aliased kernel plugin dictionaries now fail during server construction, and create_mcp_server_from_functions does not honor its plugin_name contract for existing KernelPlugin inputs when qualified naming is enabled.

Reviewed the supplied pull-request change set across correctness, security/reliability, architecture, and failure behavior.
2 verified findings remained after source verification (2 medium) across 1 file. Details are attached to the affected lines below.

Affected areas: python/semantic_kernel/connectors/mcp.py