From c43d4d7c38f3cf574ffae1f410a6c2ae1ed98084 Mon Sep 17 00:00:00 2001 From: SinuS Von SifriduS Date: Sat, 29 Aug 2026 09:12:38 +0200 Subject: [PATCH] v0.3.1-pre.006-fix.001 --- Cargo.toml | 2 +- .../ksp-store-api/src/model/raw_pagination.rs | 8 +- .../tests/dependency_boundary.rs | 2 +- deltas/0.3.1/pre.006-fix.001.md | 95 +++++++++++++++++++ 4 files changed, 101 insertions(+), 6 deletions(-) create mode 100644 deltas/0.3.1/pre.006-fix.001.md diff --git a/Cargo.toml b/Cargo.toml index 9a85252..c1220b9 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -6,7 +6,7 @@ resolver = "3" members = ["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-logging-lib", "crates/ksp-offchain-transport-lib", "crates/ksp-onchain-transport-lib", "crates/ksp-program-api", "crates/ksp-store-api", "crates/ksp-wallet-lib"] [workspace.package] -version = "0.3.1-pre.6" +version = "0.3.1-pre.6-fix.1" edition = "2024" license = "MIT" repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project" diff --git a/crates/ksp-store-api/src/model/raw_pagination.rs b/crates/ksp-store-api/src/model/raw_pagination.rs index 2d8ffdb..0c19938 100644 --- a/crates/ksp-store-api/src/model/raw_pagination.rs +++ b/crates/ksp-store-api/src/model/raw_pagination.rs @@ -142,10 +142,10 @@ pub struct RawSlotRange { impl RawSlotRange { /// Creates one optional inclusive slot range and rejects reversed bounds. pub fn new(start_inclusive: std::option::Option, end_inclusive: std::option::Option) -> crate::Result { - if let (std::option::Option::Some(start), std::option::Option::Some(end)) = (start_inclusive, end_inclusive) { - if start > end { - return std::result::Result::Err(raw_query_error("slot_range")); - } + if let (std::option::Option::Some(start), std::option::Option::Some(end)) = (start_inclusive, end_inclusive) + && start > end + { + return std::result::Result::Err(raw_query_error("slot_range")); } return std::result::Result::Ok(Self { end_inclusive, start_inclusive }); } diff --git a/crates/ksp-store-api/tests/dependency_boundary.rs b/crates/ksp-store-api/tests/dependency_boundary.rs index 2ebf639..5d75f6f 100644 --- a/crates/ksp-store-api/tests/dependency_boundary.rs +++ b/crates/ksp-store-api/tests/dependency_boundary.rs @@ -124,7 +124,7 @@ fn pre_006_source_boundary_keeps_models_and_capabilities_backend_free() { assert!(!raw_retention_capability.contains(forbidden)); assert!(!raw_transaction_capability.contains(forbidden)); } - assert!(raw_pagination.contains("u64::MAX") == false); + assert!(!raw_pagination.contains("u64::MAX")); assert!(!raw_pagination.contains("MAX_RAW_PAGE_ITEMS")); return; } diff --git a/deltas/0.3.1/pre.006-fix.001.md b/deltas/0.3.1/pre.006-fix.001.md new file mode 100644 index 0000000..4e3874c --- /dev/null +++ b/deltas/0.3.1/pre.006-fix.001.md @@ -0,0 +1,95 @@ + + + +# Delta `0.3.1-pre.006-fix.001` — conformité Clippy pagination + +## Base + +```text +0.3.1-pre.006 +``` + +Le gate opérateur de `pre.006` confirme : + +- `cargo fmt --all` propre ; +- audits Rust/Markdown propres ; +- `cargo check --workspace` propre ; +- `cargo test -p ksp-store-api` propre : 14 unitaires, 2 dependency-boundary, 1 external-backend, 6 public-api ; +- `cargo clippy --workspace --all-targets` termine avec exactement deux warnings ciblés dans `ksp-store-api` : `clippy::collapsible_if` et `clippy::bool_comparison`. + +## Objet + +Corriger exclusivement les deux warnings Clippy remontés par le gate opérateur : + +```text +crates/ksp-store-api/src/model/raw_pagination.rs +crates/ksp-store-api/tests/dependency_boundary.rs +``` + +Le premier warning concerne la validation d'un `RawSlotRange` inversé ; le second concerne uniquement l'écriture d'une assertion de canari. + +## Changement fonctionnel + +```text +aucun +``` + +La validation conserve exactement la même règle : lorsqu'une borne de début et une borne de fin sont toutes deux présentes, `start > end` reste rejeté. + +Le canari conserve exactement la même assertion : la source de pagination ne doit pas contenir le littéral `u64::MAX`, afin de ne pas transformer cette valeur en plafond KSP implicite. + +Aucun contrat de pagination, outcome, rétention, capability, modèle RAW ou export public n'est modifié. + +## Version Cargo + +Ce fix modifie du code Rust. Conformément au workflow KSP, l'identité de build devient : + +```text +0.3.1-pre.6-fix.1 +``` + +Cette identité distincte garantit notamment que les commandes Cargo exécutées après application du fix ne réutilisent pas ambiguëment l'identité de build de `0.3.1-pre.6`. + +## Fichiers inclus dans le payload + +```text +Cargo.toml +crates/ksp-store-api/src/model/raw_pagination.rs +crates/ksp-store-api/tests/dependency_boundary.rs +deltas/0.3.1/pre.006-fix.001.md +``` + +`Cargo.toml` porte : + +```text +workspace.package.version = "0.3.1-pre.6-fix.1" +``` + +## Hors scope + +```text +API publique +queries/pagination +outcomes d'idempotence +rétention/tombstones +capabilities Store +ksp-store-lib +backend PostgreSQL +plans/validation/ROADMAP +``` + +## Validations exécutées dans l'environnement de génération + +```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.1 +``` + +`cargo`, `rustc` et `rustfmt` ne sont pas disponibles dans l'environnement de génération. Le gate opérateur doit donc rejouer : + +```text +cargo fmt --all +cargo check --workspace +cargo clippy --workspace --all-targets +cargo test -p ksp-store-api +```