ci: add ai-code-review caller#27
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
|
||
| jobs: | ||
| review: | ||
| uses: smkwlab/.github/.github/workflows/ai-review.yml@v1 |
There was a problem hiding this comment.
🚨 [HIGH] [HIGH] 呼び出し先のreusable workflowのrefが @v1 と固定されています。v1 タグが強制的に更新（force push）された場合、予期しない破壊的変更が取り込まれるリスクがあります。セキュリティおよび再現性の観点から、@v1 の代わりに特定のコミットSHA（例: @abc1234）を使用することを検討してください。
|
|
||
| on: | ||
| pull_request: | ||
| types: [opened, reopened, ready_for_review] # no synchronize: avoid re-reviewing every push |
There was a problem hiding this comment.
synchronize イベントが意図的に除外されていますが、これによりPRに追加のコミットがプッシュされた際にレビューが再実行されません。コメントには「every push を避けるため」と記載されていますが、差分のみを対象とした再レビューが必要なユースケースでは問題になる可能性があります。運用方針として意図的であれば問題ありませんが、ドキュメントやREADMEに明記しておくことを推奨します。
| anthropic_api_key: ${{ secrets.ANTHROPIC_API_KEY }} | ||
| gemini_api_key: ${{ secrets.GEMINI_API_KEY }} | ||
| with: | ||
| model_code: claude-sonnet-4-6 |
There was a problem hiding this comment.
ℹ️ [LOW] [LOW] model_code: claude-sonnet-4-6 というモデル名が直接ハードコードされています。PRの説明では「配布時にトークンが置換される」と記載されていますが、このファイルが配布後のものであれば、モデルのバージョンが変わった際に各リポジトリのファイルを個別に更新する必要が生じます。モデル名をorg/repoレベルの変数（vars.AI_REVIEW_MODEL）として外部化することで、一元管理が容易になります。
Adds the shared
ai-code-reviewworkflow as a caller ofsmkwlab/.github/.github/workflows/ai-code-review.yml@v1.PR への自動コードレビュー（ワンショット・inline）を有効化します。
ANTHROPIC_API_KEY（claude モデル時）またはGEMINI_API_KEY（gemini モデル時）がこのリポジトリで利用可能である必要があります（未配布なら安全にスキップ）ready_for_reviewまで、fork PR は secret 不在のため、いずれも安全にスキップします