Skip to content

fix: Exit the MCP server immediately on stdin EOF or SIGTERM - #282

Merged
Pringled merged 4 commits into
mainfrom
fix/mcp-shutdown-hang
Oct 5, 2026
Merged

Pringled merged 4 commits into
mainfrom
fix/mcp-shutdown-hang

Conversation

@Pringled

@Pringled Pringled commented Oct 3, 2026

Copy link
Copy Markdown
Member

MCP hosts stop the server by closing stdin or sending SIGTERM. Closing stdin during an index build or model load left the process running until that work finished, and SIGTERM exited with code 143, which Go-based hosts like Antigravity report as an error. The server now exits immediately with code 0 in both cases. This resolves #280

@codecov

codecov Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Files with missing lines Coverage Δ
src/semble/cache.py 100.00% <100.00%> (ø)
src/semble/cli.py 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@greptile-apps

greptile-apps Bot commented Oct 3, 2026 •

Copy link
Copy Markdown

RetriggerConfidence Score: 5/5

[High risk] Changes how the server shuts down and handles signals.

The PR appears safe to merge; no new actionable issue was identified.

Reviews (2) · Last reviewed commit: "Register SIGTERM handler earlier and tre..."

Comment thread src/semble/cli.py Outdated
Comment thread src/semble/cli.py
@Pringled
Pringled merged commit f1daa42 into main Oct 5, 2026
15 checks passed
@Pringled
Pringled deleted the fix/mcp-shutdown-hang branch October 5, 2026 08:34
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.

Any Quickstart for Antigravity integration?

2 participants