Skip to content

docs(adr-028): 原則 6 — multi-line PR body は --body-file で渡す - #188

Merged
aloekun merged 1 commit into
masterfrom
claude/next-task-code-web-docs-J9tZQ
May 31, 2026
Merged

docs(adr-028): 原則 6 — multi-line PR body は --body-file で渡す#188
aloekun merged 1 commit into
masterfrom
claude/next-task-code-web-docs-J9tZQ

Conversation

@aloekun

@aloekun aloekun commented May 30, 2026

Copy link
Copy Markdown
Owner

概要

docs/claude-code-web-tasks.md の順位 166 (Tier 3) タスクを実装しました。

ADR-028 に PR body 複数行時の --body-file 推奨 + shell argument truncation の why/how 補足追記 (PR #181 T3-#1 採用)

PR #134 / #181 で 2 回観測された pnpm create-pr の PR body truncation を、ADR-028 に permanent reference として codify します。順位 165 が test 層で防御するのに対し、本タスクは docs 層で後発の AI / reviewer が逆引き可能な構造的予防策を作る相補関係です。

変更内容

  • docs/adr/adr-028-pnpm-create-pr-gate.md
  • トラッキング更新(着手フローに準拠): claude-code-web-tasks.md / todo-summary.md の順位 166 行、todo9.md の詳細セクションを削除

完了基準の充足

docs-only の変更で副作用はありません。


Generated by Claude Code

Summary by CodeRabbit

  • Documentation

    • アーキテクチャ決定記録(ADR-028)にPR本体の複数行対応に関する運用規範を追記しました。
  • Chores

    • 完了済みタスクをタスク管理ドキュメントから削除し、リストを整理しました。

Review Change Stack

PR #134 / #181 で観測された PR body truncation を ADR-028 に codify。
シェル引数直渡し時に改行が delimiter 化して body が partial 化する
why、scratch file 経由 (--body-file / gh pr edit --body-file) の how を
原則 6 として追記し、memory feedback_pnpm_create_pr_body と相互参照。

claude-code-web-tasks / todo-summary / todo9 の順位 166 entry を削除。
@coderabbitai

coderabbitai Bot commented May 30, 2026

Copy link
Copy Markdown

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: a30a03dd-c4e7-4863-b8b7-145c8eaea1c2

📥 Commits

Reviewing files that changed from the base of the PR and between cd747b4 and efa896f.

📒 Files selected for processing (4)
  • docs/adr/adr-028-pnpm-create-pr-gate.md
  • docs/claude-code-web-tasks.md
  • docs/todo-summary.md
  • docs/todo9.md
💤 Files with no reviewable changes (3)
  • docs/claude-code-web-tasks.md
  • docs/todo-summary.md
  • docs/todo9.md

📝 Walkthrough

Walkthrough

ADR-028ドキュメントに「原則6」を追加し、PR body複数行時の--body-file推奨をcodify。対応するタスク管理ドキュメント(claude-code-web-tasks.md、todo-summary.md、todo9.md)から完了済みタスク記録を削除して同期。

Changes

ADR-028「原則6」の追加とタスク同期

Layer / File(s) Summary
ADR-028「原則6」の追加と参照整備
docs/adr/adr-028-pnpm-create-pr-gate.md
ADR-028ドキュメントのステータス表記に2026-05-30の改訂を追記し、PR bodyが複数行の場合は--bodyのシェル引数直渡しを避け、スクラッチファイル経由で--body-fileを使うという原則6を追加。一次防衛memoryと根拠PR(#134#181)への参照も併記。
タスク管理ドキュメントの同期削除
docs/claude-code-web-tasks.md, docs/todo-summary.md, docs/todo9.md
採用タスク表、todo-summary、todo9から、ADR-028補足追記タスク(順位166)の記録を削除し、タスク管理ドキュメント間の一貫性を保つ。

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related PRs

  • aloekun/claude-code-hook-test#56: ADR-028の初版導入を行ったPRであり、本PRはその後続の原則6追加として関連。
  • aloekun/claude-code-hook-test#182: PR #182がtodo-summary.md・todo9.mdに順位166のタスクを追加したのに対し、本PRがその同一タスク記録を削除しており、ライフサイクル管理で対応。
  • aloekun/claude-code-hook-test#57: ADR-028のPR body処理ルール(--body-fileによる一時ファイル経由)を実装したPRであり、本PRはその運用規範をADRレベルで正式化。
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed PRタイトルはADR-028に原則6を追加し、マルチラインPR bodyを--body-fileで渡すという主要な変更を明確に要約している。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aloekun
aloekun merged commit 32d3269 into master May 31, 2026
1 check passed
@aloekun
aloekun deleted the claude/next-task-code-web-docs-J9tZQ branch May 31, 2026 04:00
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.

2 participants