feat(adr): ADR-033 todo numbering simplification (試験運用)#92
Merged
Conversation
PR #85/#86/#88/#89/#90/#91/Bundle 1 で連続発生した renumber 作業の構造的負債を 解消する ADR を起案。 問題: - docs/todo.md の本文中 `順位 N` cross-reference が線形コストで増加 - PR #91 では 4 件追加に対し本文 8 箇所修正、過去 PR で stale reference の 追従漏れによる CodeRabbit Minor 指摘が発生 (a15b263 で修正) - 採番のみが情報源で、本文と表の往復が発生 決定 (選択肢 B 採用): - 絶対番号 (`順位 N`) は推奨実行順序サマリー table のみに保持 - 本文での参照は task 名 (heading text or 略称) で行う - table の `依存` 列のみ絶対番号を許可 (table 内なので renumber 同期可能) - 「Tier N (順位 X/Y)」表記の `(順位 X/Y)` 部分は除去 → `Tier N` のみ残す 却下した選択肢: - 選択肢 A (renumber script の自動化): 保守コスト新規発生、問題のすり替え - 選択肢 C (現状維持): convergence loop の一因、線形増加は止まらない 本 commit は ADR と CLAUDE.md リンク追加のみ。本文 cross-ref の一括変換と 新規 entry template は次の commit で実施。 References: - docs/adr/adr-013, adr-022, adr-028 - .claude/feedback-reports/86.md Tier 3 #3 (起案動機の起源) - PR #91 a15b263 (stale reference 起因の Minor 指摘実例)
ADR-033 のガイドライン適用。docs/todo.md / todo2.md / todo3.md の本文中 `順位 N` cross-reference をすべて task 名参照に変換。 変更対象: - docs/todo.md: 戦略 section の 12 箇所、Tier 4/5 entry の (順位 X/Y) 表記、 週次レビュー entry の cross-ref - docs/todo2.md: ADR-032 series の Phase ごと優先度表記、reviewer facet / 各 Tier 1/2/3 entry の (順位 X/Y) と inline 参照 - docs/todo3.md: Stop hook lint:md / AI 生成一時スクリプト / cli-pr-monitor polling 延長 / post-pr-review rate-limit / .failed marker / Recovery 経路 / REJECT-ESCALATE entry の inline 参照を全て task 名に変換 維持した absolute number: - docs/todo.md の推奨実行順序サマリー table の `順位` 列 (source of truth) - table の `依存` 列の絶対番号 (table 内なので renumber と同期可能) - table 内 cell の歴史的 `旧順位 N` 表記 (完了済み task の参照) 未変換の例外: - docs/todo2.md line 696 の採番管理 ADR 元 entry の `(順位 24/26)` は 本 PR の Commit 3 で entry 全体を削除するため未変換 (削除待ち) 検証: ```sh grep -nE '順位 [0-9]+' docs/todo.md docs/todo2.md docs/todo3.md # 期待: docs/todo.md table 列のみ + docs/todo2.md 採番管理 ADR 元 entry のみ ``` References: - docs/adr/adr-033-todo-numbering-simplification.md (本 ADR の決定)
ADR-033 land により採番管理簡素化 task が完了したため、運用ルール (memory feedback_todo_no_history.md) に従い元 entry を削除する。 変更: - docs/todo2.md: "todo.md 採番管理の簡素化 ADR 起案 (PR #86 T3-3)" entry 全体 (40 行) を削除 - docs/todo.md summary table: - 旧順位 27 (採番管理 ADR) 行を削除 - 旧順位 28 (ADR-030 Phase E/F) → 順位 27 に renumber - 旧順位 29 (ADR-030 takt-test-vc) → 順位 28 に renumber、依存表記の `順位 28 Phase F` も `順位 27 Phase F` に追従 - サマリー header 日付ラベルを "Bundle 1 完了後" → "ADR-033 land 後" に更新 検証: - 本文 cross-ref は Commit 2 で除去済のため、本 commit の renumber は table 行と `依存` 列のみで完結 (ADR-033 の効果を本 PR 内で実証) - markdownlint pass (PostToolUse hook で各 Edit ごとに検証)
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
✅ Files skipped from review due to trivial changes (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthroughADR-033を追加し、todo参照規約を「本文中での絶対番号（ Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ 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. Review rate limit: 0/1 reviews remaining, refill in 60 minutes.Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 2
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@docs/adr/adr-033-todo-numbering-simplification.md`:
- Around line 111-113: The grep currently uses "順位 [0-9]+" which only matches
numeric ranks and misses non-numeric occurrences like "順位 X"; change the regex
in the command to match any non-empty token after "順位" (for example use "順位
[^[:space:]\|]+" with grep -E) so the check detects both numeric and non-numeric
references; update the first grep pattern in the existing pipeline (the one that
currently reads "順位 [0-9]+") to this broader pattern and keep the rest of the
pipeline/exclusions unchanged.
In `@docs/todo3.md`:
- Line 511: Line 511 に残っているプレースホルダ「順位
X」を削除して、本文方針「本文はタスク名参照」に合わせて該当タスク名に置き換えてください（ここは `.claude/` filter、takt
harness、supervise や fix_supervisor の文脈をそのまま保持しつつ、明確に対象タスク名を記述するようにしてください）。
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: 2656b823-1ec7-464d-87b3-95dfa0ef18d9
📒 Files selected for processing (5)
CLAUDE.mddocs/adr/adr-033-todo-numbering-simplification.mddocs/todo.mddocs/todo2.mddocs/todo3.md
PR #92 で受けた CodeRabbit Minor findings 2 件を child commit として修正。 両 finding は self-consistent: Finding 1 の改善 regex が Finding 2 の placeholder 残存を機械的に検出可能になる。 1. Minor: ADR-033 line 111 の検証 regex が数値以外の placeholder を見逃す - 旧: `順位 [0-9]+` (数値のみマッチ) - 新: `順位 [0-9A-Za-z_-]+` (英字 placeholder `順位 X` / `順位 N` も検出) - 期待コメントも更新し、placeholder 検出意図を明記 2. Minor: docs/todo3.md line 511 (REJECT-ESCALATE entry の動機セクション) で `.claude/ filter (順位 X = T2-1+T3-2 Bundle)` という placeholder が残存 - 旧: `.claude/ filter (順位 X = T2-1+T3-2 Bundle)` - 新: `.claude/ filter + ADR-030 制約明記 task (PR #91 T2-1 + T3-2 Bundle)` - ADR-033 「本文はタスク名参照」方針に準拠 検証 (改善 regex で 0 件達成): ```sh grep -nE "順位 [0-9A-Za-z_-]+" docs/todo.md docs/todo2.md docs/todo3.md | grep -vE "推奨実行順序サマリー|^[^:]+:[0-9]+:\| [0-9]+ \|" # 結果: 0 行 ```
Owner
Author
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
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.
Summary
docs/todo.mdの renumber 作業で連続発生していた 本文 cross-reference 追従コスト を構造的に解消する ADR を起案。3 commits で「ADR 起案 → 既存ファイルの一括変換 → 完了 task の bookkeeping」を順次 land。動機 (PR 履歴の累積データから)
PR #85/#86/#88/#89/#90/#91/Bundle 1 で連続発生した renumber 作業は、本文中の
順位 Ncross-ref 追従更新まで含めると毎回 5〜10 件の Edit を要し、過去には PR #91 で「順位 25/26が25/25に更新されないまま merge → CodeRabbit Minor 指摘 → fix commit (a15b263)」という convergence loop も発生した。タスク数が 29 まで成長した時点で、cross-ref 追従コストは 線形増加 が明確で構造的負債として顕在化。
変更内容
Commit 1:
6cb3ee77— feat(adr): ADR-033 起案新規 ADR
docs/adr/adr-033-todo-numbering-simplification.md:順位 N) は推奨実行順序サマリー table のみに保持、本文ではタスク名で参照grepでセルフチェック)CLAUDE.md の Architecture Decisions リストに ADR-033 リンク追加。
Commit 2:
63620f07— refactor(docs): cross-ref 一括変換docs/todo.md/docs/todo2.md/docs/todo3.mdの本文中順位 Ncross-reference をすべて task 名参照に変換 (累計 30+ 箇所):docs/todo.md: 戦略 section 12 箇所、Tier 4/5 entry の(順位 X/Y)表記、週次レビュー entry の cross-refdocs/todo2.md: ADR-032 Phase ごと優先度表記、reviewer facet / 各 Tier 1/2/3 entry の(順位 X/Y)と inline 参照docs/todo3.md: 7 entry の inline 参照を全て task 名に変換維持した absolute number:
順位列 (source of truth)依存列の絶対番号 (table 内なので renumber と同期可能)旧順位 N表記 (完了済み task の歴史的参照)Commit 3:
59d1c981— docs(todo): bookkeepingADR-033 land により採番管理簡素化 task が完了したため:
docs/todo2.mdの元 entry (40 行) を削除docs/todo.mdsummary table から旧順位 27 (採番管理 ADR) 行を削除、旧 28/29 を 27/28 に renumberCommit 3 の renumber 作業は table 行と
依存列のみで完結 — 本 PR 自身が ADR-033 の効果を実証する形になっている。期待される収束効果
docs/todo.md系列の docs PR review 時間Test plan
grep -nE '順位 [0-9]+' docs/todo.md docs/todo2.md docs/todo3.mdの結果が table 依存列以外で 0 件Summary by CodeRabbit
ドキュメンテーション