diff --git a/Cargo.toml b/Cargo.toml index d8d8f13..141d61e 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -1,12 +1,12 @@ # file: Cargo.toml -# version: 155 +# version: 156 [workspace] resolver = "3" members = ["crates/ksp-app-config-desk", "crates/ksp-config-lib", "crates/ksp-core-lib", "crates/ksp-logging-lib", "crates/ksp-onchain-transport-lib", "crates/ksp-wallet-lib"] [workspace.package] -version = "0.2.5-pre.7" +version = "0.2.5-pre.7.fix.1" edition = "2024" license = "MIT" repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project" diff --git a/crates/ksp-wallet-lib/src/persistence.rs b/crates/ksp-wallet-lib/src/persistence.rs index 0932f55..83d0bbc 100644 --- a/crates/ksp-wallet-lib/src/persistence.rs +++ b/crates/ksp-wallet-lib/src/persistence.rs @@ -1,5 +1,5 @@ // file: crates/ksp-wallet-lib/src/persistence.rs -// version: 2 +// version: 3 //! Async-first native Wallet V1 filesystem persistence. @@ -156,6 +156,7 @@ fn replace_wallet_file_checked_blocking( return replace_wallet_file_with_hook(destination, content, || return verify_expected_wallet_state(destination, expected_current)); } +#[cfg(test)] fn replace_wallet_file_blocking(destination: &std::path::Path, content: &[u8]) -> ksp_core_lib::Result<()> { return replace_wallet_file_with_hook(destination, content, || return std::result::Result::Ok(())); } diff --git a/deltas/0.2.5/pre.007-fix.001.md b/deltas/0.2.5/pre.007-fix.001.md new file mode 100644 index 0000000..d919fbc --- /dev/null +++ b/deltas/0.2.5/pre.007-fix.001.md @@ -0,0 +1,97 @@ + + + +# Delta `0.2.5-pre.007-fix.001` — suppression du warning `dead_code` persistence + +## Base + +```text +0.2.5-pre.7 +``` + +Version workspace cible : + +```text +0.2.5-pre.7.fix.1 +``` + +## Objectif + +Corriger le warning `dead_code` signalé par `cargo check`, `cargo clippy --workspace --all-targets` et `cargo test --workspace` sur `replace_wallet_file_blocking`, sans modifier la persistence administrative, le format `.kspwallet` V1 ni la surface publique. + +## Cause + +`replace_wallet_file_blocking(destination, content)` est une primitive de remplacement brut sans vérification d'état. Elle n'est appelée que par le helper de test `replace_wallet_for_test`, lui-même compilé sous `#[cfg(test)]`. + +Dans la bibliothèque de production, le vrai chemin administratif utilise exclusivement : + +```text +replace_wallet_file_v1 +-> replace_wallet_file_checked_blocking +-> replace_wallet_file_with_hook +``` + +avec vérification de l'état OWNER authentifié avant publication. La primitive brute restait néanmoins compilée en production, où elle était inutilisée, d'où le warning. + +## Correction + +`replace_wallet_file_blocking` est désormais lui-même protégé par : + +```rust +#[cfg(test)] +``` + +Le helper reste disponible pour les tests de la primitive de remplacement, mais la fonction brute n'existe plus dans le build de production. + +Cette correction renforce la frontière de `pre.007` : aucune primitive générique de remplacement non vérifié n'est compilée dans la bibliothèque de production. + +## Fichiers + +Modifiés : + +```text +Cargo.toml +crates/ksp-wallet-lib/src/persistence.rs +``` + +Ajouté : + +```text +deltas/0.2.5/pre.007-fix.001.md +``` + +Aucun fichier supprimé. + +## Validation opérateur observée avant le fix + +Sur `0.2.5-pre.7` : + +```text +cargo fmt --all OK +cargo check --workspace OK avec 1 warning dead_code +cargo clippy --workspace --all-targets OK avec 1 warning dead_code +cargo test -p ksp-wallet-lib OK : 46 passed, 1 ignored + intégration/doc-tests +cargo test --workspace OK avec 1 warning dead_code +cargo tree -p ksp-wallet-lib exécuté +cargo tree -p ksp-wallet-lib -d exécuté +``` + +Le warning concernait uniquement `replace_wallet_file_blocking` dans `persistence.rs`. + +## Validation opérateur requise après application + +```bash +cargo fmt --all +cargo check --workspace +cargo clippy --workspace --all-targets +cargo test -p ksp-wallet-lib +cargo test --workspace +``` + +Aucun nouvel audit `cargo tree` n'est requis pour ce fix : aucune dépendance n'est ajoutée, supprimée ou reconfigurée. + +## Commit attendu + +```text +v0.2.5-pre.007-fix.001 +```