Skip to content

fix(mcp-server): expose graph_expand / graph_hops in proxy search schema#158

Merged
liplus-lin-lay merged 1 commit into
mainfrom
157-proxy-graph-schema
Jun 14, 2026
Merged

fix(mcp-server): expose graph_expand / graph_hops in proxy search schema#158
liplus-lin-lay merged 1 commit into
mainfrom
157-proxy-graph-schema

Conversation

@liplus-lin-lay

Copy link
Copy Markdown
Member

概要

v0.9.0 で Worker(src/mcp.ts) の search に追加された graph_expand / graph_hops を、client proxy(mcp-server/server/index.js) の静的 TOOLS スキーマにも追加する。

背景 / 原因

  • proxy は tools/list を Worker へ転送せず静的 TOOLS 定数を返す(setRequestHandler(ListToolsRequestSchema, () => ({tools: TOOLS})))。
  • v0.9.0 は Worker 側にのみ param を足し、proxy 静的スキーマを更新し忘れた。
  • MCP クライアントが stale スキーマに対し additionalProperties:false を適用し未知 param を黙って除去 → Worker に届かず graph_expanded:false
  • 転送経路 callRemoteTool は引数を Worker へ素通しするため、proxy スキーマに足すだけで届く(Worker は既に受理可能)。

変更

  • mcp-server/server/index.jsTOOLS.search.inputSchema.propertiesgraph_expand(boolean) / graph_hops(number, minimum 1 / maximum 2) を追加。文言は Worker の .describe() と一致。
  • Worker 変更なし。node --check 通過。

検証

  • OAuth 直叩き(スキーマ迂回)で Worker が graph_expand=true を受理し graph_expanded:true / neighbors 14 を返すことを実証済み。
  • マージ後、再公開版 proxy で通常の search ツールから発火することを実機確認する(release → 検証 → latest)。

release

  • type = patch(既出荷機能の client 到達性回復、新規挙動の追加ではない)。タグ v0.9.1

Closes #157

Worker (src/mcp.ts) added graph_expand / graph_hops in v0.9.0, but the
client proxy's static TOOLS schema was not updated. MCP clients applied
additionalProperties:false against the stale schema and silently stripped
the unknown params before they reached the Worker, so graph expansion
never fired through `npx github-rag-mcp` (graph_expanded:false).

proxy の静的 TOOLS スキーマに graph_expand / graph_hops を追加し、通常の
search ツールからグラフ展開を呼べるようにする。転送経路は引数を素通しし、
Worker 側は既に受理可能なため proxy 一箇所の変更で到達性が回復する。

Refs #157

@liplus-lin-lay liplus-lin-lay left a comment

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

セルフレビュー (auto モード)

  • 変更範囲: mcp-server/server/index.js 1ファイル・16行追加のみ。Worker / CI / 他経路に副作用なし(node --check 通過、CI green)。
  • 文言整合: 追加した graph_expand / graph_hops の description は Worker src/mcp.ts.describe() と一致。graph_hops は minimum 1 / maximum 2 で Worker の zod .min(1).max(2) と整合。
  • 正当性検証: マージ前に OAuth 直叩き(スキーマ迂回)で Worker が当該 param を受理し graph_expanded:true / neighbors 14 を返すことを実証済み。proxy 転送 callRemoteTool は引数素通しのため、スキーマ追加で到達性が回復する。
  • release type = patch: 既出荷機能(v0.9.0 Worker 側 graph 実装)の client 到達性回復であり、新規挙動の追加でも user/system observable な構造変更でもない。
  • brake1 非該当: 本 PR は USER_REPO(github-rag-mcp) の通常 bug fix であり Li+ source(LI_PLUS_REPO) の self-evolution PR ではないため parallel-subagent-eval は対象外。

承認可、マージへ。

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.

fix(mcp-server): add graph_expand / graph_hops to proxy search tool schema

1 participant