docs: Phase D dogfood + 順位 174-182 採用 (週次レビュー Phase D + 既存変更 bundle) - #186
Merged
Conversation
…+ todo10.md 新設 PR #185 post-merge-feedback 採用 (2026-05-29 ユーザー承認): - 順位 175 (T1-#1): cargo clippy を stop_quality ステップに追加 (Rust lint structural gap 補填) - 順位 176 (T2-#4): check-ci-coderabbit format extraction 関数への variant fixture 追加 user 追加要望 (2026-05-29): - 順位 177: PostToolUse hook — Edit/Write したファイルのサイズ閾値超過検出 + 分割促進 Phase D dogfood 採用 (2026-05-30 ユーザー承認、週次レビュー 2026-05-30 実行): - 順位 178 (週次レビュー S02): state.rs behavioral invariant test を ADR-041 pattern で追加 - 順位 179 (週次レビュー S03): rate-limit retry decision boundary test を rstest parameterized で追加 - 順位 180 (週次レビュー C01): lib-report-formatter に markdown pipe / newline escape を追加 (prompt injection 防御) - 順位 181 (Phase D dogfood D-A): aggregate-weekly facet の findings.json 出力を raw JSON にする (facet output bug fix) - 順位 182 (Phase D dogfood D-B): /weekly-review skill に重複検出 (簡易 grep + 3 択 AskUserQuestion) を Phase 4 で追加 Phase D dogfood 実行サマリー (handoff doc § 4 e2e 検証): - takt workflow weekly-review 起動 → 22m 52s 完走 → 12 findings 検出 (high 1 / medium 5 / low 6) - Phase C skill flow を実観測: - Phase 0 (pending file + failed marker 検出): clean state - Phase 1 (takt 起動): pre-state 書込 → background 実行 - Phase 2 (成果物 copy): .takt/runs/ → .claude/weekly-reviews/2026-05-30.md + pending JSON 構築 - Phase 3 (採否選択): user 承認経由で 5 件採用 (S02/S03/C01/D-A/D-B) - Phase 4 (todo 展開): 順位 178-182 として docs/todo10.md に追加 (順位 N system pattern 採用、ADR-031 「週次レビュー採用 (date)」section ではなく既存 priority table integration 重視) - Phase 5 (state file 更新): pending JSON consume + last-run timestamp 書込 - 観測した skill 改善点 (= 順位 181 / 182 として codify): - findings.json が フェンス wrap で出力 (facet 出力 bug、skill 内手動 strip workaround) - WR-2026-05-30-S05 が既存 順位 173 と完全重複 (重複検出 MVP 未実装の実証) 不採用 / 様子見 (analyzer 推奨を user 承認、2026-05-30): - 週次レビュー S01 / C02 / C03: 様子見 (large refactor / overflow practical mitigated / future-only risk) - 週次レビュー S04 / S05 / S06 / S07 / S08 / A01: 却下 (YAGNI / 重複 / 既存 comment で intent 明確 / serde idiomatic / docs trivial drift) todo10.md 状態: - 17KB → 34KB (5 entries 追加後)、50KB threshold まで余裕 - 既存 todo.md / todo2.md 〜 todo9.md は引き続き有効、相互独立 メタ評価: 本セッション (#181 → #182 → #183 → #184 → #185) 5 PR chain + Phase D dogfood で計 28 件の todo 採用 (post-merge-feedback 22 件 + user 主導 1 件 + 週次レビュー dogfood 5 件)、Phase B 設計の self-reinforcing detection loop が **full session 単位での dogfood cycle complete** を実証。本 commit は Phase D 完了 + Phase E (試験運用 1-2 週 dogfood + ADR-031 本採用化 + handoff retire) に carry-forward する状態。
|
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 (11)
📝 WalkthroughWalkthroughこのPRは、2026-05-30のWeekly Review実行結果を記録し、肥大化したtodo9を分割してtodo10を新規作成し、既存のtodo3-9の参照ルールを統一更新しています。 ChangesWeekly Review実行記録とtodo10分割追加
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 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 |
Merged
4 tasks
6 tasks
5 tasks
aloekun
added a commit
that referenced
this pull request
Jun 6, 2026
本 PR は以下 4 つの作業を 1 コミットに統合: ## A. PR #196 post-merge-feedback 採用 2 件 (順位 193, 194 登録) PR #196 (Bundle 195-FB) post-merge-feedback 8 件のうち 2 件採用、6 件却下/様子見。 採用 (todo10.md に entry 追加、todo-summary.md table に行追加): - 順位 193 (T2): Companion helper group 署名整合 compile-time validation test - 順位 194 (T3): development-workflow.md \"1. Plan First\" に Codification 重複確認 step 追記 却下/様子見の詳細は .claude/feedback-reports/196.md 参照。 ## B. queue 棚卸し (順位 ≤ 100 の 32 件を audit) 8 件削除 + 7 件改訂で queue の signal/noise 改善: 削除 8 件 (既存 land 確認、または動機失効): - 順位 41 (Bundle Y2 効果定量計測): 動機の主軸失効 (Bundle Z 完成 + Z2 不採用) - 順位 42, 43, 46 (rate-limit auto-retry 系): PR #97/#113/#129/#185 段階 land 完了 - 順位 45 (--list-findings Rust モード): PR #101 で land 済 - 順位 57 (Aggregation cap integration test): PR #171 で land 済 - 順位 93 (coding-style.md partial fix 例追加): ~/.claude/rules/common/ coding-style.md に既に section 存在 - 順位 97 (with_num_ctx serialization test): lib.rs:494 で mockito test 実体存在 改訂 7 件 (Status update 2026-06-06 を front-matter に追加、現状反映): - 順位 11: ADR-018 park / ADR-030 短命プロセス移行後の再現確認が前段 - 順位 19: ADR-037/043/PR #194 land 後の残余 case baseline 観測が前段 - 順位 27: Phase D-7 = PR #154 land 済を反映 - 順位 38: ADR-031 採用昇格済 (PR #192) → Bundle W/X land のみ残依存 - 順位 40: PR #175 push-runner bookmark_check 実装済 → skill 側は二重防御に縮小 - 順位 51: 採用案 C (fix.md instruction 追加) は land 済、残作業 = dogfood 観測のみ - 順位 92: ADR-038 採用昇格済 (PR #156) で Phase d 運用入り、動機書き換え cross-reference 修復: - 順位 49: 旧依存 Bundle a Sub-PR 2 (順位 42/43/46) 消滅を反映 - 順位 61: 旧依存 順位 45 land 済を反映 ## C. todo9.md → todo11.md 分割 todo9.md が 75KB / 890 行に到達し読み取り安定性閾値 (50KB) 超過のため分割: - todo9.md (37KB / 454 行): 既存ルール仕組み化バンドル (順位 146-151) + 週次 レビュー拡張 (順位 152-154) を保持 - todo11.md (41KB / 453 行、新規): PR-specific follow-up entries 10 件 (順位 157, 160, 161, 162, 163, 165, 170, 171, 172, 173) theme-based split で意味的分離 + 両ファイルとも 50KB 閾値以下に収まる。 todo-summary.md table の file 参照を Python script で一括更新 (10 件)、 todo-summary.md 冒頭の \"追加先ファイル\" 説明を todo10.md に更新。 ## D. 順位 177 優先度引上げ PostToolUse hook ファイルサイズ検出 task が 4 回目の同型観測に到達 (PR #133 + #172 + #186 + 本セッション = Very High frequency)。CLAUDE.md code-review.md \"同型 finding の閾値判定\" (3 観測 = Tier 1 昇格) を超え systemic risk 閾値到達。 3 箇所同期更新: - todo10.md entry 本体に Status update 2026-06-06 blockquote 追加、優先度を \"Tier 1\" → \"Tier 1 (優先実装)\" に格上げ - todo-summary.md table 行で Tier 列を太字 + 注記、dependency 列に urgency note - todo-summary.md 末尾の戦略 note に \"直近優先 (2026-06-06 ユーザー指示)\" 段落を新設、Bundle 195-FB-Followup (順位 193 + 194) の次の PR で消化推奨と specific call-out ## 統計 - 10 ファイル変更 (1 新規) - ~670 lines insertions, ~910 lines deletions (net ~240 行削減) - 全 todo*.md が 50KB 閾値以下に収まる (todo9: 37KB, todo10: 36KB, todo11: 41KB) ## 参照 - .claude/feedback-reports/196.md (採否確定 commit、PR #196 由来) - memory feedback_post_merge_feedback_adoption_requires_user_approval per ユーザー承認済 - memory feedback_todo_no_history (削除は痕跡なし、コメントマーカー不使用) - ADR-035 (docs-only PR 評価ポリシー) - ADR-033 (採番管理簡素化、順位 renumber は避けて semantic markers で表現)
6 tasks
aloekun
added a commit
that referenced
this pull request
Jun 6, 2026
…順位 177) (#198) PostToolUse Edit / Write 直後にファイルサイズ閾値超過を検出して分割を促す mechanical layer を実装。ファイル分割を user 判断ベース → mechanical layer に移管し、認知負荷削減 + 早期検出 を実現する。 ## 背景 4 PR の同型観測 (Very High frequency) で systemic risk 閾値到達: - PR #133: todo.md → todo2.md split - PR #172: todo8.md → todo9.md split - PR #186: todo9.md → todo10.md split - PR #197: todo9.md → todo11.md split (本日 land) 順位 177 として todo10.md に登録済み、PR #197 で Tier 1 (優先実装) に格上げ。 ユーザー指示 (2026-06-07) で次の PR で消化と決定。 ## 設計 配置: option B = 既存 hooks-post-tool-linter (generic linter) に Layer 0.5 として統合。 option A (新 binary) に対する優位性: - 新 binary 追加なし → deploy 簡素 (Effort S 寄り) - PostToolUse hook 数の最小化 (1 PR push で発火する hook 数を抑制) - ADR-039 bounded lifetime dogfood サイクルを最短で開始可能 ## 実装 ### Layer 0.5 配置 main() を 4 layer helper に refactor (関数長 50 行制限内に収まる構造): - Layer 0: UTF-8 整合性 (既存) - Layer 0.5: file_size_check (新規、metadata-only) ★ - Layer 1: 正規表現 custom-rules (既存) - Layer 2: 外部ツール pipeline (既存) ### Config schema [post_tool_use.file_size_check] section (ADR-039 opt-in pattern 準拠): - enabled = false (default OFF) - threshold_bytes = 51200 (50KB、Claude Code 読み取り安定性閾値) - paths = ["docs/**/*.md", "src/**/*.rs"] (default 対象 glob) - touch_trigger = true (ratchet、既存超過は触られるまで grandfather) Kill-switch: enabled = false で完全停止。 ### Pure function design check_file_size_threshold(file, size_bytes, config) は I/O フリーの純粋関数として実装、 file system access は呼び出し側 (run_file_size_layer) に分離。tests が deterministic に 書け、memory feedback_test_dry_antipattern に従い各 variant 独立 setup が trivial。 ### touch_trigger ratchet の MVP 制限 touch_trigger = false (strict mode = 全 enabled paths を毎回スキャン) は ADR-039 bounded lifetime dogfood 後の判定に倒し、MVP では config field 受理のみ (true/false 同挙動)。 3-5 PR の dogfood 観測後に default-ON 昇格 or strict mode 実装 or 却下を判定。 ## ADR-007 amendment (Layer 0.5 追記) ADR-007 § Layer 0.5 追記 section を新設し、file_size_check が Q1/Q2/Q3 判断フロー対象外 であることを codify。同型 (metadata-only、content 非依存) の future check は同 Layer 0.5 に追加することで regex / AST 層との責務境界を維持する原則を明文化。 ## Tests 8 variant 独立 setup (memory feedback_test_dry_antipattern 適用): 1. enabled=false で短絡 (kill-switch 動作確認) 2. paths glob 不一致で skip 3. size 閾値内で skip 4. size 超過で feedback emit 5. docs/todo* に対する recovery hint 文脈別出し分け 6. paths が空配列の場合 skip 7. touch_trigger=false が MVP では true と同挙動 8. Windows backslash path の forward-slash 正規化 全 8 test pass、既存 141 test との non-regression 確認済 (cargo test 149/149 passed)。 cargo clippy --workspace -- -D warnings clean。 ## Dogfood threshold=30720 で hooks-config.toml を temporarily enable、docs/todo10.md (38155 bytes) に hook 発火を実観測: [file-size-check] docs/todo10.md: ファイルサイズ 38155 bytes が threshold 30720 bytes (= 30.0 KB) を超過しています。ファイル分割を推奨します (docs/todo*.md の場合は新 todo<N+1>.md を新設して entry を移管). 小ファイル (docs/todo5.md 11909 bytes) は no-op を確認。dogfood 後 enabled=false / threshold=51200 に revert 済 (production default = opt-in). ## 関連 docs 更新 - docs/todo10.md: 順位 177 entry 削除 (実装完了) - docs/todo-summary.md: 順位 177 table 行 + 「直近優先 2026-06-07」call-out 段落 削除 - docs/adr/adr-007-custom-linter-layer-boundary.md: § Layer 0.5 追記 新設 ## 参照 - memory feedback_pipeline_over_rules: rule → hook 切替の体系適用 - ADR-039: experimental feature 標準パターン (opt-in + kill-switch + bounded lifetime) - ADR-007: custom-linter layer boundary (本 PR で Layer 0.5 追記) - 順位 102 (PR #148 land): paths glob filter helper を再利用
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
ADR-031 Phase D dogfood + Phase D 観測点を todo に register する docs/config PR。本 PR で実
/weekly-reviewskill を呼出 → takt workflow を走らせ → findings 採否 → docs/todo10.md 反映までの e2e flow を実観測 (handoff doc § 4)。加えて本セッション (#181 → #182 → #183 → #184 → #185 の 5 PR chain) で蓄積した:
を bundle (ユーザー判断 2026-05-29「既存変更同 PR 許容、commit のみ分離」+ 2026-05-30「Phase D 結果も同 PR」)。さらに新規
todo10.md追加に伴う既存todo3-9.mdpreamble の数詞 update (cli-docs-lint detection 経由) も含む。含まれる変更
1. todo entries 9 件追加 (順位 174-182)
cargo clippyを stop_quality に追加 (Rust lint structural gap 補填)state.rsの behavioral invariant test を ADR-041 pattern で追加lib-report-formatterに markdown pipe / newline escape を追加 (prompt injection 防御)aggregate-weeklyfacet のfindings.json出力を raw JSON にする (facet output bug fix)/weekly-reviewskill に重複検出 (簡易 grep + 3 択 AskUserQuestion) を Phase 4 で追加2. todo10.md 新設 + todo3-9.md preamble 数詞 update
docs/todo10.md作成 (todo9.md が 100KB 到達のため split、ユーザー判断 2026-05-29)docs/todo3.md〜docs/todo9.mdの preamble 「十つすべてを確認すること (todo.md / todo2-9.md / todo-summary.md)」を「十一つすべてを確認すること (todo.md / todo2-10.md / todo-summary.md)」に更新3. todo-summary.md priority table
Phase D dogfood 実行サマリー (handoff doc § 4 = e2e 検証)
skill
/weekly-reviewを本セッションで実呼出し、Phase C 実装の全 phase を実観測:pnpm exec takt -w weekly-review -t "weekly-review 2026-05-30"→ background 実行、pre-state JSON 書込.takt/runs/20260529-150611-weekly-review-2026-05-30/reports/→.claude/weekly-reviews/2026-05-30.md+ pending JSON 構築docs/todo10.mdに展開 (本セッション pattern = 順位 N system、ADR-031 「週次レビュー採用 (date)」section は採用せず priority table 連携を優先).claude/weekly-review-last-run.json書込takt workflow 実行
観測した skill 改善点 (順位 181 / 182 に codify)
findings.jsonが```json ... ```フェンス wrap で出力 = aggregate-weekly facet 出力 bug、skill 内で手動 strip workaroundcombine_outputdead-code) が 既存 順位 173 (PR #182 S01) と完全重複 = ADR-031 § Phase 4「重複検出は MVP では実装しない」の実証ユーザー判断記録 (2026-05-29 / 2026-05-30)
feedback_no_unenforced_rules延長線上の原則)メタ評価
本セッション (#181 → #182 → #183 → #184 → #185 → 本 PR) で 計 28 件の todo 採用 (post-merge-feedback 22 件 + user 主導 1 件 + 週次レビュー dogfood 5 件)。Phase B 設計の self-reinforcing detection loop が full session 単位での dogfood cycle complete を実証:
次セッションでは Phase E (= 試験運用 1-2 週 dogfood + ADR-031 ステータス本採用化 + handoff doc retire) になります。
Pre-push 検証結果
十つ数詞不整合を 7 file で検出) → 7 file 一括 update → 再 push で PASSTest plan
pnpm push全 quality_gate + pre-push-review PASS/weekly-reviewskill の全 Phase (0-5) が e2e 実観測される🤖 Generated with Claude Code
Summary by CodeRabbit
リリースノート
New Features
Documentation
Chores