Non-blocking stdio startup, atomic cache writes, deduplicate AuthTest#259
Open
mark-liu wants to merge 2 commits intokorotovsky:masterfrom
Open
Non-blocking stdio startup, atomic cache writes, deduplicate AuthTest#259mark-liu wants to merge 2 commits intokorotovsky:masterfrom
mark-liu wants to merge 2 commits intokorotovsky:masterfrom
Conversation
…hTest Three related fixes for reliability when running multiple concurrent MCP server instances (e.g. one per Slack workspace): 1. stdio transport no longer blocks on cache warmup — serves immediately while caches build in the background, matching the existing SSE/HTTP behaviour. Previously, large workspaces (100K+ users) would time out before the server started listening. 2. Cache files are now written atomically via temp-file-and-rename, preventing concurrent readers from seeing partial/corrupt JSON. 3. AuthTest response from initial validation is passed through to NewMCPSlackClient, eliminating a redundant API call on every startup. A small random jitter (0-3s) staggers concurrent instance auth requests. Signed-off-by: Mark Liu <mark@prove.com.au>
This was referenced Mar 26, 2026
SLACK_MCP_COMPACT_OUTPUT=1 enables a slimmed-down CSV format that drops fields rarely needed by LLM agents: MsgID, UserID, ThreadTs, FileCount, AttachmentIDs, HasMedia. Merges UserName/RealName/BotName into a single User column. Truncates link preview attachments to title-only. ~40% reduction in output size. Controlled by env var, defaults to off.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Three related fixes for reliability when running multiple concurrent
MCP server instances (e.g. one per Slack workspace):
stdio transport no longer blocks on cache warmup — serves
immediately while caches build in the background, matching the
existing SSE/HTTP behaviour. Large workspaces (100K+ users)
previously timed out before the server started listening.
Cache files written atomically via temp-file-and-rename,
preventing concurrent readers from seeing partial/corrupt JSON.
AuthTest response from initial validation passed through to
NewMCPSlackClient, eliminating a redundant API call on every
startup. Small random jitter (0-3s) staggers concurrent
instance auth requests.
Supersedes #249 and #256 (squashed and rebased).