diff --git a/Cargo.toml b/Cargo.toml index 2e222f6..639da85 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -1,12 +1,12 @@ # file: Cargo.toml -# version: 205 +# version: 206 [workspace] resolver = "3" members = ["crates/ksp-app-config-desk", "crates/ksp-app-wallet-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.7-pre.9" +version = "0.2.7-pre.9.fix.1" edition = "2024" license = "MIT" repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project" diff --git a/crates/ksp-onchain-transport-lib/src/ws_accounts.rs b/crates/ksp-onchain-transport-lib/src/ws_accounts.rs index 4d01057..9dc0027 100644 --- a/crates/ksp-onchain-transport-lib/src/ws_accounts.rs +++ b/crates/ksp-onchain-transport-lib/src/ws_accounts.rs @@ -1,5 +1,5 @@ // file: crates/ksp-onchain-transport-lib/src/ws_accounts.rs -// version: 1 +// version: 2 const MAX_PROGRAM_SUBSCRIBE_FILTERS: usize = 4; const MAX_PROGRAM_SUBSCRIBE_RAW_MEMCMP_BYTES: usize = 128; @@ -163,7 +163,9 @@ impl crate::WsSession { { params.push(config.to_json_value()); } - return self.subscribe_typed(crate::WsSubscriptionKind::Account, params, |value| decode_account_notification("accountSubscribe", value)).await; + return self + .subscribe_typed(crate::WsSubscriptionKind::Account, params, |value| return decode_account_notification("accountSubscribe", value)) + .await; } /// Subscribes to account changes owned by one Solana program through standard `programSubscribe`. @@ -184,7 +186,9 @@ impl crate::WsSession { { params.push(config.to_json_value()); } - return self.subscribe_typed(crate::WsSubscriptionKind::Program, params, |value| decode_program_notification("programSubscribe", value)).await; + return self + .subscribe_typed(crate::WsSubscriptionKind::Program, params, |value| return decode_program_notification("programSubscribe", value)) + .await; } } diff --git a/crates/ksp-onchain-transport-lib/src/ws_transactions.rs b/crates/ksp-onchain-transport-lib/src/ws_transactions.rs index 4f227c3..a0fc2d4 100644 --- a/crates/ksp-onchain-transport-lib/src/ws_transactions.rs +++ b/crates/ksp-onchain-transport-lib/src/ws_transactions.rs @@ -1,5 +1,5 @@ // file: crates/ksp-onchain-transport-lib/src/ws_transactions.rs -// version: 1 +// version: 2 /// Filter accepted by the standard Solana `logsSubscribe` WebSocket method. #[derive(Clone, Debug, Eq, PartialEq)] @@ -63,7 +63,7 @@ impl crate::WsSession { { params.push(config.to_json_value()); } - return self.subscribe_typed(crate::WsSubscriptionKind::Logs, params, |value| decode_logs_notification("logsSubscribe", value)).await; + return self.subscribe_typed(crate::WsSubscriptionKind::Logs, params, |value| return decode_logs_notification("logsSubscribe", value)).await; } } diff --git a/deltas/0.2.7/pre.009-fix.001.md b/deltas/0.2.7/pre.009-fix.001.md new file mode 100644 index 0000000..3cc98ad --- /dev/null +++ b/deltas/0.2.7/pre.009-fix.001.md @@ -0,0 +1,81 @@ + + + +# Delta `0.2.7-pre.009-fix.001` — conformité Clippy des closures de décodage WebSocket + +## Base + +Base directe publiée puis validée partiellement par l'opérateur : + +```text +0.2.7-pre.009 +Cargo 0.2.7-pre.9 +``` + +Le checkpoint opérateur a confirmé `cargo fmt --all`, l'audit Python KSP et `cargo check --workspace`. `cargo clippy --workspace --all-targets` échoue uniquement sur trois violations de `clippy::implicit-return` dans les closures de décodage des nouveaux wrappers WebSocket. Les tests exécutés malgré cet échec sont verts : 294 tests unitaires Transport, 33 tests d'API publique, 22 tests de release completeness et le reste du workspace. + +## Signal technique + +```text +livraison = 0.2.7-pre.009-fix.001 +workspace.package.version = 0.2.7-pre.9.fix.1 +commit = v0.2.7-pre.009-fix.001 +``` + +Aucun tag Git n'est requis pour ce fix de prerelease. + +## Diagnostic + +Les trois nouveaux wrappers passent leur décodeur à `WsSession::subscribe_typed` via une closure. Le workspace impose `-D clippy::implicit-return`, y compris dans les closures. Les corps suivants utilisaient encore un retour implicite : + +```text +accountSubscribe -> decode_account_notification +programSubscribe -> decode_program_notification +logsSubscribe -> decode_logs_notification +``` + +Le problème est purement syntaxique. Les 294 tests Transport passent déjà sur la livraison `pre.009`, donc aucune correction fonctionnelle ou protocolaire n'est requise. + +## Correction + +Les trois closures utilisent maintenant un `return` explicite : + +```rust +|value| return decode_...(method, value) +``` + +Aucun DTO, paramètre JSON-RPC, décodage, état de subscription, remote ID, reconnect, backpressure ou contrat public n'est modifié. + +Les headers des deux fichiers Rust modifiés sont incrémentés conformément au contrat de fichiers. Le signal Cargo devient `0.2.7-pre.9.fix.1`. + +## Fichiers + +```text +Cargo.toml +crates/ksp-onchain-transport-lib/src/ws_accounts.rs +crates/ksp-onchain-transport-lib/src/ws_transactions.rs +deltas/0.2.7/pre.009-fix.001.md +``` + +`ROADMAP.md`, `CHANGELOG.md`, le plan et le document de validation restent inchangés. + +## Validation attendue + +```bash +cargo fmt --all +python3 scripts/audit_rust_workspace_rules.py +cargo check --workspace +cargo clippy --workspace --all-targets +cargo test -p ksp-onchain-transport-lib +cargo test --workspace +``` + +Comptages Transport attendus inchangés : + +```text +unit tests = 294 +public API tests = 33 +release completeness = 22 +``` + +Les validations Cargo doivent être exécutées par l'opérateur ; elles ne sont pas revendiquées depuis l'environnement de préparation du delta.