Skip to content

Add MCP health and keep-alive probes - #2723

Merged
Widthdom merged 4 commits into
mainfrom
fix-issue1740-1791
May 31, 2026
Merged

Add MCP health and keep-alive probes#2723
Widthdom merged 4 commits into
mainfrom
fix-issue1740-1791

Conversation

@Widthdom

Copy link
Copy Markdown
Owner

Summary

  • Add structured MCP ping health data and HTTP GET /healthz on the existing MCP HTTP listener.
  • Add opt-in HTTP /events keep-alive notifications via CDIDX_MCP_KEEP_ALIVE_INTERVAL_S.
  • Document the MCP health/keep-alive contracts and add bilingual changelog fragments.

Validation

  • dotnet test tests/CodeIndex.Tests/CodeIndex.Tests.csproj --filter "FullyQualifiedName=CodeIndex.Tests.McpServerTests.ProcessLineAsync_PingReturnsStructuredHealth|FullyQualifiedName=CodeIndex.Tests.HttpMcpTransportTests.HttpTransport_Healthz_ReturnsStructuredHealth|FullyQualifiedName=CodeIndex.Tests.HttpMcpTransportTests.HttpTransport_EventsStream_EmitsOptInKeepAliveNotifications" -p:UseSharedCompilation=false
  • dotnet run --project tools/CodeIndex.Changelog -- check
  • git diff --check origin/main..HEAD
  • Adversarial review: No blocking/actionable issues found.

Documentation and changelog

  • README.md
  • DEVELOPER_GUIDE.md
  • changelog.d/unreleased/1740.added.md
  • changelog.d/unreleased/1791.added.md

Notes

  • A broader McpServerTests|HttpMcpTransportTests run was blocked by existing local git signing configuration in tests that create commits (id_ed25519_signing passphrase prompt), unrelated to this PR.
  • Local cdidx status --check --json remains stale because repeated index refresh attempts ended with E012_INTERRUPTED in this environment.

Fixes #1740
Fixes #1791

@Widthdom
Widthdom merged commit 11d6192 into main May 31, 2026
11 checks passed
@Widthdom
Widthdom deleted the fix-issue1740-1791 branch May 31, 2026 06:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant