Skip to content

fix: MCP ツール未設定によるオートレビュー・アシスタント失敗を修正#113

Open
MarikoMorimoto wants to merge 1 commit into
mainfrom
fix/claude-review-remove-mcp-inline-comment
Open

fix: MCP ツール未設定によるオートレビュー・アシスタント失敗を修正#113
MarikoMorimoto wants to merge 1 commit into
mainfrom
fix/claude-review-remove-mcp-inline-comment

Conversation

@MarikoMorimoto

Copy link
Copy Markdown
Member

概要

Claude Code v2.1.183 以降、allowedTools に未設定の MCP サーバーのツールが含まれると即エラーになる挙動に変更された。これにより、オートレビューと @claude コメント両方が失敗していた問題を修正する。

原因の詳細

claude-review.yml(オートレビュー)

claude_args--allowedToolsmcp__github_inline_comment__create_inline_comment を手動指定していた。claude-code-action はこのツール名を検知してインラインコメントサーバーをセットアップする仕組みになっているが、セットアップ後に .mcp.json が trusted ブランチの内容で上書き復元され、サーバー設定が消えた状態でツールが allowedTools に残るため、v2.1.183+ が即クラッシュしていた。

- --allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),..."
+ --allowedTools "Bash(gh pr comment:*),..."

claude-assistant.yml(@claude コメント)

actions: read 権限がないため github_ci MCP サーバーのインストールがスキップされるが、そのツール群(mcp__github_ci__*)は allowedTools に残り、同様のエラーが発生していた。

  permissions:
    contents: read
    pull-requests: write
    issues: write
    id-token: write
+   actions: read

影響範囲

  • オートレビュー(CI 完了後の自動実行・@claude auto-review)が正常に動作するようになる
  • @claude コメントへの返答が正常に動作するようになる
  • @claude が CI ステータスを参照できるようになる(github_ci MCP サーバーが有効化されるため)

by Claude

Claude Code v2.1.183 以降、allowedTools に未設定の MCP サーバーのツールが
含まれると即エラーになる挙動に変更された。

- claude-review.yml: mcp__github_inline_comment__create_inline_comment を
  --allowedTools から削除。このツールは action が allowedTools への明示的な
  指定を検知してサーバーをセットアップする仕組みのため、削除することで
  セットアップ自体が行われなくなり問題を解消する。
- claude-assistant.yml: actions: read 権限を追加。これにより github_ci MCP
  サーバーが正常にセットアップされ、「Skipping CI server installation」
  警告とそれに伴う即時エラーが解消される。

by Claude
@MarikoMorimoto

Copy link
Copy Markdown
Member Author

@fumikito
#112 の修正だけでは直らなかったので、追加の修正です。
よろしくお願いします。

@MarikoMorimoto MarikoMorimoto requested a review from fumikito June 29, 2026 08:48
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.

1 participant