v0.2.5-pre.007-fix.001
This commit is contained in:
@@ -1,12 +1,12 @@
|
|||||||
# file: Cargo.toml
|
# file: Cargo.toml
|
||||||
# version: 155
|
# version: 156
|
||||||
|
|
||||||
[workspace]
|
[workspace]
|
||||||
resolver = "3"
|
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"]
|
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]
|
[workspace.package]
|
||||||
version = "0.2.5-pre.7"
|
version = "0.2.5-pre.7.fix.1"
|
||||||
edition = "2024"
|
edition = "2024"
|
||||||
license = "MIT"
|
license = "MIT"
|
||||||
repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project"
|
repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project"
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-wallet-lib/src/persistence.rs
|
// file: crates/ksp-wallet-lib/src/persistence.rs
|
||||||
// version: 2
|
// version: 3
|
||||||
|
|
||||||
//! Async-first native Wallet V1 filesystem persistence.
|
//! 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));
|
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<()> {
|
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(()));
|
return replace_wallet_file_with_hook(destination, content, || return std::result::Result::Ok(()));
|
||||||
}
|
}
|
||||||
|
|||||||
97
deltas/0.2.5/pre.007-fix.001.md
Normal file
97
deltas/0.2.5/pre.007-fix.001.md
Normal file
@@ -0,0 +1,97 @@
|
|||||||
|
<!-- file: deltas/0.2.5/pre.007-fix.001.md -->
|
||||||
|
<!-- version: 1 -->
|
||||||
|
|
||||||
|
# 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
|
||||||
|
```
|
||||||
Reference in New Issue
Block a user