[Docs] ADR の「影響」欄から変更対象ファイルの具体パス列挙を排除#187
Merged
Merged
Conversation
ADR-032 / ADR-033 の「影響範囲」セクションが、決定の帰結・トレードオフ ではなく変更対象ファイルの具体パス表になっていた。これは CLAUDE.md 「文書間参照にパスを焼き込まない」に反し、かつ ADR という「決定と根拠を 記録する文書」の抽象度から外れた実装計画寄りの情報である。ADR は決定に 至る過程で参照した文書を ID でメタデータ欄に残せば十分で、「どのファイルを どう変更するか」は実装計画（Issue・plan.yaml）の責務。 - adr_format.md: テンプレートの「4. 影響」欄に、パス列挙を書かない旨と 実装計画へ譲る旨を明記 - ADR-032 / ADR-033: 「影響範囲」の具体パス表を、決定の帰結・トレードオフ の記述（得られるもの・失うもの・新たに生じる制約）に置き換え - ADR-032 は accepted 済みだが、変更は決定内容（本文）ではなく付随する 影響記述の記述粒度の是正であり、adr_format.md の「本文（コンテキスト・ 決定・代替案）は書き換えない」原則には抵触しない Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
前コミットで plugins/forge/docs/adr_format.md を編集したため rules_toc.yaml / .toc_checksums.yaml が stale になっていた (test_toc_checksums_match_current_docs 失敗)。update-forge-toc スキルで再生成。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
/forge:review#167 (ADR-033 実装) の着手前に発見した設計文書側の問題を修正。ADR-032 / ADR-033 の「影響範囲」セクションが、決定の帰結・トレードオフではなく変更対象ファイルの具体パス列挙表になっていた。問題
変更内容
plugins/forge/docs/adr_format.md: テンプレートの「4. 影響」欄に、変更対象ファイルの具体パス列挙をここに書かない旨・実装計画へ譲る旨を明記docs/specs/forge/design/ADR-032_path_schema_unification.md: 「影響範囲」の具体パス表を、決定の帰結・トレードオフ（得られるもの・失うもの・新たに生じる制約）の記述に置き換えdocs/specs/forge/design/ADR-033_structured_finding_output.md: 同様に置き換えADR-032（accepted 済み）を修正した理由
adr_format.mdは「一度 accepted になった決定の本文（コンテキスト・決定・代替案）は書き換えない」と定めているが、今回の変更は決定内容（本文）ではなく、決定に付随する影響記述の記述粒度の是正。本文の「決定」セクション（設計原則 1・4 等）との整合性を確認した上で、その内容に基づいて「影響」欄を書き直しており、決定自体の変更ではない。検証
dprint checkpass（3 ファイルとも）🤖 Generated with Claude Code