v0.2.12-pre.005-fix.001
This commit is contained in:
@@ -1,12 +1,12 @@
|
|||||||
# file: Cargo.toml
|
# file: Cargo.toml
|
||||||
# version: 291
|
# version: 292
|
||||||
|
|
||||||
[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-logging-lib", "crates/ksp-offchain-transport-lib", "crates/ksp-onchain-transport-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-logging-lib", "crates/ksp-offchain-transport-lib", "crates/ksp-onchain-transport-lib", "crates/ksp-wallet-lib"]
|
||||||
|
|
||||||
[workspace.package]
|
[workspace.package]
|
||||||
version = "0.2.12-pre.5"
|
version = "0.2.12-pre.5.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,7 +1,7 @@
|
|||||||
{
|
{
|
||||||
"name": "ksp-app-solprices-desk",
|
"name": "ksp-app-solprices-desk",
|
||||||
"private": true,
|
"private": true,
|
||||||
"version": "0.2.12-pre.5",
|
"version": "0.2.12-pre.5.fix.1",
|
||||||
"type": "module",
|
"type": "module",
|
||||||
"scripts": {
|
"scripts": {
|
||||||
"dev": "vite",
|
"dev": "vite",
|
||||||
|
|||||||
@@ -1,7 +1,7 @@
|
|||||||
{
|
{
|
||||||
"$schema": "https://schema.tauri.app/config/2",
|
"$schema": "https://schema.tauri.app/config/2",
|
||||||
"productName": "KSP SOL Prices Desk",
|
"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",
|
"identifier": "com.sasedev.ksp-app-solprices-desk",
|
||||||
"build": {
|
"build": {
|
||||||
"beforeDevCommand": {
|
"beforeDevCommand": {
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
// file: crates/ksp-app-solprices-desk/tests/desktop_security.rs
|
// file: crates/ksp-app-solprices-desk/tests/desktop_security.rs
|
||||||
// version: 4
|
// version: 5
|
||||||
|
|
||||||
//! Security, ownership and frontend instrumentation canaries for SOL Prices Desk.
|
//! 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());
|
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") {
|
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_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());
|
assert!(!source.contains("refresh_all_market_prices"), "{} advances global refresh before pre.006", path.display());
|
||||||
|
|||||||
33
deltas/0.2.12/pre.005-fix.001.md
Normal file
33
deltas/0.2.12/pre.005-fix.001.md
Normal file
@@ -0,0 +1,33 @@
|
|||||||
|
<!-- file: deltas/0.2.12/pre.005-fix.001.md -->
|
||||||
|
<!-- version: 1 -->
|
||||||
|
|
||||||
|
# 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
|
||||||
|
```
|
||||||
@@ -1,5 +1,5 @@
|
|||||||
<!-- file: docs/plans/019-V0_2_12_SOL_PRICES_DESK_PLAN.md -->
|
<!-- file: docs/plans/019-V0_2_12_SOL_PRICES_DESK_PLAN.md -->
|
||||||
<!-- version: 10 -->
|
<!-- version: 11 -->
|
||||||
|
|
||||||
# Plan `0.2.12` — SOL Prices Desk + intégration prix Wallet Desk
|
# 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
|
### `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`.
|
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
|
### `pre.006` — Refresh multiple et global
|
||||||
|
|
||||||
**Statut : planifié.**
|
**Statut : planifié.**
|
||||||
|
|||||||
@@ -1,5 +1,5 @@
|
|||||||
<!-- file: docs/validation/015-V0_2_12_SOL_PRICES_DESK.md -->
|
<!-- file: docs/validation/015-V0_2_12_SOL_PRICES_DESK.md -->
|
||||||
<!-- version: 9 -->
|
<!-- version: 10 -->
|
||||||
|
|
||||||
# Validation `0.2.12` — SOL Prices Desk + intégration Wallet Desk
|
# 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
|
python3 scripts/audit_markdown_tables.py README.md ... deltas PASS
|
||||||
cargo fmt/check/clippy/test NOT RUN — cargo indisponible dans le sandbox
|
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`.
|
||||||
|
|||||||
Reference in New Issue
Block a user