Skip to content

Conversation

@anticomputer
Copy link
Contributor

No description provided.

Bas Alberts added 3 commits September 15, 2025 18:34
Moves the CodeQL MCP Server to a streamable transport and adds
the required process management code to start and stop local
streamable servers on demand.
Copilot AI review requested due to automatic review settings September 16, 2025 17:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds local streamable MCP server support, allowing the framework to start and manage MCP servers locally using HTTP transport instead of only stdio-based communication.

  • Implements StreamableMCPThread class for managing local streamable MCP server processes
  • Updates MCP server configurations to support the streamable transport with local process management
  • Migrates existing MCP servers from FastMCP 1.x to FastMCP 2.0 and enables HTTP transport

Reviewed Changes

Copilot reviewed 9 out of 10 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
mcp_utils.py Adds StreamableMCPThread process management class and extends mcp_client_params to handle streamable transport with local process spawning
main.py Updates deployment logic to start/stop streamable MCP server processes and handle connection waiting
toolboxes/codeql.yaml Configures CodeQL toolbox to use streamable transport with HTTP endpoint
mcp_servers/*/*.py Updates FastMCP imports to version 2.0 and enables HTTP transport for servers
mcp_servers/codeql/client.py Improves error handling and makes query server keep-alive default to true
README.md Updates documentation with configuration examples and clarifies Docker usage

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@anticomputer anticomputer merged commit 7f39a83 into main Sep 16, 2025
2 checks passed
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.

2 participants