fix: Cargo の同じ取得元・同じ版の宣言をまとめて書き換える

[owayo]

概要

Cargo.toml で同じ依存を [dependencies] と [dev-dependencies] に同じ取得元・同じ版で書くと (dev 側にだけ features を足す、Rust でよくある書き方)、書き込みの段階で Refusing to update ambiguous dependency になり、更新できずに exit code 2 になっていた。git のタグの依存・crates.io の依存・Cargo.toml を書き換えない branch の git 依存のどれでも起き、更新の行とエラーも宣言の数だけ並んでいた。

同じ取得元・同じ版を指す宣言を parse の段階で 1 依存にまとめ、1 回の書き換えで全宣言を新しい版にする。

原因

• ManifestWriter::apply_updates は書き込む前にマニフェストを parse し直し、manifest_name() が同じ宣言が 2 件以上あれば、版や取得元を比べずに拒否する。この判定は git の参照の種類で分ける処理より前にあったので、Cargo.toml を書き換えない branch / rev の git 依存も拒否していた
• CargoTomlParser::parse はセクションごとに宣言を 1 件ずつ返し、同じ依存をまとめていなかった
• 書き換える側 (update_version / update_git_tag) は、全依存セクションの同名の宣言をすべて書き換える作りになっている。宣言の版がそろっていれば、1 回の書き換えで全宣言が新しい版になる

変更内容

• CargoTomlParser::parse で、実パッケージ名・依存キー・版の指定の生表記・git の取得元 (URL・参照・公開用に併記した version) が一致する宣言を 1 依存にまとめる (CargoDeclarations)。Gradle の push_unique_declaration と同じ考え方。features などの付帯指定と is_dev はキーに含めない。is_dev は、本番の宣言が 1 つでもあれば本番にする
• writer で、Cargo.toml を書き換えない git の branch / rev / 既定ブランチの更新は曖昧判定より前に抜け、拒否しない
• 版や取得元が食い違って拒否するときも、同じ依存のエラーは 1 行にまとめる
• 文書 (docs/usage と docs/ecosystems の英日、AGENTS.md) を更新

設計の判断

codex と相談して決めた。

• 版の指定は生表記で比べる。"1.0" と "^1.0" は Cargo では同じ意味だが、まとめずに拒否のままにする。要求範囲を忠実に比べる正規化を持たないので、誤更新を避ける側に倒す
• git 依存に公開用の version を併記している場合は、それもキーに入れる。同じ URL・タグでも、要求する版が違えば同等の宣言とは言えない
• is_dev は最初の宣言を残すのではなく AND で合成する。[dev-dependencies] がターゲット固有の [dependencies] より先に読まれるので、最初の宣言を残すだけだと本番の依存が dev 扱いになる
• エラーの重複は出力の側ではなく writer でまとめる。テキストと JSON の両方に効く

範囲外

update_git_tag は依存セクションに加えて [patch.<registry>] の同名の宣言の tag も書き換える (以前からの振る舞い)。[patch] 側が別の URL (fork など) を指していても書き換わる。今回の Issue とは別の問題なので、この PR では触らない。

検証

• make ci (fmt-check・clippy・全テスト): pass (lib 2444 / e2e 26 / integration 75)
• 回帰テスト (parse):
  • test_parse_merges_declarations_with_same_source_and_version: git のタグ・crates.io・branch・引用符違いの宣言が 1 依存にまとまる
  • test_parse_keeps_declarations_with_different_source_or_version: 版・1.0 と ^1.0・タグ・URL・取得元の種類・併記した version が食い違う宣言はまとめない
  • test_parse_keeps_same_key_renamed_to_different_packages: 依存キーが同じでも package が違えばまとめない
  • test_parse_merged_declaration_is_production_when_any_declaration_is
• 回帰テスト (parse から書き込みまで):
  • test_apply_updates_rust_same_git_tag_in_dependencies_and_dev_dependencies: 両方の tag が書き換わる
  • test_apply_updates_rust_same_registry_version_in_dependencies_and_dev_dependencies: 両方の版が書き換わる
  • test_apply_updates_rust_same_git_branch_in_dependencies_and_dev_dependencies: Cargo.toml を変えず、エラーにしない
  • test_apply_updates_rust_unwritten_git_update_is_not_refused_beside_registry_declaration: branch・rev・既定ブランチの git の更新は通し、同名の crates.io の更新だけを拒否する
  • test_apply_updates_rust_mismatched_declarations_report_one_error: 食い違う宣言は拒否し、エラーは 1 行
• Issue の 3 つの Cargo.toml を、ビルドしたバイナリで実行した。git のタグ・crates.io は両方の宣言が書き換わり、branch は Cargo.toml を変えない。いずれも 1 update で exit 0。版が食い違う構成は、エラー 1 行で拒否する (exit 2)
• codex の差分レビューで出た 2 件 (rev と既定ブランチを曖昧判定から外すことのテストが無い、CargoDeclarations のコメントがその例外を書いていない) を反映した

Closes #20

🤖 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: 372c8b3e-8724-4fb4-a275-634303fc4edf

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