0.3.16-pre.006-fix.001
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-raw-transaction-ingest-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.16-pre.6"
|
||||
version = "0.3.16-pre.6.fix.1"
|
||||
edition = "2024"
|
||||
license = "MIT"
|
||||
repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project"
|
||||
|
||||
@@ -489,7 +489,7 @@ fn v0_3_16_pre_005_shared_comparator_accepts_only_proved_incoming_truncation_in_
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn v0_3_16_pre_005_shared_comparator_rejects_unproved_shapes_and_defers_more_complete_promotion() {
|
||||
fn v0_3_16_pre_006_shared_comparator_rejects_unproved_shapes_and_accepts_more_complete_promotion() {
|
||||
let stored = raw_transaction(
|
||||
9,
|
||||
br#"{"transaction":["AQID","base64"],"meta":{"err":null,"fee":5000,"logMessages":["Program Vote111 invoke [1]","Program log: stable","Program log: detail-a","Program log: detail-b","Program Vote111 success"],"rewards":[]},"version":0,"transactionIndex":7}"#,
|
||||
@@ -542,7 +542,7 @@ fn v0_3_16_pre_005_shared_comparator_rejects_unproved_shapes_and_defers_more_com
|
||||
stored_row.content_hash = stored_truncated.payload().content_hash().as_bytes().to_vec();
|
||||
stored_row.payload = std::option::Option::Some(stored_truncated.payload().bytes().to_vec());
|
||||
let comparison = super::compare_existing_transaction(&network(), stored_row, &stored);
|
||||
assert!(matches!(comparison, std::result::Result::Err(error) if error.phase() == "raw_acquisition_content_conflict"));
|
||||
assert!(matches!(comparison, std::result::Result::Ok(super::ExistingTransactionMatch::ActiveCompatibleMoreComplete)));
|
||||
return;
|
||||
}
|
||||
|
||||
|
||||
126
deltas/0.3.16/pre.006-fix.001.md
Normal file
126
deltas/0.3.16/pre.006-fix.001.md
Normal file
@@ -0,0 +1,126 @@
|
||||
<!-- file: deltas/0.3.16/pre.006-fix.001.md -->
|
||||
<!-- version: 1 -->
|
||||
|
||||
# Delta `0.3.16-pre.006-fix.001` — réalignement du canari de promotion
|
||||
|
||||
## Base requise
|
||||
|
||||
```text
|
||||
0.3.16-pre.006 appliqué
|
||||
workspace.package.version = 0.3.16-pre.6
|
||||
```
|
||||
|
||||
Le gate opérateur de `pre.006` est propre jusqu'aux tests `ksp-store-postgres-lib`. La suite unitaire exécute 87 tests et échoue sur un seul canari hérité de `pre.005` :
|
||||
|
||||
```text
|
||||
v0_3_16_pre_005_shared_comparator_rejects_unproved_shapes_and_defers_more_complete_promotion
|
||||
```
|
||||
|
||||
Les 86 autres tests PostgreSQL exécutés avant l'arrêt sont verts. Les audits Rust/export/KSP/Markdown, `cargo check --workspace`, Clippy `-D warnings` et toute la suite `ksp-store-api` sont également verts dans le gate fourni.
|
||||
|
||||
## Version
|
||||
|
||||
Le correctif modifie un test Rust et synchronise donc la version Cargo :
|
||||
|
||||
```text
|
||||
workspace.package.version = 0.3.16-pre.6.fix.1
|
||||
```
|
||||
|
||||
## Objectif
|
||||
|
||||
Réaligner le canari PostgreSQL hérité de `pre.005` sur le contrat désormais implémenté par `pre.006` sans modifier le runtime.
|
||||
|
||||
`pre.005` avait volontairement reconnu :
|
||||
|
||||
```text
|
||||
CompatibleMoreComplete
|
||||
```
|
||||
|
||||
mais différé sa promotion et attendait encore un conflit côté backend PostgreSQL.
|
||||
|
||||
`pre.006` a précisément fermé ce différé : `compare_existing_transaction(...)` projette maintenant cette relation vers :
|
||||
|
||||
```text
|
||||
ExistingTransactionMatch::ActiveCompatibleMoreComplete
|
||||
```
|
||||
|
||||
Le test ancien continuait pourtant d'exiger :
|
||||
|
||||
```text
|
||||
raw_acquisition_content_conflict
|
||||
```
|
||||
|
||||
Le défaut est donc exclusivement un canari obsolète, pas une régression de la promotion canonique.
|
||||
|
||||
## Correction
|
||||
|
||||
Le test concerné conserve toutes ses assertions fail-closed sur les formes non prouvées : mauvais préfixe, matériel après marqueur, différence d'un autre champ metadata et faux marqueur de troncature ne doivent toujours produire ni `CompatibleLessComplete` ni `CompatibleMoreComplete`.
|
||||
|
||||
Pour le seul cas bidirectionnel réellement prouvé — canonique tronqué et entrant strictement plus complet — le test exige désormais :
|
||||
|
||||
```text
|
||||
std::result::Result::Ok(
|
||||
ExistingTransactionMatch::ActiveCompatibleMoreComplete
|
||||
)
|
||||
```
|
||||
|
||||
Le nom du test est avancé vers `pre.006` afin de ne plus documenter le comportement différé de `pre.005`.
|
||||
|
||||
## Fichiers modifiés
|
||||
|
||||
```text
|
||||
Cargo.toml
|
||||
crates/ksp-store-postgres-lib/unit_tests/raw_transaction.rs
|
||||
```
|
||||
|
||||
## Fichier ajouté
|
||||
|
||||
```text
|
||||
deltas/0.3.16/pre.006-fix.001.md
|
||||
```
|
||||
|
||||
## Fichiers supprimés
|
||||
|
||||
```text
|
||||
aucun
|
||||
```
|
||||
|
||||
## Validations exécutées lors de la génération
|
||||
|
||||
```bash
|
||||
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
|
||||
```
|
||||
|
||||
Le toolchain Rust n'est pas installé dans l'environnement de génération. `cargo fmt`, `cargo check`, Clippy et les tests Cargo ne sont donc pas déclarés exécutés ici.
|
||||
|
||||
## Validations opérateur demandées
|
||||
|
||||
```bash
|
||||
cargo fmt --all
|
||||
cargo fmt --all -- --check
|
||||
|
||||
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-store-api --all-targets --all-features
|
||||
cargo test -p ksp-store-postgres-lib --all-targets --all-features
|
||||
```
|
||||
|
||||
## Décisions prises
|
||||
|
||||
- aucune logique runtime de `pre.006` n'est modifiée ;
|
||||
- les cas non prouvés restent fail-closed ;
|
||||
- `CompatibleMoreComplete` doit être accepté par la comparaison backend afin que le chemin atomique de promotion de `pre.006` puisse s'exécuter ;
|
||||
- le correctif reste strictement borné au canari obsolète qui bloquait le gate.
|
||||
|
||||
## Questions ouvertes
|
||||
|
||||
Aucune question bloquante.
|
||||
|
||||
## Suite
|
||||
|
||||
Après gate propre : `0.3.16-pre.007` — conflit durable minimal + intégration Worker non terminale.
|
||||
Reference in New Issue
Block a user