66 lines
2.4 KiB
Markdown
66 lines
2.4 KiB
Markdown
<!-- file: deltas/0.1.4/pre.015-fix.003.md -->
|
|
<!-- version: 1 -->
|
|
|
|
# Delta `0.1.4-pre.015-fix.003` — nettoyage Clippy et ownership des fixtures Logging
|
|
|
|
## Statut
|
|
|
|
Correctif de `0.1.4-pre.015-fix.002` livré pour validation locale.
|
|
|
|
Le hot reload Logging introduit par `fix.002` est conservé sans modification fonctionnelle. La validation locale montre des générations runtime successives et l'utilisateur constate que le basculement console paraît fonctionner.
|
|
|
|
Deux défauts de qualité empêchent néanmoins de considérer `fix.002` comme propre :
|
|
|
|
1. `cargo clippy --workspace --all-targets` signale un helper `assert_file` devenu inutilisé après l'isolation des fixtures Logging ;
|
|
2. l'audit ownership détecte dans `ksp-app-config-desk/unit_tests/logging_editor.rs` des noms physiques Config (`std.logging.json` / `std.logging.schema.json`) écrits en dur dans la fixture de rollback.
|
|
|
|
## Correction Clippy
|
|
|
|
Le helper `assert_file` devenu orphelin est supprimé de `ksp-config-lib/unit_tests/logging.rs`.
|
|
|
|
Aucun `#[allow(dead_code)]` n'est ajouté : le test reste nettoyé plutôt que de masquer le warning.
|
|
|
|
## Correction ownership
|
|
|
|
La fixture de rollback de Config Desk ne connaît plus les noms physiques du document Logging et de son schéma.
|
|
|
|
Elle utilise désormais exclusivement les constantes publiques de `ksp-config-lib` :
|
|
|
|
```text
|
|
DEFAULT_STD_LOGGING_FILENAME
|
|
DEFAULT_STD_LOGGING_SCHEMA_FILENAME
|
|
```
|
|
|
|
Le test continue donc à construire son répertoire temporaire et à vérifier le rollback source, mais la propriété des noms physiques reste dans Config.
|
|
|
|
`config/std.logging.json` n'est pas livré par ce correctif et la configuration locale éditée pendant les essais n'est pas remplacée.
|
|
|
|
## Hot reload
|
|
|
|
Aucun changement de logique n'est apporté à la transaction `Sauvegarder et appliquer` de `fix.002` :
|
|
|
|
- persistence Config ;
|
|
- résolution effective ;
|
|
- `ksp_logging_lib::reinitialize()` ;
|
|
- rollback source si l'application runtime échoue ;
|
|
- incrément de `logging_generation` seulement après succès.
|
|
|
|
## Version technique
|
|
|
|
```text
|
|
0.1.4-pre.15.fix.3
|
|
```
|
|
|
|
## Validation attendue
|
|
|
|
```bash
|
|
cargo fmt --all
|
|
cargo check --workspace
|
|
cargo clippy --workspace --all-targets
|
|
cargo test -p ksp-app-config-desk
|
|
cargo test -p ksp-config-lib
|
|
cargo tauri dev -c crates/ksp-app-config-desk/tauri.conf.json
|
|
```
|
|
|
|
Résultat attendu : aucun warning Clippy et `5/5` tests ownership.
|