Skip to content

Commit 1ab66a6

Browse files
committed
attempt #3
1 parent ef6c6dd commit 1ab66a6

2 files changed

Lines changed: 9 additions & 13 deletions

File tree

tests/integration/all-resource-types/expected-structure.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -601,7 +601,7 @@
601601
"spotChecks": {
602602
"apiInformation.json": {
603603
"properties.displayName": "KS MCP from External Server",
604-
"properties.path": "",
604+
"properties.path": "ks/mcp-external",
605605
"properties.type": "mcp",
606606
"properties.subscriptionRequired": false,
607607
"properties.backendId": "src-backend-mcp-external"
@@ -612,7 +612,7 @@
612612
"properties.backendId": "src-backend-mcp-external"
613613
}
614614
},
615-
"notes": "MCP API repackaging an external MCP server: empty path on the API, backendId points to the backend that holds the upstream URL (https://api.githubcopilot.com/mcp), and mcpProperties.endpoints.mcp.uriTemplate addresses the MCP endpoint exposed by that backend"
615+
"notes": "MCP API repackaging an external MCP server: backendId points to the backend that holds the upstream URL (https://api.githubcopilot.com/mcp), and mcpProperties.endpoints.mcp.uriTemplate addresses the MCP endpoint exposed by that backend"
616616
}
617617
]
618618
},

tests/integration/all-resource-types/source-apim.bicep

Lines changed: 7 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -930,17 +930,13 @@ resource apiMcpFromApi 'Microsoft.ApiManagement/service/apis@2025-09-01-preview'
930930
})
931931
}
932932

933-
// Explicit operation on the MCP-from-API surface for ApiOperation BVT coverage
934-
resource apiMcpFromApiPing 'Microsoft.ApiManagement/service/apis/operations@2025-09-01-preview' = {
935-
parent: apiMcpFromApi
936-
name: 'mcpPing'
937-
properties: {
938-
displayName: 'MCP Ping'
939-
method: 'GET'
940-
urlTemplate: '/ping'
941-
description: 'Lightweight health probe for MCP-from-API'
942-
}
943-
}
933+
// NOTE: MCP-typed APIs cannot host their own ApiOperation child resources.
934+
// APIM rejects PUT apis/{mcpApi}/operations/* with
935+
// "Operation entity cannot be defined for MCP API type".
936+
// MCP tools are surfaced via the parent API's properties.mcpTools array
937+
// (each entry's operationId references operations on a different,
938+
// non-MCP API \u2014 e.g. src-rest-openapi above). ApiOperation BVT coverage
939+
// is therefore provided by the REST APIs in this template, not by the MCP APIs.
944940

945941
// Backend for external MCP server (backendId pattern requires a backend resource)
946942
resource backendMcpExternal 'Microsoft.ApiManagement/service/backends@2025-09-01-preview' = {

0 commit comments

Comments
 (0)