fix-to-merge

Files (1)

• SKILL.md

SKILL.md (first lines)

# Fix to Merge

PRを「A評価 + コンフリクトなし + CI通過」の状態にするまで、レビューと修正を自動で繰り返します。

## 引数について

- **$ARGUMENTS**: 対象PRの指定
  - **PR番号**: `#123` または `123`
  - **指定なし**: 現在のブランチに紐づくPRを自動検出

## 実行フロー

```
┌─────────────────┐
│ 1. PR特定        │
└────────┬────────┘
         ▼
┌─────────────────┐
│ 2. レビュー方式  │
│    判定          │
└────────┬────────┘
         ▼
┌─────────────────┐
│ 3. コンフリクト  │──── あり ──→ 解消してコミット
│    チェック      │
└────────┬────────┘
         ▼ なし
┌─────────────────┐
│ 4. コードレビュー │◄──────────────────┐
│   （別コンテキスト）│                    │
└────────┬────────┘                    │
         ▼                             │
┌─────────────────┐                    │
│ 5. 評価判定      │                    │
│   A → Step 8     │                    │
│   B/C → Step 6   │                    │
│   D → 中止       │                    │
└────────┬────────┘                    │
         ▼ B/C                         │
┌─────────────────┐                    │
│ 6. 指摘修正      │                    │
│   + コミット＆push │  （最大5回ループ） │
└────────┬────────┘                    │
         ▼                             │
┌─────────────────┐                    │
│ 7. CI結果待機    │────────────────────┘
│  （最大10分）     │  pass→8, fail→修正→4
└─────────────────┘
         
┌─────────────────┐
│ 8. 最終確認      │
│   + マージ準備完了 │
└─────────────────┘
```

## Step 1: PR特定と既存フィードバックの収集

```bash
# 引数からPR番号を取得、なければ現在のブランチから検出
PR_NUMBER=$ARGUMENTS
if [ -z "$PR_NUMBER" ]; then
  PR_NUMBER=$(gh pr view --json number -q '.number')
fi

# PR情報を取得
gh pr view $PR_NUMBER --json title,body,baseRefName,headRefName,additions,deletions,changedFiles,mergeable,mergeStateStatus
```

PR が見つからない場合はエラーメッセージを表示して終了。

### 既存のレビューフィードバックを取得（必須）

**重要**: ローカルで再レビューする前に、PR上に既に投稿されているコメント・レビュースレッドを必ず取得する。人間レビュアーや CodeRabbit / Copilot などが既に指摘している内容を見落として「ユーザーに確認してください」と返すのを防ぐため。

```bash
# Issue風コメント（PR本文へのコメント）
gh pr view $PR_NUMBER --json comments \
  --jq '.comments[] | "[" + .author.login + "] " + .body'

# Review本体（approve/request_changes/comment のサマリーボディ）
gh api repos/{owner}/{repo}/pulls/$PR_NUMBER/reviews \
  --jq '.[] | select(.body != "") | "[" + .user.login + ":" + .state + "] " + .body'

# 行単位のレビューコメント（diff上にぶら下がる指摘）
gh api repos/{owner}/{repo}/pulls/$PR_NUMBER/comments \
  --jq '.[] | "[" + .user.login + "] " + .path + ":" + (.line|tostring) + " - " + .body'
```

`{owner}/{repo}` は `gh repo view --json nameWithOwner -q '.nameWithOwner'` で取得できる。

取得したコメントは Step 4 のレビュー前にチェックリスト化し、「未対応の指摘」として後続のレビュー結果とマージする。既に PR 上で指摘済みの内容を見落とさないこと。

## Step 2: レビュー方式の判定

CI に AI PR reviewer が設定されているか確認する:

```bash
gh pr checks $PR_NUMBER 2>/dev/null | grep -iE '(ai.review|code.review|pr.review)' || echo "NOT_FOUND"
```

| 結果 | レビュー方式 |
|------|-------------|
| AI reviewer チェックが存在 | CI の AI reviewer 結果を待ち、その指摘を修正 |
| `NOT_FOUND` | **ローカルで Task を使い `app-reviewer` を別コンテキスト起動**（デフォルト） |

大半のプロジェクトでは CI に AI reviewer は未設定のため、ローカル Task によるレビューがデフォルト動作です。

## Step 3: コンフリクトチェック＆解消

```bash
# マージ可能性を確認
gh pr view $PR_NUMBER --json mergeable,mergeStateStatus
```

**mergeable が CONFLICTING の場合:**

`Skill` ツールで `pr-conflict-resolution` スキルを読み込み、コンフリクトを解消する。

```bash

From ohzono/native-app-fullcycle, MIT.

More from ohzono/native-app-fullcycle