Skip to content

feat(mcp): Phase 6 — add stdio ping - #372

Merged
erishforG merged 1 commit into
release/1.0from
feat/auto-293-phase6
Jun 30, 2026
Merged

feat(mcp): Phase 6 — add stdio ping#372
erishforG merged 1 commit into
release/1.0from
feat/auto-293-phase6

Conversation

@erishforG

Copy link
Copy Markdown
Owner

무엇

MCP stdio JSON-RPC dispatcher에 lifecycle ping 메서드를 추가합니다.

Refs #293. Claude Desktop/Cursor 같은 MCP clients가 transport health check로 ping을 보낼 때 protocol-clean한 빈 object result를 받을 수 있게 합니다. @erishforG

변경

  • src/mcp/mod.rs에서 ping 메서드를 {} result로 응답
  • stdio smoke fixture에 ping recording 추가
  • MCP fixture README와 client registration doc의 smoke contract 갱신

다음 Phase 힌트

  • client installer hook 경로가 확정되면 docs/mcp/clients.md의 Phase 5 항목을 실제 설치 절차로 확장
  • delegated auth 전달 형식이 정해지면 tools/call auth context fixture 추가

리스크

Low. MCP 전용 dispatcher에 non-mutating lifecycle method를 추가했고 기존 CLI surface와 worktree/git 핵심 로직은 변경하지 않았습니다.

롤백

이 PR revert 시 ping support와 fixture/doc 계약만 제거됩니다.

Test plan

  • cargo test --quiet mcp::tests::ping_returns_empty_result_for_client_health_checks
  • cargo test --quiet mcp::tests::stdio_recording_fixtures_match_dispatcher
  • cargo build --quiet
  • cargo fmt --check
  • cargo clippy --all-targets -- -D warnings
  • cargo test --quiet

Refs #293

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@erishforG erishforG added the auto-improvement Daily auto-generated improvement PR/issue label Jun 29, 2026
@erishforG
erishforG merged commit effcb78 into release/1.0 Jun 30, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-improvement Daily auto-generated improvement PR/issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant