v0.2.7-pre.009-fix.001
This commit is contained in:
@@ -1,12 +1,12 @@
|
|||||||
# file: Cargo.toml
|
# file: Cargo.toml
|
||||||
# version: 205
|
# version: 206
|
||||||
|
|
||||||
[workspace]
|
[workspace]
|
||||||
resolver = "3"
|
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"]
|
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]
|
[workspace.package]
|
||||||
version = "0.2.7-pre.9"
|
version = "0.2.7-pre.9.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-onchain-transport-lib/src/ws_accounts.rs
|
// 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_FILTERS: usize = 4;
|
||||||
const MAX_PROGRAM_SUBSCRIBE_RAW_MEMCMP_BYTES: usize = 128;
|
const MAX_PROGRAM_SUBSCRIBE_RAW_MEMCMP_BYTES: usize = 128;
|
||||||
@@ -163,7 +163,9 @@ impl crate::WsSession {
|
|||||||
{
|
{
|
||||||
params.push(config.to_json_value());
|
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`.
|
/// 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());
|
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;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-onchain-transport-lib/src/ws_transactions.rs
|
// file: crates/ksp-onchain-transport-lib/src/ws_transactions.rs
|
||||||
// version: 1
|
// version: 2
|
||||||
|
|
||||||
/// Filter accepted by the standard Solana `logsSubscribe` WebSocket method.
|
/// Filter accepted by the standard Solana `logsSubscribe` WebSocket method.
|
||||||
#[derive(Clone, Debug, Eq, PartialEq)]
|
#[derive(Clone, Debug, Eq, PartialEq)]
|
||||||
@@ -63,7 +63,7 @@ impl crate::WsSession {
|
|||||||
{
|
{
|
||||||
params.push(config.to_json_value());
|
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;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
81
deltas/0.2.7/pre.009-fix.001.md
Normal file
81
deltas/0.2.7/pre.009-fix.001.md
Normal file
@@ -0,0 +1,81 @@
|
|||||||
|
<!-- file: deltas/0.2.7/pre.009-fix.001.md -->
|
||||||
|
<!-- version: 1 -->
|
||||||
|
|
||||||
|
# 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.
|
||||||
Reference in New Issue
Block a user