test: run MCP contract checks with Node - #573
Conversation
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
|
@codex review The description now includes the final QA results and merge order. Please review the current head, |
|
Codex Review: Didn't find any major issues. Keep them coming! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
Summary
Run the MCP contract tests with Node, which is what the shipped plugin uses. This is the independent test-only part of #567.
Changes
The two tests now share one async helper for starting the server, sending the initialize and tools/list messages, and reading its replies. Launch errors reject the test directly. No production code changes.
Testing
From
sdk/typescript, run:Both tests should inspect the real bundled MCP server without using credentials or making model calls.
Checks already run:
git diff --checkpassed.7b8bae9ec0.Risk and rollout
This can merge independently of the finding-matching work. It changes only the test launcher and keeps the existing assertions. The normal Windows CI jobs also exercise this file.
Public disclosure review