fix: 書き込めなかった更新を更新の件数に数えない

[owayo]

概要

マニフェストへの書き込みに失敗した更新も、出力では更新済みとして数えていた。テキストの Summary の N package(s) updated、マニフェストごとの見出しの N updates、--json の summary.updates のどれにも失敗した件が入り、エラー欄とは件数が食い違っていた。

書き込めなかった更新を更新ごとに記録し、件数は「judge が決めた更新 − 書き込めなかった更新」で数える。書き込めなかった更新は failed として別に数え、テキスト・JSON・diff の 3 つで同じ数え方にする。

原因

• 件数は check の段階で作った UpdateSummary から数えていた。ManifestUpdateResult::update_count / UpdateSummary::total_updates は UpdateResult::Update の数で、書き込みの結果を見ていなかった
• 書き込みの結果は ManifestWriter::apply_updates が WriteResult の updates_failed (件数だけ) と errors に入れていたが、Orchestrator::write_phase はエラーの文字列だけを errors へ移し、どの更新が失敗したかは summary に届いていなかった

変更内容

• writer: 書き込めなかった更新を results 上の位置と理由で記録する (WriteResult::failed_updates。件数は updates_failed())。保存 (I/O) に失敗しても Err にせず、書き換えた更新を書き込めなかったものとして記録し、保存の前に個別の理由で失敗した更新 (曖昧として拒否、など) はその理由を残す。Err は読み込み・解析に失敗してどの更新も試せなかったときだけで、apply_all_updates がそれを全更新の失敗として記録する。Cargo.toml を書き換えない git の branch / rev / 既定ブランチの更新は、失敗にしない
• domain: UpdateResult::Update に write_error を足す。summary は書き込みの前後で意味を分ける
  • judged_updates() / has_judged_updates(): judge が決めた更新 (writer が使う)
  • updates() / update_count() / has_updates() / total_updates(): 書き込めなかった更新を除いたもの (出力・install・Rust の lock 監査が使う)
  • failed_updates() / failed_count() / total_failed(): 書き込めなかった更新
  • language_breakdown() は失敗の件数も返す (LanguageCounts)
• orchestrator: write_phase で書き込めなかった更新を summary に記録する
• 出力
  • テキスト: 見出しに K failed (失敗があるときだけ)、行末に ✗ failed (色なしは [failed])、Summary に K package(s) failed to update。-q は N updated, K failed、--verbose の言語別にも失敗の件数。全更新を書き込めなかったマニフェストも、スキップの要約にせず更新の行を出す
  • JSON: summary.failed と、各マニフェストの failed 配列 (理由は error) を常に出し、updates からは外す。summary.updates が updates 配列の長さの和と一致する
  • diff: 書き込めなかった更新の hunk は出さず、件数の行に , K failed を付ける
• install: 全更新を書き込めなかったマニフェストは install の理由にしない (Rust の install は lock 全体を上げる cargo update)。Rust の lock 監査も、書き込めなかった更新先を「表示した更新先」(差し戻し先の候補・食い違いの注記) に使わない
• 文書: docs/cli-reference と docs/usage の英日、AGENTS.md。終了コードの節にあった「書き込みに失敗しても更新として表示され、--install も実行される」を直し、JSON と diff の例を実際の出力の形に合わせた

設計の判断

codex と相談して決めた。

• 書き込みの結果は UpdateResult に持たせ、summary を唯一の正にする。出力・install・lock 監査が同じ判定を使え、各所で WriteResult と突き合わせずに済む
• install と lock 監査も書き込めなかった更新を除く (Issue の範囲は出力だが、何も書き換えていないのに cargo update が lock 全体を上げるのは意外で、書き込めなかった更新先を監査の期待値にすると、反映していない版との差を注記してしまう)。文書に「--install も実行される」と明記されていた振る舞いを変えるので、文書も直した
• JSON は失敗を updates に残して印を付けるのではなく、failed に分ける。updates の長さを更新件数として扱う利用者も誤らない
• -q の全失敗は No updates ではなく 0 updated, K failed

検証

• make ci (fmt-check・clippy・全テスト): pass (lib 2459 / e2e 26 / integration 75)
• 回帰テスト:
  • writer: test_apply_updates_records_failed_update_positions (位置と理由の記録)、test_apply_updates_write_permission_denied (保存の失敗でも個別の理由を残す)、test_apply_updates_dry_run_records_refused_updates、test_apply_all_updates_handles_missing_file、test_apply_all_updates_unreadable_manifest_keeps_unwritten_git_updates
  • domain: test_write_errors_are_counted_apart_from_updates、test_record_write_error、test_serde_update_result_write_error (write_error の無いデータも読み込める)
  • 出力: test_format_counts_write_failures_apart_from_updates・test_format_shows_manifest_whose_updates_all_failed・test_format_without_write_failures_keeps_counts_unchanged (テキスト)、test_format_json_separates_write_failures・test_format_json_always_lists_failed (JSON)、test_format_diff_excludes_write_failures (diff)
  • orchestrator: write_failures_are_recorded_in_summary (曖昧として拒否した更新が summary に記録され、件数から外れる。dry-run)
  • install: build_install_map_skips_manifest_whose_updates_all_failed、audit_does_not_treat_an_unwritten_update_as_displayed
• ビルドしたバイナリで、版が食い違う宣言 (書き込めない 2 件) と branch の git 依存 (Cargo.lock だけが変わる 1 件) を並べた Cargo.toml を実行した。— 1 update, 2 failed、Summary は 1 package(s) updated と 2 package(s) failed to update、-q は 1 updated, 2 failed、exit code 2。package.json の曖昧な宣言でも、テキスト・JSON (summary.updates 1・failed 1)・diff (# 1 package(s) would be updated, 1 failed) の件数がそろうことを確かめた
• codex の差分レビュー: 正しさ・件数・install と lock 監査への反映・テストは問題なし。指摘 1 件 (終了コードの節で、保存の失敗と一部の更新だけの失敗を区別していなかった) を文書に反映した

Closes #21

🤖 Generated with Claude Code

[coderabbitai]

Important

• 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

• Configuration used: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: e36b4cda-413b-4162-ac0d-55240a257e6f

• Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

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