v0.3.12-pre.003-fix.002
This commit is contained in:
@@ -6,7 +6,7 @@ resolver = "3"
|
||||
members = ["crates/ksp-app-backfill-desk", "crates/ksp-app-config-desk", "crates/ksp-app-solprices-desk", "crates/ksp-app-store-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-raw-transaction-lib", "crates/ksp-store-api", "crates/ksp-store-lib", "crates/ksp-store-postgres-lib", "crates/ksp-wallet-lib", "crates/ksp-worker-api", "crates/ksp-worker-raw-transaction-ingest-lib"]
|
||||
|
||||
[workspace.package]
|
||||
version = "0.3.12-pre.3.fix.1"
|
||||
version = "0.3.12-pre.3.fix.2"
|
||||
edition = "2024"
|
||||
license = "MIT"
|
||||
repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project"
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
// file: crates/ksp-worker-raw-transaction-ingest-lib/tests/hardening.rs
|
||||
// version: 4
|
||||
// version: 5
|
||||
|
||||
//! External public, security, redaction and release-boundary hardening canaries for `pre.010`.
|
||||
|
||||
@@ -287,7 +287,7 @@ fn v0_3_12_pre_003_private_signal_debug_and_shape_do_not_expose_signature_filter
|
||||
assert!(!resources.contains(".field(\"matched_filter_fingerprint\", &self.matched_filter_fingerprint)"));
|
||||
assert!(!resources.contains(".field(\"signature\", &self.signature)"));
|
||||
let signal_struct = match resources.split_once("struct RawTransactionIngestSourceSignal {") {
|
||||
std::option::Option::Some((_, tail)) => match tail.split_once("}\n\nimpl std::fmt::Debug for RawTransactionIngestSourceSignal") {
|
||||
std::option::Option::Some((_, tail)) => match tail.split_once("impl std::fmt::Debug for RawTransactionIngestSourceSignal") {
|
||||
std::option::Option::Some((value, _)) => value,
|
||||
std::option::Option::None => "",
|
||||
},
|
||||
|
||||
130
deltas/0.3.12/pre.003-fix.002.md
Normal file
130
deltas/0.3.12/pre.003-fix.002.md
Normal file
@@ -0,0 +1,130 @@
|
||||
<!-- file: deltas/0.3.12/pre.003-fix.002.md -->
|
||||
<!-- version: 1 -->
|
||||
|
||||
# Delta `0.3.12-pre.003-fix.002` — correction du canari source sous `#[cfg(test)]`
|
||||
|
||||
## Base requise
|
||||
|
||||
```text
|
||||
0.3.12-pre.003-fix.001
|
||||
workspace.package.version = 0.3.12-pre.3.fix.1
|
||||
```
|
||||
|
||||
Le gate opérateur reçu pour `pre.003-fix.001` montre :
|
||||
|
||||
```text
|
||||
audit Rust : clean
|
||||
audit Markdown : clean (340 tables, 804 fichiers)
|
||||
cargo check --workspace : PASS
|
||||
cargo clippy --workspace --all-targets --all-features -- -D warnings : PASS
|
||||
unit tests Worker : 45 PASS
|
||||
integration dependency_boundary : 4 PASS
|
||||
integration public_api : 8 PASS
|
||||
integration release_completeness : 3 PASS
|
||||
integration hardening : 9 PASS, 1 FAIL
|
||||
```
|
||||
|
||||
L'unique échec est `v0_3_12_pre_003_private_signal_debug_and_shape_do_not_expose_signature_filters_or_payload`, qui rapporte à tort `required private signal field missing: created_at:`.
|
||||
|
||||
## Objectif
|
||||
|
||||
Corriger exclusivement le canari source devenu incompatible avec l'attribut `#[cfg(test)]` introduit par `pre.003-fix.001`, sans modifier le contrat runtime ni avancer `pre.004`.
|
||||
|
||||
## Version
|
||||
|
||||
La correction modifie un test Rust ; conformément aux règles de version du workspace :
|
||||
|
||||
```text
|
||||
delivery id = 0.3.12-pre.003-fix.002
|
||||
workspace.package.version = 0.3.12-pre.3.fix.2
|
||||
```
|
||||
|
||||
## Correction
|
||||
|
||||
Le scanner du test isolait `RawTransactionIngestSourceSignal` avec une terminaison littérale qui supposait que son `impl Debug` suivait immédiatement la structure :
|
||||
|
||||
```text
|
||||
}\n\nimpl std::fmt::Debug for RawTransactionIngestSourceSignal
|
||||
```
|
||||
|
||||
Après `pre.003-fix.001`, la forme correcte est :
|
||||
|
||||
```text
|
||||
}
|
||||
|
||||
#[cfg(test)]
|
||||
impl std::fmt::Debug for RawTransactionIngestSourceSignal
|
||||
```
|
||||
|
||||
Le scanner est donc rendu indépendant de cet attribut : après avoir trouvé la déclaration de la structure, il borne désormais le segment sur le début de `impl std::fmt::Debug for RawTransactionIngestSourceSignal` lui-même.
|
||||
|
||||
Cette forme fonctionne avec les adapters actuellement test-only et restera compatible lorsque `pre.004` remettra ces helpers dans le build productif.
|
||||
|
||||
## Décisions prises
|
||||
|
||||
```text
|
||||
aucune modification du signal privé
|
||||
aucune modification de runtime_resources.rs
|
||||
aucun allow/expect lint
|
||||
aucune activation Yellowstone/HTTP/Store
|
||||
aucune avance fonctionnelle vers pre.004
|
||||
correction limitée au canari qui inspecte la source
|
||||
```
|
||||
|
||||
## Questions ouvertes
|
||||
|
||||
Aucune.
|
||||
|
||||
## Fichiers ajoutés
|
||||
|
||||
```text
|
||||
deltas/0.3.12/pre.003-fix.002.md
|
||||
```
|
||||
|
||||
## Fichiers modifiés
|
||||
|
||||
```text
|
||||
Cargo.toml
|
||||
crates/ksp-worker-raw-transaction-ingest-lib/tests/hardening.rs
|
||||
docs/validation/029-V0_3_12_YELLOWSTONE_HYDRATION_CONTINUITY.md
|
||||
```
|
||||
|
||||
## Fichiers supprimés
|
||||
|
||||
Aucun.
|
||||
|
||||
## 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
|
||||
```
|
||||
|
||||
`cargo`, `rustc` et `rustfmt` ne sont pas installés dans cet environnement ; aucun gate Cargo local n'est déclaré PASS.
|
||||
|
||||
## Gate opérateur demandé
|
||||
|
||||
```bash
|
||||
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
|
||||
cargo check --workspace
|
||||
cargo clippy --workspace --all-targets --all-features -- -D warnings
|
||||
cargo test -p ksp-worker-raw-transaction-ingest-lib
|
||||
```
|
||||
|
||||
## Non-claims
|
||||
|
||||
Ce fix ne revendique toujours pas :
|
||||
|
||||
```text
|
||||
stream Yellowstone ouvert
|
||||
update réseau consommée
|
||||
hydration HTTP
|
||||
Common RAW issu du réseau
|
||||
provenance composite finale
|
||||
persistence réseau
|
||||
Block / BlockMeta / Slot
|
||||
coalescence
|
||||
frontier / from_slot / ReplayInfo
|
||||
```
|
||||
@@ -1,5 +1,5 @@
|
||||
<!-- file: docs/validation/029-V0_3_12_YELLOWSTONE_HYDRATION_CONTINUITY.md -->
|
||||
<!-- version: 5 -->
|
||||
<!-- version: 6 -->
|
||||
|
||||
# Validation v0.3.12 — Yellowstone + hydration HTTP + continuité de run du Worker RawTransaction
|
||||
|
||||
@@ -801,3 +801,14 @@ Le correctif applique directement `RUST-API-008` : ces helpers strictement priv
|
||||
Le champ privé `route` de `RawTransactionIngestYellowstoneSource` reste, lui, productif : sa validation appartient déjà au contrat de construction `pre.003`. Il est désormais lu par le `Debug` sûr de la source, qui n'expose que les `RawProvenanceCode` déjà bornés et validés.
|
||||
|
||||
Le correctif ne change ni la surface publique, ni le graphe Cargo, ni le scope fonctionnel de `pre.003`, et n'active toujours aucun stream Yellowstone, HTTP ou Store réseau.
|
||||
|
||||
## 33. Correctif `pre.003-fix.002` — canari de forme compatible avec `#[cfg(test)]`
|
||||
|
||||
Le gate opérateur de `pre.003-fix.001` confirme que rustfmt, les audits, `cargo check --workspace`, Clippy strict et les 45 unit tests Worker passent. Le seul échec restant est le test externe `v0_3_12_pre_003_private_signal_debug_and_shape_do_not_expose_signature_filters_or_payload`.
|
||||
|
||||
Le signal privé lui-même est correct. L'échec vient du scanner source du canari : il bornait la structure avec la séquence littérale `}\n\nimpl std::fmt::Debug ...`, alors que `pre.003-fix.001` a placé légitimement `#[cfg(test)]` entre la fermeture de la structure et son `impl Debug` afin de respecter `RUST-API-008`. Le scanner obtenait donc une chaîne vide et signalait `created_at:` comme absent.
|
||||
|
||||
Le correctif ne touche pas au contrat de production. Le canari recherche désormais directement le début de `impl std::fmt::Debug for RawTransactionIngestSourceSignal` après la déclaration de la structure. Il reste ainsi valide avec les helpers sous `#[cfg(test)]` et restera valide lorsque `pre.004` remettra ces adapters dans le build productif.
|
||||
|
||||
Aucun lint n'est neutralisé, aucun helper n'est remis prématurément en production et aucune fonctionnalité de `pre.004` n'est avancée.
|
||||
|
||||
|
||||
Reference in New Issue
Block a user