fix(agent): Reconnect signed-commit MCP server after session refresh#2724
fix(agent): Reconnect signed-commit MCP server after session refresh#2724charlesvien wants to merge 4 commits into
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
React Doctor found no issues in the changed files. 🎉 Reviewed by React Doctor for commit |
There was a problem hiding this comment.
The PR exceeds the auto-review size limit and has zero reviews. The behavioral changes — in-flight mutation of session MCP state, pre-prompt self-healing before every cloud turn, and cache clearing on reconnect — are non-trivial and warrant a domain expert sign-off before auto-approval.
|
Reviews (1): Last reviewed commit: "simplify signed-commit heal and dedup te..." | Re-trigger Greptile |
There was a problem hiding this comment.
The PR exceeds the auto-review size limit (589 lines across 9 files), and the bot review flagged substantive behavioral concerns: in-flight mutation of session MCP state, pre-prompt self-healing before every cloud turn, and cache clearing on reconnect. These are non-trivial agent session management changes that need a domain expert sign-off.
joshsny
left a comment
There was a problem hiding this comment.
Should we allow attempts for non-signed commits? We've had quite a few issues with the signed commit tool and it's not required on all repos, on the other hand it's nice to have this available as an MCP tool that we can track and later restrict the permissions for, will leave it for you to decide

Problem
In cloud runs, a session refresh disconnected the signed-commit MCP server (posthog-code-tools), so the agent could no longer commit and a turn's work appeared lost while raw git stays blocked.
Changes
How did you test this?
Manually
Automatic notifications