All Skills > Workflow > Code Review

Sentry Code Review

GitHub Pull Request のレビューコメント内で Sentry が特定した問題を分析・解決するための専門的なスキルです。

Sentry PR レビューコメント形式

Sentry は PR 内のコード変更に対して行単位のレビューコメントを投稿します。各コメントには以下が含まれます。

コメントメタデータ（API から）

• author: ボットのユーザー名（例："sentry[bot]"）
• file: コメント対象のファイル（例："src/sentry/seer/explorer/tools.py"）
• line: コード内の行番号（ファイル全体へのコメントの場合は null）
• body: コメント本文（HTML の details タグを含む Markdown）

Body の構造

body フィールドには折りたたみ可能なセクションを含む Markdown が含まれます。

ヘッダー:

**Bug:** [問題の説明]
<sub>Severity: CRITICAL | Confidence: 1.00</sub>

分析セクション（<details> タグ内）:

<details>
<summary>🔍 <b>Detailed Analysis</b></summary>
技術的な問題とその影響を説明します
</details>

修正提案セクション（<details> タグ内）:

<details>
<summary>💡 <b>Suggested Fix</b></summary>
具体的な解決策を提案します
</details>

AI エージェントプロンプト（<details> タグ内）:

<details>
<summary>🤖 <b>Prompt for AI Agent</b></summary>
問題を確認し修正するための具体的な指示
含むもの: 位置情報（file#line）、潜在的な問題の説明
</details>

問題の例

1. None 値による TypeError

   • リスト返却時に None を返す関数
   • イテレーション前の null チェック不足

2. 入力検証の問題

   • 過度に寛容な入力検証
   • 無効なデータが通過している

3. エラーハンドリングの抜け

   • エラーはログされるが再スロー されない
   • 重要なパスでの無声な失敗

ワークフロー

1. PR コメントの取得

PR 番号または URL を指定された場合:

# GitHub API を使用して PR レビューコメント（行単位のコード コメント）を取得
gh api repos/{owner}/{repo}/pulls/<PR_NUMBER>/comments --jq '.[] | select(.user.login | startswith("sentry")) | {author: .user.login, file: .path, line: .line, body: .body}'

または WebFetch を使用して PR URL から直接取得してください。

2. Sentry コメントの解析

• のみ Sentry からのコメントを処理（ユーザー名が "sentry" で始まる、例："sentry[bot]"）
• "cursor[bot]" または他のボットからのコメントは無視
• 各コメントから以下を抽出:
  • file: コメント対象のファイルパス
  • line: 特定の行番号（利用可能な場合）
  • body: Markdown/HTML body から以下を解析:
    • バグの説明（"Bug:" で始まるヘッダー行から）
    • 重大度レベル（<sub>Severity: X タグから）
    • 信頼度スコア（sub タグ内の Confidence: X.XX から）
    • 詳細な分析（<summary>🔍 <b>Detailed Analysis</b></summary> details ブロック内のテキスト）
    • 修正提案（<summary>💡 <b>Suggested Fix</b></summary> details ブロック内のテキスト）
    • AI エージェントプロンプト（<summary>🤖 <b>Prompt for AI Agent</b></summary> details ブロック内のテキスト）

3. 各問題の分析

各 Sentry コメントについて:

1. コメントメタデータから file と line に注目 - ここが確認場所です
2. コメント内に記載された特定のファイルを読み取る
3. 行番号にナビゲートして問題のあるコードを確認
4. 「🤖 Prompt for AI Agent」セクションで問題の具体的なコンテキストを読む
5. 現在のコード内で問題が存在するかどうかを確認
6. 詳細な分析から根本原因を理解
7. 修正提案を評価

4. 修正の実装

各検証済みの問題について:

1. 影響を受けるファイルを読み取る
2. 提案された修正またはあなた自身の解決策を実装
3. 修正が根本原因に対応していることを確認
4. エッジケースと副作用を考慮
5. Edit ツールを使用して正確な変更を加える

5. サマリーの提供

問題を分析・修正した後、レポートを提供してください:

## Sentry Code Review サマリー

**PR:** #[番号] - [タイトル]
**発見された Sentry コメント:** [件数]

### 解決された問題

#### 1. [問題タイトル] - [重大度]
- **信頼度:** [スコア]
- **位置:** [file:line]
- **問題:** [簡潔な説明]
- **適用された修正:** [実施内容]
- **ステータス:** 解決済み

#### 2. [問題タイトル] - [重大度]
- **信頼度:** [スコア]
- **位置:** [file:line]
- **問題:** [簡潔な説明]
- **適用された修正:** [実施内容]
- **ステータス:** 解決済み

### 手動レビューが必要な問題

#### 1. [問題タイトル] - [重大度]
- **理由:** [手動レビューが必要な理由]
- **推奨事項:** [提案するアプローチ]

### サマリー
- **合計問題数:** [件数]
- **解決済み:** [件数]
- **手動レビュー必要:** [件数]

重要なガイドライン

1. Sentry のみ: Sentry からのコメント（ユーザー名が "sentry" で始まる）に焦点を当てる
2. まず検証: 修正を試みる前に問題の存在を常に確認
3. 読み取ってから編集: Edit ツールを使用する前に常に Read ツールを使用
4. 正確さ: 根本原因に対応したターゲット限定の修正を実施
5. 安全性: 修正について不確実な場合、AskUserQuestion を使用してユーザーに指導を求める
6. テスト: 修正適用後にテストを実行するようユーザーに通知

よくある Sentry ボット問題カテゴリ

ビルド設定の問題

• ビルド出力で欠落しているファイル
• 不正な tsconfig 設定
• ビルドスクリプトのファイルコピーステップが欠落

エラーハンドリングの問題

• キャッチされたエラーが再スローされない
• 重要なパスでの無声な失敗
• エラーバウンダリが欠落

ランタイム設定の問題

• 欠落している環境変数
• パス解決の不正
• 欠落している必須の依存関係

タイプセーフティの問題

• null チェック欠落
• 失敗する可能性のあるタイプアサーション
• 欠落している入力検証