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
|
| Section | Status | Recommendation |
|---|---|---|
| Title | ✅ | Keep as-is or optionally add a short scope prefix |
| Commit Type | ✅ | None |
| Risk Level | ✅ | None (advised risk = Medium) |
| What & Why | ✅ | Optionally expand with short user scenarios |
| Impact of Change | ✅ | Consider listing key touched modules/files |
| Test Plan | ✅ | Add/clarify E2E/integration testing plan |
| Contributors | ✅ | Optionally list PMs/Designers if applicable |
| Screenshots/Videos | Optional: screenshot of connection UI if helpful |
Summary / Next Steps
This PR follows the required PR body template and includes unit tests for the new consumption MCP logic. The provided risk label (risk:medium) matches the scope of the changes and the code diff, so no change to the risk label is required.
Recommendations to improve reviewer confidence before merging:
- Add a short note in the Test Plan explaining whether/when E2E or integration tests will be added for the listMcpTools flow (native vs managed connection shapes). If E2E is intentionally omitted, explain why.
- Optionally list the primary files/modules touched in the Impact of Change to help reviewers locate the main changes quickly (e.g., libs/logic-apps-shared/src/designer-client-services/lib/consumption/{connection.ts, connector.ts, operationmanifest.ts, manifests/*}, new tests under tests).
- (Optional) Add a screenshot showing the connection creation UI for the MCP connection types if you made visible UI changes to the connection dialog; otherwise you can ignore.
Please update the PR body only if you want to add the optional clarifications above; otherwise this PR is good to proceed from a title/body compliance perspective. Thank you for the thorough PR and the included unit tests!
Last updated: Wed, 11 Mar 2026 10:28:52 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
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