[Copilot]

rand 0.10 no longer re-exports RngCore at the crate root, breaking the build when crypto.rs imported it as rand::RngCore.

Changes

• src/crypto.rs: import RngCore via aes_gcm::aead::rand_core and consolidate it into the existing use aes_gcm::{ ... } block, removing the split import
• Cargo.toml: remove the now-unused rand = "0.10" direct dependency — RngCore is sourced from aes_gcm's re-exported rand_core, so the top-level rand crate is no longer needed

// before
use aes_gcm::{
    aead::{Aead, KeyInit, OsRng},
    Aes256Gcm, Nonce,
};
use rand::RngCore;

// after
use aes_gcm::{
    aead::{rand_core::RngCore, Aead, KeyInit, OsRng},
    Aes256Gcm, Nonce,
};

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

This PR updates the RngCore import in crypto.rs to account for rand 0.10 no longer providing RngCore via rand::RngCore, restoring build compatibility for nonce generation in the AES-GCM encryption module.

Changes:

• Switch RngCore import from rand::RngCore to aes_gcm::aead::rand_core::RngCore.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

After this change, the crate no longer appears to use the rand dependency anywhere (no remaining rand::... imports/usages across the Rust sources), but Cargo.toml still declares rand = "0.10". Consider removing the unused dependency (or switching to a direct rand_core dependency) to reduce dependency surface and avoid carrying extra rand/rand_core versions.

[Copilot]

RngCore is now imported from aes_gcm::aead::rand_core, but OsRng is still imported separately from aes_gcm::aead above, resulting in split/duplicated aes_gcm imports. Consider grouping these together (e.g., bring rand_core::RngCore into the existing use aes_gcm::{ ... } block) to keep imports consistent and easier to scan.

[Copilot]

@copilot コメントの確認と対応を日本語で提案してください

レビューコメントを踏まえて以下の2点を修正しました（コミット 7fc7606）：

1. インポートの統合：aes_gcm::aead::rand_core::RngCore を独立した use 文にするのではなく、既存の use aes_gcm::{ aead::{...} } ブロック内にまとめました。
2. 未使用依存関係の削除：RngCore は aes_gcm 経由で取得するようになったため、Cargo.toml から rand = "0.10" を削除しました。

[Copilot]

Pull request overview

Copilot reviewed 2 out of 3 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.