fix(mcp): support ndjson stdio clients#14
Merged
Conversation
Allow kmsg mcp-server to speak either MCP Content-Length framing or newline-delimited JSON-RPC, returning responses with the same transport shape used by the request. Also make startup readiness less brittle for slow KakaoTalk AX state and surface FOCUS_FAIL as an actionable MCP error code. Tested: python3 -m unittest discover -s tests -v Tested: swift build Not-tested: live KakaoTalk send/read via Accessibility
Add a visible MIT license badge and a License section that points readers to the repository LICENSE file. Tested: git diff --check
Add an animation-style cyberpunk visual to the README MCP section so the PR shows the kmsg philosophy at a glance: human intent, chat, protocol bridge, fail-closed safety, and automation. Constraint: Use the generated local image artifact rather than the OpenAI Images API path because the API-key lane hit billing limits and the OAuth API token lacked image request scope. Rejected: Use the second-newest candidate image | it includes visible English text and is less suitable for a clean README asset. Confidence: high Scope-risk: narrow Tested: git diff --check Tested: file assets/kmsg-mcp-cyberpunk-hero.png Tested: sips -g pixelWidth -g pixelHeight assets/kmsg-mcp-cyberpunk-hero.png Not-tested: GitHub README rendering after push
Refine the Rust comparison to separate narrow stdio/JSON and cold-start costs from the end-to-end latency dominated by macOS AX and KakaoTalk UI response time. Constraint: Keep the answer aligned with the current Swift MCP implementation and avoid overstating unmeasured Rust gains. Confidence: high Scope-risk: narrow Tested: python3 -m unittest discover -s tests -v Tested: swift build Tested: git diff --check Tested: MCP NDJSON and Content-Length smoke checks against .build/debug/kmsg
Removed image tag related to MCP integration from README.
channprj
approved these changes
Jun 5, 2026
channprj
left a comment
channprj
left a comment
Owner
There was a problem hiding this comment.
I removed vibe image from your pr directly.
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
2 participants
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.
Summary
kmsg mcp-serverTesting
python3 -m unittest discover -s tests -vswift buildNotes