Skip to content

Commit 4bc6758

Browse files
authored
fix: drop stale mcpServerInformation.json references (#205)
PR #173 moved MCP server configuration into apiInformation.json and stopped producing the mcpServerInformation.json sidecar, but the all-resource-types integration manifest and reference docs still expected it. This caused the round-trip test to fail (2 checks) on the src-mcp-todos MCP API. - expected-structure.json: remove the sidecar from src-mcp-todos and fold its spot-checks (mcpProperties.endpoints.mcp.uriTemplate, mcpTools) into apiInformation.json where the data now lives. - artifact-format.md / resource-types.md: mark McpServer config as embedded in apiInformation.json instead of the removed sidecar file.
1 parent 03fe887 commit 4bc6758

3 files changed

Lines changed: 4 additions & 7 deletions

File tree

docs/reference/artifact-format.md

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,6 @@ apim-artifacts/
2727
│ ├── policy.xml # API-level policy
2828
│ ├── specification.yaml # OpenAPI spec (if applicable)
2929
│ ├── wiki.md # API wiki (if exists)
30-
│ ├── mcpServerInformation.json # MCP server config (if exists)
3130
│ ├── operations/
3231
│ │ └── get-pets/
3332
│ │ └── policy.xml # Operation-level policy
@@ -151,7 +150,7 @@ All 34 APIM resource types and their artifact mappings:
151150
| ApiWiki | `apis/{api}` | `wiki.md` |
152151
| GraphQLResolver | `apis/{api}/resolvers/{resolver}` | `resolverInformation.json` |
153152
| GraphQLResolverPolicy | `apis/{api}/resolvers/{resolver}` | `policy.xml` |
154-
| McpServer | `apis/{api}` | `mcpServerInformation.json` |
153+
| McpServer | `apis/{api}` | _embedded in `apiInformation.json`_ |
155154

156155
### Gateway Child Resources
157156

docs/reference/resource-types.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ Resources scoped to a specific API.
6767
| ApiWiki | `/apis/{name}/wikis/default` | `apis/{0}` | `wiki.md` | Markdown documentation page for an API |
6868
| GraphQLResolver | `/apis/{name}/resolvers/{resolver}` | `apis/{0}/resolvers/{1}` | `resolverInformation.json` | GraphQL field resolver configuration |
6969
| GraphQLResolverPolicy | `/apis/{name}/resolvers/{resolver}/policies/policy` | `apis/{0}/resolvers/{1}` | `policy.xml` | Policy applied to a GraphQL resolver |
70-
| McpServer | `/apis/{name}/mcpServers/default` | `apis/{0}` | `mcpServerInformation.json` | MCP (Model Context Protocol) server linked to an API |
70+
| McpServer | `/apis/{name}/mcpServers/default` | `apis/{0}` | _embedded in `apiInformation.json`_ | MCP (Model Context Protocol) server linked to an API |
7171

7272
## Gateway Resources
7373

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

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -614,15 +614,13 @@
614614
},
615615
{
616616
"name": "src-mcp-todos",
617-
"files": ["apiInformation.json", "mcpServerInformation.json", "policy.xml"],
617+
"files": ["apiInformation.json", "policy.xml"],
618618
"spotChecks": {
619619
"apiInformation.json": {
620620
"properties.displayName": "KS MCP Todos Server",
621621
"properties.path": "ks/mcp-todos",
622622
"properties.type": "mcp",
623-
"properties.subscriptionRequired": false
624-
},
625-
"mcpServerInformation.json": {
623+
"properties.subscriptionRequired": false,
626624
"properties.mcpProperties.endpoints.mcp.uriTemplate": "/mcp",
627625
"properties.mcpTools": "exists"
628626
}

0 commit comments

Comments
 (0)