diff --git a/Cargo.toml b/Cargo.toml index 222bb6e..b257fd7 100644 --- a/Cargo.toml +++ b/Cargo.toml @@ -1,12 +1,12 @@ # file: Cargo.toml -# version: 291 +# version: 292 [workspace] 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-logging-lib", "crates/ksp-offchain-transport-lib", "crates/ksp-onchain-transport-lib", "crates/ksp-wallet-lib"] [workspace.package] -version = "0.2.12-pre.5" +version = "0.2.12-pre.5.fix.1" edition = "2024" license = "MIT" repository = "https://git.sasedev.com/Sasedev/khadhroony-solana-project" diff --git a/crates/ksp-app-solprices-desk/package.json b/crates/ksp-app-solprices-desk/package.json index 2d1e767..b4e81de 100644 --- a/crates/ksp-app-solprices-desk/package.json +++ b/crates/ksp-app-solprices-desk/package.json @@ -1,7 +1,7 @@ { "name": "ksp-app-solprices-desk", "private": true, - "version": "0.2.12-pre.5", + "version": "0.2.12-pre.5.fix.1", "type": "module", "scripts": { "dev": "vite", diff --git a/crates/ksp-app-solprices-desk/tauri.conf.json b/crates/ksp-app-solprices-desk/tauri.conf.json index c6eb571..b3a6b1f 100644 --- a/crates/ksp-app-solprices-desk/tauri.conf.json +++ b/crates/ksp-app-solprices-desk/tauri.conf.json @@ -1,7 +1,7 @@ { "$schema": "https://schema.tauri.app/config/2", "productName": "KSP SOL Prices Desk", - "version": "0.2.12-pre.5", + "version": "0.2.12-pre.5.fix.1", "identifier": "com.sasedev.ksp-app-solprices-desk", "build": { "beforeDevCommand": { diff --git a/crates/ksp-app-solprices-desk/tests/desktop_security.rs b/crates/ksp-app-solprices-desk/tests/desktop_security.rs index 250a36b..d41c22c 100644 --- a/crates/ksp-app-solprices-desk/tests/desktop_security.rs +++ b/crates/ksp-app-solprices-desk/tests/desktop_security.rs @@ -1,5 +1,5 @@ // file: crates/ksp-app-solprices-desk/tests/desktop_security.rs -// version: 4 +// version: 5 //! Security, ownership and frontend instrumentation canaries for SOL Prices Desk. @@ -83,7 +83,7 @@ fn pre_005_tauri_commands_remain_centralized_and_only_single_refresh_is_present( assert_eq!(count, 0, "{} declares a Tauri command outside tauri.rs", path.display()); } if path.file_name().and_then(std::ffi::OsStr::to_str) != std::option::Option::Some("tauri.rs") { - assert!(!source.contains("refresh_market_price"), "{} declares refresh command plumbing outside tauri.rs", path.display()); + assert!(!source.contains("tauri::generate_handler!"), "{} registers Tauri handlers outside tauri.rs", path.display()); } assert!(!source.contains("refresh_market_prices"), "{} advances multi-refresh before pre.006", path.display()); assert!(!source.contains("refresh_all_market_prices"), "{} advances global refresh before pre.006", path.display()); diff --git a/deltas/0.2.12/pre.005-fix.001.md b/deltas/0.2.12/pre.005-fix.001.md new file mode 100644 index 0000000..49cface --- /dev/null +++ b/deltas/0.2.12/pre.005-fix.001.md @@ -0,0 +1,33 @@ + + + +# Delta `0.2.12-pre.005-fix.001` — Canari de centralisation Tauri rendu structurel + +## Contexte + +Le gate opérateur de `pre.005` confirme que le refresh individuel compile, fonctionne manuellement et que le Desk démarre. Le seul échec est le canari `pre_005_tauri_commands_remain_centralized_and_only_single_refresh_is_present`, qui traite à tort `AppState::refresh_market_price` comme du plumbing de commande simplement parce que son nom contient `refresh_market_price`. + +## Correction + +- `workspace.package.version` devient `0.2.12-pre.5.fix.1` ; +- `package.json` et `tauri.conf.json` sont synchronisés ; +- le test continue d'interdire tout `#[tauri::command]` hors `src/tauri.rs` ; +- il interdit aussi `tauri::generate_handler!` hors `src/tauri.rs` ; +- il n'interdit plus les méthodes applicatives portant le même nom métier que la commande Tauri ; +- les gardes `refresh_market_prices` et `refresh_all_market_prices` restent actives jusqu'à `pre.006`. + +## Hors scope + +Aucun changement n'est apporté au runtime prix, à `MarketPriceService`, au comportement réseau, aux DTOs, aux providers, au frontend, au logging ou au contrat du refresh individuel. + +## Gate attendu + +```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.2.12 +cargo check --workspace +cargo clippy --workspace --all-targets +cargo test -p ksp-app-solprices-desk +cargo test --workspace +``` diff --git a/docs/plans/019-V0_2_12_SOL_PRICES_DESK_PLAN.md b/docs/plans/019-V0_2_12_SOL_PRICES_DESK_PLAN.md index 7eb43eb..c756e87 100644 --- a/docs/plans/019-V0_2_12_SOL_PRICES_DESK_PLAN.md +++ b/docs/plans/019-V0_2_12_SOL_PRICES_DESK_PLAN.md @@ -1,5 +1,5 @@ - + # Plan `0.2.12` — SOL Prices Desk + intégration prix Wallet Desk @@ -514,10 +514,16 @@ Le gate de `pre.004-fix.001` a révélé que `desktop_contract.rs` comparait enc ### `pre.005` — Refresh individuel -**Statut : implémenté ; gate opérateur à confirmer.** +**Statut : implémenté ; corrigé par `pre.005-fix.001`, gate opérateur à confirmer.** Branchement du refresh d'une ligne via `MarketPriceService::refresh`, projection de l'observation exacte, de la fraîcheur et de l'availability, gestion d'erreur sûre et maintien du dernier état applicatif uniquement en mémoire backend. Le backend marque la ligne `loading` sans conserver le mutex de présentation pendant l'`await`, puis applique l'outcome provider-neutral ; une tentative sans nouvelle observation conserve la dernière observation réussie. L'UI ajoute un bouton `Refresh` par ligne, journalise le clic en `debug`, et n'expose encore aucun refresh multiple/global avant `pre.006`. +#### `pre.005-fix.001` — Canari de centralisation Tauri rendu structurel + +**Statut : implémenté ; gate opérateur à confirmer.** + +Le gate de `pre.005` confirme que le refresh individuel fonctionne et que le runtime Tauri démarre, mais révèle un faux positif dans `desktop_security.rs` : le canari assimilait toute occurrence du nom métier `refresh_market_price` hors `tauri.rs` à du plumbing de commande. Le fix conserve l'interdiction structurelle de `#[tauri::command]` hors `tauri.rs`, ajoute l'interdiction de `tauri::generate_handler!` hors `tauri.rs`, et autorise les méthodes applicatives homonymes dans `AppState`. Aucun comportement réseau, DTO, provider, row, frontend ou commande n'est modifié. + ### `pre.006` — Refresh multiple et global **Statut : planifié.** diff --git a/docs/validation/015-V0_2_12_SOL_PRICES_DESK.md b/docs/validation/015-V0_2_12_SOL_PRICES_DESK.md index ac05c96..51f58b3 100644 --- a/docs/validation/015-V0_2_12_SOL_PRICES_DESK.md +++ b/docs/validation/015-V0_2_12_SOL_PRICES_DESK.md @@ -1,5 +1,5 @@ - + # Validation `0.2.12` — SOL Prices Desk + intégration Wallet Desk @@ -457,3 +457,32 @@ python3 scripts/audit_rust_workspace_rules.py PASS python3 scripts/audit_markdown_tables.py README.md ... deltas PASS cargo fmt/check/clippy/test NOT RUN — cargo indisponible dans le sandbox ``` + +## 21. Fix `0.2.12-pre.005-fix.001` + +Le gate opérateur de `pre.005` confirme que le runtime de refresh individuel fonctionne : + +```text +cargo fmt --all PASS +python3 scripts/audit_rust_workspace_rules.py PASS +python3 scripts/audit_markdown_tables.py ... deltas/0.2.12 PASS +cargo check --workspace PASS +cargo test -p ksp-app-solprices-desk FAIL — canari desktop_security uniquement +cargo tauri dev SOL Prices Desk PASS +refresh individuel manuel PASS +``` + +L'unique échec est `pre_005_tauri_commands_remain_centralized_and_only_single_refresh_is_present`. Le test recherchait la chaîne `refresh_market_price` dans tous les fichiers Rust hors `tauri.rs`, ce qui classe à tort `AppState::refresh_market_price` comme plumbing Tauri alors que cette méthode est la frontière applicative appelée par la commande centralisée. + +Le correctif vérifie désormais la centralisation sur les marqueurs structurels Tauri : + +```text +#[tauri::command] hors tauri.rs interdit +tauri::generate_handler! hors tauri.rs interdit +méthode applicative refresh_market_price dans AppState autorisée +refresh_market_prices toujours absent avant pre.006 +refresh_all_market_prices toujours absent avant pre.006 +runtime prix / frontend / providers inchangés +``` + +La version technique devient `0.2.12-pre.5.fix.1`. Le gate attendu doit confirmer le test ciblé puis le workspace complet avant d'ouvrir `pre.006`.