Skip to content

ci: add ai-code-review caller#82

Open
toshi0806 wants to merge 1 commit into
mainfrom
add-ai-code-review
Open

ci: add ai-code-review caller#82
toshi0806 wants to merge 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:50

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 というモデルが実際に利用可能か確認してください。誤ったモデル名を指定するとワークフローが実行時エラーになります。

# 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.
# The ref / model / language tokens below are substituted at distribution time.

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] コメント冒頭に「Caller template」「Distributed by scripts/distribute-workflow.sh」とあり、このファイルが配布スクリプトによって生成されるテンプレートであることが示されています。しかし実際にリポジトリに配置されているこのファイルは既に配布済みの成果物であるため、テンプレート用のコメント(1〜4行目)はそのまま残すと混乱を招く可能性があります。配布後のファイルにはテンプレートメタ情報を残さないか、明確に「このファイルは自動生成されました」と書き換えることを検討してください。

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