Suivis de la revue de #460 : CSV compilés hors des chemins de check-rust, cargo-xwin et toolchain non épinglés à la release, commentaire périmé de release.yml #461

New issue

Open

opened 2026-10-18 12:56:04 +00:00 by maximus · 1 comment

maximus commented

2026-10-18 12:56:04 +00:00

Owner

Constats de la revue de la PR #460 (commentaire 24517), hors du périmètre de #456.

1. CSV compilés hors des `paths` de `check-rust`

src-tauri/src/commands/fs_commands.rs:218-221 compile par include_str! les fichiers src/__fixtures__/samples/exemple-desjardins.csv et exemple-tangerine.csv. Ils ne sont pas dans les paths de check-rust.yml (src-tauri/**, .cargo/**, le workflow lui-même). Une PR qui les renomme, les déplace ou les supprime casse donc cargo check sans que check-rust tourne. check-frontend tourne, lui, et vitest passe si samples/index.ts suit. La casse n'apparaît qu'au run du lundi ou au pré-vol de /release.

Piste : ajouter src/__fixtures__/samples/** aux paths de check-rust.yml (un déclencheur ajouté reste de niveau normal selon REVIEW.md), puis retirer l'exception du piège « CI d'une pile » de REVIEW.md.

2. `cargo-xwin` et la toolchain Rust non épinglés dans `release.yml`

release.yml:46 installe cargo-xwin par cargo install --locked cargo-xwin, donc la dernière version publiée à chaque tag ; release.yml:63 le lance par npx tauri build --runner cargo-xwin, avec TAURI_SIGNING_PRIVATE_KEY dans l'environnement. La toolchain Rust vient de rustup sans version, donc stable. Les deux s'exécutent avec la clé de signature.

Piste : épingler cargo-xwin (--version x.y.z) et, à décider, la toolchain ; chaque montée devient alors un diff relu. Cette PR sera de niveau haut (release.yml).

3. Commentaire périmé de `release.yml:104`

« One key per Linux package, a .deb fallback and the NSIS installer » : la clé générique, ce .deb de repli, a été retirée le 2026-10-11 (#320, PR #418).

Critères d'acceptation

check-rust tourne sur une PR qui ne touche qu'un CSV de src/__fixtures__/samples/, preuve par le statut de commit de cette PR, et REVIEW.md perd l'exception.
cargo-xwin est épinglé dans release.yml, et la décision sur la toolchain est consignée.
Le commentaire de release.yml:104 est corrigé.

Constats de la revue de la PR #460 (commentaire 24517), hors du périmètre de #456. ## 1. CSV compilés hors des `paths` de `check-rust` `src-tauri/src/commands/fs_commands.rs:218-221` compile par `include_str!` les fichiers `src/__fixtures__/samples/exemple-desjardins.csv` et `exemple-tangerine.csv`. Ils ne sont pas dans les `paths` de `check-rust.yml` (`src-tauri/**`, `.cargo/**`, le workflow lui-même). Une PR qui les renomme, les déplace ou les supprime casse donc `cargo check` sans que `check-rust` tourne. `check-frontend` tourne, lui, et vitest passe si `samples/index.ts` suit. La casse n'apparaît qu'au run du lundi ou au pré-vol de `/release`. Piste : ajouter `src/__fixtures__/samples/**` aux `paths` de `check-rust.yml` (un déclencheur ajouté reste de niveau normal selon `REVIEW.md`), puis retirer l'exception du piège « CI d'une pile » de `REVIEW.md`. ## 2. `cargo-xwin` et la toolchain Rust non épinglés dans `release.yml` `release.yml:46` installe `cargo-xwin` par `cargo install --locked cargo-xwin`, donc la dernière version publiée à chaque tag ; `release.yml:63` le lance par `npx tauri build --runner cargo-xwin`, avec `TAURI_SIGNING_PRIVATE_KEY` dans l'environnement. La toolchain Rust vient de `rustup` sans version, donc `stable`. Les deux s'exécutent avec la clé de signature. Piste : épingler `cargo-xwin` (`--version x.y.z`) et, à décider, la toolchain ; chaque montée devient alors un diff relu. Cette PR sera de niveau haut (`release.yml`). ## 3. Commentaire périmé de `release.yml:104` « One key per Linux package, a .deb fallback and the NSIS installer » : la clé générique, ce `.deb` de repli, a été retirée le 2026-10-11 (#320, PR #418). ## Critères d'acceptation - [ ] `check-rust` tourne sur une PR qui ne touche qu'un CSV de `src/__fixtures__/samples/`, preuve par le statut de commit de cette PR, et `REVIEW.md` perd l'exception. - [ ] `cargo-xwin` est épinglé dans `release.yml`, et la décision sur la toolchain est consignée. - [ ] Le commentaire de `release.yml:104` est corrigé.

maximus added the

status:triage

type:infra

source:human

labels 2026-10-18 12:56:05 +00:00

maximus referenced this issue

2026-10-18 12:56:31 +00:00

docs(review): REVIEW.md, calibrated risk levels and review rules per zone (#456) #460

maximus referenced this issue

2026-10-18 12:56:42 +00:00

docs(review): REVIEW.md, calibrated risk levels and review rules per zone (#456) #460

maximus referenced this issue

2026-10-18 13:33:23 +00:00

Brancher les skills de revue sur REVIEW.md et les niveaux de risque #457

maximus referenced this issue

2026-10-18 17:01:42 +00:00

fix(release): refuse an empty or multi-document latest.json, test the Publish step #464

maximus referenced this issue from a commit

2026-10-18 17:01:43 +00:00

fix(release): refuse an empty or multi-document latest.json, test the Publish step

maximus referenced this issue

2026-10-18 17:07:07 +00:00

fix(release): refuse an empty or multi-document latest.json, test the Publish step #464

maximus commented

2026-10-18 17:38:01 +00:00

Author

Owner

Point 3 (commentaire périmé de release.yml:104, repli .deb) réglé par la PR #464, mergée le 2026-10-18 (main c859405) : le commentaire de l'étape « Generate latest.json » décrit désormais les trois clés, sans clé générique. Restent les points 1 et 2 de cette issue.

Point 3 (commentaire périmé de `release.yml:104`, repli `.deb`) réglé par la PR #464, mergée le 2026-10-18 (`main` `c859405`) : le commentaire de l'étape « Generate latest.json » décrit désormais les trois clés, sans clé générique. Restent les points 1 et 2 de cette issue.

maximus referenced this issue

2026-10-18 21:42:27 +00:00

ci: --locked in check-rust and the release builds, cargo-xwin pinned, cargo metadata --locked in /release (#449) #469

maximus referenced this issue

2026-10-18 21:48:05 +00:00

ci: --locked in check-rust and the release builds, cargo-xwin pinned, cargo metadata --locked in /release (#449) #469