docs(todo): WP-16 系 post-merge feedback 採用 5 件を todo14 に登録 (順位 359-363) - #346
Merged
Conversation
PR #342/#343/#344 の post-merge feedback 採用候補 14 件のうち、2026-08-02 の 方針決定 (文書系 10 件 = 1 docs バッチ PR、実装系 4 件 = A/B 系統別 PR) に従い 5 エントリとして登録: - 順位 359: 文書系 10 件の docs バッチ (dev-conventions 集中、Severity High 2 件含む) - 順位 360: cargo test コマンド等価性検証テスト (#342/#343 で連続指摘 = Frequency High) - 順位 361: JJ_VERSION の 2 ファイル一致検証テスト (順位 360 と同一 PR 可) - 順位 362: git subprocess ブランチ名依存引数の custom lint rule (#343 incident 由来) - 順位 363: check-ci-coderabbit の detached HEAD 回帰統合テスト (順位 362 と同一 PR 可)
📝 WalkthroughWalkthroughWP-16 系の post-merge feedback に基づく TODO を追加しました。文書整備、Cargo と ChangesWP-16 TODO 更新
Estimated code review effort: 1 (Trivial) | ~5 minutes Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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 |
Owner
Author
|
@coderabbitai review |
✅ Action performedReview finished.
|
![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 current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@docs/todo14.md`:
- Around line 774-792: 新設する detached HEAD 統合テストで、実際に fetch_ci の repo と pr を使う gh
pr view 経路を実行し、偽の gh 実行ファイルで「pr view <pr> --repo
<repo>」の引数と呼び出しを検証してください。parse_ci_rollup
やソース構造だけに依存せず、空のブランチ名へ回帰すると失敗するテストにし、通常の cargo test で実行可能にするか、CI で cargo test --
--ignored --test-threads=1 を明示的に実行できる設定を追加してください。
🪄 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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro Plus
Run ID: 5e3ea232-9ed5-4a0b-a780-52cbf5106358
📒 Files selected for processing (2)
docs/todo-summary2.mddocs/todo14.md
Comment on lines
+774
to
+792
| ### check-ci-coderabbit の detached HEAD 回帰統合テスト | ||
|
|
||
| > **動機**: PR #343 で修正した「jj colocated (detached HEAD) 環境で CI 状態が恒久 pending 化する」バグの regression test が皆無 (`src/check-ci-coderabbit/tests/` 自体が不在)。再発時は監視の自律ループが再び silent に破綻する。#343 T2-1 採用。 | ||
| > | ||
| > **対処案**: temp dir に jj colocated repo (detached HEAD) を組み、CI 状態解決が statusCheckRollup ベースで機能すること (旧経路のようにブランチ名解決依存で空にならないこと) を検証する統合テストを新設する。gh 呼び出しは実 API に依存しない形 (parse 層の既存単体テスト + 経路の構造検証) を基本とし、実 jj spawn が必要な部分は lib-jj-helpers の `#[ignore]` + 直列実行パターンを踏襲する。 | ||
| > | ||
| > **参照**: `.claude/feedback-reports/343.md` Tier2 #1、`src/check-ci-coderabbit/src/main.rs` (`fetch_ci`) / `src/check-ci-coderabbit/src/parsers.rs` (`parse_ci_rollup` 単体テスト群 = 既存資産)。順位 362 と同一 PR (B 系統) にまとめてよい。 | ||
| > | ||
| > **実行優先度**: 🔧 Tier 2 — Severity Medium / Frequency Low / Effort M / Adoption Risk None。 | ||
|
|
||
| #### 作業計画 | ||
|
|
||
| - [ ] detached HEAD 環境での CI 状態解決を検証する統合テストを新設 | ||
| - [ ] ブランチ名依存の旧経路への回帰が fail することを確認 | ||
| - [ ] 本エントリ削除 + todo-summary2.md 行削除 | ||
|
|
||
| #### 完了基準 | ||
|
|
||
| - detached HEAD 環境で CI 状態が pending に固着する回帰が、テスト失敗として land 前に検出されること。 |
There was a problem hiding this comment.
🎯 Functional Correctness | 🟠 Major | 🏗️ Heavy lift
🧩 Analysis chain
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
rg -n -C 4 '#\[ignore\]|--ignored|--include-ignored|test-threads|cargo test' \
.github scripts src/check-ci-coderabbit Cargo.tomlRepository: aloekun/claude-code-hook-test
Length of output: 159
🏁 Script executed:
#!/usr/bin/env bash
set -euo pipefail
echo "== files =="
git ls-files | rg '(^|/)(docs/todo14\.md|src/check-ci-coderabbit/src/main\.rs|src/check-ci-coderabbit/src/.*\.rs|Cargo\.toml|README\.md)' || true
echo "== doc excerpt =="
sed -n '760,802p' docs/todo14.md 2>/dev/null || true
echo "== outline check-ci-coderabbit =="
ast-grep outline src/check-ci-coderabbit/src/main.rs --view compact 2>/dev/null || true
echo "== main relevant excerpt =="
sed -n '320,410p' src/check-ci-coderabbit/src/main.rs 2>/dev/null || true
echo "== tests directory =="
git ls-files src/check-ci-coderabbit | sort
echo "== grep ignore/test hints minimal =="
rg -n -C 2 '#\[ignore\]|--ignored|--include-ignored|test-threads|cargo test|fetch_ci|derive_ci|get_repo|repo.*pr|pr view' \
src/check-ci-coderabbit docs/todo14.md Cargo.toml 2>/dev/null || trueRepository: aloekun/claude-code-hook-test
Length of output: 159
detached HEAD の実行経路を実際に検証してください。
parse_ci_rollup の単体テストとソース構造の検証だけでは、呼び出し側が空のブランチ名を repo または pr の代わりに渡す回帰を検出できません。
fetch_ci が repo と pr を使って gh pr view を実行する経路を使い、偽の gh 実行ファイルで pr view <pr> --repo <repo> の引数を検証してください。
#[ignore] 付きテストは通常の cargo test では実行されません。完了基準を満たすには、CI で cargo test -- --ignored --test-threads=1 を明示的に実行するか、通常のテストスイートで実行できる構成にしてください。
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In `@docs/todo14.md` around lines 774 - 792, 新設する detached HEAD 統合テストで、実際に
fetch_ci の repo と pr を使う gh pr view 経路を実行し、偽の gh 実行ファイルで「pr view <pr> --repo
<repo>」の引数と呼び出しを検証してください。parse_ci_rollup
やソース構造だけに依存せず、空のブランチ名へ回帰すると失敗するテストにし、通常の cargo test で実行可能にするか、CI で cargo test --
--ignored --test-threads=1 を明示的に実行できる設定を追加してください。
Contributor
🤖 PR Monitor 分析 (GitHub Actions バックストップ)
Applicable Findings (Critical / High / Major)該当なし Applicable Findings (Medium 以下)該当なし Filtered (not applicable)
次のアクション
|
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
docs/todo14.md+docs/todo-summary2.mdに登録Context
Why: WP-16 関連 4 PR (#342-#345) の post-merge feedback レポートから採用候補 14 件を系統別に整理し、「文書系 10 件 = 1 docs バッチ PR、実装系 4 件 = A/B 系統別 PR」の方針でユーザー承認を得た。本 PR はその todo 登録 (ADR-033: 詳細 = todo14.md、順位 table = todo-summary2.md 末尾)。
重複統合: #343 T2-3 は #342 T2-1 と同一提案のため順位 360 に統合 (連続指摘の事実は Frequency High の根拠として反映)。#345 は採用候補 0 件。様子見 6 件・却下推奨 9 件は本 PR の対象外 (未確定のまま)。
Validation
pnpm lint:docsOK (preamble + cross-ref + priority-inversion、summary 両ファイル統合検査)pnpm pushpre-push review: simplicity / security 両 approved (docs-only routing、pr_size 126 行)References
.claude/feedback-reports/342.md/343.md/344.md/345.md(採用候補の出典、gitignore 対象の内部 artifact)Summary by CodeRabbit
cargo testの対象範囲とコマンド等価性、JJ_VERSIONの同期状態を検証する計画を記録しました。