tomevault-io PR Feedback Complete Check
このスキルは、PRに関連する全種類のフィードバックを取得し、漏れなく対応するための手順を提供します。
重要な教訓
CodeQL/Analyzeジョブ成功 ≠ Code Scanningチェック成功
CodeQL Analyzeジョブが成功しても、Code Scanningアラートが残っていればPRチェックは失敗します。 必ず
gh pr checksで全件passを確認してください。
1. Code Scanningアラート取得
PRに関連する全種類のCode Scanningアラートを取得:
# PRマージコミットに対するopenアラート（全種類）
gh api "repos/:owner/:repo/code-scanning/alerts?ref=refs/pull/{PR_NUMBER}/merge&state=open" \
--jq '.[] | {number, rule: .rule.id, severity: .rule.severity, file: .most_recent_instance.location.path, line: .most_recent_instance.location.start_line}'
確認すべきアラート種類
| ルールID | 説明 | 対応方法 |
|---|---|---|
java/log-injection |
ログにユーザー入力を直接出力 | SanitizeUtil.forLog() |
java/polynomial-redos |
正規表現のReDoS脆弱性 | .*の連続を避ける |
java/sql-injection |
SQLインジェクション | パラメータバインド使用 |
java/path-injection |
パストラバーサル | パス検証 |
2. Copilot/セキュリティレビューコメント取得
# PRのレビューコメント（Copilot、セキュリティボット含む）
gh api "repos/:owner/:repo/pulls/{PR_NUMBER}/comments" \
--jq '.[] | {author: .user.login, path, line, body: (.body | split("\n")[0])}'
3. PRチェック全件確認
# 全チェックの状態確認
gh pr checks {PR_NUMBER}
重要: 以下が全てsuccessになるまで完了としない:
CodeQL(Code Scanningチェック)CodeQL/Analyze (java)CodeQL/Analyze (javascript)E2E Tests/End-to-End- その他全てのチェック
4. 対応チェックリスト
PRレビュー対応時は以下を確認:
- Code Scanningアラート全件取得済み
- 全アラート種類（log-injection, redos等）を確認
- Copilotレビューコメント全件確認
- 手動レビューコメント確認
-
gh pr checksで全件pass確認 - 修正コードをビルド・テスト検証済み
Converted and distributed by TomeVault — claim your Tome and manage your conversions.