fix(cli-push-runner): 新規 bookmark push を --all で修正 + clippy/W1 doc 同梱

[aloekun]

概要

fix/clippy-and-w1-doc ブランチに溜まった 3 コミットをまとめた PR。push 環境整備セッションで
push-runner の新規 bookmark push バグ修正が加わったため、元の clippy / doc 変更と同梱している。

変更内容

1. fix(cli-push-runner): 新規 bookmark の push を --all で修正（中核）

• bare jj git push は tracking 済み bookmark のみ push するため、新規 bookmark が push されず、
  jj は exit 0 のまま Refusing to create new remote bookmark を出力して何もしない。
  push-runner はこれを false success（実 push 未完了を「成功」）と誤報告していた。
• push-runner-config.toml / templates/push-runner-config.toml: command を jj git push --all に変更
  （jj 0.42 に --allow-new フラグは無いため --all を採用）。
• src/cli-push-runner/src/stages/push.rs: push_was_refused() を追加し Refusing to 出力を
  検知して失敗扱いに変更（+ unit test 4 件）。

2. fix(lib-report-formatter): clippy 解消

• unnecessary_sort_by を sort_by_key に解消。

3. docs(file-length-enforcement-plan): W1 doc 更新

• PR-W1 (refactor(hooks-post-tool-comment-lint-rust): main.rs を 7 module に分割 (PR-W1、self-host irony 解消) #220) land を status に反映 + 実績節を追加。

検証

• pre-push ゲート（lint / test / build / clippy / cargo-test）+ AI レビュー（simplicity / security）all approved。
• cli-push-runner: clippy clean / 168 tests pass。
• push 成功を origin で確認（fix/clippy-and-w1-doc@origin = 0e78fd22）。

補足

• 本 PR はスコープの異なる 3 トピックを同梱している。分割が望ましければ push-runner 修正を別ブランチに切り出し可能。

🤖 Generated with Claude Code

Summary by CodeRabbit

• New Features

  • 新しい bookmark も含めて一括で push されるようになりました。
  • push の失敗判定が強化され、リモート側で拒否された場合に見逃しにくくなりました。

• Bug Fixes

  • 失敗しても成功扱いになっていた一部の push を正しく検知するようになりました。

• Documentation

  • 作業計画の進捗と完了状況を更新しました。

• Refactor

  • 並び替え処理をより簡潔な実装に整理しました。

[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: c19304d5-3670-49f3-ae1a-4440b5759c50

📥 Commits

Reviewing files that changed from the base of the PR and between 08f173f and 0e78fd2.

📒 Files selected for processing (5)

• docs/file-length-enforcement-plan.md
• push-runner-config.toml
• src/cli-push-runner/src/stages/push.rs
• src/lib-report-formatter/src/lib.rs
• templates/push-runner-config.toml

---

📝 Walkthrough

Walkthrough

jj git push --all への変更、push 出力の拒否検知、進捗ドキュメントの更新、severity ソートの簡略化が含まれています。

Changes

ファイル長制限計画の進捗更新

Layer / File(s)	Summary
進捗と実績の更新 docs/file-length-enforcement-plan.md	PR-W0 と PR-W1 の status が land 済になり、PR-W1 の実績セクションと進捗追跡行が更新された。

push runner の全 bookmark push と拒否検知

Layer / File(s)	Summary
push コマンドを --all に変更 templates/push-runner-config.toml, push-runner-config.toml	[push] の command が jj git push --all に変わり、挙動を説明するコメントが追加された。
push 拒否の検知 src/cli-push-runner/src/stages/push.rs	run_push が push 出力を判定して拒否ケースを false 扱いにし、拒否検知関数とテストが追加された。

severity ソートの簡略化

Layer / File(s)	Summary
severity 順のキーソート src/lib-report-formatter/src/lib.rs	findings の並び替えが sort_by から sort_by_key に置き換えられた。

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• aloekun/claude-code-hook-test#49: src/cli-push-runner/src/stages/push.rs の run_push フローに関わる変更で、この PR の push 出力判定と同じ実行経路を扱っています。

🚥 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	主要な変更である新規 bookmark の push 修正と、clippy・ドキュメント更新の同梱を簡潔に示しています。
Docstring Coverage	✅ Passed	Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.
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.

✨ 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.}