Add MCP server support for Consumption Logic Apps#8901
Add MCP server support for Consumption Logic Apps#8901
Conversation
🤖 AI PR Validation ReportPR Review ResultsThank you for your submission! Here's detailed feedback on your PR title and body compliance:✅ PR Title
✅ Commit Type
✅ Risk Level
✅ What & Why
✅ Impact of Change
✅ Test Plan
|
| Section | Status | Recommendation |
|---|---|---|
| Title | ✅ | Title is good; optional expansion of MCP acronym for clarity |
| Commit Type | ✅ | Correct (feature) |
| Risk Level | ✅ | risk:medium label matches the PR body and scope |
| What & Why | ✅ | Good; consider listing the main functional surface areas changed |
| Impact of Change | ✅ | Good; optionally note rollout/feature-flag info if any |
| Test Plan | ✅ | Unit tests present in diff — add E2E/manual test notes or rationale |
| Contributors | OK, but add additional contributors if applicable | |
| Screenshots/Videos | Add if there are UI changes visible to users |
Final notes / next steps
- This PR passes the PR title/body template checks.
- Advised risk level:
risk:medium(matches submitter's selection). No escalation required. - Recommendations to improve reviewability before merge:
- In Test Plan, either add E2E tests or include a short justification for why unit tests are sufficient and document any manual test steps run.
- Add brief manual test steps to the Test Plan (e.g., create built-in MCP connection, verify listMcpTools returns expected values, confirm workflow payloads filter builtin MCP connections) to help reviewers validate behavior quickly.
- If the UX/flow changed visually (connection creation panel logic was modified), attach screenshots showing the new flow (especially the logic that hides the name input for built-in MCP on Consumption SKU).
- Consider adding a short migration/rollout note if consumers of manifests or connector ids need to be aware of any naming/compatibility considerations.
Please update the PR body with any of the optional clarifying items above (Test Plan manual steps or E2E plans, screenshots if UX changed) if you can — otherwise this is good to land from a PR description/template perspective. Thanks!
Last updated: Wed, 18 Mar 2026 15:17:01 GMT
There was a problem hiding this comment.
Pull request overview
Adds Consumption SKU support for MCP (Model Context Protocol) by introducing a consumption MCP connector + built-in operation manifest wiring, and updating the connection creation UI/service to recognize MCP connections.
Changes:
- Add consumption MCP connector + built-in MCP Client operation manifest and wire them into the consumption operation manifest service.
- Extend
ConsumptionConnectionServiceto recognize MCP connectors and create MCP connections. - Update Designer V2 create-connection panel behavior for consumption MCP connections and add unit tests for the new connection logic.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 5 comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Tweaks lint-staged ESLint invocation to reduce ignored-file warnings. |
| libs/logic-apps-shared/src/designer-client-services/lib/consumption/operationmanifest.ts | Adds MCP built-in operation support (mcpclienttool → nativemcpclient) and returns the MCP operation manifest/operation metadata. |
| libs/logic-apps-shared/src/designer-client-services/lib/consumption/manifests/mcpclientconnector.ts | Introduces the consumption MCP connector manifest (auth parameter sets, metadata). |
| libs/logic-apps-shared/src/designer-client-services/lib/consumption/manifests/builtinmcpclient.ts | Adds the built-in MCP Client operation manifest including listMcpTools dynamic list for tool selection. |
| libs/logic-apps-shared/src/designer-client-services/lib/consumption/connection.ts | Adds MCP connector lookup and MCP-specific connection creation paths (built-in vs managed). |
| libs/logic-apps-shared/src/designer-client-services/lib/consumption/tests/connection.spec.ts | Adds unit tests for MCP connection creation helpers/paths. |
| libs/designer-v2/src/lib/ui/panel/connectionsPanel/createConnection/createConnection.tsx | Hides the “name” input for Consumption MCP connections via a runtime service check. |
libs/logic-apps-shared/src/designer-client-services/lib/consumption/connection.ts
Show resolved
Hide resolved
libs/logic-apps-shared/src/designer-client-services/lib/consumption/connection.ts
Outdated
Show resolved
Hide resolved
...logic-apps-shared/src/designer-client-services/lib/consumption/manifests/builtinmcpclient.ts
Show resolved
Hide resolved
libs/designer-v2/src/lib/ui/panel/connectionsPanel/createConnection/createConnection.tsx
Show resolved
Hide resolved
libs/logic-apps-shared/src/designer-client-services/lib/consumption/operationmanifest.ts
Show resolved
Hide resolved
📊 Coverage CheckThe following changed files need attention:
Please add tests for the uncovered files before merging. |
…cAppsUX into bhavya/mcpconsumption
…ya/mcpconsumption
…cAppsUX into bhavya/mcpconsumption
Commit Type
Risk Level
What & Why
Adds MCP (Model Context Protocol) server support for Consumption Logic Apps. This enables consumption SKU workflows to connect to custom MCP servers and use MCP tools as actions, bringing feature parity with the Standard SKU's existing MCP support.
Impact of Change
Test Plan
Contributors
@Bhavd13
Screenshots/Videos
mcp.recording.mp4