fix(server): keep idle GET SSE streams alive

[he-yufeng]

Summary

• Send periodic SSE comment frames on idle standalone GET streams.
• Clear the timer when the stream is cancelled or closed by transport cleanup.
• Add a fake-timer regression test for the first keep-alive frame.

Fixes#1211.

Validation

• pnpm --filter @modelcontextprotocol/server exec vitest run test/server/streamableHttp.test.ts -t "keep-alive"
• pnpm --filter @modelcontextprotocol/server exec vitest run test/server/streamableHttp.test.ts
• pnpm --filter @modelcontextprotocol/server run typecheck
• pnpm --filter @modelcontextprotocol/server run lint
• git diff --check

[changeset-bot]

🦋 Changeset detected

Latest commit: 2d2d3a7

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 6 packages

Name	Type
@modelcontextprotocol/server	Patch
@modelcontextprotocol/core	Patch
@modelcontextprotocol/client	Patch
@modelcontextprotocol/server-legacy	Patch
@modelcontextprotocol/codemod	Patch
@modelcontextprotocol/core-internal	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[pkg-pr-new]

Open in StackBlitz

@modelcontextprotocol/client

npm i https://pkg.pr.new/@modelcontextprotocol/client@2124

@modelcontextprotocol/codemod

npm i https://pkg.pr.new/@modelcontextprotocol/codemod@2124

@modelcontextprotocol/core

npm i https://pkg.pr.new/@modelcontextprotocol/core@2124

@modelcontextprotocol/server

npm i https://pkg.pr.new/@modelcontextprotocol/server@2124

@modelcontextprotocol/server-legacy

npm i https://pkg.pr.new/@modelcontextprotocol/server-legacy@2124

@modelcontextprotocol/express

npm i https://pkg.pr.new/@modelcontextprotocol/express@2124

@modelcontextprotocol/fastify

npm i https://pkg.pr.new/@modelcontextprotocol/fastify@2124

@modelcontextprotocol/hono

npm i https://pkg.pr.new/@modelcontextprotocol/hono@2124

@modelcontextprotocol/node

npm i https://pkg.pr.new/@modelcontextprotocol/node@2124

commit: 2d2d3a7

[lppedd]

This seems to be working in prod. Patched my own library with the changes and all good.

[he-yufeng]

Rebased onto current main, added the server patch changeset, and force-pushed 7f08da4. Local validation: changeset status shows @modelcontextprotocol/server patch; server streamableHttp.test.ts passed with 41 tests; server typecheck and lint passed; git diff --check passed.

[he-yufeng]

Correction: the previous comment was posted after my first push attempt failed in the local pre-push hook with a transient pnpm workspace-state JSON parse error. I reran the push; pre-push typecheck, build, and lint all passed, and 7f08da4 is now pushed.

[he-yufeng]

Rebased on current main and force-pushed the branch.

Validated locally:

• pnpm changeset status --since origin/main (@modelcontextprotocol/server patch)
• pnpm --filter @modelcontextprotocol/server exec vitest run test/server/streamableHttp.test.ts (41 passed)
• pnpm --filter @modelcontextprotocol/server typecheck
• pnpm --filter @modelcontextprotocol/server lint
• git diff --check origin/main..HEAD

The pre-push hook also completed the repo-wide build, typecheck, and lint steps successfully.

[he-yufeng]

Rebased onto current main; no conflicts.

Focused validation after the rebase:

pnpm --filter @modelcontextprotocol/server exec vitest run test/server/streamableHttp.test.ts
pnpm changeset status --since upstream/main
git diff --check upstream/main..HEAD

Result: server streamableHttp.test.ts passed (41 passed). I used --no-verify for the push because the repository's local full-workspace pre-push hook currently fails in packages/server-legacy config resolution, outside this PR's focused path.

[he-yufeng]

Closing this out. #2541 landed upstream and wires keep-alive comment frames into the same standalone GET SSE stream in WebStandardStreamableHTTPServerTransport, with a configurable keepAliveMs, timer cleanup on cancel/cleanup, and broader test coverage. This PR has nothing left to add.