v0.2.9-pre.004-fix.001
This commit is contained in:
@@ -1,12 +1,12 @@
|
||||
# file: Cargo.toml
|
||||
# version: 243
|
||||
# version: 244
|
||||
|
||||
[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.9-pre.4"
|
||||
version = "0.2.9-pre.4.fix.1"
|
||||
edition = "2024"
|
||||
license = "MIT"
|
||||
repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project"
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
// file: crates/ksp-onchain-transport-lib/src/grpc_subscribe.rs
|
||||
// version: 1
|
||||
// version: 2
|
||||
|
||||
const MAX_GRPC_SUBSCRIBE_DATA_SLICE_COUNT: usize = 128;
|
||||
const MAX_GRPC_SUBSCRIBE_DATA_SLICE_LENGTH_BYTES: u64 = 64 * 1024 * 1024;
|
||||
@@ -72,6 +72,7 @@ impl YellowstoneAccountsDataSlice {
|
||||
return self.length;
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(self) -> yellowstone_grpc_proto::geyser::SubscribeRequestAccountsDataSlice {
|
||||
return yellowstone_grpc_proto::geyser::SubscribeRequestAccountsDataSlice { offset: self.offset, length: self.length };
|
||||
}
|
||||
@@ -96,6 +97,7 @@ impl YellowstoneSubscribePing {
|
||||
return self.id;
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(self) -> yellowstone_grpc_proto::geyser::SubscribeRequestPing {
|
||||
return yellowstone_grpc_proto::geyser::SubscribeRequestPing { id: self.id };
|
||||
}
|
||||
@@ -116,6 +118,7 @@ impl YellowstoneSubscribeAccountFilter {
|
||||
return Self { _private: () };
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(&self) -> yellowstone_grpc_proto::geyser::SubscribeRequestFilterAccounts {
|
||||
return yellowstone_grpc_proto::geyser::SubscribeRequestFilterAccounts::default();
|
||||
}
|
||||
@@ -136,6 +139,7 @@ impl YellowstoneSubscribeSlotFilter {
|
||||
return Self { _private: () };
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(&self) -> yellowstone_grpc_proto::geyser::SubscribeRequestFilterSlots {
|
||||
return yellowstone_grpc_proto::geyser::SubscribeRequestFilterSlots::default();
|
||||
}
|
||||
@@ -156,6 +160,7 @@ impl YellowstoneSubscribeTransactionFilter {
|
||||
return Self { _private: () };
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(&self) -> yellowstone_grpc_proto::geyser::SubscribeRequestFilterTransactions {
|
||||
return yellowstone_grpc_proto::geyser::SubscribeRequestFilterTransactions::default();
|
||||
}
|
||||
@@ -176,6 +181,7 @@ impl YellowstoneSubscribeBlockFilter {
|
||||
return Self { _private: () };
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(&self) -> yellowstone_grpc_proto::geyser::SubscribeRequestFilterBlocks {
|
||||
return yellowstone_grpc_proto::geyser::SubscribeRequestFilterBlocks::default();
|
||||
}
|
||||
@@ -194,6 +200,7 @@ impl YellowstoneSubscribeBlocksMetaFilter {
|
||||
return Self { _private: () };
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(self) -> yellowstone_grpc_proto::geyser::SubscribeRequestFilterBlocksMeta {
|
||||
return yellowstone_grpc_proto::geyser::SubscribeRequestFilterBlocksMeta::default();
|
||||
}
|
||||
@@ -212,6 +219,7 @@ impl YellowstoneSubscribeEntryFilter {
|
||||
return Self { _private: () };
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(self) -> yellowstone_grpc_proto::geyser::SubscribeRequestFilterEntry {
|
||||
return yellowstone_grpc_proto::geyser::SubscribeRequestFilterEntry::default();
|
||||
}
|
||||
@@ -465,6 +473,7 @@ impl YellowstoneSubscribeRequest {
|
||||
return std::result::Result::Ok(());
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn to_wire(&self) -> ksp_core_lib::Result<yellowstone_grpc_proto::geyser::SubscribeRequest> {
|
||||
let validation = self.validate();
|
||||
if let std::result::Result::Err(error) = validation {
|
||||
@@ -522,6 +531,7 @@ impl YellowstoneSubscribeRequest {
|
||||
}
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
fn commitment_to_wire(commitment: std::option::Option<crate::SolanaCommitment>) -> std::option::Option<i32> {
|
||||
return commitment.map(|value| {
|
||||
return match value {
|
||||
|
||||
64
deltas/0.2.9/pre.004-fix.001.md
Normal file
64
deltas/0.2.9/pre.004-fix.001.md
Normal file
@@ -0,0 +1,64 @@
|
||||
<!-- file: deltas/0.2.9/pre.004-fix.001.md -->
|
||||
<!-- version: 1 -->
|
||||
|
||||
# Delta `0.2.9-pre.004-fix.001` — suppression des warnings `dead_code` des conversions wire test-only
|
||||
|
||||
## 1. Déclencheur
|
||||
|
||||
Le premier gate opérateur `pre.004` est fonctionnellement vert :
|
||||
|
||||
```text
|
||||
cargo fmt --all PASS
|
||||
python3 scripts/audit_rust_workspace_rules.py PASS / clean
|
||||
cargo check --workspace PASS + 10 warnings dead_code
|
||||
cargo clippy --workspace --all-targets PASS + 10 warnings dead_code
|
||||
Transport unit 359/359 PASS
|
||||
Transport public_api 44/44 PASS
|
||||
Transport release_completeness 37/37 PASS
|
||||
Transport doctests 4/4 PASS
|
||||
Core workspace_dependencies 3/3 PASS
|
||||
cargo test --workspace PASS + mêmes warnings de compilation
|
||||
```
|
||||
|
||||
Les dix warnings concernent exclusivement les conversions protobuf privées de `grpc_subscribe.rs` (`to_wire` et `commitment_to_wire`).
|
||||
|
||||
## 2. Diagnostic
|
||||
|
||||
`pre.004` matérialise le contrat `SubscribeRequest` et vérifie son wire dans les unit tests, mais le stream bidi n'est volontairement ouvert qu'en `pre.008`. Ces helpers ont donc un consommateur test aujourd'hui et aucun consommateur runtime encore.
|
||||
|
||||
Contrairement au channel N1 de `pre.002`, ces fonctions ne représentent pas un état runtime devant exister avant leur consommation. Les compiler en production maintenant ne fait qu'introduire du code mort temporaire.
|
||||
|
||||
## 3. Correction
|
||||
|
||||
```text
|
||||
#[cfg(test)] sur les 9 méthodes to_wire de grpc_subscribe.rs
|
||||
#[cfg(test)] sur commitment_to_wire
|
||||
aucune modification des types publics
|
||||
aucune modification des validations/bornes
|
||||
aucune modification du protobuf produit par les tests
|
||||
aucune dépendance/feature Cargo modifiée
|
||||
```
|
||||
|
||||
Lors de `pre.008`, quand la session bidi consommera réellement ces conversions, le `cfg(test)` sera retiré des helpers nécessaires au runtime.
|
||||
|
||||
## 4. Version
|
||||
|
||||
```text
|
||||
workspace.package.version = 0.2.9-pre.4.fix.1
|
||||
commit attendu = v0.2.9-pre.004-fix.001
|
||||
aucun tag prerelease
|
||||
```
|
||||
|
||||
## 5. Validation requise
|
||||
|
||||
```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 -p ksp-core-lib --test workspace_dependencies
|
||||
cargo test --workspace
|
||||
```
|
||||
|
||||
Aucun `cargo tree` supplémentaire n'est requis : le fix ne modifie ni dépendance ni feature.
|
||||
@@ -1,9 +1,9 @@
|
||||
<!-- file: docs/plans/016-V0_2_9_YELLOWSTONE_GRPC_PLAN.md -->
|
||||
<!-- version: 10 -->
|
||||
<!-- version: 11 -->
|
||||
|
||||
# Plan `0.2.9` — moteur Yellowstone gRPC + standard Solana + PublicNode
|
||||
|
||||
> **Statut : `0.2.9-pre.004` candidate. `pre.003-fix.001` est fermée sur gate opérateur intégralement vert : fmt/audit/check/Clippy/workspace PASS, Transport 354 unit + 43 public API + 36 completeness + 4 doctests, et graphes Cargo inspectés sans seconde version Tonic/Prost/Solana. `pre.004` matérialise uniquement la fondation `SubscribeRequest` standard : sept maps nommées, commitment, account-data slices ordonnées, ping, `from_slot`, bornes communes, unicité globale des noms et Debug sans payload arbitraire. Les champs détaillés Accounts/Slots/Transactions/Blocks restent réservés à `pre.005–007` et aucun stream bidi n'est encore ouvert. PublicNode et Config V3 restent hors tranche. `0.2.9` reste bornée à un moteur client Yellowstone partagé, une façade Solana Yellowstone standard et une première intégration concrète PublicNode. Seuls OrbitFlare puis Helius LaserStream gRPC sont actuellement planifiés comme releases provider suivantes ; les autres providers restent en TODO/IDEAS sans numéro réservé. Chaque prerelease vise 15–20 minutes de travail effectif et la release complète doit rester clôturable dans une seule session de chat.**
|
||||
> **Statut : `0.2.9-pre.004-fix.001` candidate. Le premier gate opérateur de `pre.004` est fonctionnellement vert (359 unit + 44 public API + 37 completeness + 4 doctests, workspace PASS) mais `cargo check`/Clippy/workspace émettent 10 warnings `dead_code` sur les conversions protobuf encore utilisées uniquement par les tests. Le fix compile ces helpers sous `#[cfg(test)]` jusqu'à leur consommation runtime lors de l'ouverture du stream. Aucun contrat public, filtre ou wire n'est modifié. `pre.003-fix.001` est fermée sur gate opérateur intégralement vert : fmt/audit/check/Clippy/workspace PASS, Transport 354 unit + 43 public API + 36 completeness + 4 doctests, et graphes Cargo inspectés sans seconde version Tonic/Prost/Solana. `pre.004` matérialise uniquement la fondation `SubscribeRequest` standard : sept maps nommées, commitment, account-data slices ordonnées, ping, `from_slot`, bornes communes, unicité globale des noms et Debug sans payload arbitraire. Les champs détaillés Accounts/Slots/Transactions/Blocks restent réservés à `pre.005–007` et aucun stream bidi n'est encore ouvert. PublicNode et Config V3 restent hors tranche. `0.2.9` reste bornée à un moteur client Yellowstone partagé, une façade Solana Yellowstone standard et une première intégration concrète PublicNode. Seuls OrbitFlare puis Helius LaserStream gRPC sont actuellement planifiés comme releases provider suivantes ; les autres providers restent en TODO/IDEAS sans numéro réservé. Chaque prerelease vise 15–20 minutes de travail effectif et la release complète doit rester clôturable dans une seule session de chat.**
|
||||
|
||||
## 1. Objet, base et état d'ouverture
|
||||
|
||||
@@ -919,8 +919,8 @@ pre.002 DONE — moteur Yellowstone : proto/dependencies + settings/errors + ch
|
||||
pre.003 DONE — moteur TLS/metadata + façade N2 unary + fixture locale + 7 unary RPCs
|
||||
budget : 15–20 min ; gate final fix.001 : fmt/audit/check/Clippy/workspace PASS + graphes Cargo inspectés
|
||||
|
||||
pre.004 CANDIDATE — standard Solana : Subscribe foundation + maps/commitment/ping/from_slot/data slices/bounds
|
||||
budget : 15–20 min ; preuve : map vide/entrée nommée vide/common wire exact + rejets déterministes avant I/O
|
||||
pre.004 FIX.001 CANDIDATE — standard Solana : Subscribe foundation + maps/commitment/ping/from_slot/data slices/bounds
|
||||
budget : 15–20 min ; premier gate fonctionnel PASS, fix warning-only sur helpers wire test-only
|
||||
|
||||
pre.005 standard Solana : Accounts + Slots filters/updates
|
||||
budget : 15–20 min ; preuve : fixtures exactes + enum/optional/malformed/adversarial
|
||||
|
||||
@@ -1,9 +1,9 @@
|
||||
<!-- file: docs/validation/012-V0_2_9_YELLOWSTONE_GRPC.md -->
|
||||
<!-- version: 11 -->
|
||||
<!-- version: 12 -->
|
||||
|
||||
# Validation `0.2.9` — moteur Yellowstone + standard Solana + PublicNode
|
||||
|
||||
> **Statut : `pre.003-fix.001` est fermé sur gate opérateur intégralement vert et graphes Cargo inspectés. `0.2.9-pre.004` est candidate et ajoute uniquement la fondation standard `SubscribeRequest` : sept maps, commitment, account-data slices, ping, `from_slot`, bornes communes et redaction Debug. Aucun stream bidi n'est encore ouvert ; les filtres détaillés Accounts/Slots/Transactions/Blocks restent réservés à `pre.005–007`. PublicNode et Config V3 restent hors tranche. OrbitFlare et Helius sont les seules releases provider suivantes planifiées ; les autres providers restent en TODO/IDEAS sans numéro réservé.**
|
||||
> **Statut : `pre.003-fix.001` est fermé sur gate opérateur intégralement vert et graphes Cargo inspectés. `0.2.9-pre.004-fix.001` est candidate après un premier gate `pre.004` fonctionnellement vert mais non silencieux : 10 warnings `dead_code` concernent uniquement les conversions protobuf privées encore utilisées seulement par les unit tests. Le fix les place sous `#[cfg(test)]` sans changement de runtime/public API. `0.2.9-pre.004` ajoute uniquement la fondation standard `SubscribeRequest` : sept maps, commitment, account-data slices, ping, `from_slot`, bornes communes et redaction Debug. Aucun stream bidi n'est encore ouvert ; les filtres détaillés Accounts/Slots/Transactions/Blocks restent réservés à `pre.005–007`. PublicNode et Config V3 restent hors tranche. OrbitFlare et Helius sont les seules releases provider suivantes planifiées ; les autres providers restent en TODO/IDEAS sans numéro réservé.**
|
||||
|
||||
## 1. Autorités du gate
|
||||
|
||||
@@ -631,3 +631,24 @@ Graphe pertinent confirmé : `tonic 0.14.6`, `tonic-prost 0.14.6`, `prost/prost-
|
||||
Le proto publié `yellowstone-grpc-proto 12.6.0` a été recontrôlé avant implémentation : les onze champs top-level retenus restent `accounts`, `slots`, `transactions`, `transactions_status`, `blocks`, `blocks_meta`, `entry`, `commitment`, `accounts_data_slice`, `ping`, `from_slot`. Aucun `SubscribeDeshred` n'entre dans la surface KSP.
|
||||
|
||||
**Verdict `pre.004` : candidate source prête ; fermeture après gate Cargo opérateur.**
|
||||
|
||||
|
||||
### 19.1 Premier gate opérateur `pre.004` et `pre.004-fix.001`
|
||||
|
||||
| Gate / observation | Résultat `pre.004` | État `fix.001` |
|
||||
|------------------------------------------|----------------------------------------|--------------------------------------------|
|
||||
| workspace version | `0.2.9-pre.4` | `0.2.9-pre.4.fix.1` |
|
||||
| `cargo fmt --all` | PASS | à réexécuter |
|
||||
| audit Rust workspace | PASS / clean | audit local PASS / clean |
|
||||
| `cargo check --workspace` | PASS + 10 warnings `dead_code` | helpers wire test-only sous `#[cfg(test)]` |
|
||||
| `cargo clippy --workspace --all-targets` | PASS + mêmes 10 warnings | idem |
|
||||
| Transport unit | 359/359 PASS | inchangé |
|
||||
| Transport `public_api` | 44/44 PASS | inchangé |
|
||||
| Transport `release_completeness` | 37/37 PASS | inchangé |
|
||||
| Transport doctests | 4/4 PASS | inchangé |
|
||||
| Core dependency canary | 3/3 PASS | inchangé |
|
||||
| `cargo test --workspace` | PASS + mêmes warnings à la compilation | à réexécuter sans warning |
|
||||
|
||||
Les dix symboles concernés (`to_wire` et `commitment_to_wire`) servent uniquement aux tests de projection protobuf de `pre.004`. Le stream bidi n'étant pas ouvert avant `pre.008`, ils ne font pas encore partie du runtime. `#[cfg(test)]` évite donc un faux code mort de production sans créer une seconde implémentation ni modifier le contrat public. Lors de l'intégration runtime du stream, ces helpers seront naturellement retirés du `cfg(test)` au moment où ils auront un consommateur de production.
|
||||
|
||||
**Verdict `pre.004-fix.001` : correctif warning-only prêt ; fermeture de `pre.004` après gate opérateur sans warning.**
|
||||
|
||||
Reference in New Issue
Block a user