Bundle 195-FB: jj revset hardcode lint rule + helper parameterize

[aloekun]

Summary

PR #195 post-merge-feedback で採用された 4 件 (Bundle 195-FB) を mechanical + 実装 + docs の 3 層構造で land。前セッションで land 済の feedback registration commit に 3 commits を追加して同 PR で消化。

各層 (4 commits)

Commit	Layer	順位 / Tier
ouxpmynm (既 push 済)	docs(todo): PR #195 post-merge-feedback 採用 4 件登録	前セッション、Bundle 195-FB 計画書
uoxuyxrp	feat(lint): rule⑫ no-hardcoded-jj-revset-range 追加	順位 189 (T1) — mechanical
qntnzyxt	refactor(cli-pr-monitor): count_empty_in_pr_range を default_branch 引数化	順位 190 (T2) — 実装
nootrxyl	docs(todo): Bundle 195-FB 完了で 順位 189-192 削除 + global rule edit 同伴	順位 191/192 (T3) — cleanup + docs

3 層が 1 PR で循環する dogfood

• Layer 1 (189) で rule⑫ を追加 → 同 PR の Layer 2 (190) で production の count_empty_in_pr_range の hardcode を修正 → rule⑫ は 0 fire の clean baseline に遷移
• grep -rn 'master\.\.@' src/ = 0 hit (verify 済)
• companion helper group (3 関数: assert_descriptions_absent_* / assert_descriptions_present_* / count_empty_in_pr_range) の API signature が (repo_dir, default_branch, ...) で整合

Global rule edits (本 PR の diff 外、要確認)

順位 191/192 は ~/.claude/rules/common/ 配下の global rule docs を編集するため本リポジトリの diff には現れません。派生プロジェクト (techbook-ledger / auto-review-fix-vc) に自動波及するため、別途確認をお願いします:

• ~/.claude/rules/common/code-review.md (順位 191): § Non-blocking findings の留置基準 を追加
  • current impact: none / low / medium+ の 3 段階 table
  • 残置時の理由明記原則 + PR description 形式の例
  • 同型 finding 複数観測 → medium+ 昇格 ルール
• ~/.claude/rules/common/patterns.md (順位 192): § Companion Helper Group の API Cohesion を追加
  • 「同 PR で全 companion 関数を揃える」原則
  • grep -n "fn <prefix>_" による事前確認手順
  • PR docs/test: PR #194 follow-up bundle — fail-closed ADR + jj test/revset patterns + experimental feature checklist #195 anti-pattern を inline cite
• ~/.claude/rules/common/testing.md (順位 192): sentinel pattern section 末尾に ADR-041 (Test Isolation Patterns for Multi-Condition Guards) への inline cite を追加 (派生プロジェクトでは ADR-041 が無い場合があるため markdown link ではなく inline 引用で完結)

Snapshot: ~/.claude-backups/20260605-131309-bundle-195-fb/ (memory feedback_global_config_backup per)

設計判断

rule⑫ の pattern 緩和 ("master\.\.@" → master\.\.@)

当初設計は引用符付き ("master\.\.@") で false positive を抑制する案でしたが、実 production の hardcode は "empty() & (master..@)" のように 内側 literal が outer string scope 内に embed されている形式 だったため、引用符付きでは fire しない問題が dogfood で判明。pattern を引用符なしに緩和し、FP リスクは Bundle Z #B-α (comment-lint-rust) が非 doc comments を既に block するため軽微 (jj revset 固有 token) と判断。

Self-exclusion 戦略

src/hooks-post-tool-linter/src/main.rs 自身が extensions = ["rs"] の lint 対象であるため、test fixture は build_*_fixture(branch: &str) 関数で runtime 組立 (= source 内に literal master..@ を書かない)。grep -n 'master\.\.@' src/hooks-post-tool-linter/src/main.rs で 0 hit を verify 済。

doc comments の書き換え

production の count_empty_in_pr_range 直近の doc comments (line 621, 644) も master..@ literal を含んでいたため、<default_branch>..@ placeholder に書き換えて FP を構造的に排除。

Verification

Check	Result
cargo clippy --workspace -- -D warnings	✅ clean
cargo test --workspace	✅ all green (199 + lib doc tests)
cargo test -p hooks-post-tool-linter no_hardcoded_jj_revset_range	✅ 4 tests pass (positive 2 / negative 2)
cargo test -p hooks-post-tool-linter rule_test_coverage_check	✅ pass (TOML meta field 整合)
cargo test -p cli-pr-monitor 'fix_commit::tests::integration_sweep' -- --ignored --test-threads=1	✅ 5 integration tests pass
grep -rn 'master\.\.@' src/	✅ 0 hit (clean baseline)

Test plan

• rule⑫ unit tests (4 variants) pass: simple hardcode / empty() filter scope / parameterized format skip / other branch literal skip
• count_empty_in_pr_range が新 signature (repo_dir, default_branch: &str) で動作
• integration_sweep_respects_alternative_default_branch で count_empty_in_pr_range(repo_dir, "main") が helper の "main" 動作を保護
• companion helper group 3 関数の API signature 整合性
• global rule (~/.claude/rules/common/*.md) edits が派生プロジェクトに波及 (= 次の派生プロジェクトでのセッションで rule が見える)
• 次回 rule⑫ が production の新規 hardcode (もしあれば) を fire (dogfood 継続観測)

References

• .claude/feedback-reports/195.md (post-merge-feedback report)
• PR docs/test: PR #194 follow-up bundle — fail-closed ADR + jj test/revset patterns + experimental feature checklist #195 (Bundle 195-FB の root cause = test helper API cohesion 違反)
• ADR-007 (custom-linter layer boundary)
• ADR-021 § Revset Composability
• ADR-041 (Test Isolation Patterns for Multi-Condition Guards)
• Memory: feedback_global_config_backup, feedback_pnpm_create_pr_body, feedback_test_dry_antipattern

Summary by CodeRabbit

リリースノート

• 新機能

  • Rustコード内のハードコードされたリポジトリブランチ範囲を検出する新規カスタムリントルールを追加しました。

• テスト

  • 新規リントルールと既存のテストヘルパーに対応するテストケースを追加しました。

[coderabbitai]

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: b933958d-bce4-48c7-9da3-af0e3f444a08

📥 Commits

Reviewing files that changed from the base of the PR and between 82d64ec and 870bb0b.

📒 Files selected for processing (3)

• .claude/custom-lint-rules.toml
• src/cli-pr-monitor/src/fix_commit.rs
• src/hooks-post-tool-linter/src/main.rs

---

📝 Walkthrough

Walkthrough

This PR introduces a new custom lint rule no-hardcoded-jj-revset-range to detect hardcoded jj revset branch ranges in Rust code, parametrizes test infrastructure to support configurable default branches, and adds comprehensive test coverage for the new rule with multiple revset syntax variants.

Changes

カスタムリントルール「ハードコード revset 範囲」の追加と test 基盤のパラメータ化

Layer / File(s)	Summary
リントルール定義と test 期待値 \.claude/custom-lint-rules.toml	新規ルール no-hardcoded-jj-revset-range が master..@ パターンを Rust ファイルで警告として検出し、default_branch を用いた修正戦略と test カバレッジ期待値を定義しています。
Test ヘルパーの default_branch パラメータ化 src/cli-pr-monitor/src/fix_commit.rs	count_empty_in_pr_range ヘルパーが default_branch 引数をサポートするよう拡張され、revset 生成がパラメータ化されました。統合テストのコメントと assert が更新され、master と main 両方のブランチに対する動作検証が明示的になっています。
ハードコード revset 検出の test カバレッジ src/hooks-post-tool-linter/src/main.rs	新規ルール用に複数の test ケースが追加され、ハードコード、empty() 構文内、format! パラメータ化、代替ブランチリテラルなど異なる revset 表現パターンの正・負ケースを検証しています。

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#194: 本 PR の src/cli-pr-monitor/src/fix_commit.rs への default_branch パラメータ化は、PR #194 で新たに追加された空 commit sweep ロジックと同じモジュール内の default_branch 配線に直結しています。
• aloekun/claude-code-hook-test#165: 本 PR の no-hardcoded-jj-revset-range ルール追加（.claude/custom-lint-rules.toml の rules.test_coverage と src/hooks-post-tool-linter/src/main.rs の新規 test ケース）は、PR #165 で導入された rule_test_coverage_check 機構と rules.test_coverage メタ配線に直結しています。
• aloekun/claude-code-hook-test#195: 本 PR の src/cli-pr-monitor/src/fix_commit.rs と統合テストへの変更（default_branch による revset 範囲パラメータ化）は、PR #195 の sweep revset バリアント test・ヘルパーと直接重複しており、revset scope overreach 回帰防止を目的としています。

🚥 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	PR タイトルは、新しいカスタムリントルール（no-hardcoded-jj-revset-range）の追加と、ヘルパー関数のパラメータ化という、PR の主要な変更内容を明確に要約しており、スキャン時に主要な変更を理解できる内容になっています。
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.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}