docs: PR #135/#136 land 反映 — 順位 97 追加 + Phase d gate 解消の stale-doc cleanup#137
Merged
Conversation
…eanup PR #135 (Bundle i) と PR #136 (§8.D / num_ctx 8192) merge 後の post-merge-feedback / 状態整合に伴う doc 更新を 1 PR で集約。 ## 順位 97 追加 (PR #136 T2-#1 採用) PR #136 post-merge-feedback で ✅ 採用された Tier 2 #1 (`with_num_ctx(X)` override 値 serialization 検証テスト、Effort S / Adoption Risk None) を todo に登録。 - docs/todo-summary.md: 順位 97 行を table に追加 (Tier 2 / S / 依存なし) - docs/todo6.md: 詳細エントリ (動機 / 設計決定 / 作業計画 / 完了基準) を追加 CodeRabbit nitpick ではなく post-merge-feedback agent が独立に発見した test gap。 既存 num_ctx_is_serialized_into_request_body は default 値 (8192) のみ検証で with_num_ctx(X) の wiring が壊れた場合の silent degrade を捕捉できない。 ## Phase d gate 解消の stale-doc cleanup PR #136 で §8.D (= num_ctx 8192) が land したため、analysis.md 内で「Phase d 着手の 最終 gate」と記述していた箇所が stale 化。4 箇所を更新: - L5 (状態 banner): §8.D 完了表記を追加、'最終 gate' を 'kickoff 待機' に書き換え - L129 (§1 Phase c+ 残る最終 gate): 'land 完了' に書き換え + root cause pivot 経緯を保存 - L153 (§2 §8.E Phase d 着手前提): ✅ 完了マーク + pivot 経緯 - L185 (§4 再開チェックリスト): '§8.D v4 改訂ループ' → 'Phase d dogfood' - L200 (§4 次に何をするか): Phase d kickoff prep / 実 dogfood / 結果集約 の 3 段階に再構成 ## Phase d 着手前提の確認 (a) [lint_screen] config silent failure 防止 ✅ PR #135 (b) scale-aware fixtures による reproducible measurement ✅ PR #135 (c) cross-file partial fix anti-pattern global rule 化 ✅ PR #135 (d) JSON 完全性問題への一次対策 ✅ PR #136 (num_ctx 8192) → Phase d kickoff 可能。dogfood 自体は実 PR 3-5 件で long-running。
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
📝 WalkthroughWalkthroughPhase c完了と§8.D最終ゲート確認（num_ctx拡張対応）に基づき、ドキュメント群を更新した。Phase d着手前提を明確化し、実行順序（kickoff prep → dogfood → 結果集約と並行独立タスク）を整理し、with_num_ctx override値serialization検証テストを新規タスクとして定義。 Changesフェーズ状態・実行順序・テストタスク統合
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 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. 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: 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/todo-summary.md`:
- Line 75: Update the heading date in docs/todo-summary.md to reflect the new
addition (item 97): locate the heading text "推奨実行順序サマリー (2026-04-29 更新...)" and
change the date to the current update date that corresponds to when you added
the row for item 97 so the heading and the table remain consistent.
🪄 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: 2633217c-1d0c-48dc-8f7f-e987ffe8e971
📒 Files selected for processing (3)
docs/local-llm-offload-analysis.mddocs/todo-summary.mddocs/todo6.md
…#r3213575272) 順位 97 を 2026-05-10 に追加したが見出しの「(2026-04-29 更新)」が stale 化していた ため修正。 外部参照は併設の `<a id="recommended-order-summary"></a>` 経由のため anchor は break しない (`coding-style.md` § 日付入り見出しアンカー — 安定識別子優先 に準拠)。 CodeRabbit PR #137 review #r3213575272 採用。
4 tasks
aloekun
added a commit
that referenced
this pull request
May 10, 2026
…#138) * docs(todo): 順位 98 (PR #137 T1-#1 採用) を Bundle i 系列に追加 PR #137 (docs cleanup) post-merge-feedback で ✅ 採用された Tier 1 #1 (`num_ctx` overflow detection — JSON parse error 検知時の context window 診断ログ、 Severity Medium / Effort M / Adoption Risk: 派生プロジェクト deploy コストのみ) を todo に登録。 - docs/todo-summary.md: 順位 98 行を table に追加 (Tier 1 / M / 依存なし) - docs/todo6.md: 詳細エントリ (動機 / 設計決定 / 作業計画 / 完了基準 / 詰まっている箇所) PR #136 セッションで Claude が「prompt 設計の問題」と誤診し v4 prompt 改訂ループ という名前の誤った means に向かいかけた事例 (実 root cause は num_ctx default 4096 超過) から抽出された learning。lib-ollama-client の response validation 層に warn log を追加することで、将来の同型事故を decisive に診断できる runtime hint を整備。 サイズ: todo6.md 32.6KB / todo-summary.md 36.8KB (いずれも 50KB ガイドライン内)。 * docs(local-llm-offload): Phase d kickoff prep 完了 — 運用ガイド独立 doc + analysis.md 状態同期 ユーザー判断 (Phase d kickoff、2026-05-10) で以下を確定: - (a) config switch: session-only opt-in (commit せず session 内のみ enable、kill-switch 即可) - (b) metrics: lint_screen latency p50/p95 + fallback rate + Claude session input token 削減効果 (質的傾向) - (c) kill-switch: fallback rate > 50% で停止 成果物: - 新規: docs/local-llm-offload-phase-d-guide.md (Phase d 運用ガイド、≈170 行) — Setup / 計測 / Kill-switch / Scope / 結果集約フロー の 5 section - 更新: docs/local-llm-offload-analysis.md - 状態 banner: Phase d kickoff prep 完了表記を追加 (2026-05-10、guide link 付き) - §4 「次に何をするか」 #1: Phase d kickoff prep を ✅ 完了表示、決定事項 (a)/(b)/(c) を記録、3 obstacles の scope 外確定を併記 過去 dogfood 阻害要因 3 種 (findings ゼロ / review body 抽出漏れ / rate-limit) は classifier 専用で lint_screen は CR 非依存のため scope 外と確定。advisor 指摘で 事前検証し、対応設計を不要化。 guide は history.md (50KB cap 到達済) ではなく独立 doc として stable 配置、引退条件は Phase d 完了 → §8.E 採否判定 → ADR-038 採用/却下時に削除。 * docs(local-llm-offload-analysis): Phase d 実 dogfood の 5 PRs roster を §4 に反映 Phase d kickoff prep 完了 (前 commit、vqtuonzu) に基づく具体的な dogfood 対象 PR roster を「次に何をするか #2」に追加。`docs/todo-summary.md` から選定。 | Order | 構成 | Effort | Diff Profile | |---|---|---|---| | P-1 | Bundle h (89+90) + Bundle g-2 (87+88) | M | docs only | | P-2 | Bundle j-1 (順位 94) | S | TOML + Rust | | P-3 | Bundle g-1 (順位 85+86) | M | Rust impl + test | | P-4 | Bundle d (順位 68) | S | Rust test only | | P-5 | Bundle c-1 (順位 63+64+67) | L | Rust impl ×2 + ADR | 設計判断: - Effort 分布 M→S→M→S→L で size ramp-up: kill-switch (fallback > 50%) signal の 質を切り分け可能 (小規模 fallback = 設計 issue、大規模 fallback = num_ctx 再到達) - Bundle h + g-2 を 1 PR に統合: 共通テーマ "global rules consolidation" - Bundle f 除外: `(defer)` 表記 = systemic 性未確認 具体運用 (Setup / 計測 / Kill-switch) は phase-d-guide.md §1-3 に従う。
Merged
8 tasks
This was referenced May 11, 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
PR #135 (Bundle i) と PR #136 (§8.D / num_ctx 8192) merge 後の doc 反映を 1 PR で集約。docs only、コード変更なし。
with_num_ctx(X)override 値 serialization 検証テスト) を todo に登録変更内容
順位 97 追加 (PR #136 T2-#1 採用)
CodeRabbit nitpick ではなく post-merge-feedback agent が独立に発見した test gap。既存
num_ctx_is_serialized_into_request_bodyは default 値 (8192) のみ検証で、with_num_ctx(X)の wiring が壊れた場合の silent degrade を捕捉できない。Phase d gate 解消 (4 箇所の stale-doc cleanup)
PR #136 で §8.D (= num_ctx 8192) が land したため、analysis.md 内で「Phase d 着手の最終 gate」と記述していた箇所が stale 化:
Phase d は未着手で §8.D v4 prompt 改訂が最終 gate§8.D 完了 = PR #136 land 2026-05-09 (num_ctx 8192、agreement 86.7% / verdict GO)、Phase d 着手前提条件は全て充足済、kickoff 待機§8.D v4 prompt 改訂で eval13/15 の JSON 完全性問題に一次対策PR #136 で land 完了+ root cause pivot 経緯 (prompt → num_ctx) を保存大規模 diff の JSON 不完全問題への一次対策 = §8.D v4 prompt 改訂✅ PR #136 で land 完了+ pivot 経緯§8.D v4 改訂ループPhase d dogfoodPhase d 着手前提の現状
[lint_screen]config silent failure 防止→ Phase d kickoff 可能。dogfood 自体は実 PR 3-5 件で long-running。
Test plan
本 PR は docs only、機能変更なし。
markdownlint-cli2clean (PostToolUse hook で確認)grep "§8.D v4|prompt 改訂が最終 gate|prompt 改訂ループ"で stale references が「過去経緯としての記述」のみに収まることを確認Out of scope
docs/todo-summary.md+docs/todo6.mdの Bundle i 詳細エントリ) は PR feat(cli-finding-classifier, cli-push-runner): Bundle i — Phase d 着手前必須 follow-up (順位 91 + 92 + 93) #135 land 済のためfeedback_todo_no_history違反。本 PR では scope を「順位 97 追加 + analysis.md cleanup」に絞ったため未対応。Phase d kickoff PR or 別 cleanup PR で対処予定push-runner-config.tomlのenabled切替判断 / metrics 計測スクリプト整備 / dogfood 運用ガイド作成は別 PRSummary by CodeRabbit
リリースノート
本リリースには、ユーザーに見える機能追加やバグ修正はありません。ドキュメントの内部更新のみが含まれています。