docs(ledger): 夜間ループでマージした順位を台帳から削除する運用を明文化 - #383
Conversation
|
Important Review skippedAuto incremental reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
📝 WalkthroughWalkthroughPR マージ時のタスク台帳更新手順を追加しました。順位 163 の cross_ref validator テストタスクを関連する一覧から削除しました。 Changesタスク台帳の保守
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
@coderabbitai review |
✅ Action performedReview finished.
|
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/claude-code-web-tasks.md`:
- Line 50: Update the Japanese connective at the end of the explanatory
paragraph in docs/claude-code-web-tasks.md, replacing the sentence-ending
“したがって:” with wording that explicitly introduces the following problems, such as
“そのため、次の問題が起きます。”.
🪄 Autofix
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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: dd0daebe-6043-4563-84ca-1c96bfa7e701
📒 Files selected for processing (3)
docs/claude-code-web-tasks.mddocs/todo-summary.mddocs/todo11.md
💤 Files with no reviewable changes (2)
- docs/todo11.md
- docs/todo-summary.md
🤖 PR Monitor 分析 (GitHub Actions バックストップ)
Applicable Findings (Critical / High / Major)該当なし Applicable Findings (Medium 以下)
Filtered (not applicable)該当なし 次のアクション
|
--- CodeRabbit レビュー対応 (#383、1 件) --- 「したがって:」で文を終えて箇条書きを導く形が唐突、という指摘 (Minor)。観察は採るが 提案文面はそのまま採らない — 提案の「そのため、次の問題が起きます。」は です・ます体で、 本ファイルは である体で統一されている (実測: です・ます 0 件 / である体 18 件) ため、 そのまま入れると文体が混在する。 文書の体裁に合わせて「したがって、後始末をしないと次の 2 つが起きる。」へ直した。 箇条書きが 2 項目であることも明示して、後続との接続を分かりやすくしている。
e22b0ac to
b87fb38
Compare
Summary
Context
Why: 夜間ループの「着手済み」判定は
claude/nightly-*ブランチの存在だけを見る (ADR-072 決定 3)。台帳に完了を示す列は無く、無人可列の✅は着手可否の印であって進捗ではない。したがって PR をマージしてブランチを削除すると、除外マーカーが消えて同じ順位が再選択される。しかも完了済みタスクの重複実装になる。
先行事例: #365 のブランチを手で削除した際、順位 203 が再選択されて #373 が作られた。あれはクローズ由来だったが、マージ由来でも同じ構造で起きる。
なぜ夜間ループ自身が後始末できないか: Guard step が台帳の書き換えを禁止している (ADR-072 決定 6)。自律 actor が自分の選択元を書き換える経路を塞ぐための意図的な制約なので、完了の記録は人間側の責務として残る。今回この責務が明文化されていなかったため、#381 のマージ直前に発覚した (2026-08-10 ユーザー判断で本 PR を先行させた)。
既存の「着手フロー」との関係: 手順 3〜4 が既に「完了後に台帳と todo から削除する」と定めていたが、Web セッションで人間が着手する場合を想定した記述で、夜間ループ経由のケースが読み取れなかった。同じ削除をマージ時に行う旨を専用の節として追加した。
Scope decision: docs のみ。PR #381 のマージはこの PR が land した後に行う (順位 163 が master の台帳から消えていないと、次の夜間 run で再選択されうる)。
Validation
pnpm lint:md(127 files) /pnpm lint:docs: 0 errorpnpm pushpre-push review: approvedgrep '^| 163 ' docs/*.mdが 0 件、詳細エントリの見出しも消滅)---が重複していたため整形済みReferences
Summary by CodeRabbit