Skip to content

feat(build-pr-description): restructure Why as claims backed by evidence - #16

Merged
gitt510 merged 1 commit into
mainfrom
20260731-rework-pr-why-as-argument
Jul 31, 2026
Merged

feat(build-pr-description): restructure Why as claims backed by evidence#16
gitt510 merged 1 commit into
mainfrom
20260731-rework-pr-why-as-argument

Conversation

@gitt510

@gitt510 gitt510 commented Jul 31, 2026

Copy link
Copy Markdown
Owner

Why

  • 現行ルールは Why を「客観事実の集合」として書かせるため、並んだ fact がどの結論を支えるのかの復元が reviewer の宿題になる — Why は主張(判断)とそれを支える根拠の論証として書く規律が必要
    • 現行 allowlist の 1 は動機を「変更を必要とする客観事実」と定義しており、変更を必要とする判断そのものを書く区分が存在しない
    • 現行の完了チェックには「Why が主張であるか」「fact がどの主張を支えるか」を検査する項目が無い

What

allowlist

  • 「動機(客観事実)」を「主張」と「根拠」の2種に分割し、allowlist を4種 → 5種に変更
  • 主張の判定基準を追加: 「この主張が偽なら、この PR は不要になるか」
  • 主張の上限を追加: 1 PR に 1〜3 個
  • 根拠の配置ルールを追加: 支える主張の直下に置く

形式

  • Why を「主張 → 根拠」の2段 nest bullet に変更(nest の使用は Why 限定)
  • paragraph 禁止を Why 特記から全 section 共通ルールに変更

手順・完了チェック

  • 落ちる典型に「事実の陳列」(主張のない Why)を追加
  • 「意思の混入」の変換先を「根拠に支えられる判断」に更新
  • 完了チェックに主張の検査4項目を追加し、既存の「Why が user 確認済み」項目を統合
  • frontmatter の description を新構造(Why is written as an argument)に同期

Test

コマンド 結果
just test 37 pass / 0 fail
just validate-plugins Validation passed

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@gitt510
gitt510 merged commit cdbdd82 into main Jul 31, 2026
2 checks passed
@gitt510
gitt510 deleted the 20260731-rework-pr-why-as-argument branch July 31, 2026 01:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant