docs(todo): split summary and todo5.md tasks to keep files under 50KB#133
Merged
docs(todo): split summary and todo5.md tasks to keep files under 50KB#133
Conversation
Claude Code の読み取り安定性閾値 (~50KB) を超過していた docs/todo.md (68KB) と docs/todo5.md (67KB) を分割。 - docs/todo.md (68KB → 36KB): 推奨実行順序サマリー section を独立した docs/todo-summary.md (新規 33KB) に切り出し。todo.md には pointer のみ残す - docs/todo5.md (67KB → 25KB): PR #101〜#109 由来の古い半分 13 タスクを docs/todo7.md (新規 45KB) に分離。todo5.md には PR #110 以降が残存 - recommended-order-summary anchor 参照 7 箇所 (todo2-6.md / custom-lint-rules.toml) を todo-summary.md# に更新 - ADR-034 / local-llm-offload-history.md の priority table 参照 4 箇所も todo-summary.md に追従更新 - todo-summary.md table の「ファイル」列で 順位 47-67 を todo5.md → todo7.md に変更、todo7.md 内部チェックリストの「本 todoN.md エントリを削除」も整合 全 8 ファイル (todo.md / todo2-7.md / todo-summary.md) が 50KB 以下に収まる。
|
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 (2)
📝 WalkthroughWalkthroughPR ChangesTODO ドキュメント構造再編成
推定レビュー工数🎯 3 (Moderate) | ⏱️ ~25 分 関連 PR
🚥 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. 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: 3
🤖 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/todo6.md`:
- Line 5: The preamble in docs/todo6.md incorrectly states "六つすべてを確認すること" even
though the repo currently has eight todo files; update that sentence to reflect
eight files and list them explicitly (todo.md, todo2.md, todo3.md, todo4.md,
todo5.md, todo6.md, todo7.md, todo-summary.md) and ensure the parallel note in
todo7.md is also updated so both preambles are consistent with the PR summary.
In `@docs/todo7.md`:
- Line 5: Update the sentence that reads "新セッションでは七つすべてを確認すること" to reflect the
current count after adding todo-summary.md by replacing "七つ" with "八つ" (or
otherwise adjusting the number to match the actual files to be checked), and
verify any neighboring text that enumerates files still matches the new total so
the statement and any counts remain consistent.
- Line 103: The relative ADR link
"../docs/adr/adr-036-bundle-z-three-layer-review.md" in docs/todo7.md is
incorrect (it resolves to docs/docs/...), so update that link text to the
correct path from the docs/ folder (e.g.
"adr/adr-036-bundle-z-three-layer-review.md" or
"./adr/adr-036-bundle-z-three-layer-review.md") so the reference to ADR-036
resolves properly.
🪄 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: d01e3962-dc52-4675-b430-b47cd196d9a3
📒 Files selected for processing (11)
.claude/custom-lint-rules.tomldocs/adr/adr-034-coderabbit-auto-monitoring.mddocs/local-llm-offload-history.mddocs/todo-summary.mddocs/todo.mddocs/todo2.mddocs/todo3.mddocs/todo4.mddocs/todo5.mddocs/todo6.mddocs/todo7.md
PR #133 で CodeRabbit が検出した 3 件 Minor finding を修正: - docs/todo6.md:5: 「六つすべてを確認すること」→「八つすべてを確認すること (todo.md / todo2-7.md / todo-summary.md)」 - docs/todo7.md:5: 「七つすべてを確認すること」→「八つすべてを確認すること (todo.md / todo2-7.md / todo-summary.md)」 (todo-summary.md カウント漏れ修正) - docs/todo7.md:103: ADR-036 link を `../docs/adr/...` → `adr/...` に修正 (todo7.md は docs/ 配下なので `../docs/` は誤り。todo5.md からの分割で表面化 した pre-existing bug) cli-finding-classifier (ADR-038) は ADR-035 docs-only filter で not_applicable 判定したが、GitHub thread state は依然 unresolved のため fold-in fix で thread を closeable な状態にする。
Owner
Author
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
4 tasks
aloekun
added a commit
that referenced
this pull request
May 9, 2026
…#134) - todo6.md / todo-summary.md に Bundle j 3 タスク追加 (順位 94-96) - 94: docs/ 内 ../docs/ 相対パストラップ検出 lint rule (Tier 1, S) - 95: docs/todo*.md preamble file count 自動照合 (Tier 2, S) - 96: Markdown cross-reference validator CI step (Tier 2, M) - local-llm-offload-analysis.md に Phase c MVP 完了 (PR #132) を反映 - 実装方針変更 (takt facet -> cli-push-runner stage) を §1 Phase c に記述 - Phase c+ (Bundle i) として PR #132 post-merge-feedback 採用 3 件 - §8.E を MVP land 済みステータスに更新 - 再開チェックリストに Phase c smoke / Bundle i 着手手順を追加 - todo-summary.md retrospective に Bundle j (PR #133 post-merge-feedback) 追記
This was referenced May 9, 2026
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
Claude Code の読み取り安定性閾値 (~50KB) を超過していた
docs/todo.md(68KB) とdocs/todo5.md(67KB) を分割。全 todo*.md / todo-summary.md (8 ファイル) を 50KB 以下に整列。docs/todo.md(68KB → 36KB): 推奨実行順序サマリー section を新規docs/todo-summary.md(33KB) に切り出し。todo.md には pointer のみ残すdocs/todo5.md(67KB → 25KB): PR feat(check-ci-coderabbit): --list-findings モード追加 (Bundle a Sub-PR 1) #101〜feat(docs): ADR-035 docs-only PR 評価ポリシー集約 (順位 59/31/32) #109 由来の古い半分 13 タスクを新規docs/todo7.md(45KB) に分離。todo5.md には PR feat(hooks): Bundle "docs quality pre-write" — Stop hook lint:md + ephemeral todo ref detection (順位 4/29) #110 以降のエントリが残存todo-summary.md#recommended-order-summaryに追従更新 (docs/todo2-6.mdline 7 +.claude/custom-lint-rules.tomlexample)docs/adr/adr-034-coderabbit-auto-monitoring.mdの section 名・参照先 +docs/local-llm-offload-history.mdの priority table 参照 3 箇所)todo5.md→todo7.mdに変更、todo7.md内部チェックリスト「本 todoN.md エントリを削除」も整合File Sizes (50KB Threshold Check)
docs/todo.mddocs/todo2.mddocs/todo3.mddocs/todo4.mddocs/todo5.mddocs/todo6.mddocs/todo7.mddocs/todo-summary.mdSplit Boundary Rationale
docs/todo5.mdは古い半分 (PR #101〜#109、順位 47-67、13 タスク) をdocs/todo7.mdに移動し、新しい半分 (PR #110 以降、順位 68-88、11 タスク) を残す形で分割。これによりdocs/local-llm-offload-history.mdの "末尾"・"順位 80-88 = todo5.md" 参照が破壊されない。Pre-push Review
simplicity-review / security-review 両方 APPROVE (run:
20260509-054442-pre-push-review)。Non-blocking findings 2 件 (preamble カウント不整合):
docs/todo6.mdline 5:「六つすべてを確認すること」→ 八つに更新が必要docs/todo7.mdline 5:「七つすべてを確認すること」→todo-summary.mdを含めて八つに更新が必要機能影響なしの ephemeral tracking なので別 commit で fold-in 検討。
Test Plan
recommended-order-summaryanchor が GitHub 上で正しく解決すること (markdown プレビュー確認)docs/todo-summary.mdtable 内のファイル列リンク (todo2.md〜todo7.md) が解決することSummary by CodeRabbit
Documentation
Chores