v0.3.7-pre.006-fix.001
This commit is contained in:
@@ -1,12 +1,12 @@
|
||||
# file: Cargo.toml
|
||||
# version: 423
|
||||
# version: 424
|
||||
|
||||
[workspace]
|
||||
resolver = "3"
|
||||
members = ["crates/ksp-app-backfill-desk", "crates/ksp-app-config-desk", "crates/ksp-app-solprices-desk", "crates/ksp-app-wallet-desk", "crates/ksp-config-lib", "crates/ksp-core-lib", "crates/ksp-interface-lib", "crates/ksp-job-api", "crates/ksp-job-backfill-lib", "crates/ksp-logging-lib", "crates/ksp-offchain-transport-lib", "crates/ksp-onchain-transport-lib", "crates/ksp-program-api", "crates/ksp-store-api", "crates/ksp-store-lib", "crates/ksp-store-postgres-lib", "crates/ksp-wallet-lib"]
|
||||
|
||||
[workspace.package]
|
||||
version = "0.3.7-pre.6"
|
||||
version = "0.3.7-pre.6.fix.1"
|
||||
edition = "2024"
|
||||
license = "MIT"
|
||||
repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project"
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
// file: crates/ksp-app-backfill-desk/tests/config_composition.rs
|
||||
// version: 5
|
||||
// version: 6
|
||||
|
||||
//! Config composite, Transport readiness and Store-network composition contracts for Backfill Desk through pre.006.
|
||||
|
||||
@@ -252,13 +252,13 @@ fn pre_006_default_mainnet_profile_exposes_pool_and_targeted_http_routes_without
|
||||
};
|
||||
let providers = endpoints
|
||||
.iter()
|
||||
.filter_map(|endpoint| endpoint.get("provider").and_then(serde_json::Value::as_str))
|
||||
.filter_map(|endpoint| return endpoint.get("provider").and_then(serde_json::Value::as_str))
|
||||
.collect::<std::collections::BTreeSet<_>>();
|
||||
assert_eq!(providers, std::collections::BTreeSet::from(["publicnode", "solana-public"]));
|
||||
let roles = endpoints
|
||||
.iter()
|
||||
.flat_map(|endpoint| endpoint.get("roles").and_then(serde_json::Value::as_array).into_iter().flatten())
|
||||
.filter_map(|role| role.get("role").and_then(serde_json::Value::as_str))
|
||||
.flat_map(|endpoint| return endpoint.get("roles").and_then(serde_json::Value::as_array).into_iter().flatten())
|
||||
.filter_map(|role| return role.get("role").and_then(serde_json::Value::as_str))
|
||||
.collect::<std::collections::BTreeSet<_>>();
|
||||
assert!(roles.contains("backfill_pool"));
|
||||
assert!(roles.contains("backfill_publicnode"));
|
||||
|
||||
98
deltas/0.3.7/pre.006-fix.001.md
Normal file
98
deltas/0.3.7/pre.006-fix.001.md
Normal file
@@ -0,0 +1,98 @@
|
||||
<!-- file: deltas/0.3.7/pre.006-fix.001.md -->
|
||||
<!-- version: 1 -->
|
||||
|
||||
# Delta `0.3.7-pre.006-fix.001` — Conformité Clippy du canari HTTP Mainnet
|
||||
|
||||
## 1. Base requise
|
||||
|
||||
```text
|
||||
0.3.7-pre.006
|
||||
workspace.package.version = 0.3.7-pre.6
|
||||
```
|
||||
|
||||
## 2. Objectif
|
||||
|
||||
Corriger l'unique blocage du gate opérateur `pre.006` : trois closures du test de composition Mainnet/pool HTTP doivent respecter le lint workspace `clippy::implicit_return`.
|
||||
|
||||
## 3. Cause
|
||||
|
||||
Le test `pre_006_default_mainnet_profile_exposes_pool_and_targeted_http_routes_without_secret_provider_data` utilisait trois closures expression-only dans les chaînes `filter_map`/`flat_map`. Le workspace compile Clippy avec `-D clippy::implicit-return`, donc ces closures doivent utiliser un `return` explicite même si leur comportement Rust est sinon valide.
|
||||
|
||||
## 4. Correction
|
||||
|
||||
- ajouter `return` dans la closure qui lit `provider` ;
|
||||
- ajouter `return` dans la closure qui lit `roles` ;
|
||||
- ajouter `return` dans la closure qui lit le nom logique `role` ;
|
||||
- ne modifier aucun profil Config, endpoint, rôle Transport, DTO, frontend, runtime Store/Transport ou contrat Job.
|
||||
|
||||
## 5. Version
|
||||
|
||||
Le correctif modifie un fichier Rust compilé. Conformément à `VER-ID-007` et `VER-ID-010` :
|
||||
|
||||
```text
|
||||
workspace.package.version = 0.3.7-pre.6.fix.1
|
||||
label = 0.3.7-pre.006-fix.001
|
||||
```
|
||||
|
||||
## 6. Fichiers ajoutés
|
||||
|
||||
```text
|
||||
deltas/0.3.7/pre.006-fix.001.md
|
||||
```
|
||||
|
||||
## 7. Fichiers modifiés
|
||||
|
||||
```text
|
||||
Cargo.toml
|
||||
crates/ksp-app-backfill-desk/tests/config_composition.rs
|
||||
docs/validation/024-V0_3_7_BACKFILL_DESK.md
|
||||
```
|
||||
|
||||
## 8. Fichiers supprimés
|
||||
|
||||
Aucun.
|
||||
|
||||
## 9. Gate opérateur ayant déclenché le fix
|
||||
|
||||
```text
|
||||
cargo fmt --all exécuté
|
||||
python3 scripts/audit_rust_workspace_rules.py PASS
|
||||
python3 scripts/audit_markdown_tables.py ... deltas/0.3.7 PASS
|
||||
cargo check --workspace PASS
|
||||
cargo clippy --workspace --all-targets FAIL — 3 x implicit_return dans config_composition.rs
|
||||
cargo test -p ksp-config-lib PASS
|
||||
cargo test -p ksp-app-backfill-desk PASS sur les suites exécutées après l'arrêt Clippy
|
||||
```
|
||||
|
||||
## 10. Validations exécutées dans l'environnement d'assemblage
|
||||
|
||||
```text
|
||||
python3 scripts/audit_rust_workspace_rules.py
|
||||
python3 scripts/audit_markdown_tables.py README.md RULES.md ROADMAP.md CHANGELOG.md docs prompts crates deltas/0.3.7
|
||||
```
|
||||
|
||||
Contrôles ciblés :
|
||||
|
||||
- les trois closures signalées contiennent un `return` explicite ;
|
||||
- le diff fonctionnel de `pre.006` hors test/version/validation/delta est vide ;
|
||||
- aucun fichier Config/Transport/frontend/runtime n'est modifié.
|
||||
|
||||
## 11. Validations non exécutées dans l'environnement d'assemblage
|
||||
|
||||
`cargo` et `rustfmt` ne sont pas disponibles dans l'environnement d'assemblage. Gate opérateur :
|
||||
|
||||
```text
|
||||
cargo fmt --all
|
||||
python3 scripts/audit_rust_workspace_rules.py
|
||||
python3 scripts/audit_markdown_tables.py README.md RULES.md ROADMAP.md CHANGELOG.md docs prompts crates deltas/0.3.7
|
||||
cargo check --workspace
|
||||
cargo clippy --workspace --all-targets
|
||||
cargo test -p ksp-config-lib
|
||||
cargo test -p ksp-app-backfill-desk
|
||||
cargo tree -p ksp-app-backfill-desk --edges normal
|
||||
cargo tree -p ksp-app-backfill-desk -e features
|
||||
```
|
||||
|
||||
## 12. Questions ouvertes
|
||||
|
||||
Aucune pour ce correctif.
|
||||
@@ -1,5 +1,5 @@
|
||||
<!-- file: docs/validation/024-V0_3_7_BACKFILL_DESK.md -->
|
||||
<!-- version: 7 -->
|
||||
<!-- version: 8 -->
|
||||
|
||||
# Validation v0.3.7 — Backfill Desk
|
||||
|
||||
@@ -288,3 +288,17 @@ Le support Yellowstone gRPC de `ksp-job-backfill-lib`/Backfill Desk est retiré
|
||||
- [X] aucun Start ni dépendance `ksp-job-backfill-lib` n'est ouvert par la Desk ;
|
||||
- [X] audits statiques Rust/Markdown, JSON Schema et syntaxe TypeScript propres dans l'environnement d'assemblage ;
|
||||
- [ ] replay opérateur `cargo fmt/check/clippy/test` de `pre.006` à exécuter.
|
||||
|
||||
## Correctif `pre.006-fix.001`
|
||||
|
||||
Le gate opérateur de `pre.006` confirme que les audits Rust/Markdown et `cargo check --workspace` sont propres, puis Clippy isole trois closures de `tests/config_composition.rs` qui violaient `clippy::implicit_return`. Les tests exécutés après cet arrêt restent propres, mais le gate global est bloqué tant que Clippy échoue.
|
||||
|
||||
Le correctif reste strictement dans la responsabilité de `pre.006` : il ajoute uniquement les `return` explicites exigés par les lints workspace dans le canari Mainnet/pool HTTP. Aucun profil Config, rôle Transport, DTO, frontend, runtime Store/Transport ou contrat Job n'est modifié.
|
||||
|
||||
### Gate du fix
|
||||
|
||||
- [X] trois closures `filter_map`/`flat_map` utilisent désormais un `return` explicite ;
|
||||
- [X] aucun contenu fonctionnel de `pre.006` n'est modifié ;
|
||||
- [X] version Cargo synchronisée en `0.3.7-pre.6.fix.1` conformément à `VER-ID-007` et `VER-ID-010` ;
|
||||
- [X] audits statiques Rust/Markdown rejoués dans l'environnement d'assemblage ;
|
||||
- [ ] replay opérateur `cargo fmt/check/clippy/test` du fix à exécuter.
|
||||
|
||||
Reference in New Issue
Block a user