mcpdo: run the daemon test suites on Windows by porting the harness to named pipes
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 4/5
- 見積もり時間
- 3〜5日
- 初心者へのやさしさ
- 53/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- github-actions, node.js, typescript
調査の方向性
Start with clients/mcpdo/src/daemon/paths.ts, clients/mcpdo/__tests__/helpers/platform.ts, and the daemon test suites gated by unixDaemonOnly; trace how they construct endpoints and wait during teardown. Run the tests on the Windows CI job and identify cases that depend on POSIX behavior. Done means the daemon suites run on Windows, only explicitly justified POSIX cases remain skipped, and the Windows coverage carve-out is removed from clients/mcpdo/vitest.config.ts.
索引モデルが issue の本文から書いたものです。
説明
Problem
mcpdo's Windows daemon path has never been run by a test. Production code already uses a named pipe on win32 (getDaemonSocketPath in clients/mcpdo/src/daemon/paths.ts returns \\.\pipe\mcp-conn-<hash>, and clients/mcpdo/src/daemon/client.ts handles the pipe path when locating the token file). But the daemon test harness was written for a Unix domain socket:
- suites bind or poll
<dir>/mcpdod.sockdirectly. Windows refuses that withEACCES, and the tests then time out waiting on the daemon. - teardown waits for the socket file to vanish. A pipe never appears there, so a live daemon keeps the temp dir open and
rmSyncfails withEBUSY. - some assertions assume POSIX premises (
0700directories, thesun_pathlimit).
#2748 (PR #2906) therefore skips 127 mcpdo tests on win32 (unixDaemonOnly in clients/mcpdo/__tests__/helpers/platform.ts). It also turns off mcpdo's per-file 90% coverage threshold on Windows only (clients/mcpdo/vitest.config.ts), because the daemon modules fall below it there. The new windows CI job is green, but it doesn't show that the daemon works on Windows.
Expected
- The daemon test helpers derive the endpoint from
getDaemonSocketPathinstead of a hard-coded.sockfile, and tear down a named-pipe daemon cleanly on Windows. - The suites gated by
unixDaemonOnlyrun on thewindowsCI job. Only cases that assert genuinely POSIX-only behaviour stay skipped, each with its reason at the site. - mcpdo's per-file 90% threshold is enforced on Windows again, and the win32 carve-out in
clients/mcpdo/vitest.config.tsis removed.
Verification has to come from the windows CI job; maintainers develop on macOS.
- 主要言語
- TypeScript
- スター
- 11.2k
- フォーク
- 1.6k
- 平均マージ
- 3時間 36分
- マージ済み PR（30日）
- 154
環境構築
- Dockerfile または Docker Compose ファイルあり
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
modelcontextprotocol/inspector のほかの issue
-
Document load testing and Active Sessions on modelcontextprotocol.io対応中かも @cliffhall が 2 日前に担当しました。 オープンdocumentation v2 waiting-on-docs
modelcontextprotocol/inspector#2869 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
CLI: OAuth denials and other non-usage failures exit 1 with code "error", the same as a usage errorオープンbug v2
難易度 3/5 1〜2日 初心者へのやさしさ 68/100
modelcontextprotocol/inspector#2842 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
blocked chore v2
難易度 4/5 3〜5日 初心者へのやさしさ 48/100
modelcontextprotocol/inspector#2548 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
blocked chore v2
難易度 4/5 3〜5日 初心者へのやさしさ 52/100
modelcontextprotocol/inspector#2547 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Create the GitHub App for board writes and security-alert reads, and store its credentials as org secrets対応中かも @cliffhall が 18 日前に担当しました。 オープンchore v2
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
modelcontextprotocol/inspector#2544 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
modelcontextprotocol/inspector の issue をすべて見る
似ている issue
-
bug needs-triage potential-regression
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
aws/aws-sdk-js-v3#8408 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
colinhacks/zod#6743 ·
メンテナーはふだん 1 日以内に返信
-
[BUG] Packing list: the weight field drops digits on a slow connection and saves the wrong weightオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 79/100
openai/openai-node#3124 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 73/100
anthropics/anthropic-sdk-typescript#1296 ·
メンテナーはふだん 8 日以内に返信