Skip to content

MCPの未知オプションを暗黙の既定値へ切り替えない - #97

Merged
opaopa6969 merged 1 commit into
masterfrom
fix/issue-96-mcp-backend-fail-fast
Aug 31, 2026
Merged

MCPの未知オプションを暗黙の既定値へ切り替えない#97
opaopa6969 merged 1 commit into
masterfrom
fix/issue-96-mcp-backend-fail-fast

Conversation

@opaopa6969

Copy link
Copy Markdown
Owner

Closes #96

変更

  • backend の省略は AST_EVALUATOR を維持し、明示された未知値は JSON-RPC Invalid params (-32602) にする
  • resultType の省略は float を維持し、明示された未知値は evaluate / validate / execute_batch / parity_check で統一して拒否する
  • 未知 tool の Method not found (-32601) と不正引数を区別する
  • エラーへ未知値と許可候補を含める

判断

省略値は公開済みの default なので互換維持する。一方、明示した typo を成功扱いにすると人間にも LLM にも実行条件を偽るため fail-fast にした。これはアプリケーション入力検証であり、parser generator へ移す責務ではない。

検証

  • McpServerTest の新規・関連7テスト成功
  • mvn -P p4-smoke test 成功
  • McpServerTest 全体は19件成功。既存 oversizedBody_rejectedWith413 のみローカルで connection reset(最新 master CI は同テストを含め成功)

@opaopa6969
opaopa6969 merged commit d63d3e3 into master Aug 31, 2026
2 checks passed
@opaopa6969
opaopa6969 deleted the fix/issue-96-mcp-backend-fail-fast branch August 31, 2026 23:30
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.

MCPで未知のbackend/resultTypeを既定値へ暗黙fallbackしない

1 participant