Skip to content

feat(adr+rules): Bundle h+g-2 — ADR-039 + global rules 4 件 codify (P-1 dogfood with caveat)#139

Merged
aloekun merged 2 commits intomasterfrom
feature/bundle-h-g2-adr-039
May 10, 2026
Merged

feat(adr+rules): Bundle h+g-2 — ADR-039 + global rules 4 件 codify (P-1 dogfood with caveat)#139
aloekun merged 2 commits intomasterfrom
feature/bundle-h-g2-adr-039

Conversation

@aloekun
Copy link
Copy Markdown
Owner

@aloekun aloekun commented May 10, 2026 •
edited by coderabbitai Bot
Loading

Summary

Bundle h (順位 89+90) と Bundle g-2 (順位 87+88) を 1 PR で land。Phase d の「dogfood 対象 5 PRs roster」P-1 に該当 (analysis.md §4 #2)。

  • ADR-039 新設 (順位 89): 試験運用 feature の 標準パターン 3 点セット (config opt-in / kill-switch / bounded lifetime) を独立 ADR に codify。既存試験運用 ADR (031/036/038) に back-link を追加 (残り 8 ADR は後続 PR で追補)
  • Global rules に 4 件追記 (順位 87/88/90): ~/.claude/rules/common/* 4 ファイルを更新 (本 PR diff 外、PR body §「Global rule 編集の enumeration」で詳細列挙)
  • 完了 todo 削除: 順位 87/88 (todo5.md) + 順位 89/90 (todo6.md) + 4 行 (todo-summary.md) を削除、Bundle 解説の land 状況を反映

変更内容

Commit 1: 順位 99 (PR #138 T3-#1+#2 採用) を Bundle i 系列に追加 (前 PR からの carry-over)

PR #138 merge 直前に登録した feedback-derived タスク。ADR-038 に PR #138 の learning 2 件 (cost-aware 実装層選択 + attention dilution pitfall) を追記する Tier 3。本 PR とは独立な commit として保持。

Commit 2: ADR-039 新設 + 既存 ADR cross-link + global rule 4 件 codify + todo cleanup

ADR-039 (新規、102 行)

docs/adr/adr-039-experimental-feature-standard-pattern.md:

  • lineage table: 既存試験運用 ADR 11 件の網羅列挙 (014/023/025/029/030/031/033/034/036/037/038)
  • 共通パターン table: ADR-031 / ADR-036 / ADR-038 の 3 点セット適合状況分析
  • 決定: 3 点セット (config opt-in / kill-switch / bounded lifetime) の運用規定
  • 帰結 / 想定運用: 新規試験運用 ADR 策定時の参照点 (status 記述テンプレ + kill-switch table 必須化)

ADR-039 自体も試験運用扱い (3-5 個の新規試験運用 ADR で適用後に本採用判定)。

Cross-link 追加 (031 / 036 / 038)

各 ADR のステータス直下に blockquote を追加: > 本 ADR の運用パターンは ADR-039 の 3 点セットの対象。本採用判定または却下時に ADR-039 の retirement workflow に従う。

残り 8 ADR (014/023/025/029/030/033/034/037) は本 PR では遡及更新せず、後続 PR で追補。ADR-039 lineage table の冒頭注記でこの非対称性を明示。

Global rule 編集の enumeration (本 PR diff 外、reviewer は手動確認推奨)

~/.claude/rules/common/* への編集は project worktree 外のため CodeRabbit / git diff には現れない。以下が今回追記された section の一覧:

File Section (新規 / 拡張) 概要 順位
git-workflow.md § Multi-PR chaining ベストプラクティス (新規、Pull Request Workflow 配下) 論理ユニット (init/integrate/organize) + diff 250-800 lines/PR を PR #119/#120/#121 dogfood 知見として codify 87
development-workflow.md § Edge case 観測頻度による優先度判定 (新規、巻末追加) post-merge-feedback の Frequency table と「3 観測 = Tier 1 昇格」を明文化 (Bundle f #80 事例引用) 88
docs-governance.md § Ephemeral 大規模コンテンツの ADR 昇格基準 (新規、§References 直前に追加) 50 行超 + 永続性 → ADR 昇格検討の判断基準を retirement workflow Step 1 と接続 90 (a)
coding-style.md § Concrete anti-pattern examples の TOML コメント節を拡張 + 由来補足を追加 *-analysis.md 等 ephemeral 計画書を config から参照する anti-pattern (PR #123 pr-monitor-config.toml 事例) 90 (b)

reviewer が verify したい場合: 各 file の git log (project 外、ローカル ~/.claude/) を参照 / または codification 完了確認として ADR-039 + 本 PR body のクロスリファレンスで担保。

完了 todo cleanup

  • docs/todo5.md: 順位 87 (Multi-PR chaining) + 順位 88 (Frequency 3 = Tier 1) section 削除 (-45 行)
  • docs/todo6.md: 順位 89 (Experimental feature 標準パターン) + 順位 90 (Ephemeral lifecycle) section 削除 (-89 行)
  • docs/todo-summary.md: 順位 87/88/89/90 行を priority table から削除 + Bundle h paragraph を ✅ 完了化 + Bundle g paragraph で g-2 land 済明記

順位 99 の table 行は preserve (PR #138 carry-over commit で参照するため)。

Phase d dogfood signal (P-1 = 本 PR、classifier preview のみ)

重要 caveat: 本 PR は Phase d guide §1 「session-only opt-in (config commit せず)」方針に従い、push-runner-config.toml の [lint_screen] enabled = true を commit に乗せていない。そのため pnpm push 経由の lint_screen pipeline は 走っていない (= real P-1 dogfood metric は未取得)。

代替として cli-finding-classifier.exe を直接呼び出した classifier preview を取得:

Metric Value 解釈
Latency 23s eval baseline (p95=8.4s) の ~3x、337-line diff のサイズ起因と推定
Findings 0 空配列
screen_decision human_review (fallback) mistral:7b 出力崩壊 → fallback 経路 activated
fallback_reason JSON parse error: missing field 'screen_decision' num_ctx=8192 でも 337-line diff で出力 truncate の可能性

→ 順位 98 (num_ctx overflow detection — diagnostic warn log) の 必要性を再確認 する signal。本 PR では実装せず、Phase d 進行中に該当 PR で対応予定。

real pipeline 経由の P-1 metric は P-2 移行時に再検討。analysis.md §4 #2 の P-1 行に caveat を明記済。

Test plan

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

  • markdownlint-cli2 clean (PostToolUse hook で各 edit 後に確認)
  • takt pre-push-review: 2 iter / 23m 5s / convergence_verdict: fully_resolved
  • サイズチェック: todo5.md 22KB / todo6.md 30KB / todo-summary.md 35KB (50KB ガイドライン内)
  • ADR-039 の lineage table 非対称性を冒頭注記で明示 (advisor 指摘反映)
  • global rule edits の enumeration を PR body に明記 (advisor 指摘反映)
  • P-1 = classifier preview only / real pipeline 未実行を明示 (advisor 指摘反映)

Out of scope

  • 残り 8 試験運用 ADR (014/023/025/029/030/033/034/037) の back-link 追加: 後続 PR で追補
  • 順位 98 実装 (num_ctx overflow detection): classifier preview で再確認した重要性は本 PR でも記述したが、実装は将来 PR
  • real Phase d P-1 pipeline run: P-2 移行時に再検討

Summary by CodeRabbit

  • Documentation
    • 実験的機能の運用パターンを標準化し、新しいドキュメント規格を定義しました。
    • 既存の複数のドキュメントを更新し、新しい標準パターンへの準拠を明記しました。
    • 運用ステータスと関連する判定フローに関する説明を追加しました。

Review Change Stack

aloekun added 2 commits May 10, 2026 16:21
@aloekun
docs(todo): 順位 99 (PR #138 T3-#1+#2 採用) を Bundle i 系列に追加
b59fede 
PR #138 (Phase d kickoff prep) post-merge-feedback で ✅ 採用された Tier 3 #1 + #2
を 1 task に bundle して todo に登録。両者とも ADR-038 への追記で 1 ファイル編集、
land 単位として bundle 推奨。

- Tier 3 #1: cost-aware 実装層選択の判断基準 (lint_screen が takt facet → Rust stage
  に pivot した cost 根拠) を ADR-038 §architecture decision に codify
- Tier 3 #2: attention dilution pitfall (Phase b' v2 で diff header full 追加 →
  agreement 75% → 50% 33pt 低下) を ADR-038 §prompt engineering に codify

不採用 (Tier 3 #4): coding-style.md §Markdown に「重複表現 grep チェック手順」追加は
ユーザー判断で見送り。重複ワードのバリエーションが多すぎて grep pattern 列挙では網羅
できないため、`feedback_no_unenforced_rules.md` 方針 (機械検知不可なルールは追加
しない) と整合的に却下相当。

- docs/todo-summary.md: 順位 99 行を table に追加 (Tier 3 / S / 依存なし)
- docs/todo6.md: 詳細エントリ (動機 / 設計決定案 / 作業計画 / 完了基準 / 不採用理由)

サイズ: todo6.md 36.9KB / todo-summary.md 37.2KB (いずれも 50KB ガイドライン内)。
@aloekun
feat(adr+rules): Bundle h + g-2 — ADR-039 (試験運用標準パターン) + global rules…
4e26979 
… 4 件 codify (順位 87+88+89+90)
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 10, 2026 •
edited
Loading

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: 2acd9931-0b6c-4caa-99b7-396cf74d4f9d

📥 Commits

Reviewing files that changed from the base of the PR and between 83d2ff2 and 4e26979.

📒 Files selected for processing (9)
  • CLAUDE.md
  • docs/adr/adr-031-weekly-review-pipeline.md
  • docs/adr/adr-036-bundle-z-three-layer-review.md
  • docs/adr/adr-038-local-llm-finding-classification.md
  • docs/adr/adr-039-experimental-feature-standard-pattern.md
  • docs/local-llm-offload-analysis.md
  • docs/todo-summary.md
  • docs/todo5.md
  • docs/todo6.md
💤 Files with no reviewable changes (1)
  • docs/todo5.md
📝 Walkthrough

Walkthrough

本PRは、実験的機能の運用パターンを統一するADR-039を新規作成し、既存の試験運用ADR（031、036、038）をこの標準に参照させるとともに、タスク管理ドキュメントを整理する変更です。

Changes

ADR-039 標準パターンの導入と既存ADRの整合

Layer / File(s) Summary
ADR-039 標準パターン定義
docs/adr/adr-039-experimental-feature-standard-pattern.md		 実験的機能に必須の「3点セット」（config opt-in/デフォルトOFF、kill-switch/明示的なリバートパス、bounded lifetime/試験期限・採用/却下/延長ワークフロー）を形式的に定義。運用前提、関連ADR、governance参照を記述。
ADR-039 索引登録
CLAUDE.md		 アーキテクチャ決定リストにADR-039エントリを追加。
既存試験運用ADRの標準参照化
docs/adr/adr-031-weekly-review-pipeline.md
docs/adr/adr-036-bundle-z-three-layer-review.md
docs/adr/adr-038-local-llm-finding-classification.md		 ADR-031、036、038それぞれに対して、ADR-039の「3点セット」パターンへの明示的な参照と、採用/却下時にADR-039の retirement workflow に従うことを追記。
運用・dogfood ドキュメント更新
docs/local-llm-offload-analysis.md		 Phase d P-1の dogfood 対象PR roster における実運用方針（config switch を commit しない、cli-finding-classifier 直叩き、num_ctx overflow detection 確認）を追記。
タスク管理の整理
docs/todo-summary.md
docs/todo5.md
docs/todo6.md		 todo-summary.md では順位87〜90削除・順位99追加、Bundle h/g 説明更新。todo5.md ではグローバルルール関連エントリ2件削除。todo6.md では Tier 3エントリを削除・再構成し、PR #138 learning 追記エントリを追加。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • aloekun/claude-code-hook-test#124: 本PR と同じ「実験的機能の3点セット」パターン codify を扱う—本PRはADR-039を追加して既存ADRに参照させる一方、当該PRはその todo6 タスク定義を追加したため関連。
  • aloekun/claude-code-hook-test#138: 本PRと同じ docs/local-llm-offload-analysis.md を変更し、同一の「実験的機能」運用パターン（config opt-in / kill-switch / bounded lifetime）に整合するため関連。
  • aloekun/claude-code-hook-test#123: 本PRが codify するADR-039の「config opt-in / kill-switch / bounded lifetime」パターンを、local-LLM classifier に実装する（pr-monitor-config opt-in + kill-switch/docs追加）ため関連。
🚥 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 プルリクエストのタイトルは、ADR-039の追加とグローバルルール4件の成文化を明確に示しており、Bundle h+g-2のランディングという主要な変更内容を正確に要約している。
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.

Tip

💬 Introducing Slack Agent: The best way for teams to turn conversations into code.

Slack Agent is built on CodeRabbit's deep understanding of your code, so your team can collaborate across the entire SDLC without losing context.

  • Generate code and open pull requests
  • Plan features and break down work
  • Investigate incidents and troubleshoot customer tickets together
  • Automate recurring tasks and respond to alerts with triggers
  • Summarize progress and report instantly

Built for teams:

  • Shared memory across your entire org—no repeating context
  • Per-thread sandboxes to safely plan and execute work
  • Governance built-in—scoped access, auditability, and budget controls

One agent for your entire SDLC. Right inside Slack.

👉 Get started

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 d5a3af2 into master May 10, 2026
1 check passed
@aloekun aloekun deleted the feature/bundle-h-g2-adr-039 branch May 10, 2026 09:36
@aloekun aloekun mentioned this pull request May 10, 2026
Merged
6 tasks
aloekun added a commit that referenced this pull request May 10, 2026
@aloekun
feat(lint): rule⑧ no-docs-relative-back-to-docs (Phase d P-2 / Bundle…
b6b61dd 
… j-1 / 順位 94) (#140)

* docs: 順位 100 (PR #139 T3-#1 採用) 登録 + analysis.md P-1 完了反映 (classifier preview metrics + post-merge-feedback 結果)

* feat(lint): rule⑧ no-docs-relative-back-to-docs — `docs/**/*.md` 内 `](../docs/` 検出 (順位 94 / Bundle j-1)
@aloekun aloekun mentioned this pull request May 10, 2026
Open
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@aloekun