docs: ADR運用文書を現行ルールへ合わせる - #61
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: eadf5fe855
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| 実際に検討した案と却下理由を書く。外部制約により選択肢がなかった場合は、この節を削除する。 | ||
|
|
||
| ## 結果 | ||
| ## 受け入れた代償 |
There was a problem hiding this comment.
ADR-0017 を置換してからテンプレートを変更してください
この見出し変更は、承認済みの docs/adr/0017-adr-in-repo.md が固定している形式(背景 / 決定 / 検討した代替 / 結果 / 再評価のサイン)を変更しますが、0017 は承認済みのままで置換 ADR も追加されていません。同 ADR が定める「判断変更時は新しい ADR を作り、旧 ADR を廃止にする」という履歴も残らないため、現行テンプレートと承認済み決定のどちらに従うべきかが矛盾します。新しい運用判断を ADR として記録し、0017 のステータスから参照してください。
AGENTS.md reference: AGENTS.md:L159-L162
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
対応
ご説明の方針を確認しました。この指摘は採用せず、コードおよび文書の変更は行っていません。
- Issue docs: ADR運用文書を現行ルールへ合わせる #60 では、既存 ADR の本文やステータスの再評価は対象外です。
- ADR-0017 は当時の運用を記録した歴史的文書として維持します。現在も、設計判断を
docs/adr/に保存するという当時の決定内容を記録しています。 docs/adr/0017-adr-in-repo.mdL26-L38 - 現在の作成基準・書式・更新手順の正は
adrskill であり、リポジトリ内の入口はdocs/adr/README.mdであることが明記されています。 docs/adr/README.mdL3-L8 - テンプレートには現行のセクションと、置換時の取り扱いが反映されています。 docs/adr/template.mdL7-L29
- 置換 ADR に同じ書式規範を複製すると二重管理になるため、追加 ADR や ADR-0017 のステータス変更は不要と判断しました。
変更がないため、コミットおよび新しいプルリクエストは作成していません。
確認
- ✅
git status --short --branch - ✅
git log -1 --oneline - ✅
nl -ba docs/adr/README.md | sed -n '1,35p' - ✅
nl -ba docs/adr/template.md | sed -n '1,40p' - ✅
nl -ba docs/adr/0017-adr-in-repo.md | sed -n '1,90p'
domain-modeling skill が用語とコンテキスト境界を担い、CONTEXT.md を必要時に作成・更新する責務を明確にする。
概要
adrskill に一本化docs/adr/template.mdを現在の skill テンプレートへ同期検証
cmp -s docs/adr/template.md ~/.agents/skills/adr/assets/template.mdgit diff --checkbun run cf-typegenbun run check(format、lint、3系統の型検査、208 tests、production build)対象外
AGENTS.md全体の再構成Closes #60