Skip to content

ci: add ai-code-review caller#131

Merged
toshi0806 merged 1 commit into
mainfrom
add-ai-code-review
Jun 23, 2026
Merged

ci: add ai-code-review caller#131
toshi0806 merged 1 commit into
mainfrom
add-ai-code-review

Conversation

@toshi0806

Copy link
Copy Markdown
Member

Adds the shared ai-code-review workflow as a caller of smkwlab/.github/.github/workflows/ai-code-review.yml@v1.

PR への自動コードレビュー(ワンショット・inline)を有効化します。

  • 動作には org シークレット ANTHROPIC_API_KEY(claude モデル時)または GEMINI_API_KEY(gemini モデル時)がこのリポジトリで利用可能である必要があります(未配布なら安全にスキップ)
  • draft PR は ready_for_review まで、fork PR は secret 不在のため、いずれも安全にスキップします

Copilot AI review requested due to automatic review settings June 23, 2026 01:49

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

全体的にワークフローの構成は適切です。いくつか確認・改善すべき点を指摘します。

anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }}
gemini_api_key: ${{ secrets.GEMINI_API_KEY }}
with:
model_code: claude-sonnet-4-6

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🚨 [HIGH] モデルコード claude-sonnet-4-6 は存在しないモデル名の可能性があります。2025年時点での正式なモデル識別子は claude-sonnet-4-5claude-opus-4 などです。claude-sonnet-4-6 というモデルが実際に存在するか確認してください。誤ったモデル名を指定するとAPIエラーが発生し、レビューが常に失敗します。

@@ -0,0 +1,27 @@
# Caller template: AI Code Review (one-shot, inline) — calls the ai-review reusable.
# Distributed by scripts/distribute-workflow.sh as
# .github/workflows/ai-code-review.yml in each target repository.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ℹ️ [LOW] コメント冒頭(1〜4行目)に「Distributed by scripts/distribute-workflow.sh」とあり、このファイルが配布スクリプトによって生成されるテンプレートであることが示されています。しかし、このリポジトリに scripts/distribute-workflow.sh が存在するか、またはこのファイルが手動で配置されたものかが不明です。テンプレートとして管理するのであれば、実際の配布元リポジトリとの同期方法を明確にしておくことを推奨します。


on:
pull_request:
types: [opened, reopened, ready_for_review, synchronize] # synchronize: re-review on each push. concurrency below cancels superseded runs. Drop synchronize if cost is a concern.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

ℹ️ [LOW] 9行目のインラインコメントが非常に長く、YAMLの可読性を損なっています。コメントは別行に分けるか、ワークフローの説明として README や上部のブロックコメントにまとめることを検討してください。

@toshi0806 toshi0806 merged commit 3012fb3 into main Jun 23, 2026
1 check passed
@toshi0806 toshi0806 deleted the add-ai-code-review branch June 23, 2026 01:50
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