Skip to content

docs: Phase d kickoff prep + 順位 98 追加 + dogfood roster 反映 (3 commits)#138

Merged
aloekun merged 3 commits intomasterfrom
docs/phase-d-kickoff
May 10, 2026
Merged

docs: Phase d kickoff prep + 順位 98 追加 + dogfood roster 反映 (3 commits)#138
aloekun merged 3 commits intomasterfrom
docs/phase-d-kickoff

Conversation

@aloekun
Copy link
Copy Markdown
Owner

@aloekun aloekun commented May 10, 2026

Summary

Phase d (lint_screen 実環境 dogfood) の kickoff prep を完了し、PR #136 / #137 post-merge-feedback で採用された 2 件 (順位 97/98) を todo に登録。3 commit に分離した docs-only PR。

変更内容

Commit 1: 順位 98 追加 (PR #137 T1-#1 採用)

PR #136 セッションで Claude が「prompt 設計の問題」と誤診し v4 prompt 改訂ループという誤った means に向かいかけた事例から抽出された learning。lib-ollama-client の response validation 層に warn log を追加することで、将来の同型事故を decisive に診断できる runtime hint を整備。

  • docs/todo-summary.md: 順位 98 行 (Tier 1 / M / 依存なし)
  • docs/todo6.md: 詳細エントリ (動機 / 設計決定案 A/B/C / 作業計画 / 完了基準 / 詰まっている箇所)

Commit 2: Phase d kickoff prep 完了 — 運用ガイド独立 doc + analysis.md 状態同期

ユーザー判断 (Phase d kickoff、2026-05-10) で確定した決定事項を新規 stable doc として独立化:

項目 決定
(a) Config switch session-only opt-in (config commit せず session 内のみ enable、kill-switch 即可)
(b) Metrics lint_screen latency p50/p95 + fallback rate + Claude session input token 削減効果 (質的傾向)
(c) Kill-switch fallback rate > 50% で停止

成果物:

advisor 指摘で事前検証した結果、過去 dogfood 阻害要因 3 種 (findings ゼロ / review body 抽出漏れ / rate-limit) は classifier 専用で lint_screen は CR 非依存のため scope 外と確定。対応設計を不要化。

guide は history.md (50KB cap 到達済) ではなく独立 doc として stable 配置、引退条件は Phase d 完了 → §8.E 採否判定 → ADR-038 採用/却下時に削除。

Commit 3: 5 PRs roster を analysis.md §4 #2 に反映

docs/todo-summary.md から選定した dogfood 対象 PR roster を analysis.md に明記。別セッションで「次に何の PR を作るか」が doc レベルで決まっている状態に。

Order 構成 Effort Diff Profile dogfood signal
P-1 Bundle h (順位 89+90) + Bundle g-2 (順位 87+88) M global rules markdown 4 file docs-only で informational 期待、false-positive 検証 baseline
P-2 Bundle j-1 (順位 94 — ../docs/ 相対パス detect lint rule) S TOML config + 軽い Rust regex 小規模 mixed diff
P-3 Bundle g-1 (順位 85+86 — cli-pr-monitor verdict guard + transition test) M Rust impl + Rust test 中規模 Rust、auto_fix 期待
P-4 Bundle d (順位 68 — no-ephemeral-todo-reference self-exclusion test) S Rust test only 狭 scope test diff
P-5 Bundle c-1 (順位 63+64+67 — cli-merge-pipeline Drop guard + reaper + ADR) L Rust impl ×2 + ADR 大規模 Rust (PR #132 868 行 stress 再現候補)

設計判断のポイント

  • Effort 分布 M→S→M→S→L (size ramp-up): kill-switch (fallback > 50%) signal の質を切り分け可能 (小規模で発動 = 設計 issue / 大規模で発動 = num_ctx 再到達)
  • Bundle h + g-2 を 1 PR に統合: 共通テーマ「global rules consolidation (process/lifecycle codification)」、reviewer も「rule 追加 4 件まとめ」として認識しやすい
  • Bundle f 除外: (defer) 表記 = systemic 性未確認のため Phase d で push 圧力を加えない
  • 3 obstacles 対処設計を不要化: classifier 専用で lint_screen は CR 非依存のため scope 外確定 (advisor 指摘で事前検証)

Test plan

本 PR は docs only、機能変更なし。

  • markdownlint-cli2 clean (PostToolUse hook で各 edit 後に確認)
  • takt pre-push-review 1 iteration / APPROVE (4m 36s)
  • サイズチェック: todo6.md 32.6KB / todo-summary.md 36.8KB (50KB ガイドライン内)
  • 新規 phase-d-guide.md は ≈170 行 (advisor 推奨の ≤150 を概ね満足)

Out of scope

  • Phase d 実 dogfood の実行: PR-1 〜 P-5 の実装は本 PR land 後の後続 PR で実施 (long-running、数日〜数週間)
  • Bundle i 完了タスクの todo entries 削除 (順位 91/92/93): 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 外。Phase d kickoff PR or 別 cleanup PR で対処予定
  • 順位 98 (num_ctx overflow detection) の実装: todo 登録のみ、実装は将来 PR
  • 派生プロジェクト (techbook-ledger / auto-review-fix-vc) への deploy: Phase d 結果集約後の判定対象

Summary by CodeRabbit

リリースノート

  • ドキュメント
    • Phase d 運用ガイドの新規追加:lint_screen ステージのテスト手順、収集メトリクス、停止条件を明記
    • Phase d 計画の詳細化:実 dogfood の運用方針と PR 対象リストを追記
    • タスク追跡の更新:コンテキスト ウィンドウ診断ログ機能をロードマップに追加

Review Change Stack

aloekun added 3 commits May 10, 2026 06:41
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 ガイドライン内)。
…ysis.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 採用/却下時に削除。
…§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 に従う。
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 10, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 0157a873-137b-4482-ba65-ab9592fbba61

📥 Commits

Reviewing files that changed from the base of the PR and between 3968591 and 05c9d9e.

📒 Files selected for processing (4)
  • docs/local-llm-offload-analysis.md
  • docs/local-llm-offload-phase-d-guide.md
  • docs/todo-summary.md
  • docs/todo6.md

📝 Walkthrough

Walkthrough

このプルリクエストはPhase d(ADR-038に基づくlint_screen段階の試験的運用段階)の完全なドキュメント定義と実行計画を追加します。新しいガイド、状態更新、実dogfood手順、メトリクス定義、キルスイッチ基準、および後続のnum_ctx溢出検出タスク計画を含みます。

Changes

Phase D 運用ドキュメント統合

Layer / File(s) Summary
Phase D ガイド導入
docs/local-llm-offload-phase-d-guide.md
新しいドキュメントページを追加し、Phase dをADR-038試験的運用段階として定義、cli-push-runnerlint_screenスコープを指定、CR非依存性を確立。
セットアップとメトリクス計画
docs/local-llm-offload-phase-d-guide.md
セッション限定オプトイン手順を定義(一時的にenabled = trueを設定、コミット不要、実行後リバート)。3つのメトリクスを指定:lint_screenレイテンシ(p50/p95)、フォールバック率検知、トークン削減の定性評価(/cost比較およびマニュアル確認による)。
キルスイッチと完了ワークフロー
docs/local-llm-offload-phase-d-guide.md
フォールバック率>50%(3/5 PR)でキルスイッチ発動、設定リバート、フォールバック理由分類、履歴更新、次アクション決定(プロンプトv2改善またはモデル切替)を定義。Phase d完了時のメトリクス集約、ADR-038採択条件との比較、最終決定パス(採択/却下/続行)。
Phase d キックオフ状態と実dogfood詳細
docs/local-llm-offload-analysis.md
Phase d キックオフ prep完了(2026-05-10)と新ガイドへのリンクで状態行を更新。session-only opt-in、計測対象、キルスイッチ条件を明確化。実dogfoof PR roster(P-1〜P-5)を構成・Effort・Diff Profile・dogfood signalまで拡充、設計判断ポイント(Effort分布、Bundle統合、Bundle f除外理由)を記載。
タスク計画
docs/todo-summary.md, docs/todo6.md
優先度98の新規タスク「num_ctx溢出検出」を追加(Tier 1、工数M、依存なし)。lib-ollama-clientの JSON生成/検証層に、serde_json::from_str解析エラー時のwarn ログ出力方針、構造体フィールド追加、テスト観点、完了基準を定義。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed タイトルは3つのコミットで実施された変更内容を正確に反映しており、Phase d キックオフ準備完了、タスク順位 98 追加、dogfood ロスター反映という主要な変更を明確に示しています。
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ 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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@aloekun aloekun merged commit 83d2ff2 into master May 10, 2026
1 check passed
@aloekun aloekun deleted the docs/phase-d-kickoff branch May 10, 2026 06:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant