v0.3.2-pre.002-fix.001
This commit is contained in:
@@ -1,12 +1,12 @@
|
|||||||
# file: Cargo.toml
|
# file: Cargo.toml
|
||||||
# version: 332
|
# version: 333
|
||||||
|
|
||||||
[workspace]
|
[workspace]
|
||||||
resolver = "3"
|
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-store-lib", "crates/ksp-store-postgres-lib", "crates/ksp-wallet-lib"]
|
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-store-lib", "crates/ksp-store-postgres-lib", "crates/ksp-wallet-lib"]
|
||||||
|
|
||||||
[workspace.package]
|
[workspace.package]
|
||||||
version = "0.3.2-pre.2"
|
version = "0.3.2-pre.2.fix.1"
|
||||||
edition = "2024"
|
edition = "2024"
|
||||||
license = "MIT"
|
license = "MIT"
|
||||||
repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project"
|
repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project"
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-store-lib/src/constants.rs
|
// file: crates/ksp-store-lib/src/constants.rs
|
||||||
// version: 1
|
// version: 2
|
||||||
|
|
||||||
//! Store facade-owned constants.
|
//! Store facade-owned constants.
|
||||||
|
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-store-lib/src/lib.rs
|
// file: crates/ksp-store-lib/src/lib.rs
|
||||||
// version: 1
|
// version: 2
|
||||||
|
|
||||||
#![warn(missing_docs)]
|
#![warn(missing_docs)]
|
||||||
#![deny(unreachable_pub)]
|
#![deny(unreachable_pub)]
|
||||||
@@ -19,3 +19,7 @@ mod constants;
|
|||||||
|
|
||||||
/// Crate-owned tracing target reserved for later Store runtime behavior.
|
/// Crate-owned tracing target reserved for later Store runtime behavior.
|
||||||
pub(crate) use self::constants::TRACING_TARGET;
|
pub(crate) use self::constants::TRACING_TARGET;
|
||||||
|
|
||||||
|
// Keep the mandatory crate-owned tracing target part of the compiled scaffold
|
||||||
|
// without inventing runtime logging before the first behavioral tranche.
|
||||||
|
const _: &str = TRACING_TARGET;
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-store-lib/tests/dependency_boundary.rs
|
// file: crates/ksp-store-lib/tests/dependency_boundary.rs
|
||||||
// version: 1
|
// version: 2
|
||||||
|
|
||||||
#![warn(missing_docs)]
|
#![warn(missing_docs)]
|
||||||
#![deny(unreachable_pub)]
|
#![deny(unreachable_pub)]
|
||||||
@@ -34,6 +34,7 @@ fn pre_002_scaffold_keeps_backend_types_and_future_runtime_surface_private() {
|
|||||||
let crate_root = include_str!("../src/lib.rs");
|
let crate_root = include_str!("../src/lib.rs");
|
||||||
assert!(crate_root.contains("mod constants;"));
|
assert!(crate_root.contains("mod constants;"));
|
||||||
assert!(crate_root.contains("pub(crate) use self::constants::TRACING_TARGET;"));
|
assert!(crate_root.contains("pub(crate) use self::constants::TRACING_TARGET;"));
|
||||||
|
assert!(crate_root.contains("const _: &str = TRACING_TARGET;"));
|
||||||
for forbidden in [
|
for forbidden in [
|
||||||
"pub mod ",
|
"pub mod ",
|
||||||
"pub use ksp_store_postgres_lib",
|
"pub use ksp_store_postgres_lib",
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-store-postgres-lib/src/constants.rs
|
// file: crates/ksp-store-postgres-lib/src/constants.rs
|
||||||
// version: 1
|
// version: 2
|
||||||
|
|
||||||
//! PostgreSQL Store backend-owned constants.
|
//! PostgreSQL Store backend-owned constants.
|
||||||
|
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-store-postgres-lib/src/lib.rs
|
// file: crates/ksp-store-postgres-lib/src/lib.rs
|
||||||
// version: 1
|
// version: 2
|
||||||
|
|
||||||
#![warn(missing_docs)]
|
#![warn(missing_docs)]
|
||||||
#![deny(unreachable_pub)]
|
#![deny(unreachable_pub)]
|
||||||
@@ -19,3 +19,7 @@ mod constants;
|
|||||||
|
|
||||||
/// Crate-owned tracing target reserved for later PostgreSQL backend behavior.
|
/// Crate-owned tracing target reserved for later PostgreSQL backend behavior.
|
||||||
pub(crate) use self::constants::TRACING_TARGET;
|
pub(crate) use self::constants::TRACING_TARGET;
|
||||||
|
|
||||||
|
// Keep the mandatory crate-owned tracing target part of the compiled scaffold
|
||||||
|
// without inventing runtime logging before the first behavioral tranche.
|
||||||
|
const _: &str = TRACING_TARGET;
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-store-postgres-lib/tests/dependency_boundary.rs
|
// file: crates/ksp-store-postgres-lib/tests/dependency_boundary.rs
|
||||||
// version: 1
|
// version: 2
|
||||||
|
|
||||||
#![warn(missing_docs)]
|
#![warn(missing_docs)]
|
||||||
#![deny(unreachable_pub)]
|
#![deny(unreachable_pub)]
|
||||||
@@ -26,6 +26,7 @@ fn pre_002_backend_does_not_advance_connection_pool_tls_or_migrations() {
|
|||||||
let crate_root = include_str!("../src/lib.rs");
|
let crate_root = include_str!("../src/lib.rs");
|
||||||
assert!(crate_root.contains("mod constants;"));
|
assert!(crate_root.contains("mod constants;"));
|
||||||
assert!(crate_root.contains("pub(crate) use self::constants::TRACING_TARGET;"));
|
assert!(crate_root.contains("pub(crate) use self::constants::TRACING_TARGET;"));
|
||||||
|
assert!(crate_root.contains("const _: &str = TRACING_TARGET;"));
|
||||||
for forbidden in ["pub mod ", "pub struct", "pub enum", "pub trait", "tokio_postgres", "deadpool_postgres", "mod migration", "SELECT ", "CREATE TABLE"] {
|
for forbidden in ["pub mod ", "pub struct", "pub enum", "pub trait", "tokio_postgres", "deadpool_postgres", "mod migration", "SELECT ", "CREATE TABLE"] {
|
||||||
assert!(!crate_root.contains(forbidden), "premature PostgreSQL backend surface detected: {forbidden}");
|
assert!(!crate_root.contains(forbidden), "premature PostgreSQL backend surface detected: {forbidden}");
|
||||||
}
|
}
|
||||||
|
|||||||
122
deltas/0.3.2/pre.002-fix.001.md
Normal file
122
deltas/0.3.2/pre.002-fix.001.md
Normal file
@@ -0,0 +1,122 @@
|
|||||||
|
<!-- file: deltas/0.3.2/pre.002-fix.001.md -->
|
||||||
|
<!-- version: 1 -->
|
||||||
|
|
||||||
|
# Delta `0.3.2-pre.002-fix.001` — ancrage compile-time des targets de tracing
|
||||||
|
|
||||||
|
## 1. Déclencheur
|
||||||
|
|
||||||
|
Le gate opérateur de `pre.002` est fonctionnellement vert :
|
||||||
|
|
||||||
|
```text
|
||||||
|
cargo fmt --all PASS
|
||||||
|
audit Rust général / exports / workspace PASS
|
||||||
|
audit Markdown PASS — 186 tables / 125 files
|
||||||
|
cargo check --workspace PASS avec warnings
|
||||||
|
cargo clippy --workspace --all-targets PASS avec warnings
|
||||||
|
cargo test -p ksp-store-api PASS
|
||||||
|
cargo test -p ksp-store-lib PASS
|
||||||
|
cargo test -p ksp-store-postgres-lib PASS
|
||||||
|
cargo check -p ksp-store-lib --no-default-features PASS
|
||||||
|
cargo tree Store facade/backend fourni et conforme
|
||||||
|
```
|
||||||
|
|
||||||
|
L'écart est limité aux deux nouveaux scaffolds :
|
||||||
|
|
||||||
|
```text
|
||||||
|
unused import: self::constants::TRACING_TARGET
|
||||||
|
constant TRACING_TARGET is never used
|
||||||
|
```
|
||||||
|
|
||||||
|
Le même défaut existe dans `ksp-store-lib` et `ksp-store-postgres-lib`.
|
||||||
|
|
||||||
|
## 2. Diagnostic
|
||||||
|
|
||||||
|
`pre.002` ne contient encore aucun comportement Store/PostgreSQL et n'ajoute volontairement pas `ksp-logging-lib`. Le simple réexport crate-private du `TRACING_TARGET` obligatoire ne possède donc aucun site d'usage réel à cette tranche, ce qui produit les warnings observés.
|
||||||
|
|
||||||
|
Supprimer les targets n'est pas possible : l'audit workspace impose déjà :
|
||||||
|
|
||||||
|
```text
|
||||||
|
KSP-TRACE-102 — constants.rs possède le TRACING_TARGET canonique
|
||||||
|
KSP-TRACE-103 — le crate root réexporte TRACING_TARGET
|
||||||
|
```
|
||||||
|
|
||||||
|
Ajouter un faux log, assouplir les règles ou poser un `allow(dead_code/unused_imports)` masquerait le staging réel.
|
||||||
|
|
||||||
|
## 3. Correction
|
||||||
|
|
||||||
|
Les deux targets et leurs réexports sont conservés à l'identique. Chaque crate ajoute uniquement un canari compile-time privé :
|
||||||
|
|
||||||
|
```rust
|
||||||
|
const _: &str = TRACING_TARGET;
|
||||||
|
```
|
||||||
|
|
||||||
|
Ce canari :
|
||||||
|
|
||||||
|
```text
|
||||||
|
marque le réexport et la constante comme réellement consommés par le scaffold compilé
|
||||||
|
ne crée aucun comportement runtime
|
||||||
|
n'émet aucun log artificiel
|
||||||
|
n'ajoute pas ksp-logging-lib
|
||||||
|
n'assouplit aucun lint
|
||||||
|
préserve KSP-TRACE-102/103
|
||||||
|
```
|
||||||
|
|
||||||
|
Les tests de frontière figent aussi la présence de ce canari pour empêcher le retour des warnings tant que le premier comportement réel n'a pas remplacé cet usage de scaffold.
|
||||||
|
|
||||||
|
## 4. Version
|
||||||
|
|
||||||
|
Le correctif touche des sources Rust ; conformément à `VER-ID-007` et `VER-ID-010`, la version workspace devient :
|
||||||
|
|
||||||
|
```text
|
||||||
|
0.3.2-pre.2.fix.1
|
||||||
|
```
|
||||||
|
|
||||||
|
Identifiant de livraison :
|
||||||
|
|
||||||
|
```text
|
||||||
|
0.3.2-pre.002-fix.001
|
||||||
|
```
|
||||||
|
|
||||||
|
## 5. Documentation durable
|
||||||
|
|
||||||
|
Le plan 023 et la validation 019 sont synchronisés :
|
||||||
|
|
||||||
|
```text
|
||||||
|
graphe pre.002 confirmé par le gate opérateur
|
||||||
|
--no-default-features confirmé
|
||||||
|
cycle backend absent confirmé
|
||||||
|
warning TRACING_TARGET documenté puis corrigé par fix.001
|
||||||
|
TRACING_TARGET conservé et ancré compile-time sans faux log
|
||||||
|
```
|
||||||
|
|
||||||
|
Le delta `pre.002.md` n'est pas réécrit : une livraison déjà émise reste immuable et son correctif est tracé séparément.
|
||||||
|
|
||||||
|
## 6. Hors scope
|
||||||
|
|
||||||
|
Aucun changement de périmètre fonctionnel :
|
||||||
|
|
||||||
|
```text
|
||||||
|
pas de StoreSettings
|
||||||
|
pas de Store runtime public
|
||||||
|
pas de Config std.store
|
||||||
|
pas de ksp-logging-lib
|
||||||
|
pas de Tokio/tokio-postgres/Deadpool/Rustls
|
||||||
|
pas de connexion PostgreSQL
|
||||||
|
pas de migration
|
||||||
|
pas de SQL métier
|
||||||
|
```
|
||||||
|
|
||||||
|
## 7. Validation opérateur requise
|
||||||
|
|
||||||
|
```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/0.3.2
|
||||||
|
cargo check --workspace
|
||||||
|
cargo clippy --workspace --all-targets
|
||||||
|
cargo test -p ksp-store-lib
|
||||||
|
cargo test -p ksp-store-postgres-lib
|
||||||
|
cargo check -p ksp-store-lib --no-default-features
|
||||||
|
```
|
||||||
|
|
||||||
|
Le passage à `pre.003` reste interdit si un warning ou un nouvel écart apparaît.
|
||||||
@@ -1,5 +1,5 @@
|
|||||||
<!-- file: docs/plans/023-V0_3_2_STORE_POSTGRES_FOUNDATION_PLAN.md -->
|
<!-- file: docs/plans/023-V0_3_2_STORE_POSTGRES_FOUNDATION_PLAN.md -->
|
||||||
<!-- version: 2 -->
|
<!-- version: 3 -->
|
||||||
|
|
||||||
# Plan `0.3.2` — Store/PostgreSQL runtime foundation
|
# Plan `0.3.2` — Store/PostgreSQL runtime foundation
|
||||||
|
|
||||||
@@ -882,7 +882,7 @@ Livrer le présent plan, la matrice validation, l'audit externe/kbot3, les déci
|
|||||||
|
|
||||||
Créer les crates, manifests, modules minimaux, `postgres` default, `--no-default-features`, constants/logging targets et canaris de dépendances. Ajouter seulement les dépendances nécessaires au scaffold retenu.
|
Créer les crates, manifests, modules minimaux, `postgres` default, `--no-default-features`, constants/logging targets et canaris de dépendances. Ajouter seulement les dépendances nécessaires au scaffold retenu.
|
||||||
|
|
||||||
**Statut : matérialisé par `0.3.2-pre.002`, gate Cargo opérateur à exécuter.**
|
**Statut : matérialisé par `0.3.2-pre.002`. Le gate opérateur confirme le graphe et les tests ; `pre.002-fix.001` corrige les warnings `TRACING_TARGET` sans modifier le scope avant passage à `pre.003`.**
|
||||||
|
|
||||||
La tranche matérialise exactement :
|
La tranche matérialise exactement :
|
||||||
|
|
||||||
@@ -893,7 +893,7 @@ ksp-store-postgres-lib -> ksp-store-api
|
|||||||
ksp-store-postgres-lib -X-> ksp-store-lib
|
ksp-store-postgres-lib -X-> ksp-store-lib
|
||||||
```
|
```
|
||||||
|
|
||||||
Les deux crates possèdent déjà leur `src/constants.rs` et leur futur `TRACING_TARGET`, mais `ksp-logging-lib` n'est pas encore ajouté : aucun comportement de production n'émet de log dans le scaffold. De même, Tokio, tokio-postgres, Deadpool, Rustls et SHA-256 restent absents tant que leurs tranches d'usage réel ne sont pas ouvertes. Cette matérialisation respecte la règle selon laquelle une dépendance de package doit correspondre à un usage réel plutôt qu'à une anticipation du graphe final.
|
Les deux crates possèdent déjà leur `src/constants.rs` et leur `TRACING_TARGET`, comme l'exigent les règles KSP-TRACE-102/103. Le gate opérateur de `pre.002` a toutefois révélé que le simple réexport privé produisait `unused_imports` et `dead_code` tant qu'aucun comportement de production n'émet encore de log. `pre.002-fix.001` conserve donc l'invariant de tracing et ajoute uniquement un canari compile-time privé `const _: &str = TRACING_TARGET;` : le target participe au scaffold compilé sans faux log, sans `allow(...)` et sans ajouter prématurément `ksp-logging-lib`. Tokio, tokio-postgres, Deadpool, Rustls et SHA-256 restent absents tant que leurs tranches d'usage réel ne sont pas ouvertes.
|
||||||
|
|
||||||
### `pre.003` — Settings + backend selection + lifecycle contracts
|
### `pre.003` — Settings + backend selection + lifecycle contracts
|
||||||
|
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
<!-- file: docs/validation/019-V0_3_2_STORE_POSTGRES_FOUNDATION.md -->
|
<!-- file: docs/validation/019-V0_3_2_STORE_POSTGRES_FOUNDATION.md -->
|
||||||
<!-- version: 2 -->
|
<!-- version: 3 -->
|
||||||
|
|
||||||
# Validation `0.3.2` — Store/PostgreSQL runtime foundation
|
# Validation `0.3.2` — Store/PostgreSQL runtime foundation
|
||||||
|
|
||||||
@@ -68,6 +68,8 @@ refinery 0.9.2 audité/rejeté
|
|||||||
|
|
||||||
Le gate opérateur après application de `pre.001`, fourni le 29 août 2026, est vert : audits Rust/Markdown, `cargo check --workspace` et `cargo clippy --workspace --all-targets` passent sur `0.3.2-pre.1`.
|
Le gate opérateur après application de `pre.001`, fourni le 29 août 2026, est vert : audits Rust/Markdown, `cargo check --workspace` et `cargo clippy --workspace --all-targets` passent sur `0.3.2-pre.1`.
|
||||||
|
|
||||||
|
Le gate opérateur de `pre.002`, fourni le 29 août 2026, confirme également les tests ciblés, `--no-default-features` et les `cargo tree`. `cargo check`/Clippy terminent avec succès mais signalent deux warnings par nouvelle crate (`TRACING_TARGET` importé mais inutilisé et constant `dead_code`) ; `pre.002-fix.001` traite cet écart par un canari compile-time privé, tout en conservant les invariants KSP-TRACE-102/103 et sans ajouter de logging runtime.
|
||||||
|
|
||||||
## 3. Frontières Cargo
|
## 3. Frontières Cargo
|
||||||
|
|
||||||
### V32-DEP-001 — Façade -> API
|
### V32-DEP-001 — Façade -> API
|
||||||
@@ -78,7 +80,7 @@ ksp-store-lib -> ksp-store-api
|
|||||||
|
|
||||||
Matérialisé par `0.3.2-pre.002` dans le manifest de `ksp-store-lib`.
|
Matérialisé par `0.3.2-pre.002` dans le manifest de `ksp-store-lib`.
|
||||||
|
|
||||||
Statut : `TODO gate opérateur pre.002` jusqu'au test ciblé et au `cargo tree`.
|
Statut : `PASS pre.002 opérateur` — test ciblé et `cargo tree` fournis.
|
||||||
|
|
||||||
### V32-DEP-002 — Backend -> API
|
### V32-DEP-002 — Backend -> API
|
||||||
|
|
||||||
@@ -88,7 +90,7 @@ ksp-store-postgres-lib -> ksp-store-api
|
|||||||
|
|
||||||
Matérialisé par `0.3.2-pre.002` dans le manifest de `ksp-store-postgres-lib`.
|
Matérialisé par `0.3.2-pre.002` dans le manifest de `ksp-store-postgres-lib`.
|
||||||
|
|
||||||
Statut : `TODO gate opérateur pre.002` jusqu'au test ciblé et au `cargo tree`.
|
Statut : `PASS pre.002 opérateur` — test ciblé et `cargo tree` fournis.
|
||||||
|
|
||||||
### V32-DEP-003 — Pas de cycle backend
|
### V32-DEP-003 — Pas de cycle backend
|
||||||
|
|
||||||
@@ -98,9 +100,9 @@ ksp-store-postgres-lib -X-> ksp-store-lib
|
|||||||
|
|
||||||
Preuves : manifest scanner + cargo tree.
|
Preuves : manifest scanner + cargo tree.
|
||||||
|
|
||||||
Le canari source de `pre.002` interdit explicitement la dépendance inverse ; la preuve Cargo reste à exécuter puis sera durcie à nouveau en `pre.009`.
|
Le canari source de `pre.002` interdit explicitement la dépendance inverse ; le `cargo tree` opérateur confirme l'absence de cycle. La frontière sera durcie à nouveau en `pre.009`.
|
||||||
|
|
||||||
Statut : `TODO gate opérateur pre.002/pre.009`.
|
Statut : `PASS pre.002 opérateur / TODO pre.009 final`.
|
||||||
|
|
||||||
### V32-DEP-004 — Feature PostgreSQL
|
### V32-DEP-004 — Feature PostgreSQL
|
||||||
|
|
||||||
@@ -114,7 +116,7 @@ cargo check -p ksp-store-lib --no-default-features passe
|
|||||||
|
|
||||||
Le manifest de `ksp-store-lib` matérialise `default = ["postgres"]` et l'optional dependency `ksp-store-postgres-lib`. La compilation sans default feature est une preuve opérateur obligatoire.
|
Le manifest de `ksp-store-lib` matérialise `default = ["postgres"]` et l'optional dependency `ksp-store-postgres-lib`. La compilation sans default feature est une preuve opérateur obligatoire.
|
||||||
|
|
||||||
Statut : `TODO gate opérateur pre.002`.
|
Statut : `PASS pre.002 opérateur` — default feature, optional backend edge et compilation `--no-default-features` confirmés.
|
||||||
|
|
||||||
### V32-DEP-005 — Firewall domaines
|
### V32-DEP-005 — Firewall domaines
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user