From b5a5caa2a4067fc95323cd1fd5c459bc6b998f64 Mon Sep 17 00:00:00 2001 From: strnh Date: Thu, 9 Jul 2026 14:48:31 +0900 Subject: [PATCH 1/2] =?UTF-8?q?docs:=20PR=E3=83=AC=E3=83=93=E3=83=A5?= =?UTF-8?q?=E3=83=BC=E6=99=82=E3=81=AE=E3=82=B9=E3=82=B3=E3=83=BC=E3=83=97?= =?UTF-8?q?=E7=85=A7=E5=90=88=E3=83=AB=E3=83=BC=E3=83=AB=E3=82=92=20AGENTS?= =?UTF-8?q?.md=20=E3=81=AB=E8=BF=BD=E5=8A=A0=20(#57)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit issue #57 の調査結果: - PDF取込機能は PR #56 には含まれておらず、専用の PR #51 (Issue #1) で 2026-07-07 に別途マージ済み（git diff 68a7eca..65b265a および gh pr diff 56 --name-only で PDF/Import 関連 0 件を確認） - 「PR #56 に混在」に見えたのは、古いローカル main（68a7eca 未取得） を比較基準にしたことによる誤認 再発防止として、レビュー時に最新 origin/main を基準にした スコープ照合（PR本文 vs 実 diff）を行う運用ルールを明文化する。 Closes #57 Co-Authored-By: Claude Fable 5 --- AGENTS.md | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/AGENTS.md b/AGENTS.md index c4b0135..4aece74 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -37,6 +37,12 @@ Privacy_Policy.md - tools.md を読むこと。 +## PRレビュー時のスコープ照合 + +- レビュー前に `git fetch origin` を実行し、diff の比較基準は必ず最新の `origin/main`（merge-base）にする。ローカル main が古いと、他PRでマージ済みの変更が「このPRの新規追加」に見える（issue #57 の誤認の原因）。 +- PR本文に宣言されたスコープと `gh pr diff --name-only` の実ファイル一覧を突合し、本文に記載のない差分があれば指摘する。 +- 独立した複数機能を1つのPRにまとめない。作業中に別機能の必要が生じたら、別issue・別PRに切り出す。 + ## worklogs - prompt/handovers/*.md について、金曜日の夜/土曜日の朝、いずれかで要約し、worklogsに保存する。 From 80f0952b4e7f4f3793c13cc8a7bdc0fa9440b7c4 Mon Sep 17 00:00:00 2001 From: strnh Date: Thu, 9 Jul 2026 14:59:20 +0900 Subject: [PATCH 2/2] =?UTF-8?q?docs:=20merge-base=20=E3=81=AE=E8=AA=AC?= =?UTF-8?q?=E6=98=8E=E3=82=92=E6=AD=A3=E7=A2=BA=E5=8C=96=E3=81=97=E3=82=B3?= =?UTF-8?q?=E3=83=9E=E3=83=B3=E3=83=89=E4=BE=8B=E3=82=92=E6=98=8E=E8=A8=98?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Copilot レビュー指摘に対応: merge-base は origin/main 自体ではなく 共通祖先コミットであるため、表現を修正し `git diff origin/main...HEAD` の具体例を追記。 Co-Authored-By: Claude Fable 5 --- AGENTS.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/AGENTS.md b/AGENTS.md index 4aece74..5b965c4 100644 --- a/AGENTS.md +++ b/AGENTS.md @@ -39,7 +39,7 @@ Privacy_Policy.md ## PRレビュー時のスコープ照合 -- レビュー前に `git fetch origin` を実行し、diff の比較基準は必ず最新の `origin/main`（merge-base）にする。ローカル main が古いと、他PRでマージ済みの変更が「このPRの新規追加」に見える（issue #57 の誤認の原因）。 +- レビュー前に `git fetch origin` を実行し、diff は最新の `origin/main` と PR ブランチの merge-base（共通祖先コミット）を基準に取る（例: `git diff origin/main...HEAD`、これは `git diff $(git merge-base origin/main HEAD) HEAD` と同等）。ローカル main が古いと、他PRでマージ済みの変更が「このPRの新規追加」に見える（issue #57 の誤認の原因）。 - PR本文に宣言されたスコープと `gh pr diff --name-only` の実ファイル一覧を突合し、本文に記載のない差分があれば指摘する。 - 独立した複数機能を1つのPRにまとめない。作業中に別機能の必要が生じたら、別issue・別PRに切り出す。