Kadomori Report
Evidence / Semantic Assist
- Deterministic evidence: git diff / default rules / team policy から機械的に提示した情報
- Semantic assist: ACP LLM が意味的理解で追加した補助観点(事実・判定の上書きではありません)
- 提示項目の source 内訳: rule 49 / acp_agent 0 / merged 0
このレポートの読み方
- Executive Summary — verdict と reviewability score で全体像を把握
- Review Map — Human-required → Context-needed → Machine-verified の順に確認対象を特定
- 著者が確認すること(含まれる場合) — PR 作成前に完了すべきアクション
- レビュアーが判断すること(含まれる場合) — レビュー時に提示する論点
Executive Summary
- Reviewability: 55/100
- 結論: NotReady(PR作成前に要対応)
- 主な理由: 目的との関係説明が必要
- Slop リスク: Medium(中)
- PR 粒度: TooBroad(広すぎる)
- 人間判断が必要な項目: 4
- ドメイン確認: 必要(人間によるドメイン判断が必要な項目があります)
- 差分規模: 35ファイル / +1174 / -201
- Top 3 Review Focus:
- Scope drift / 目的外変更:
.github/workflows/ci.yml— CI workflow: インフラファイルの変更が goal に含まれていません(ほか 2 件)。この PR に含めるべき変更か確認してください - Slop リスク Medium(中): 代表検出 R006 (Medium):
crates/kadomori-cli/src/commands/init.rs:369: catch-all エラーを握りつぶしている可能性があります。意図した fallback か著者確認してください — スタブが意図的でないことを確認 - PR 粒度 TooBroad(広すぎる): 20ファイル超過(コアレビュー対象 23 / 20) — 分割要否の判断材料です
- Scope drift / 目的外変更:
スコア内訳
| 項目 | 減点 | 改善アクション |
|---|---|---|
| 差分サイズ | -20 | ファイル数/行数を policy 上限内に収める |
| Scope Drift | -12 | 目的外変更を別 PR に分離するか、goal との関連を説明 |
| Slop | -8 | Slop 候補を確認・修正 |
| ドメイン確認 | -5 | ドメイン判断が必要な項目を担当者と確認 |
確認負担の内訳
- Human-required(人間の判断が必要): 0ファイル
- Context-needed(背景確認が必要): 23ファイル
- Machine-verified(機械的に確認しやすい変更): 12ファイル
- 人間判断が必要な項目: 4
- 著者確認項目: 5
- 分割検討: はい
- PR 前に提示した確認項目: 4 件
- handoff レビュアー判断の省略: 14 件(レビュアー判断 3 / 背景確認 9 / 分割・粒度 2) / source内訳: rule 由来 14
- explain 重点確認の省略: 12 件(レビュアー判断 2 / 背景確認 9 / 分割・粒度 1) / source内訳: rule 由来 12
Policy Coverage
- default rule 由来: 32 件
- team policy 由来: 0 件
- policy seed 由来: 0 件
- ACP 由来: 0 件
Git Summary
- Base:
main - Head:
cursor/resolve-onboarding-friction-dfcd - 変更ファイル数: 35
- Lines: +1174 / -201
- Diff mode:
commit-to-commit - Base SHA:
c11827512b841e48f493de394c1543f76e1ad8a1 - Head SHA:
29ae2579831acdee29bbc2b734af55a6192d3d58
Warnings
なし
詳細証跡
Slop Candidates
リスク: Medium(中)
代表的な検出理由:
- R006 (Medium)
crates/kadomori-cli/src/commands/init.rs:369: catch-all エラーを握りつぶしている可能性があります。意図した fallback か著者確認してください — stub implementation: スタブが意図的でないことを確認
Scope Drift
Goal: 初期オンボーディングの摩擦を解消
- CI workflow: インフラファイルの変更が goal に含まれていません: 2ファイル(代表:
.github/workflows/ci.yml) CHANGELOG.md: リリースノート / changelog の変更は goal との関係説明が必要です
PR Granularity
- 判定: TooBroad(広すぎる)
- 理由:
- 20ファイル超過(コアレビュー対象 23 / 20)
- 800行超過(コアレビュー対象 1273 / 800、raw total 1375)
- 検出トピック:
- CI workflow / permissions: CI workflow 変更は token 権限・secrets / OIDC・deploy 経路を確認してください
- CLI UX / commands: CLI 引数・stdout / stderr・dry-run 導線の変更意図を確認してください
- Markdown / handoff rendering: Markdown / handoff 出力の変更は snapshot と実出力を確認してください
Suggested PR Split
インフラ / CI
インフラ / CI の変更は独立してレビューできます
.github/workflows/ci.yml.github/workflows/release.yml
crates/kadomori-core の変更
ディレクトリ crates/kadomori-core 配下のファイルをまとめて分割できます
crates/kadomori-core/src/config/loader.rscrates/kadomori-core/src/config/mod.rscrates/kadomori-core/src/config/seed/mod.rs
rule-based analysis / pipeline
判定ロジックと pipeline の変更は verdict / score / report 影響をまとめて確認できます
crates/kadomori-core/src/analysis/scope_drift.rscrates/kadomori-core/src/analysis/split_plan.rs
Markdown / handoff rendering
Markdown / handoff 出力の変更は snapshot と実出力をまとめて確認できます
crates/kadomori-core/src/render/pr_body.rscrates/kadomori-core/src/render/responsibility_contract.rs
CLI UX / commands
CLI 引数・対話 UX の変更は利用者導線としてまとめて確認できます リリースノートは本体変更の随伴です。記載内容と変更内容の一致を確認してください。
crates/kadomori-cli/src/commands/doctor.rscrates/kadomori-cli/src/commands/init.rscrates/kadomori-cli/src/commands/mod.rsCHANGELOG.md
scripts/fixtures の変更
ディレクトリ scripts/fixtures 配下のファイルをまとめて分割できます
scripts/fixtures/fake-gh.sh
テスト
テストのみの変更は独立して検証できます
crates/kadomori-cli/tests/cli_integration.rs
ドキュメント
ドキュメント変更は本体コードと分割できます
README.en.mdREADME.mddocs/distribution-archive-readme.mddocs/distribution-repository-readme.mddocs/kadomori-config-authoring.mddocs/release-distribution.mddocs/test-user-onboarding/README.en.mddocs/test-user-onboarding/README.mddocs/test-user-onboarding/release-readiness.mddocs/test-user-onboarding/user-guide.md
補助差分 / snapshots
snapshot 更新は本体差分の後に、対応するテスト結果と期待値差分を確認できます
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
scripts の変更
ディレクトリ scripts 配下のファイルをまとめて分割できます
scripts/install.shscripts/publish-release.shscripts/test-release-distribution.sh
Review Map
Human-required(人間の判断が必要)
なし
Context-needed(背景確認が必要)
- CI workflow 文脈: token 権限、fork PR、secrets / OIDC、release / deploy 経路を確認: 2ファイル(代表:
.github/workflows/ci.yml) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください CLI コマンド変更: 引数・出力・終了コード・後方互換を確認: 3ファイル(代表:
crates/kadomori-cli/src/commands/doctor.rs) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてくださいcrates/kadomori-cli/src/commands/doctor.rscrates/kadomori-cli/src/commands/init.rscrates/kadomori-cli/src/commands/mod.rs
crates/kadomori-cli/tests/cli_integration.rs: CLI テスト: 出力回帰・終了コード・非対話導線を確認 — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください- pipeline 判定影響: verdict / score / review map の代表出力を確認: 2ファイル(代表:
crates/kadomori-core/src/analysis/scope_drift.rs) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください - rendering 出力影響: handoff / report.md / snapshot の読みやすさを確認: 2ファイル(代表:
crates/kadomori-core/src/render/pr_body.rs) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください コンテキスト確認推奨: リポジトリ固有の意図を確認: 4ファイル(代表:
scripts/fixtures/fake-gh.sh) — 著者アクション: リポジトリ固有の意図と PR 目的との整合を確認し、確認内容を PR 本文に書いてくださいscripts/fixtures/fake-gh.shscripts/install.shscripts/publish-release.shscripts/test-release-distribution.sh
CHANGELOG.md: リリースノート / changelog 文脈: 実装差分、互換性表記、ユーザー向け説明が一致するか確認 — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてください実行手順 / hook / config snippet: 公式仕様・ローカル実行・fail-open/blocking 意図を確認: 5ファイル(代表:
README.en.md) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてくださいREADME.en.mdREADME.mddocs/distribution-archive-readme.mddocs/distribution-repository-readme.mddocs/test-user-onboarding/README.md
設定値の変更: 影響範囲・デフォルト値・環境差分を確認: [Config]: 3ファイル(代表:
crates/kadomori-core/src/config/loader.rs) — 著者アクション: PR 目的との整合と代表ケースの挙動や出力差分を確認し、確認内容を PR 本文に書いてくださいcrates/kadomori-core/src/config/loader.rscrates/kadomori-core/src/config/mod.rscrates/kadomori-core/src/config/seed/mod.rs
Machine-verified(機械的に確認しやすい変更)
注: 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
Human-only Decision Points
- スコープ外の変更が検出されました (Author): goal「初期オンボーディングの摩擦を解消」に対して
.github/workflows/ci.ymlなどのスコープ外候補が含まれるため、この PR に含める理由を著者が説明してください - Markdown / handoff 出力の意図確認 (Reviewer):
crates/kadomori-core/src/render/pr_body.rs,crates/kadomori-core/src/render/responsibility_contract.rsの変更: rendering 変更はレビュアーの判断材料に影響するため、golden snapshot と実際の読みやすさを確認してください - レポート判定ロジックの出力影響確認 (Reviewer):
crates/kadomori-core/src/analysis/scope_drift.rs,crates/kadomori-core/src/analysis/split_plan.rsの変更: pipeline / analysis 変更は verdict、score、review map、handoff の出力に波及するため、代表ケースで差分を確認してください - CLI 対話 / 非対話 UX の確認 (Author):
crates/kadomori-cli/src/commands/mod.rs,crates/kadomori-cli/tests/cli_integration.rsの変更: CLI UX 変更は TTY prompt、非 TTY / pipe / CI、--goal 指定時、--non-interactive 指定時の挙動を確認してください
Responsibility Map
著者とレビュアー双方で確認すること
- Markdown / handoff 出力の意図確認(判断理由は Human-only Decision Points 参照): crates/kadomori-core/src/render/pr_body.rs, crates/kadomori-core/src/render/responsibility_contract.rs
- レポート判定ロジックの出力影響確認(判断理由は Human-only Decision Points 参照): crates/kadomori-core/src/analysis/scope_drift.rs, crates/kadomori-core/src/analysis/split_plan.rs
- CLI 対話 / 非対話 UX の確認(判断理由は Human-only Decision Points 参照): crates/kadomori-cli/src/commands/mod.rs, crates/kadomori-cli/tests/cli_integration.rs
著者が確認すること
設定 / インフラ変更の動作確認: 5ファイル(代表: .github/workflows/ci.yml)
.github/workflows/ci.yml.github/workflows/release.ymlcrates/kadomori-core/src/config/loader.rscrates/kadomori-core/src/config/mod.rscrates/kadomori-core/src/config/seed/mod.rs
Author Checklist(著者確認)
- この PR は単一の目的を持っていることを確認しました
- テストが追加または更新されていることを確認しました
- handoff / report.md の snapshot と実出力を確認しました
- 代表ケースで verdict / score / review map の変化を確認しました
- TTY 実行時、非 TTY / pipe / CI、--goal、--non-interactive の代表挙動を確認しました
生成元
- rule-based: true
- llm_attempted: false
- llm_succeeded: false
- llm_outcome:
NotAttempted - analysis_mode:
Default rules(LLMなし) - signals:
default_rules