Skip to content

fix(protocol-resolver): map CAP 'mcp' protocol to ORD apiProtocol#519

Merged
zongqichen merged 2 commits into
mainfrom
fix/mcp-protocol-mapping
Jul 16, 2026
Merged

fix(protocol-resolver): map CAP 'mcp' protocol to ORD apiProtocol#519
zongqichen merged 2 commits into
mainfrom
fix/mcp-protocol-mapping

Conversation

@zongqichen

@zongqichen zongqichen commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@zongqichen zongqichen added the run-e2e Trigger e2e test pipeline label Jul 15, 2026

@hyperspace-pr-bot hyperspace-pr-bot Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR is clean and well-structured. The core change — adding "mcp": ORD_API_PROTOCOL.MCP to CAP_TO_ORD_PROTOCOL_MAP — is correct and follows the exact same pattern used for GraphQL. The test refactoring in api-resource.test.js simplifies the MCP test by replacing the brittle jest.doMock/jest.resetModules approach with a proper beforeEach/afterEach protocol registration, which is a clear improvement. The only gap I flagged is a missing "silent skip without plugin" test case to match the symmetry with the GraphQL test suite.

PR Bot Information

Version: 1.27.6

  • Correlation ID: 5290cc40-8056-11f1-8968-26e18ef9f158
  • Agent Instructions:
  • LLM: anthropic--claude-4.6-sonnet
  • File Content Strategy: Full file content
  • Event Trigger: pull_request.opened

Comment thread __tests__/unit/protocol-resolver.test.js
@zongqichen zongqichen added the retry-e2e When the e2e pipeline failed, add retry-e2e label Jul 15, 2026
@ord-e2e-bot ord-e2e-bot Bot removed the retry-e2e When the e2e pipeline failed, add retry-e2e label Jul 15, 2026
@zongqichen zongqichen added the retry-e2e When the e2e pipeline failed, add retry-e2e label Jul 15, 2026
@ord-e2e-bot ord-e2e-bot Bot removed the retry-e2e When the e2e pipeline failed, add retry-e2e label Jul 15, 2026
@zongqichen zongqichen added the retry-e2e When the e2e pipeline failed, add retry-e2e label Jul 15, 2026
@zongqichen
zongqichen requested a review from mlakov July 15, 2026 16:20
@ord-e2e-bot ord-e2e-bot Bot removed the retry-e2e When the e2e pipeline failed, add retry-e2e label Jul 15, 2026
@zongqichen zongqichen added the retry-e2e When the e2e pipeline failed, add retry-e2e label Jul 15, 2026
@ord-e2e-bot ord-e2e-bot Bot removed the retry-e2e When the e2e pipeline failed, add retry-e2e label Jul 15, 2026
@zongqichen
zongqichen merged commit 2ffb818 into main Jul 16, 2026
5 checks passed
@zongqichen
zongqichen deleted the fix/mcp-protocol-mapping branch July 16, 2026 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-e2e Trigger e2e test pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants