Skip to content

Commit 70ab5ca

Browse files
committed
Move conformance client/server scripts to scripts/conformance/
Address review feedback: the conformance client is CI test infrastructure, not a user-facing example. Move it to scripts/conformance/client.py as a standalone script and colocate it with the server script (renamed from scripts/run-server-conformance.sh to scripts/conformance/run-server.sh). This removes the mcp-conformance-client workspace member and its lockfile entry. The client script is now run directly via uv run python scripts/conformance/client.py.
1 parent 8d139d7 commit 70ab5ca

File tree

6 files changed

+6
-83
lines changed

6 files changed

+6
-83
lines changed

.github/workflows/conformance.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ jobs:
2727
with:
2828
node-version: 24
2929
- run: uv sync --frozen --all-extras --package mcp-everything-server
30-
- run: ./scripts/run-server-conformance.sh
30+
- run: ./scripts/conformance/run-server.sh
3131

3232
client-conformance:
3333
runs-on: ubuntu-latest
@@ -41,5 +41,5 @@ jobs:
4141
- uses: actions/setup-node@v4
4242
with:
4343
node-version: 24
44-
- run: uv sync --frozen --all-extras --package mcp-conformance-client
45-
- run: npx @modelcontextprotocol/conformance client --command 'uv run --frozen mcp-conformance-client' --suite all
44+
- run: uv sync --frozen --all-extras --package mcp
45+
- run: npx @modelcontextprotocol/conformance client --command 'uv run --frozen python scripts/conformance/client.py' --suite all

examples/clients/conformance-client/mcp_conformance_client/__main__.py

Lines changed: 0 additions & 6 deletions
This file was deleted.

examples/clients/conformance-client/pyproject.toml

Lines changed: 0 additions & 42 deletions
This file was deleted.

examples/clients/conformance-client/mcp_conformance_client/__init__.py renamed to scripts/conformance/client.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,8 @@
2929
from urllib.parse import parse_qs, urlparse
3030

3131
import httpx
32+
from pydantic import AnyUrl
33+
3234
from mcp import ClientSession, types
3335
from mcp.client.auth import OAuthClientProvider, TokenStorage
3436
from mcp.client.auth.extensions.client_credentials import (
@@ -39,7 +41,6 @@
3941
from mcp.client.streamable_http import streamable_http_client
4042
from mcp.shared.auth import OAuthClientInformationFull, OAuthClientMetadata, OAuthToken
4143
from mcp.shared.context import RequestContext
42-
from pydantic import AnyUrl
4344

4445
# Set up logging to stderr (stdout is for conformance test output)
4546
logging.basicConfig(
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ PORT="${PORT:-3001}"
55
SERVER_URL="http://localhost:${PORT}/mcp"
66

77
SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
8-
cd "$SCRIPT_DIR/.."
8+
cd "$SCRIPT_DIR/../.."
99

1010
# Start everything-server
1111
uv run --frozen mcp-everything-server --port "$PORT" &

uv.lock

Lines changed: 0 additions & 30 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)