Skip to content

fix: claude-review の /review コマンドを削除#112

Merged
fumikito merged 1 commit into
mainfrom
fix/claude-review-remove-slash-review-command
Jun 29, 2026
Merged

fix: claude-review の /review コマンドを削除#112
fumikito merged 1 commit into
mainfrom
fix/claude-review-remove-slash-review-command

Conversation

@MarikoMorimoto

Copy link
Copy Markdown
Member

概要

Claude Code v2.1.186 のリリースで /review コマンドの動作が変更されたため、オートレビューが失敗するようになった問題を修正します。

原因

  • v2.1.186 で /review/code-review エンジンに委譲されるよう変更された
  • 非インタラクティブな SDK モード(CI)では、プロンプト先頭の /review が即エラーになる
  • claude-code-action@v1 が内部でインストールする Claude Code が v2.1.181 → v2.1.195 に上がったことで発現

修正内容

prompt: の先頭にあった /review(と直後の空行)を削除。

-            /review
-
             REPO: ${{ github.repository }}

確認方法

修正後、PRに対してオートレビューが正常に動作することを確認する。

by Claude

Claude Code v2.1.186 以降、/review が /code-review エンジンに委譲されるよう変更され、
非インタラクティブな SDK モード(CI)では即エラーになる。
prompt 先頭の `/review` を削除することで修正する。

by Claude
@MarikoMorimoto MarikoMorimoto requested a review from fumikito June 29, 2026 08:14
@MarikoMorimoto

Copy link
Copy Markdown
Member Author

補足:影響範囲について

@claude コメントと /code-review コマンドへの影響についても整理します。

今回の修正が直すもの

CI の自動実行(workflow_call)のプロンプト先頭に /review がハードコードされていたことが原因です。今回の修正で解消されます。

@claude コメントについて

claude-code-action@claude コメントの本文をそのままプロンプトとして SDK モードで渡します。

コメント内容 結果
@claude このPRをレビューして ✅ 動く(自然言語なのでOK)
@claude /review ❌ 失敗(スラッシュコマンドはSDKモード非対応)
@claude /code-review ❌ 失敗(同上)

まとめ

@claude でレビュー依頼するときは、スラッシュコマンドではなく自然言語で書けば問題ありません。

by Claude

@MarikoMorimoto

Copy link
Copy Markdown
Member Author

@fumikito
https://github.com/tarosky/autosport-web-www/pull/271 で本ワークフローを使用したGitHub Actionsが動かなかったため修正しました。
Slack でも相談させていただいた件です。: https://tarosky.slack.com/archives/C097EAJE6JZ/p1782718878124319

よろしくお願いします!

@fumikito fumikito merged commit 81e4c65 into main Jun 29, 2026
9 checks passed
@fumikito fumikito deleted the fix/claude-review-remove-slash-review-command branch June 29, 2026 08:20
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.

2 participants