Kadomori Pre-PR Handoff
この文書は PR description の完成文ではなく、PR 作成前に著者が責任を持って確認し、レビュアーや責任者へ引き継ぐための確認証跡です。
| 結論 | NotReady(PR作成前に要対応) / Reviewability 55/100 |
|---|---|
| 主な理由 | 目的との関係説明が必要 |
| Slop リスク | Medium(中) |
| PR 粒度 | TooBroad(広すぎる) |
| ドメイン確認 | 必要(人間によるドメイン判断が必要な項目あり) |
| 差分規模 | 35ファイル / +1174 / -201 |
| 証跡の読み方 | Deterministic evidence は git diff / rule / policy に基づく情報、Semantic assist は ACP の意味的補助(事実・判定の上書きではありません) |
| スコア内訳 | 差分サイズ -20 / Scope Drift -12 / Slop -8 / ドメイン確認 -5 |
目的
初期オンボーディングの摩擦を解消
著者がPR前に責任を持って確認すること
- 目的外変更候補をこの PR に含める理由を説明してください
- この PR は単一の目的を持っていることを確認しました
- テストが追加または更新されていることを確認しました
- handoff / report.md の snapshot と実出力を確認しました
- 代表ケースで verdict / score / review map の変化を確認しました
- ほか 2 件(rule 由来 2)は詳細証跡(
.kadomori/report.md/.kadomori/report.json)を参照してください
PR本文に転記する Kadomori 確認候補
以下は PR description へ転記するための下書きです。確認できた事実だけを残し、未確認の項目は削除せず「未確認」として扱ってください。
**Kadomori 確認** - Context-needed: CI workflow 文脈: token 権限、fork PR、secrets / OIDC、release / deploy 経路を確認(2ファイル、代表: `.github/workflows/ci.yml`)について、PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認した事実または未確認点を PR 本文に残してください。 - Context-needed: CLI コマンド変更: 引数・出力・終了コード・後方互換を確認(3ファイル、代表: `crates/kadomori-cli/src/commands/doctor.rs`)について、PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認した事実または未確認点を PR 本文に残してください。 - Context-needed: CLI テスト: 出力回帰・終了コード・非対話導線を確認(`crates/kadomori-cli/tests/cli_integration.rs`)について、PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認した事実または未確認点を PR 本文に残してください。 - Scope drift: `.github/workflows/ci.yml` などの目的外変更候補について、PR 目的との関係とこの PR に含める理由を PR 本文に残してください。 - Machine-verified: `crates/kadomori-cli/tests/snapshots/cli_integration__dogfood_pr38_rendering_detailed_evidence.snap` ほか 11ファイルについて、テスト / 生成コマンド / snapshot 更新などの機械的な確認根拠を PR 本文に残してください。 **レビュアー向け補足候補** - 必要に応じて、未確認項目や責任者判断が必要な論点をここに残してください。
NotReady — PR 作成前に検討すること
Kadomori の判定は NotReady です。これは PR 作成を禁止するものではなく、以下の点について対応を検討したうえで PR を作成することを推奨する合図です。
主な理由: 目的との関係説明が必要
PR 本文への転記候補:
**Kadomori NotReady 判定への対応** - Context-needed 23ファイル: PR 目的との整合を確認してください。 - [ ] 背景・意図を PR 本文に補足しました - 目的外変更候補: スコープ外の変更が検出されています。 - [ ] 目的との関係を説明するか、別 PR に分割しました
この候補テキストを PR description に転記し、対応した項目にチェックを入れてください。未対応の項目は削除せず残してください。
レビュアーに判断してほしいこと
Top 3 Review Focus(確認論点カード):
- Scope drift / 目的外変更:
.github/workflows/ci.yml— CI workflow: インフラファイルの変更が goal に含まれていません(ほか 2 件)。この PR に含めるべき変更か確認してください- Deterministic evidence: git diff / rule / team policy に基づく確認候補
- Semantic assist: なし
- Slop リスク Medium(中): 代表検出 R006 (Medium):
crates/kadomori-cli/src/commands/init.rs:369: catch-all エラーを握りつぶしている可能性があります。意図した fallback か著者確認してください — スタブが意図的でないことを確認- Deterministic evidence: git diff / rule / team policy に基づく確認候補
- Semantic assist: なし
- PR 粒度 TooBroad(広すぎる): 20ファイル超過(コアレビュー対象 23 / 20) — 分割要否の判断材料です
- Deterministic evidence: git diff / rule / team policy に基づく確認候補
- Semantic assist: なし
- ほか 14 件(レビュアー判断 3 / 背景確認 9 / 分割・粒度 2、rule 由来 14)は詳細証跡(
.kadomori/report.md/.kadomori/report.json)を参照してください
省略された確認項目の内訳:
- handoff レビュアー判断の省略: 14 件(レビュアー判断 3 / 背景確認 9 / 分割・粒度 2) / source内訳: rule 由来 14
- explain 重点確認の省略: 12 件(レビュアー判断 2 / 背景確認 9 / 分割・粒度 1) / source内訳: rule 由来 12
詳細証跡への導線
- 詳細レポート:
.kadomori/report.md - 機械処理用の事実データ:
.kadomori/report.json - Policy coverage / gap candidates:
.kadomori/report.md/.kadomori/report.jsonで確認できます - 分割検討の詳細:
kadomori split-plan --goal "..."
詳細証跡
Scope / Review Map の詳細証跡
Scope
含めた変更
crates/kadomori-cli/tests/snapshots/cli_integration__dogfood_pr38_rendering_detailed_evidence.snapcrates/kadomori-cli/tests/snapshots/cli_integration__history_summary_markdown.snapcrates/kadomori-cli/tests/snapshots/cli_integration__pr_body_no_llm_custom_output_summary.snapcrates/kadomori-cli/tests/snapshots/cli_integration__pr_body_no_llm_default_summary.snapcrates/kadomori-cli/tests/snapshots/cli_integration__pr_body_no_save_stdout_summary.snapcrates/kadomori-core/src/render/snapshots/kadomori_core__render__pr_body__tests__pr_body_md.snapcrates/kadomori-core/src/render/snapshots/kadomori_core__render__pr_body__tests__pr_body_md_en.snapdocs/kadomori-config-authoring.mddocs/release-distribution.mddocs/test-user-onboarding/README.en.mddocs/test-user-onboarding/release-readiness.mddocs/test-user-onboarding/user-guide.md.github/workflows/ci.yml.github/workflows/release.ymlCHANGELOG.mdREADME.en.mdREADME.mdcrates/kadomori-cli/src/commands/doctor.rscrates/kadomori-cli/src/commands/init.rscrates/kadomori-cli/src/commands/mod.rscrates/kadomori-cli/tests/cli_integration.rscrates/kadomori-core/src/analysis/scope_drift.rscrates/kadomori-core/src/analysis/split_plan.rscrates/kadomori-core/src/config/loader.rscrates/kadomori-core/src/config/mod.rscrates/kadomori-core/src/config/seed/mod.rscrates/kadomori-core/src/render/pr_body.rscrates/kadomori-core/src/render/responsibility_contract.rsdocs/distribution-archive-readme.mddocs/distribution-repository-readme.mddocs/test-user-onboarding/README.mdscripts/fixtures/fake-gh.shscripts/install.shscripts/publish-release.shscripts/test-release-distribution.sh
目的外変更候補
- CI workflow: インフラファイルの変更が goal に含まれていません: 2ファイル(代表:
.github/workflows/ci.yml).github/workflows/ci.yml.github/workflows/release.yml
CHANGELOG.md: リリースノート / changelog の変更は goal との関係説明が必要です
Review Map
Context-needed(背景確認が必要):
- CI workflow 文脈: token 権限、fork PR、secrets / OIDC、release / deploy 経路を確認: 2ファイル(代表:
.github/workflows/ci.yml、ほか 1ファイル) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください - CLI コマンド変更: 引数・出力・終了コード・後方互換を確認: 3ファイル(代表:
crates/kadomori-cli/src/commands/doctor.rs、ほか 2ファイル) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください - CLI テスト: 出力回帰・終了コード・非対話導線を確認: 1ファイル(代表:
crates/kadomori-cli/tests/cli_integration.rs) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください - pipeline 判定影響: verdict / score / review map の代表出力を確認: 2ファイル(代表:
crates/kadomori-core/src/analysis/scope_drift.rs、ほか 1ファイル) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください - rendering 出力影響: handoff / report.md / snapshot の読みやすさを確認: 2ファイル(代表:
crates/kadomori-core/src/render/pr_body.rs、ほか 1ファイル) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください - コンテキスト確認推奨: リポジトリ固有の意図を確認: 4ファイル(代表:
scripts/fixtures/fake-gh.sh、ほか 3ファイル) — 著者アクション: リポジトリ固有の意図と PR 目的との整合を確認し、確認内容を PR 本文に書いてください - リリースノート / changelog 文脈: 実装差分、互換性表記、ユーザー向け説明が一致するか確認: 1ファイル(代表:
CHANGELOG.md) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください - 実行手順 / hook / config snippet: 公式仕様・ローカル実行・fail-open/blocking 意図を確認: 5ファイル(代表:
README.en.md、ほか 4ファイル) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください - 設定値の変更: 影響範囲・デフォルト値・環境差分を確認: [Config]: 3ファイル(代表:
crates/kadomori-core/src/config/loader.rs、ほか 2ファイル) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください
Machine-verified(機械的に確認しやすい変更):
- snapshot 差分は golden test で機械的に検証しやすい: 7ファイル(代表:
crates/kadomori-core/src/render/snapshots/kadomori_core__render__pr_body__tests__pr_body_md.snap)crates/kadomori-cli/tests/snapshots/cli_integration__dogfood_pr38_rendering_detailed_evidence.snapcrates/kadomori-cli/tests/snapshots/cli_integration__history_summary_markdown.snapcrates/kadomori-cli/tests/snapshots/cli_integration__pr_body_no_llm_custom_output_summary.snapcrates/kadomori-cli/tests/snapshots/cli_integration__pr_body_no_llm_default_summary.snapcrates/kadomori-cli/tests/snapshots/cli_integration__pr_body_no_save_stdout_summary.snapcrates/kadomori-core/src/render/snapshots/kadomori_core__render__pr_body__tests__pr_body_md.snapcrates/kadomori-core/src/render/snapshots/kadomori_core__render__pr_body__tests__pr_body_md_en.snap
- ドキュメントのみの変更: 記述の正確性を確認(挙動への影響なし): 5ファイル(代表:
docs/kadomori-config-authoring.md)docs/kadomori-config-authoring.mddocs/release-distribution.mddocs/test-user-onboarding/README.en.mddocs/test-user-onboarding/release-readiness.mddocs/test-user-onboarding/user-guide.md
Slop リスク詳細
- R006 (Medium):
crates/kadomori-cli/src/commands/init.rs:369: catch-all エラーを握りつぶしている可能性があります。意図した fallback か著者確認してください — stub implementation: スタブが意図的でないことを確認
Notes for Reviewers
特記事項なし
生成元
- rule-based: true
- llm_attempted: false
- llm_succeeded: false
- llm_outcome:
NotAttempted - analysis_mode:
Default rules(LLMなし) - signals:
default_rules