feat(rules): Bundle e (convention long-tail) — 7 件 global rules 集約 (順位 23/24/25/26/30/33/70)#111
Merged
Merged
Conversation
…位 23/24/25/26/30/33/70) ## Bundle e 実装 global rules (~/.claude/) は project repo 外のため diff に現れないが、本 PR 完了で全 project に即時適用される: - 順位 23 (XS): coding-style.md Markdown 節新設 — 日付入り見出しアンカーは安定 ASCII identifier 優先、grep -rn でクロスファイル参照確認 - 順位 24 (XS): git-workflow.md jj Operations 拡張 — jj conflict リカバリ手順 (規模別の使い分け、大規模は jj abandon + jj new master + 再 edit が高速) - 順位 25 (XS): CLAUDE.md Personal Preferences > Code Style 拡張 — __ prefix scratch 規約 + jj auto-snapshot 特性 (PR #85 事例由来) - 順位 26 (XS): development-workflow.md 「背景タスクの待機方針」節新設 — polling 禁止 (token 浪費防止、ADR-018 補強) - 順位 30 (XS): coding-style.md Cross-File Reference Lifecycle に具体例追記 — Rust raw string / TOML / JSONC の 3 種類 anti-pattern + raw string 編集時補助 - 順位 33 (XS): code-review.md Multi-point synchronization 節新設 — 新 verdict 経路追加時の 3 点同期チェック (PR #95 由来) - 順位 70 (XS): code-review.md 同節内に追記 — 設計 doc / 実装の同期チェック (PR #110 OBS-2 由来、commit message に乖離理由を明記) ## 副次変更 (project repo 内) - docs/todo2.md: 順位 23/24/25/26 詳細エントリ削除 - docs/todo3.md: 順位 30/33 詳細エントリ削除 - docs/todo5.md: 順位 70 詳細エントリ削除 - docs/todo.md: priority table から 7 行削除、Bundle U/V/d/e 脚注更新 (Bundle U/V は完全消化) - docs/docs-pr-iteration-efficiency.md: 📋 LOW IMPACT + 後回し可セクションを Bundle "e" ✅ 完了に集約 ## 同梱 (Bundle d 登録、PR #110 post-merge-feedback 採用案、前ターンで registered) - docs/todo5.md: 順位 68/69 詳細エントリ追加 (順位 70 は本 PR で land のため別途削除) - docs/todo.md: priority table に 順位 68/69/70 追加済み (70 は本 PR で削除に転じた) - 残る 順位 68 (test infra) と 順位 69 (config コメント) は scope 異なるため独立 PR 推奨 ## Bundle U/V/d 終了状態 - Bundle U (順位 29 = PR #110、順位 30 = 本 PR) 完全消化 - Bundle V (順位 31/32 = PR #109、順位 33 = 本 PR) 完全消化 - Bundle d (順位 70 = 本 PR、順位 68/69 残存) 部分消化 ## 期待効果 (検証は dogfood) - AI / 人間が新セッションで convention に自然に follow する (rule 自動参照) - PR #110 で導入した no-ephemeral-todo-reference lint rule (machine 層) と本 PR の Cross-File Reference Lifecycle 具体例 (guideline 層) で二層防御 - spec-impl drift パターンが code-review.md checklist で意識化される
|
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 (1)
🚧 Files skipped from review as they are similar to previous changes (1)
📝 WalkthroughWalkthrough複数のドキュメントファイルに対する包括的なアップデートで、完了したバンドル（U、V、e）と対応するタスク行を削除し、Bundle e の完了状態を明記し、PR Changesプロジェクト追跡ドキュメントの更新
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 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. Review rate limit: 0/1 reviews remaining, refill in 60 minutes.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 the current code and only fix it if needed.
Inline comments:
In `@docs/docs-pr-iteration-efficiency.md`:
- Line 49: Fix the mismatch between docs/docs-pr-iteration-efficiency.md and
~/.claude/CLAUDE.md: either update docs/docs-pr-iteration-efficiency.md (remove
or correct the claim on Line 49 and Line 92 that "Bundle e item 25" was codified
under "Personal Preferences > Code Style") or add the missing "Personal
Preferences > Code Style" section to ~/.claude/CLAUDE.md and include the
codified item 25 there; ensure the doc text and the actual CLAUDE.md content are
consistent and that references to "Bundle e (item 25)" appear only where the
target section actually exists.
🪄 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: 86e3dc92-271f-4c1a-8152-40d759e0d600
📒 Files selected for processing (5)
docs/docs-pr-iteration-efficiency.mddocs/todo.mddocs/todo2.mddocs/todo3.mddocs/todo5.md
💤 Files with no reviewable changes (2)
- docs/todo3.md
- docs/todo2.md
Resolved findings: - [Critical] docs/docs-pr-iteration-efficiency.md:49 docs/docs-pr-iteration-efficiency.md の CLAUDE.md へのcodify 内容について記述と実ファイルの齟齬を修正してください
Owner
Author
|
@coderabbitai review |
✅ Actions performedReview triggered.
|
Merged
6 tasks
aloekun
added a commit
that referenced
this pull request
May 5, 2026
…ion-efficiency.md retire (PR #111 post-merge-feedback) (#112) ## Bundle f (PR #111 post-merge-feedback、4 件採用) + 順位 62 (Document Governance) + 計画書 retirement の 1 PR 集約 land ### 順位 62 (Document Governance、Effort XS、新規 global rule) - 新規 ~/.claude/rules/common/docs-governance.md を作成 - 3 ルール codify: 1. Document Lifecycle Classification (permanent / ephemeral) 2. Cross-File Reference Lifecycle (再掲、coding-style.md と cross-link) 3. Retirement 4-step workflow (migrate value → migrate tasks → update refs → delete file) - 本 PR 自身が retirement workflow の dogfood (docs/docs-pr-iteration-efficiency.md retire = 第一実例) ### 順位 71 (codification claims 検証手順、Effort XS) - ~/.claude/rules/common/coding-style.md の Cross-File Reference Lifecycle に新 sub-section ### Codification claims の検証手順 追加 - 「file X の section Y に codified」と書く前に grep で実在確認、階層表現 > は ambiguous なため避ける - PR #111 自己違反事例を実例引用 (Personal Preferences > Code Style 拡張 → CodeRabbit Critical 指摘) ### 順位 72 + 73 (ADR-033 拡張、Effort XS×2、同一新セクション内) - docs/adr/adr-033-todo-numbering-simplification.md に新セクション ## priority table 運用ルール 追加 - 順位 72: 「依存」列の二層追跡 (technical 依存 vs strategic 順序)、順位 10 broken-link を実例引用 - 順位 73: Bundle 分割時の land 期限明記ルール、Bundle U/V/d を過去事例として引用 - 本 PR 自身が順位 73 の dogfood (Bundle f 分割推奨を統合実行した判断記録は commit message + PR body に残す) ### 順位 74 (ADR-030 拡張、Effort XS) - docs/adr/adr-030-deterministic-post-merge-feedback.md の 入力源 / 出力 直後に新セクション ### Global rules editing の追跡 追加 - global rules (~/.claude/rules/, ~/.claude/CLAUDE.md, ~/.claude/skills/) の編集は VCS 管理外、transcript 経由で post-merge-feedback が捕捉 - PR #111 を実例引用 (5 ファイル global edit + project diff 0 行 + post-merge-feedback で全件 visible) - pre-merge 補完層として coding-style.md (順位 71) と docs-governance.md (順位 62) を cross-link ### Retirement (docs/docs-pr-iteration-efficiency.md 削除) - bundle 全消化 (Bundle docs-PR-streamline / docs-quality-pre-write / e で 12 件 land、残 順位 10 は priority table で継続追跡) で本ファイルが自己宣言した end condition 成立 - ADR-035 + global rules + lint rule + priority table 脚注で永続価値が全て分散保存済 → 新規 ADR 作成は不要 - 参照解除: docs/coderabbit-monitoring-efficiency.md:117 のリンクを ADR-035 リンクに置換 ### Cleanup - docs/todo5.md: 順位 62/71/72/73/74 の 5 詳細エントリ削除 - docs/todo.md: priority table から 5 行削除 + Bundle f 脚注を全 4 件 land + retirement 完了に更新 ## Sub-PR 分割可否の dogfood 結果 (順位 73 への self-feedback) 順位 73 が codify した「Bundle 分割時の land 期限明記」ルールを本 PR の作業計画段階で適用検討した。結果、以下の理由で **統合 1 PR を選択**: - ファイル削除 + 順位 62 (新規 global rule) + Bundle f 4 件 が全て docs-PR governance 領域で scope が完全重複 - retirement が global rule (順位 62) の dogfood として最も価値が高く、両者を分離すると意味的連続性が壊れる - 5 ファイル touch は review 容易性を著しく下げるほどではない 順位 73 ルールは「分割時に期限を書く」を default としているが、本 PR のように統合に強い意味的根拠がある場合は統合を許容する。判断プロセス自体が今後の Bundle 運用への参照記録となる。 ## 期待効果 (検証は dogfood) - 計画書 ephemeral lifecycle が global rule (順位 62) で codify され、再発時の判断ぶれ消滅 - Cross-File Reference Lifecycle の self-application 強化で PR description / commit message での codification claim が事前検証される - priority table 運用の二層追跡 + 期限管理が ADR-033 で formalize、長期滞留 detection - global rules 編集の transparency が ADR-030 で formalize、post-merge-feedback による事後検証経路が明示化
This was referenced May 7, 2026
aloekun
added a commit
that referenced
this pull request
May 9, 2026
…必須 follow-up (順位 91 + 92 + 93) (#135) * feat(cli-finding-classifier, cli-push-runner): Bundle i — Phase d 着手前必須 follow-up (順位 91 + 92 + 93) PR #132 (Phase c MVP land) の post-merge-feedback で採用された 3 件を 1 PR にまとめて land。 ## 順位 91 (Tier 2 #4): [lint_screen] config parse test - src/cli-push-runner/src/config.rs に 5 tests を追加 - silent field rename / 追加で None fallback する failure mode を unit test で防止 - full fields / minimal only enabled / absent yields None / numeric defaults / string defaults の 5 軸独立検証 ## 順位 92 (Tier 2 #5): scale-aware eval fixtures (200+ 行) - eval13-large-refactor-real.diff (5 file / 280 行) — context 限界 + JSON 完全性 - eval14-mid-mixed.diff (3 file / 153 行) — mid-scale recall 安定性 - eval15-syntax-stress.diff (1 file / 208 行) — 単 file 長尺の schema 完全性 - lint-screen-evals.json に id 13/14/15 baseline (auto_fix lane × 13 findings 合計) 追加 - count test を rename + 上限緩和 (eval_set_loads_and_has_at_least_phase_b_prime_baseline_count) - Bundle i 実体スモーク test (eval_set_includes_bundle_i_scale_aware_fixtures) 追加 ### dogfood 結果 (mistral:7b / temperature=0) agreement = 11/15 = 73.3% (Phase b' 75% から marginal 劣化 = fixture が設計通り failure mode を再現) eval13 (280 行): JSON parse error 'missing field screen_decision' → fallback path 作動 = PR #132 smoke (868 行 diff) で観測した failure mode を decisive に再現 eval15 (208 行): JSON parse error 'missing field severity at line 38' = nested field omission の別 failure mode を新規捕捉 eval14 (153 行): JSON 完全だが recall 33% (3 baseline 中 1 件のみ TP) aggregate precision=76.2% recall=51.6% latency p50=4591ms p95=8370ms verdict CONDITIONAL-GO agreement < 75% 未達理由は eval13/15 の fallback (= fixture が設計通り作動した結果) で mechanical に説明可能。Phase d 投入前の必須 measurement を取得 (todo6.md L164 「未達理由が 文書化される」branch を満たす)。§8.D v4 prompt 改訂は別 bundle に切り出し。 ## 順位 93 (Tier 3 #8): coding-style.md partial fix anti-pattern codify - ~/.claude/rules/common/coding-style.md § Cross-File Reference Lifecycle に 「変更差分外への partial fix 再発」anti-pattern を追加 - PR #94 / #111 / #132 を inline cite (実証ベース) - family_tag を grep -rn で全 path 検索する対処手順、partial fix の意図的切り出しを明記 ## Phase d 着手の前提条件 update Bundle i land で以下が揃った: - (a) [lint_screen] config silent failure 防止 (順位 91) - (b) scale-aware fixtures による failure mode の reproducible measurement (順位 92) - (c) cross-file partial fix anti-pattern の global rule 化 (順位 93) 次は §8.D v4 prompt 改訂で大規模 diff の JSON 完全性を改善するループ (Phase d 着手前の最終 gate)。 * fix(cli-finding-classifier): CodeRabbit Major #r3213115045 — eval count 下限を Bundle i baseline 15 に固定 >=12 だと既存 fixture 削除を検出できないため >= 15 に変更し regression 防止。 将来の fixture 追加 (>15) は許容。
5 tasks
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
Bundle "e" (convention long-tail) を本セッションで land。順位 23/24/25/26/30/33/70 の 7 件を
~/.claude/rules/common/配下のグローバル rules に集約。注: グローバル rules は project repo 外 (
~/.claude/rules/...) のため diff には現れません。本 PR は完了の証跡として project 側の todo cleanup + docs-pr-iteration-efficiency.md status 更新を含みます。Bundle e 内訳
coding-style.mdMarkdown 節 (新設)git-workflow.mdjj Operations 拡張jj abandon + jj newが高速)~/.claude/CLAUDE.mdCode Style 拡張__prefix scratch 規約 + jj auto-snapshot 特性 (PR #85 由来)development-workflow.md「背景タスクの待機方針」(新設)coding-style.mdCross-File Reference Lifecycle 拡張code-review.mdMulti-point synchronization (新設)Bundle U / V / d 終了状態
同梱 (Bundle d 登録、PR #110 post-merge-feedback 採用案、前ターン分)
docs/todo5.mdに 順位 68 / 69 詳細エントリ追加 (順位 70 は本 PR で land のため別途削除)docs/todo.mdpriority table に 順位 68 / 69 追加済 (70 は本 PR で削除)Project repo 内の変更
docs/todo2.md: 順位 23/24/25/26 詳細エントリ削除 (-134 lines)docs/todo3.md: 順位 30/33 詳細エントリ削除 (-82 lines)docs/todo5.md: 順位 70 詳細エントリ削除 (Bundle d 残の 68/69 は登録)docs/todo.md: priority table から 7 行削除 + Bundle U/V/d/e 脚注更新docs/docs-pr-iteration-efficiency.md: 📋 LOW IMPACT + 後回し可セクションを Bundle "e" ✅ 完了に集約期待効果 (検証は dogfood)
no-ephemeral-todo-referencelint rule (machine 層) と本 PR の Cross-File Reference Lifecycle 具体例 (guideline 層) で 二層防御code-review.mdchecklist で意識化されるTest plan
Summary by CodeRabbit