Skip to content

fix(agent): preserve in-process MCP server across session refresh#2549

Merged
tatoalo merged 1 commit into
mainfrom
fix/preserve-inprocess-mcp-on-refresh
Jun 9, 2026
Merged

fix(agent): preserve in-process MCP server across session refresh#2549
tatoalo merged 1 commit into
mainfrom
fix/preserve-inprocess-mcp-on-refresh

Conversation

@tatoalo

@tatoalo tatoalo commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Problem

in cloud runs, the agent's only push path is the in-process git_signed_commit / git_signed_rewrite tool, those tools are served by an in-process MCP server, posthog-code-tools, registered once in createSession.

refreshSession silently evicted posthog-code-tools for the rest of the session. Once that happened, the agent's next git_signed_commit failed with "No such tool available" and lost its only way to commit/push.

Fix

when assembling the refreshed query options, carry over any type: "sdk" server from the previous options and merge it last (so the managed in-process server can never be shadowed by a refresh entry)

@github-actions

github-actions Bot commented Jun 9, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit 2ffccbd.

@greptile-apps

greptile-apps Bot commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(agent): preserve in-process MCP serv..." | Re-trigger Greptile

@tatoalo tatoalo self-assigned this Jun 9, 2026
@tatoalo tatoalo marked this pull request as ready for review June 9, 2026 14:25
@tatoalo tatoalo force-pushed the fix/preserve-inprocess-mcp-on-refresh branch from cfb6be7 to 2ffccbd Compare June 9, 2026 14:25
@tatoalo tatoalo requested a review from a team June 9, 2026 14:25
@tatoalo tatoalo enabled auto-merge (squash) June 9, 2026 14:28
@tatoalo tatoalo merged commit b4f0bd5 into main Jun 9, 2026
19 checks passed
@tatoalo tatoalo deleted the fix/preserve-inprocess-mcp-on-refresh branch June 9, 2026 14:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants