Skip to content

Conversation

@ChrisPenner
Copy link
Member

@ChrisPenner ChrisPenner commented Jul 1, 2025

Overview

  • Adds support for the mcp http protocol
    • Most agents prefer to use the stdio approach instead (Claude doesn't even support local http servers), and the stdio version is more robust since it doesn't require a UCM to be already running to connect to, but perhaps some agents prefer this method, but it's at least an option now; and it's handy to have for transcript tests.
  • Adds transcript tests against the http mcp server
  • Refactors the tool setup to something a bit more robust and consistent
  • Adds some new tools
  • Allows typechecking against a file.
  • Upgrades the hs-mcp dependency to a rewrite which should prevent orphaned mcp processes.

Implementation notes

  • Mounts the mcp http server interface at /codebase/mcp on the codeserver port.
  • Adds POST requests to the transcript parser
  • Delimits the BODY: and RESPONSE: in API transcripts accordingly (this improves clarity and also makes parsing easier)

Test coverage

  • Adds transcript tests for the MCP server

@ChrisPenner ChrisPenner force-pushed the mcp/init branch 2 times, most recently from f8c024a to d13a111 Compare August 2, 2025 06:33
@ChrisPenner ChrisPenner force-pushed the mcp/server branch 2 times, most recently from 661bbea to 7f60384 Compare August 5, 2025 17:34
@ChrisPenner ChrisPenner force-pushed the mcp/server branch 2 times, most recently from 7cdf2bb to 64f0ff8 Compare August 5, 2025 19:54
@ChrisPenner ChrisPenner marked this pull request as ready for review August 5, 2025 19:55
Base automatically changed from mcp/init to trunk August 5, 2025 20:21
@ChrisPenner ChrisPenner requested a review from aryairani August 5, 2025 20:22
@aryairani aryairani merged commit d5dc8eb into trunk Aug 7, 2025
31 checks passed
@aryairani aryairani deleted the mcp/server branch August 7, 2025 02:09
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.

3 participants