Skip to content

docs(todo): Bundle h (PR #123 post-merge) — 順位 89-90 採用 + todo6.md 新設#124

Merged
aloekun merged 1 commit intomasterfrom
docs/todo6-bundle-h
May 7, 2026
Merged

docs(todo): Bundle h (PR #123 post-merge) — 順位 89-90 採用 + todo6.md 新設#124
aloekun merged 1 commit intomasterfrom
docs/todo6-bundle-h

Conversation

@aloekun
Copy link
Copy Markdown
Owner

@aloekun aloekun commented May 7, 2026

Summary

  • PR feat(local-llm): P-0 classifier opt-in + §10 ブランチ分離運用 (ADR-038 Phase 5) #123 (ADR-038 Phase 5 P-0) merge 後の post-merge-feedback で 採用 verdict を受けた 2 タスクを docs/todo 系に登録 (Bundle h)
  • todo5.md が 50KB 閾値を超過したため、新規追加先として docs/todo6.md を新設
  • 順位 89 (Experimental Features 標準パターン codify) + 順位 90 (ephemeral lifecycle global rule) を docs/todo.md 推奨実行順序サマリーに追加 + Bundle h 解説段落

docs-only PR (実装コード変更なし、後続作業との混在を避けるため独立 PR)。

Changes

docs/todo6.md (新規)

  • 順位 89: Experimental feature 標準パターン codify — config opt-in + kill-switch + bounded lifetime の 3 点セット
    • 配置先 (case 1: CLAUDE.md / case 2: 別 ADR + リンク) は実装時に決定。memory feedback_claude_md_link_only.md との整合は case 2 推奨
  • 順位 90: グローバルルール: ephemeral 大規模コンテンツの ADR 昇格 + config コメント lifecycle
    • (a) ~/.claude/rules/common/docs-governance.md に「ephemeral 内 50 行超 governance content は ADR 昇格を検討」基準を追加
    • (b) ~/.claude/rules/common/coding-style.md § Cross-File Reference Lifecycle に「config コメント → ephemeral 計画書」 anti-pattern を追加

docs/todo.md

  • front matter 使い分けルール: 6 ファイル運用に更新 (todo5.md は編集専用に推移、todo6.md が新規追加先)
  • 推奨実行順序サマリー: 順位 89, 90 を表末尾に追加
  • Bundle h 解説段落: Bundle g 直前に挿入。採用 2 / 却下 4 / 様子見 4 件を inline 整理

Bundle h 採用判断の根拠

# タスク Effort Adoption Risk 採用根拠
順位 89 Experimental feature 標準パターン XS None 本 PR (#123) が kill-switch 経路を PR body に明記した模範例。ADR-031 / 036 / 038 で systemic 反復確認済み
順位 90 ephemeral lifecycle global rule (2 ファイル) S None PR #123 自身が §10 (約 200 行) を ephemeral 計画書に追加した自己違反 + config → ephemeral 参照の dual-violation を観測

不採用 verdict (参考)

詳細は .claude/feedback-reports/123.md (gitignore 対象内部 artifact) 参照。

Test plan

  • markdownlint clean (本 PR の全 Edit / Write で 0 errors)
  • pre-push-review APPROVE (アーティファクト疑義 2 件は実コンテンツで確認済、anomalies なし)
  • todo.md <a id="recommended-order-summary"></a> アンカーが todo6.md 末尾の参照と整合
  • テーブル行 89/90 が他行 (85-88) と同形式
  • Bundle h 段落が Bundle f/g 段落と同パターン
  • (後続セッション) 順位 89 + 90 を同 PR で land (Bundle h コア、両者 XS+S)

関連

Summary by CodeRabbit

リリースノート

  • ドキュメント
    • TODO管理ドキュメントの運用ルールを更新し、複数ファイル間の参照・管理指示を明確化
    • 新セッションでは全ファイルの確認が必要なことを明記
    • 実験的機能の標準パターン(設定オプション、キルスイッチ、有限期間)に関するタスク情報を追加
    • 短命コンテンツのADR昇格基準とコンフィグ参照ガイドラインに関する運用指示を追加

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 7, 2026

Review Change Stack
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: 5de27a9a-7797-4f78-adc9-24d83fbcd625

📥 Commits

Reviewing files that changed from the base of the PR and between 9a861dd and 5e3fd28.

📒 Files selected for processing (2)
  • docs/todo.md
  • docs/todo6.md

📝 Walkthrough

Walkthrough

PR #124 は、docs/todo.mddocs/todo6.md を更新して、ドキュメント分割管理ルールを 6 ファイル対応に拡張し、experimental feature 標準パターンと ephemeral 大規模コンテンツの ADR 昇格に関する 2 つの新たなタスク(Bundle h、順位 89-90)を追記しました。

Changes

ドキュメント管理ルール更新

Layer / File(s) Summary
運用ルール拡張
docs/todo.md
docs/todo2.mddocs/todo6.md の役割分担と新規タスク追加の条件を明記。セッション確認対象を「5つ」から「6つ」に更新。
実行中タスク定義
docs/todo6.md
Experimental feature 標準パターン(設定オプトイン、キルスイッチ、有限ライフタイム)の codify、および ephemeral→permanent ライフサイクル遷移ルール(ADR 昇格基準と config-comment 参照ガイドライン)を追加。
推奨実行順序への反映
docs/todo.md
Bundle h(PR #123)の 2 つの新タスク(順位 89-90)を推奨実行順序サマリーに追記。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • aloekun/claude-code-hook-test#89: 同一の docs/todo.md 運用ルールを変更し、セッション確認指示とタスクルーティング条件を更新。
  • aloekun/claude-code-hook-test#112: ephemeral コンテンツの ADR 昇格とconfig-comment ライフサイクルに関する同一の documentation governance ルールを扱う。
  • aloekun/claude-code-hook-test#111: Cross-File Reference Lifecycle と config-comment ライフサイクルの ephemeral 参照規制に関する同一のタスクを修正。
🚥 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 プルリクエストのタイトルは、Bundle h (PR #123 後続) に基づく順位 89-90 の採用と docs/todo6.md 新設という主要な変更を明確に反映しており、ファイル変更の中核内容を正確に要約している。
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 229356c into master May 7, 2026
1 check passed
@aloekun aloekun deleted the docs/todo6-bundle-h branch May 7, 2026 11:29
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